app_list: Fix shadow bleeds over launcher button and steals mouse events

app_list: Fix shadow bleeds over launcher button and steals mouse events

- Add hit test mask to aura Window;
- Use hit test mask to exclude shadow from hit test;

BUG=132872
TEST=Verify mouse events reaches app list launcher button when app list bubble is shown. Clicking on top 3 rect of the 3x3 grid icon should dismiss app list and mouse hover there should have a hover effect.

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

[sky, 2012-06-15 22:25:44]

I'm passing this one to Ben.

[xiyuan, 2012-06-15 22:28:32]

On 2012/06/15 22:25:44, sky wrote:
> I'm passing this one to Ben.

I was wondering this too. :)

[Ben Goodger (Google), 2012-06-15 22:57:26]

lgtm

http://codereview.chromium.org/10566009/diff/1/ui/aura/window_delegate.h
File ui/aura/window_delegate.h (right):

http://codereview.chromium.org/10566009/diff/1/ui/aura/window_delegate.h#newc...
ui/aura/window_delegate.h:89: // mask. It works similar to the vies
counterparts. That is, if the function
views

[xiyuan, 2012-06-15 23:07:41]

http://codereview.chromium.org/10566009/diff/1/ui/aura/window_delegate.h
File ui/aura/window_delegate.h (right):

http://codereview.chromium.org/10566009/diff/1/ui/aura/window_delegate.h#newc...
ui/aura/window_delegate.h:89: // mask. It works similar to the vies
counterparts. That is, if the function
On 2012/06/15 22:57:26, Ben Goodger (Google) wrote:
> views

Done.

[commit-bot: I haz the power, 2012-06-16 03:29:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/xiyuan@chromium.org/10566009/13001

[commit-bot: I haz the power, 2012-06-16 05:10:27]

Change committed as 142582