chromeos: convert more ash/wm code to aura::Window

chromeos: convert more ash/wm code to aura::Window

BUG=722553
TEST=covered by tests
R=varkha@chromium.org

Review-Url: https://codereview.chromium.org/2905893002
Cr-Commit-Position: refs/heads/master@{#474757}
Committed: https://chromium.googlesource.com/chromium/src/+/cd9d4aeceea0690b9ce7aef8b84d6ed92b7b2f87

[sky, 2017-05-25 17:21:36]

Description was changed from

==========
chromeos: convert more ash/wm code to aura::Window

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

to

==========
chromeos: convert more ash/wm code to aura::Window

BUG=722553
TEST=covered by tests
R=varkha@chromium.org
==========

[sky, 2017-05-25 17:21:37]

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

[sky, 2017-05-25 17:21:42]

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

[commit-bot: I haz the power, 2017-05-25 17:22:15]

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-25 18:38:17]

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

[commit-bot: I haz the power, 2017-05-25 18:38:18]

Dry run: This issue passed the CQ dry run.

[varkha, 2017-05-25 19:11:52]

https://codereview.chromium.org/2905893002/diff/40001/ash/wm/wm_toplevel_wind...
File ash/wm/wm_toplevel_window_event_handler.cc (right):

https://codereview.chromium.org/2905893002/diff/40001/ash/wm/wm_toplevel_wind...
ash/wm/wm_toplevel_window_event_handler.cc:531: wm::WindowState* window_state =
GetWindowState(window);
nit: Do you need wm:: (here and elsewhere in this file and .h)?

https://codereview.chromium.org/2905893002/diff/40001/ash/wm/workspace/worksp...
File ash/wm/workspace/workspace_event_handler_aura.cc (right):

https://codereview.chromium.org/2905893002/diff/40001/ash/wm/workspace/worksp...
ash/wm/workspace/workspace_event_handler_aura.cc:13:
WorkspaceEventHandlerAura::WorkspaceEventHandlerAura(WmWindow* workspace_window)
Is this coming next?

[sky, 2017-05-25 19:26:06]

https://codereview.chromium.org/2905893002/diff/40001/ash/wm/workspace/worksp...
File ash/wm/workspace/workspace_event_handler_aura.cc (right):

https://codereview.chromium.org/2905893002/diff/40001/ash/wm/workspace/worksp...
ash/wm/workspace/workspace_event_handler_aura.cc:13:
WorkspaceEventHandlerAura::WorkspaceEventHandlerAura(WmWindow* workspace_window)
On 2017/05/25 19:11:52, varkha wrote:
> Is this coming next?

Yes. I try to avoid changing everything at once.

[varkha, 2017-05-25 19:29:46]

lgtm

[sky, 2017-05-25 19:32:21]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2017-05-25 19:33: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-05-25 19:38:37]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1495740741758490,
"parent_rev": "4ce4987234f2b38382a7f3610ef195a94e5a6b12", "commit_rev":
"cd9d4aeceea0690b9ce7aef8b84d6ed92b7b2f87"}

[commit-bot: I haz the power, 2017-05-25 19:38:48]

Description was changed from

==========
chromeos: convert more ash/wm code to aura::Window

BUG=722553
TEST=covered by tests
R=varkha@chromium.org
==========

to

==========
chromeos: convert more ash/wm code to aura::Window

BUG=722553
TEST=covered by tests
R=varkha@chromium.org

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

[commit-bot: I haz the power, 2017-05-25 19:38:49]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/cd9d4aeceea0690b9ce7aef8b84d...