Make EnvInputStateController owned by aura::Env.

Make EnvInputStateController owned by aura::Env.

EnvInputStateController::touch_ids_down_ should be a global state
and EnvInputStateController is updating global input event states 
in Env so it makes sense to have Env own EnvInputStateController.

BUG=none, releated to https://codereview.chromium.org/2779093004/
TEST=covered by tests

Review-Url: https://codereview.chromium.org/2854663002
Cr-Commit-Position: refs/heads/master@{#468745}
Committed: https://chromium.googlesource.com/chromium/src/+/717d8eaea3a41525ff8c4d1d9bf06f975e1f02a1

[riajiang, 2017-05-01 20:09:35]

Description was changed from

==========
Make EnvInputStateController owned by aura::Env.

BUG=none, releated to https://codereview.chromium.org/2779093004/
TEST=covered by tests
==========

to

==========
Make EnvInputStateController owned by aura::Env.

EnvInputStateController::touch_ids_down_ should be a global state
and EnvInputStateController is updating global input event states 
in Env so it makes sense to have Env own EnvInputStateController.

BUG=none, releated to https://codereview.chromium.org/2779093004/
TEST=covered by tests
==========

[riajiang, 2017-05-01 20:10:45]

riajiang@chromium.org changed reviewers:
+ sadrul@chromium.org

[riajiang, 2017-05-01 20:10:45]

Hi sadrul@, could you take a look? Thanks!

[riajiang, 2017-05-01 20:34:47]

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

[commit-bot: I haz the power, 2017-05-01 20:35:36]

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-05-01 21:55:47]

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

[commit-bot: I haz the power, 2017-05-01 21:55:49]

Dry run: This issue passed the CQ dry run.

[sadrul, 2017-05-02 18:10:51]

lgtm

[riajiang, 2017-05-02 19:29:53]

The CQ bit was checked by riajiang@chromium.org

[commit-bot: I haz the power, 2017-05-02 19:30:23]

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-05-02 19:35:13]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493753393096660, "parent_rev":
"60e148f1da0c27818ec63406b731ca686393440c", "commit_rev":
"717d8eaea3a41525ff8c4d1d9bf06f975e1f02a1"}

[commit-bot: I haz the power, 2017-05-02 19:35:27]

Description was changed from

==========
Make EnvInputStateController owned by aura::Env.

EnvInputStateController::touch_ids_down_ should be a global state
and EnvInputStateController is updating global input event states 
in Env so it makes sense to have Env own EnvInputStateController.

BUG=none, releated to https://codereview.chromium.org/2779093004/
TEST=covered by tests
==========

to

==========
Make EnvInputStateController owned by aura::Env.

EnvInputStateController::touch_ids_down_ should be a global state
and EnvInputStateController is updating global input event states 
in Env so it makes sense to have Env own EnvInputStateController.

BUG=none, releated to https://codereview.chromium.org/2779093004/
TEST=covered by tests

Review-Url: https://codereview.chromium.org/2854663002
Cr-Commit-Position: refs/heads/master@{#468745}
Committed:
https://chromium.googlesource.com/chromium/src/+/717d8eaea3a41525ff8c4d1d9bf0...
==========

[commit-bot: I haz the power, 2017-05-02 19:35:29]

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

[sadrul, 2017-05-02 21:46:11]

In a follow up patch, can you write a test that does this:
 . Create two WindowTreeHost.
 . Send touch-press events to both.
 . Release touch on one.
 . Verify that Env::is_touch_down() returns true

Because with the old code, I think it would return false.