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
-~----------~----~----~----~------~----~------~--~---

Reply via email to