Update the button visibility when mouse enters or exits from notification surface window

Update the button visibility when mouse enters or exits from notification surface window

The outer view's ONMOUSEENTERED/EXITED are not called on toast, so we need to handle the window's event handlers.

BUG=728915
TEST=manual (issue is not happened)

Review-Url: https://codereview.chromium.org/2922643002
Cr-Commit-Position: refs/heads/master@{#477927}
Committed: https://chromium.googlesource.com/chromium/src/+/7b1b44becb55f7920992b9027307531932f83ce7

[yoshiki, 2017-06-02 03:11:02]

The CQ bit was checked by yoshiki@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-02 03:11:53]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-02 03:41:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-02 03:41:49]

Dry run: This issue passed the CQ dry run.

[yoshiki, 2017-06-06 09:26:14]

The CQ bit was checked by yoshiki@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-06 09:26:24]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yoshiki, 2017-06-06 09:32:39]

Description was changed from

==========
wip

.

BUG=
==========

to

==========
Update the button visibility when mouse enters or exits from notification
surface window

The outer view's ONMOUSEENTERED/EXITED are not called on toast, so we need to
handle the window's event handlers.

BUG=728915
TEST=manual (issue is not happened)
==========

[yoshiki, 2017-06-06 09:32:39]

yoshiki@chromium.org changed reviewers:
+ yhanada@chromium.org

[yoshiki, 2017-06-06 09:33:09]

Hanada-san, PTAL. Thanks.

[commit-bot: I haz the power, 2017-06-06 09:54:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-06 09:54:02]

Dry run: This issue passed the CQ dry run.

[yoshiki, 2017-06-06 11:08:04]

As offline discussion with Hanada-san, it looks the root cause is deeper than I
thought. Let me investigate further.

[yhanada, 2017-06-08 08:27:41]

lgtm

[yoshiki, 2017-06-08 08:34:11]

On 2017/06/08 08:27:41, yhanada wrote:
> lgtm

Thanks!

As we discussed offline, thoug, this patch looks ok.

[yoshiki, 2017-06-08 08:34:16]

The CQ bit was checked by yoshiki@chromium.org

[commit-bot: I haz the power, 2017-06-08 08:34:35]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-08 09:02:46]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1496910856128700,
"parent_rev": "d9072312c7152bcbfc21df56e7646dd2870b1dc0", "commit_rev":
"7b1b44becb55f7920992b9027307531932f83ce7"}

[commit-bot: I haz the power, 2017-06-08 09:02:58]

Description was changed from

==========
Update the button visibility when mouse enters or exits from notification
surface window

The outer view's ONMOUSEENTERED/EXITED are not called on toast, so we need to
handle the window's event handlers.

BUG=728915
TEST=manual (issue is not happened)
==========

to

==========
Update the button visibility when mouse enters or exits from notification
surface window

The outer view's ONMOUSEENTERED/EXITED are not called on toast, so we need to
handle the window's event handlers.

BUG=728915
TEST=manual (issue is not happened)

Review-Url: https://codereview.chromium.org/2922643002
Cr-Commit-Position: refs/heads/master@{#477927}
Committed:
https://chromium.googlesource.com/chromium/src/+/7b1b44becb55f7920992b9027307...
==========

[commit-bot: I haz the power, 2017-06-08 09:02:59]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/7b1b44becb55f7920992b9027307...