Don't regenerate browser frame on native theme changes.

BrowserView was calling BrowserFrame::FrameTypeChanged() whenever the
native theme changed. While Chrome theme changes can indeed cause frame
changes, native theme changes should largely be color-based and should
not require re-creation of the frame and re-composition of the entire
window. This was causing flicker, which was especially pronounced on
older machines and computers running Windows 8.x.

See attached bug for details.

(cherry picked from commit b153488cab7173359d93cc0c64af13125d857619)

Bug: 945138
Change-Id: I002a6535de507866b781643ce9afb81002c6946c
Commit-Queue: Dana Fried <>
Reviewed-by: Peter Kasting <>
Cr-Original-Commit-Position: refs/heads/master@{#644544}
Reviewed-by: Dana Fried <>
Cr-Commit-Position: refs/branch-heads/3729@{#511}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}
5 files changed