exo: Use aura::Env()->Add{Pre,Post}TargetHandler to simplfy WmHelperMus.

exo: Use aura::Env()->Add{Pre,Post}TargetHandler to simplfy WmHelperMus.

BUG=None

Committed: https://crrev.com/61e97d3dd0d1d4be51e3c2bd0cbcd3a9df1bef3e
Cr-Commit-Position: refs/heads/master@{#414704}

[Peng, 2016-08-25 20:05:34]

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

[commit-bot: I haz the power, 2016-08-25 20:06:14]

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

[Peng, 2016-08-25 20:06:28]

penghuang@chromium.org changed reviewers:
+ reveman@chromium.org

[Peng, 2016-08-25 20:06:28]

David, PTAL. Thanks.

[reveman, 2016-08-25 20:14:45]

https://codereview.chromium.org/2280763002/diff/1/components/exo/wm_helper_mu...
File components/exo/wm_helper_mus.cc (right):

https://codereview.chromium.org/2280763002/diff/1/components/exo/wm_helper_mu...
components/exo/wm_helper_mus.cc:78:
aura::Env::GetInstance()->AddPreTargetHandler(handler);
could we use the same for Ash impl and just remove this from the WMHelper
interface?

[commit-bot: I haz the power, 2016-08-25 20:55:57]

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

[commit-bot: I haz the power, 2016-08-25 20:55:58]

Dry run: This issue passed the CQ dry run.

[Peng, 2016-08-26 14:00:23]

On 2016/08/25 20:14:45, reveman wrote:
>
https://codereview.chromium.org/2280763002/diff/1/components/exo/wm_helper_mu...
> File components/exo/wm_helper_mus.cc (right):
> 
>
https://codereview.chromium.org/2280763002/diff/1/components/exo/wm_helper_mu...
> components/exo/wm_helper_mus.cc:78:
> aura::Env::GetInstance()->AddPreTargetHandler(handler);
> could we use the same for Ash impl and just remove this from the WMHelper
> interface?

I tried use arua::Env for ash. It works. But the unittests failed. The aura::Env
can not 
get events from ui::test::EventGenerator.

[reveman, 2016-08-26 14:07:52]

On 2016/08/26 at 14:00:23, penghuang wrote:
> On 2016/08/25 20:14:45, reveman wrote:
> >
https://codereview.chromium.org/2280763002/diff/1/components/exo/wm_helper_mu...
> > File components/exo/wm_helper_mus.cc (right):
> > 
> >
https://codereview.chromium.org/2280763002/diff/1/components/exo/wm_helper_mu...
> > components/exo/wm_helper_mus.cc:78:
> > aura::Env::GetInstance()->AddPreTargetHandler(handler);
> > could we use the same for Ash impl and just remove this from the WMHelper
> > interface?
> 
> I tried use arua::Env for ash. It works. But the unittests failed. The
aura::Env can not 
> get events from ui::test::EventGenerator.

ok, let's go ahead with this patch then. lgtm.

[Peng, 2016-08-26 14:09:54]

The CQ bit was checked by penghuang@chromium.org

[commit-bot: I haz the power, 2016-08-26 14:10:16]

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-08-26 14:14:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-26 14:15:54]

Description was changed from

==========
exo: Use aura::Env()->Add{Pre,Post}TargetHandler to simplfy WmHelperMus.

BUG=None
==========

to

==========
exo: Use aura::Env()->Add{Pre,Post}TargetHandler to simplfy WmHelperMus.

BUG=None

Committed: https://crrev.com/61e97d3dd0d1d4be51e3c2bd0cbcd3a9df1bef3e
Cr-Commit-Position: refs/heads/master@{#414704}
==========

[commit-bot: I haz the power, 2016-08-26 14:15:55]

Patchset 1 (id:??) landed as
https://crrev.com/61e97d3dd0d1d4be51e3c2bd0cbcd3a9df1bef3e
Cr-Commit-Position: refs/heads/master@{#414704}