Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3219)

Unified Diff: chrome/browser/ui/views/frame/browser_frame_win.cc

Issue 11273107: Fix BrowserFrameWin artifact on opaque to glass frame changes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Cleanup. Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/frame/browser_frame_win.cc
diff --git a/chrome/browser/ui/views/frame/browser_frame_win.cc b/chrome/browser/ui/views/frame/browser_frame_win.cc
index ee3a0b723b515b61b1da7618888f674c9c59364a..be2f95001b9295123ec0726ad50c439869d9f4fa 100644
--- a/chrome/browser/ui/views/frame/browser_frame_win.cc
+++ b/chrome/browser/ui/views/frame/browser_frame_win.cc
@@ -177,10 +177,9 @@ bool BrowserFrameWin::GetClientAreaInsets(gfx::Insets* insets) const {
}
void BrowserFrameWin::HandleFrameChanged() {
- // We need to update the glass region on or off before the base class adjusts
- // the window region.
- UpdateDWMFrame();
+ // Handle window frame layout changes, then set the updated glass region.
NativeWidgetWin::HandleFrameChanged();
+ UpdateDWMFrame();
}
bool BrowserFrameWin::PreHandleMSG(UINT message,
@@ -455,7 +454,7 @@ void BrowserFrameWin::UpdateDWMFrame() {
}
// In maximized mode, we only have a titlebar strip of glass, no side/bottom
// borders.
- if (!browser_view_->IsFullscreen()) {
+ if (!IsFullscreen()) {
gfx::Rect tabstrip_bounds(
browser_frame_->GetBoundsForTabStrip(browser_view_->tabstrip()));
margins.cyTopHeight = tabstrip_bounds.bottom() + kDWMFrameTopOffset;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698