Enable revealing the top-of-window views in immersive fullscreen when the owning window is inactive

Enable revealing the top-of-window views in immersive fullscreen when the mouse is hovered over the top edge of the fullscreen window but the fullscreen window is inactive. This is consistent to how hover works elsewhere on ChromeOS (e.g. the maximize/restore button's help bubble)

BUG=319959
TEST=ImmersiveFullscreenControllerTest.Inactive
R=jamescook
TBR=sky (For change to DragTabToImmersiveBrowserOnSeparateDisplay test)

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=238287

[pkotwicz, 2013-12-01 22:57:20]

James, can you please take a look?

https://codereview.chromium.org/97943002/diff/170001/ash/wm/immersive_fullscr...
File ash/wm/immersive_fullscreen_controller.cc (right):

https://codereview.chromium.org/97943002/diff/170001/ash/wm/immersive_fullscr...
ash/wm/immersive_fullscreen_controller.cc:543:
!native_window_->Contains(static_cast<aura::Window*>(event->target()))) {
I check whether |top_edge_hover_timer_| is running because the event may be at a
location where it should be ignored.

https://codereview.chromium.org/97943002/diff/170001/ash/wm/immersive_fullscr...
ash/wm/immersive_fullscreen_controller.cc:603: // top-of-window views may be
partially obscured when |widget_| is inactive.
Doing the right thing is complicated by the maximize/restore button's help
bubble. In particular, we would need to make sure that the top-of-window views
do not hide when the cursor is over the maximize/restore button's help bubble. I
have not added a TODO because I do not think that doing the right thing is a
priority.

[James Cook, 2013-12-02 17:58:01]

LGTM. Nice patch.

[commit-bot: I haz the power, 2013-12-02 18:46:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/97943002/170001

[commit-bot: I haz the power, 2013-12-02 20:27:00]

Retried try job too often on linux_chromeos for step(s) interactive_ui_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[commit-bot: I haz the power, 2013-12-02 22:58:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/97943002/210001

[commit-bot: I haz the power, 2013-12-02 23:07:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/97943002/210001

[commit-bot: I haz the power, 2013-12-03 03:18:02]

Change committed as 238287