Comment #21 on issue 5054 by [email protected]: [magic_browzr] Window
size/position wrong in XP maximized mode
http://code.google.com/p/chromium/issues/detail?id=5054
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=7976
------------------------------------------------------------------------
r7976 | [email protected] | 2009-01-13 14:34:48 -0800 (Tue, 13 Jan
2009) | 2 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/frame/aero_glass_non_client_view.h?r1=7976&r2=7975
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/frame/browser_frame.h?r1=7976&r2=7975
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/frame/browser_view.cc?r1=7976&r2=7975
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/frame/opaque_non_client_view.cc?r1=7976&r2=7975
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/frame/opaque_non_client_view.h?r1=7976&r2=7975
Pixel perfection on tabstrip for non-Aero frame. This gets both ends of
the tabstrip positioned correctly for normal and incognito windows, in both
restored and maximized modes.The chief change was to correctly transform
the bounds returned by the non-client view into the BrowserView's
coordinate system, as the Views system expected. To make this not break
the world I then had to actually return the right coordinates in the
non-client view :). Updated comments on function declarations to hopefully
match reality.For convenience this also introduces a function to get the
horizontal border size and cleans up existing code to use it.BUG=5054
Review URL: http://codereview.chromium.org/18019
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---