Comment #4 on issue 9928 by [email protected]: Regression: Status  
bubble shadow is not transparent
http://code.google.com/p/chromium/issues/detail?id=9928

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=13670

------------------------------------------------------------------------
r13670 | [email protected] | 2009-04-14 10:37:04 -0700 (Tue, 14 Apr  
2009) | 4 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/frame/browser_view.cc?r1=13670&r2=13669
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/status_bubble_views.cc?r1=13670&r2=13669
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/status_bubble_views.h?r1=13670&r2=13669

Fix status bubble shadow being opaque, which made the bubble appear to be  
off by one pixel.  This reverts Brett's fix for issue 8651 and instead  
fixes that issue by ensuring the text bounding box does not overlap the  
status bubble shadow.  This also makes kShadowThickness (renamed for  
consistency with the browser frame) a class-scoped constant so the browser  
view can access it, eliminating a magic number.

BUG=9928
Review URL: http://codereview.chromium.org/67059
------------------------------------------------------------------------


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