Fixes the problem of the status bubble obscuring the download shelf....

This fixes the problem of the status bubble obscuring the download shelf.

BUG= http://crbug.com/6590
TEST=Cause the download shelf to appear by initiating a download.  Hover
over a link in the extreme lower left corner of the browser window.  The
status bubble should avoid your mouse by moving to the right, instead of
moving down and obscuring the download shelf.



Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=17025

[Miranda Callahan, 2009-05-20 22:41:54]

This is basically a tweak of Yusuke's code to move the status bubble when the
download shelf is visible.

[Yusuke (unused), 2009-05-22 05:48:14]

http://codereview.chromium.org/113671/diff/32/1034
File chrome/browser/views/status_bubble_views.cc (right):

http://codereview.chromium.org/113671/diff/32/1034#newcode612
Line 612: offset > kBubbleCornerRadius / 2 - kShadowThickness)) {
You might want to use GetStyle() function which I recently added (e.g.,
view_->GetStyle() == StatusView::STYLE_FLOATING).

[Miranda Callahan, 2009-05-26 17:14:06]

http://codereview.chromium.org/113671/diff/32/1034
File chrome/browser/views/status_bubble_views.cc (right):

http://codereview.chromium.org/113671/diff/32/1034#newcode612
Line 612: offset > kBubbleCornerRadius / 2 - kShadowThickness)) {
On 2009/05/22 05:48:14, Yusuke wrote:
> You might want to use GetStyle() function which I recently added (e.g.,
> view_->GetStyle() == StatusView::STYLE_FLOATING).

Thanks, Yusuke!  Done.

[Yusuke (unused), 2009-05-27 01:03:29]

LGTM

LGTM, but please note that I'm not a chromium committer. Glen may have a
different opinion.

--Yusuke

[Miranda Callahan, 2009-05-27 16:21:26]

Thank you, Yusuke!  I'll commit if Glen gives the ok...

On 2009/05/27 01:03:29, Yusuke wrote:
> LGTM
> 
> LGTM, but please note that I'm not a chromium committer. Glen may have a
> different opinion.
> 
> --Yusuke

[Glen Murphy, 2009-05-27 17:54:51]

LGTM.

Erik Kay should also take a quick peek as he is adding other bottom-shelves, and
maybe we want to abstract it into something like, but less ridiculous-sounding
than "Content Area Bottom Changed".

But don't let that stop you committing now.