Revert of desktop_aura: do not restore focused view if it has modal transient child

Revert of desktop_aura: do not restore focused view if it has modal transient child (patchset #3 id:40001 of https://codereview.chromium.org/2919973002/ )

Reason for revert:
The newly-added test is flaky:

https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNySwsSBUZsYWtlIkBXaWRnZXRUZXN0SW50ZXJhY3RpdmUuRGVza3RvcE5hdGl2ZVdpZGdldFdpdGhNb2RhbFRyYW5zaWVudENoaWxkDA

Reverting the CL rather than disabling the test as it's not clear where the flake is coming from: is it a problem in the test or in the production code added in this CL?

Original issue's description:
> desktop_aura: do not restore focused view if it has modal transient child
> 
> Changes:
> RestoreFocusedView on DesktopNativeWidgetAura::HandleActivationChanged is added to process Linux desktop native widget asynchronous activation. This works when all the child widgets are managed by top level's FocusManager. However, this is not true for modal transient child, whose FocusManager is not the same as top level's FocusManager. In this case on activation, we shall not restore focused view on top level's FocusManager. Instead, the modal transient child window will get activated and restore focused view through its own NativeWidgetAura::OnWindowActivated.
> 
> BUG=727641
> TEST=emulator test saw bug fixed and added test coverage
> 
> Review-Url: https://codereview.chromium.org/2919973002
> Cr-Commit-Position: refs/heads/master@{#476835}
> Committed: https://chromium.googlesource.com/chromium/src/+/81ead2ab2380d5f624b9ce63df050a76de134f06

TBR=sky@chromium.org,warx@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=727641
Review-Url: https://codereview.chromium.org/2921253002
Cr-Commit-Position: refs/heads/master@{#476970}
Committed: https://chromium.googlesource.com/chromium/src/+/a796415316eb8e7c8a6f1e746b837f8f24a60382

[blundell, 2017-06-05 09:57:50]

The CQ bit was checked by blundell@chromium.org

[blundell, 2017-06-05 09:57:50]

Created Revert of desktop_aura: do not restore focused view if it has modal
transient child

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

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-05 10:55:55]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496656670242640, "parent_rev":
"106d7cc34ecb15837748d047fa6de4f5a40bf34e", "commit_rev":
"a796415316eb8e7c8a6f1e746b837f8f24a60382"}

[commit-bot: I haz the power, 2017-06-05 10:56:07]

Description was changed from

==========
Revert of desktop_aura: do not restore focused view if it has modal transient
child (patchset #3 id:40001 of https://codereview.chromium.org/2919973002/ )

Reason for revert:
The newly-added test is flaky:

https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9...

Reverting the CL rather than disabling the test as it's not clear where the
flake is coming from: is it a problem in the test or in the production code
added in this CL?

Original issue's description:
> desktop_aura: do not restore focused view if it has modal transient child
> 
> Changes:
> RestoreFocusedView on DesktopNativeWidgetAura::HandleActivationChanged is
added to process Linux desktop native widget asynchronous activation. This works
when all the child widgets are managed by top level's FocusManager. However,
this is not true for modal transient child, whose FocusManager is not the same
as top level's FocusManager. In this case on activation, we shall not restore
focused view on top level's FocusManager. Instead, the modal transient child
window will get activated and restore focused view through its own
NativeWidgetAura::OnWindowActivated.
> 
> BUG=727641
> TEST=emulator test saw bug fixed and added test coverage
> 
> Review-Url: https://codereview.chromium.org/2919973002
> Cr-Commit-Position: refs/heads/master@{#476835}
> Committed:
https://chromium.googlesource.com/chromium/src/+/81ead2ab2380d5f624b9ce63df05...

TBR=sky@chromium.org,warx@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=727641
==========

to

==========
Revert of desktop_aura: do not restore focused view if it has modal transient
child (patchset #3 id:40001 of https://codereview.chromium.org/2919973002/ )

Reason for revert:
The newly-added test is flaky:

https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9...

Reverting the CL rather than disabling the test as it's not clear where the
flake is coming from: is it a problem in the test or in the production code
added in this CL?

Original issue's description:
> desktop_aura: do not restore focused view if it has modal transient child
> 
> Changes:
> RestoreFocusedView on DesktopNativeWidgetAura::HandleActivationChanged is
added to process Linux desktop native widget asynchronous activation. This works
when all the child widgets are managed by top level's FocusManager. However,
this is not true for modal transient child, whose FocusManager is not the same
as top level's FocusManager. In this case on activation, we shall not restore
focused view on top level's FocusManager. Instead, the modal transient child
window will get activated and restore focused view through its own
NativeWidgetAura::OnWindowActivated.
> 
> BUG=727641
> TEST=emulator test saw bug fixed and added test coverage
> 
> Review-Url: https://codereview.chromium.org/2919973002
> Cr-Commit-Position: refs/heads/master@{#476835}
> Committed:
https://chromium.googlesource.com/chromium/src/+/81ead2ab2380d5f624b9ce63df05...

TBR=sky@chromium.org,warx@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=727641

Review-Url: https://codereview.chromium.org/2921253002
Cr-Commit-Position: refs/heads/master@{#476970}
Committed:
https://chromium.googlesource.com/chromium/src/+/a796415316eb8e7c8a6f1e746b83...
==========

[commit-bot: I haz the power, 2017-06-05 10:56:08]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/a796415316eb8e7c8a6f1e746b83...