Shuffles around immersive related classes

Shuffles around immersive related classes

All the shared classes are now in ash/shared. WmImmersive* has been
nuked as it's no longer necessary. WmShell creates ImmersiveContextAsh
as it works with mus and aura. I will make WmShell create
ImmersiveFullscreenController directly soonish. I need
https://codereview.chromium.org/2248263003/ to land first though.

BUG=548435
TEST=covered by tests
R=jamescook@chromium.org

Committed: https://crrev.com/228c74f7599b1d36139856737a0e26feccb4d372
Cr-Commit-Position: refs/heads/master@{#413227}

[sky, 2016-08-19 17:41:53]

Description was changed from

==========
Shuffles around immersive related classes

All the shared classes are now in ash/shared. WmImmersive* has been
nuked as it's no longer necessary. WmShell creates ImmersiveContextAsh
as it works with mus and aura. I will make WmShell create
ImmersiveFullscreenController directly soonish. I need
https://codereview.chromium.org/2248263003/ to land first though.

BUG=627146
TEST=covered by tests
R=jamescook@chromium.org
==========

to

==========
Shuffles around immersive related classes

All the shared classes are now in ash/shared. WmImmersive* has been
nuked as it's no longer necessary. WmShell creates ImmersiveContextAsh
as it works with mus and aura. I will make WmShell create
ImmersiveFullscreenController directly soonish. I need
https://codereview.chromium.org/2248263003/ to land first though.

BUG=548435
TEST=covered by tests
R=jamescook@chromium.org
==========

[James Cook, 2016-08-19 18:15:17]

LGTM

https://codereview.chromium.org/2265563002/diff/20001/ash/shared/immersive_fu...
File ash/shared/immersive_fullscreen_controller.cc (right):

https://codereview.chromium.org/2265563002/diff/20001/ash/shared/immersive_fu...
ash/shared/immersive_fullscreen_controller.cc:65: // static
nit: Either put the comment here for consistency with above, or move the above
comment to header file.

[sky, 2016-08-19 19:25:40]

https://codereview.chromium.org/2265563002/diff/20001/ash/shared/immersive_fu...
File ash/shared/immersive_fullscreen_controller.cc (right):

https://codereview.chromium.org/2265563002/diff/20001/ash/shared/immersive_fu...
ash/shared/immersive_fullscreen_controller.cc:65: // static
On 2016/08/19 18:15:16, James Cook (slow reviews) wrote:
> nit: Either put the comment here for consistency with above, or move the above
> comment to header file.

I moved the above comment to the header as that is generally where we put
comments.

[sky, 2016-08-19 19:26:00]

The CQ bit was checked by sky@chromium.org

[sky, 2016-08-19 19:26:01]

The patchset sent to the CQ was uploaded after l-g-t-m from
jamescook@chromium.org
Link to the patchset: https://codereview.chromium.org/2265563002/#ps60001
(title: "move comment")

[commit-bot: I haz the power, 2016-08-19 19:26:44]

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-19 20:18:32]

Description was changed from

==========
Shuffles around immersive related classes

All the shared classes are now in ash/shared. WmImmersive* has been
nuked as it's no longer necessary. WmShell creates ImmersiveContextAsh
as it works with mus and aura. I will make WmShell create
ImmersiveFullscreenController directly soonish. I need
https://codereview.chromium.org/2248263003/ to land first though.

BUG=548435
TEST=covered by tests
R=jamescook@chromium.org
==========

to

==========
Shuffles around immersive related classes

All the shared classes are now in ash/shared. WmImmersive* has been
nuked as it's no longer necessary. WmShell creates ImmersiveContextAsh
as it works with mus and aura. I will make WmShell create
ImmersiveFullscreenController directly soonish. I need
https://codereview.chromium.org/2248263003/ to land first though.

BUG=548435
TEST=covered by tests
R=jamescook@chromium.org
==========

[commit-bot: I haz the power, 2016-08-19 20:18:33]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-08-19 20:22:33]

Description was changed from

==========
Shuffles around immersive related classes

All the shared classes are now in ash/shared. WmImmersive* has been
nuked as it's no longer necessary. WmShell creates ImmersiveContextAsh
as it works with mus and aura. I will make WmShell create
ImmersiveFullscreenController directly soonish. I need
https://codereview.chromium.org/2248263003/ to land first though.

BUG=548435
TEST=covered by tests
R=jamescook@chromium.org
==========

to

==========
Shuffles around immersive related classes

All the shared classes are now in ash/shared. WmImmersive* has been
nuked as it's no longer necessary. WmShell creates ImmersiveContextAsh
as it works with mus and aura. I will make WmShell create
ImmersiveFullscreenController directly soonish. I need
https://codereview.chromium.org/2248263003/ to land first though.

BUG=548435
TEST=covered by tests
R=jamescook@chromium.org

Committed: https://crrev.com/228c74f7599b1d36139856737a0e26feccb4d372
Cr-Commit-Position: refs/heads/master@{#413227}
==========

[commit-bot: I haz the power, 2016-08-19 20:22:34]

Patchset 4 (id:??) landed as
https://crrev.com/228c74f7599b1d36139856737a0e26feccb4d372
Cr-Commit-Position: refs/heads/master@{#413227}