Skip to content

Commit

Permalink
Fix parentheses omission which led to tab misplacement in maximized m…
Browse files Browse the repository at this point in the history
…ode. Also remove an extra pair of parentheses that snuck in.

TBR=jcivelli
BUG=none
TEST=When maximized with a custom theme, tabs should be at the top of the screen
Review URL: http://codereview.chromium.org/3191006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56450 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
pkasting@chromium.org committed Aug 17, 2010
1 parent 086a9a5 commit 30be4cd
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions chrome/browser/views/frame/opaque_browser_frame_view.cc
Original file line number Diff line number Diff line change
Expand Up @@ -479,14 +479,14 @@ int OpaqueBrowserFrameView::NonClientTopBorderHeight() const {
}

return FrameBorderThickness() -
(browser_view_->IsTabStripVisible() && window->IsMaximized()) ?
kTabstripTopShadowThickness : 0;
((browser_view_->IsTabStripVisible() && window->IsMaximized()) ?
kTabstripTopShadowThickness : 0);
}

int OpaqueBrowserFrameView::CaptionButtonY() const {
// Maximized buttons start at window top so that even if their images aren't
// drawn flush with the screen edge, they still obey Fitts' Law.
return (frame_->GetWindow()->IsMaximized()) ?
return frame_->GetWindow()->IsMaximized() ?
FrameBorderThickness() : kFrameShadowThickness;
}

Expand Down

0 comments on commit 30be4cd

Please sign in to comment.