Random fixes for mash bridge classes

Random fixes for mash bridge classes

. MusLayoutManagerAdapter wasn't installed observer correctly
. Updated set of activation parents.
. WmGlobalMus::GetTopLevelAncestor consults set of activation parents.
. WmGlobalMus deals with WindowTreeConnection being deleted while alive.
. WmLookupMus installs itself as the WmLookup.
. WmRootWindowController lookup key is no longer owned (it'll be owned by
  RootController).
. Added WmRootWindowController::NotifyFullscreenStatechanged().
. Wired up WmRootWindowControllerMus observers.
. Changed WmWindowMus shell ids to be stored in WmWindowMus rather than use
  mus::Window's local_id. This way it doesn't conflict with other code that
  wants to use the local id (we store the container in the local id).
. Route WmWindowMus::SetBounds() to the layutmanager appropriately.
. WmWindowMus::GetboundsInScreen() was including the bounds of 'this' resulting
  in double counting the initial offset.
. WmWindowMus::Activate() moves to front.
. WmWindowMus::OnWindowReordered() was notifying the wrong observers.
. WmWindowMus::OnWindowSharedPropertyChanged now calls to WindowState

R=jamescook@chromium.org
BUG=603369
TEST=none

Committed: https://crrev.com/990b86fe4ea3085e6df6dd58316a2d2c34aad241
Cr-Commit-Position: refs/heads/master@{#394170}

[James Cook, 2016-05-17 16:36:04]

LGTM

https://codereview.chromium.org/1984293002/diff/20001/mash/wm/bridge/wm_windo...
File mash/wm/bridge/wm_window_mus.h (right):

https://codereview.chromium.org/1984293002/diff/20001/mash/wm/bridge/wm_windo...
mash/wm/bridge/wm_window_mus.h:187: int shell_window_id_ = -1;
nit: Is this a container id? If so, may comment that?

[sky, 2016-05-17 16:59:46]

https://codereview.chromium.org/1984293002/diff/20001/mash/wm/bridge/wm_windo...
File mash/wm/bridge/wm_window_mus.h (right):

https://codereview.chromium.org/1984293002/diff/20001/mash/wm/bridge/wm_windo...
mash/wm/bridge/wm_window_mus.h:187: int shell_window_id_ = -1;
On 2016/05/17 16:36:04, James Cook wrote:
> nit: Is this a container id? If so, may comment that?

It's one of the ids defined in wm_shell_window_ids. I added a comment.

[sky, 2016-05-17 16:59:51]

The CQ bit was checked by sky@chromium.org

[sky, 2016-05-17 16:59:51]

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/1984293002/#ps40001
(title: "comment")

[commit-bot: I haz the power, 2016-05-17 17:00:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1984293002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1984293002/40001

[commit-bot: I haz the power, 2016-05-17 18:13:32]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-17 18:17:41]

Description was changed from

==========
Random fixes for mash bridge classes

. MusLayoutManagerAdapter wasn't installed observer correctly
. Updated set of activation parents.
. WmGlobalMus::GetTopLevelAncestor consults set of activation parents.
. WmGlobalMus deals with WindowTreeConnection being deleted while alive.
. WmLookupMus installs itself as the WmLookup.
. WmRootWindowController lookup key is no longer owned (it'll be owned by
  RootController).
. Added WmRootWindowController::NotifyFullscreenStatechanged().
. Wired up WmRootWindowControllerMus observers.
. Changed WmWindowMus shell ids to be stored in WmWindowMus rather than use
  mus::Window's local_id. This way it doesn't conflict with other code that
  wants to use the local id (we store the container in the local id).
. Route WmWindowMus::SetBounds() to the layutmanager appropriately.
. WmWindowMus::GetboundsInScreen() was including the bounds of 'this' resulting
  in double counting the initial offset.
. WmWindowMus::Activate() moves to front.
. WmWindowMus::OnWindowReordered() was notifying the wrong observers.
. WmWindowMus::OnWindowSharedPropertyChanged now calls to WindowState

R=jamescook@chromium.org
BUG=603369
TEST=none
==========

to

==========
Random fixes for mash bridge classes

. MusLayoutManagerAdapter wasn't installed observer correctly
. Updated set of activation parents.
. WmGlobalMus::GetTopLevelAncestor consults set of activation parents.
. WmGlobalMus deals with WindowTreeConnection being deleted while alive.
. WmLookupMus installs itself as the WmLookup.
. WmRootWindowController lookup key is no longer owned (it'll be owned by
  RootController).
. Added WmRootWindowController::NotifyFullscreenStatechanged().
. Wired up WmRootWindowControllerMus observers.
. Changed WmWindowMus shell ids to be stored in WmWindowMus rather than use
  mus::Window's local_id. This way it doesn't conflict with other code that
  wants to use the local id (we store the container in the local id).
. Route WmWindowMus::SetBounds() to the layutmanager appropriately.
. WmWindowMus::GetboundsInScreen() was including the bounds of 'this' resulting
  in double counting the initial offset.
. WmWindowMus::Activate() moves to front.
. WmWindowMus::OnWindowReordered() was notifying the wrong observers.
. WmWindowMus::OnWindowSharedPropertyChanged now calls to WindowState

R=jamescook@chromium.org
BUG=603369
TEST=none

Committed: https://crrev.com/990b86fe4ea3085e6df6dd58316a2d2c34aad241
Cr-Commit-Position: refs/heads/master@{#394170}
==========

[commit-bot: I haz the power, 2016-05-17 18:17:42]

Patchset 3 (id:??) landed as
https://crrev.com/990b86fe4ea3085e6df6dd58316a2d2c34aad241
Cr-Commit-Position: refs/heads/master@{#394170}