arc: Fix missing notification on extended desktop

arc: Fix missing notification on extended desktop

The notification is missing because old ArcCustomNotificationView
is removed from widget first before ViewHierarchyChanged, which
causes the bail out logic be skipped. The CL updates the bail out
condition so that the bail out logic runs for the case and did not
detach notification surface from the new ArcCustomNotificationView.

BUG=661801
Committed: https://crrev.com/24290805c28dbf3da5220464d68befac854244ad
Cr-Commit-Position: refs/heads/master@{#429599}

[xiyuan, 2016-11-02 22:44:31]

Description was changed from

==========
arc: Fix missing notification on extended desktop

The notification is missing because old ArcCustomNotificationView
is removed from widget first before ViewHierarchyChanged, which
causes the bail out logic be skipped. The CL remove the widget
check on the bail out condition so that the bail out logic runs
for the case and did not detach notification surface from the new
ArcCustomNotificationView.

BUG=661801
==========

to

==========
arc: Fix missing notification on extended desktop

The notification is missing because old ArcCustomNotificationView
is removed from widget first before ViewHierarchyChanged, which
causes the bail out logic be skipped. The CL updates the bail out
condition so that the bail out logic runs for the case and did not
detach notification surface from the new ArcCustomNotificationView.

BUG=661801
==========

[xiyuan, 2016-11-02 22:47:57]

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

[xiyuan, 2016-11-02 22:48:01]

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

[commit-bot: I haz the power, 2016-11-02 22:48:45]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yoshiki, 2016-11-02 22:51:15]

lgtm. Thanks!

[commit-bot: I haz the power, 2016-11-02 23:18:05]

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

[commit-bot: I haz the power, 2016-11-02 23:18:06]

Dry run: This issue passed the CQ dry run.

[xiyuan, 2016-11-03 15:41:41]

The CQ bit was checked by xiyuan@chromium.org

[commit-bot: I haz the power, 2016-11-03 15:42:15]

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, 2016-11-03 15:50:34]

Description was changed from

==========
arc: Fix missing notification on extended desktop

The notification is missing because old ArcCustomNotificationView
is removed from widget first before ViewHierarchyChanged, which
causes the bail out logic be skipped. The CL updates the bail out
condition so that the bail out logic runs for the case and did not
detach notification surface from the new ArcCustomNotificationView.

BUG=661801
==========

to

==========
arc: Fix missing notification on extended desktop

The notification is missing because old ArcCustomNotificationView
is removed from widget first before ViewHierarchyChanged, which
causes the bail out logic be skipped. The CL updates the bail out
condition so that the bail out logic runs for the case and did not
detach notification surface from the new ArcCustomNotificationView.

BUG=661801
==========

[commit-bot: I haz the power, 2016-11-03 15:50:35]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-03 16:11:48]

Description was changed from

==========
arc: Fix missing notification on extended desktop

The notification is missing because old ArcCustomNotificationView
is removed from widget first before ViewHierarchyChanged, which
causes the bail out logic be skipped. The CL updates the bail out
condition so that the bail out logic runs for the case and did not
detach notification surface from the new ArcCustomNotificationView.

BUG=661801
==========

to

==========
arc: Fix missing notification on extended desktop

The notification is missing because old ArcCustomNotificationView
is removed from widget first before ViewHierarchyChanged, which
causes the bail out logic be skipped. The CL updates the bail out
condition so that the bail out logic runs for the case and did not
detach notification surface from the new ArcCustomNotificationView.

BUG=661801

Committed: https://crrev.com/24290805c28dbf3da5220464d68befac854244ad
Cr-Commit-Position: refs/heads/master@{#429599}
==========

[commit-bot: I haz the power, 2016-11-03 16:11:49]

Patchset 2 (id:??) landed as
https://crrev.com/24290805c28dbf3da5220464d68befac854244ad
Cr-Commit-Position: refs/heads/master@{#429599}