Moves TransientWindowClient to ui/aura/client and adds observer

Moves TransientWindowClient to ui/aura/client and adds observer

Mus is going to need to use this from within aura, so I'm moving it to
ui/aura/client. Additionally I'm moving creation of WmState to
AuraTestHelper. Again, mus is going to need it aura test code already
depends upon ui/wm, so I can't think of a reason not to do it
everywhere.

BUG=659155
TEST=covered by tests
R=ben@chromium.org

Committed: https://crrev.com/f8fcebc79eae73281b270ae26c8248f757fb0570
Cr-Commit-Position: refs/heads/master@{#428087}

[sky, 2016-10-26 23:24:15]

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

[commit-bot: I haz the power, 2016-10-26 23:24:55]

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, 2016-10-27 00:14:05]

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

[commit-bot: I haz the power, 2016-10-27 00:14:06]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sky, 2016-10-27 03:25:30]

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

[commit-bot: I haz the power, 2016-10-27 03:25:41]

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, 2016-10-27 04:43:02]

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

[commit-bot: I haz the power, 2016-10-27 04:43:03]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[sky, 2016-10-27 15:49:16]

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

[commit-bot: I haz the power, 2016-10-27 15:49:32]

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, 2016-10-27 15:52:05]

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

[commit-bot: I haz the power, 2016-10-27 15:52:06]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[sky, 2016-10-27 15:54:44]

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

[commit-bot: I haz the power, 2016-10-27 15:55:00]

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, 2016-10-27 16:58:49]

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

[commit-bot: I haz the power, 2016-10-27 16:58:51]

Dry run: This issue passed the CQ dry run.

[sky, 2016-10-27 16:59:28]

Description was changed from

==========
Moves TransientWindowClient to ui/aura/client and adds observer

Mus is going to need to use this from within aura, so I'm moving it to
ui/aura/client. Additionally I'm moving creation of WmState to
AuraTestHelper. Again, mus is going to need it aura test code already
depends upon ui/wm, so I can't think of a reason not to do it
everywhere.

BUG=659155
TEST=covered by tests
R=msw@chromium.org
==========

to

==========
Moves TransientWindowClient to ui/aura/client and adds observer

Mus is going to need to use this from within aura, so I'm moving it to
ui/aura/client. Additionally I'm moving creation of WmState to
AuraTestHelper. Again, mus is going to need it aura test code already
depends upon ui/wm, so I can't think of a reason not to do it
everywhere.

BUG=659155
TEST=covered by tests
R=ben@chromium.org
==========

[sky, 2016-10-27 16:59:28]

sky@chromium.org changed reviewers:
+ ben@chromium.org
- msw@chromium.org

[sky, 2016-10-27 17:02:23]

As part of this I'm making AuraTestHelper install WMState. This meant I could
remove creating WMState from a bunch of places in views. But there was some code
using TestViewsDelegate that isn't using AuraTestHelper, so I had to make them
explicitly create WMState.

[Ben Goodger (Google), 2016-10-27 18:25:11]

lgtm

[sky, 2016-10-27 18:25:50]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2016-10-27 18:26: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, 2016-10-27 18:34:40]

Description was changed from

==========
Moves TransientWindowClient to ui/aura/client and adds observer

Mus is going to need to use this from within aura, so I'm moving it to
ui/aura/client. Additionally I'm moving creation of WmState to
AuraTestHelper. Again, mus is going to need it aura test code already
depends upon ui/wm, so I can't think of a reason not to do it
everywhere.

BUG=659155
TEST=covered by tests
R=ben@chromium.org
==========

to

==========
Moves TransientWindowClient to ui/aura/client and adds observer

Mus is going to need to use this from within aura, so I'm moving it to
ui/aura/client. Additionally I'm moving creation of WmState to
AuraTestHelper. Again, mus is going to need it aura test code already
depends upon ui/wm, so I can't think of a reason not to do it
everywhere.

BUG=659155
TEST=covered by tests
R=ben@chromium.org
==========

[commit-bot: I haz the power, 2016-10-27 18:34:46]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-10-27 19:01:31]

Description was changed from

==========
Moves TransientWindowClient to ui/aura/client and adds observer

Mus is going to need to use this from within aura, so I'm moving it to
ui/aura/client. Additionally I'm moving creation of WmState to
AuraTestHelper. Again, mus is going to need it aura test code already
depends upon ui/wm, so I can't think of a reason not to do it
everywhere.

BUG=659155
TEST=covered by tests
R=ben@chromium.org
==========

to

==========
Moves TransientWindowClient to ui/aura/client and adds observer

Mus is going to need to use this from within aura, so I'm moving it to
ui/aura/client. Additionally I'm moving creation of WmState to
AuraTestHelper. Again, mus is going to need it aura test code already
depends upon ui/wm, so I can't think of a reason not to do it
everywhere.

BUG=659155
TEST=covered by tests
R=ben@chromium.org

Committed: https://crrev.com/f8fcebc79eae73281b270ae26c8248f757fb0570
Cr-Commit-Position: refs/heads/master@{#428087}
==========

[commit-bot: I haz the power, 2016-10-27 19:01:33]

Patchset 5 (id:??) landed as
https://crrev.com/f8fcebc79eae73281b270ae26c8248f757fb0570
Cr-Commit-Position: refs/heads/master@{#428087}