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

Unified Diff: chrome/browser/views/frame/opaque_browser_frame_view.cc

Issue 42027: Make Chromium windows not hide auto-hide taskbars, take 2.... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 11 years, 9 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 | « chrome/browser/views/frame/glass_browser_frame_view.cc ('k') | chrome/common/win_util.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/views/frame/opaque_browser_frame_view.cc
===================================================================
--- chrome/browser/views/frame/opaque_browser_frame_view.cc (revision 11284)
+++ chrome/browser/views/frame/opaque_browser_frame_view.cc (working copy)
@@ -505,7 +505,7 @@
gfx::Path* window_mask) {
DCHECK(window_mask);
- if (browser_view_->IsFullscreen())
+ if (!browser_view_->CanCurrentlyResize())
return;
// Redefine the window visible region for the new size.
@@ -649,10 +649,7 @@
// OpaqueBrowserFrameView, private:
int OpaqueBrowserFrameView::FrameBorderThickness() const {
- if (browser_view_->IsFullscreen())
- return 0;
- return frame_->IsMaximized() ?
- GetSystemMetrics(SM_CXSIZEFRAME) : kFrameBorderThickness;
+ return browser_view_->CanCurrentlyResize() ? kFrameBorderThickness : 0;
}
int OpaqueBrowserFrameView::TopResizeHeight() const {
@@ -920,13 +917,11 @@
// button to the screen corner to obey Fitts' Law.
int right_extra_width = is_maximized ?
(kFrameBorderThickness - kFrameShadowThickness) : 0;
- int right_spacing = is_maximized ?
- (GetSystemMetrics(SM_CXSIZEFRAME) + right_extra_width) : frame_thickness;
gfx::Size close_button_size = close_button_->GetPreferredSize();
- close_button_->SetBounds(width() - close_button_size.width() - right_spacing,
- caption_y,
- close_button_size.width() + right_extra_width,
- close_button_size.height() + top_extra_height);
+ close_button_->SetBounds(width() - close_button_size.width() -
+ right_extra_width - frame_thickness, caption_y,
+ close_button_size.width() + right_extra_width,
+ close_button_size.height() + top_extra_height);
// When the window is restored, we show a maximized button; otherwise, we show
// a restore button.
« no previous file with comments | « chrome/browser/views/frame/glass_browser_frame_view.cc ('k') | chrome/common/win_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698