Refactors StackingController::GetDefaultParent to ash/wm/common

Refactors StackingController::GetDefaultParent to ash/wm/common

This way I can use it in mus too.

I'm not clear that the windowmanager should handle system modals or
lock screen stuff. But for the near term we likely need it for sys_ui.

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

Committed: https://crrev.com/cb51abf81b0437f5768d1b1de48f41d6e8bc1fab
Cr-Commit-Position: refs/heads/master@{#391847}

[James Cook, 2016-05-05 02:11:40]

LGTM

https://codereview.chromium.org/1954523002/diff/20001/ash/shell_window_ids.h
File ash/shell_window_ids.h (right):

https://codereview.chromium.org/1954523002/diff/20001/ash/shell_window_ids.h#...
ash/shell_window_ids.h:169: "lock-screen-system-modal between lock-screen
status");
nit: "lock-screen and status"

https://codereview.chromium.org/1954523002/diff/20001/ash/wm/common/container...
File ash/wm/common/container_finder.cc (right):

https://codereview.chromium.org/1954523002/diff/20001/ash/wm/common/container...
ash/wm/common/container_finder.cc:20: if (bounds == gfx::Rect())
nit: #include "ui/gfx/geometry/rect.h"

[sky, 2016-05-05 15:33:17]

https://codereview.chromium.org/1954523002/diff/20001/ash/shell_window_ids.h
File ash/shell_window_ids.h (right):

https://codereview.chromium.org/1954523002/diff/20001/ash/shell_window_ids.h#...
ash/shell_window_ids.h:169: "lock-screen-system-modal between lock-screen
status");
On 2016/05/05 02:11:39, James Cook wrote:
> nit: "lock-screen and status"

Done.

https://codereview.chromium.org/1954523002/diff/20001/ash/wm/common/container...
File ash/wm/common/container_finder.cc (right):

https://codereview.chromium.org/1954523002/diff/20001/ash/wm/common/container...
ash/wm/common/container_finder.cc:20: if (bounds == gfx::Rect())
On 2016/05/05 02:11:39, James Cook wrote:
> nit: #include "ui/gfx/geometry/rect.h"

Done.

[sky, 2016-05-05 15:39:26]

The CQ bit was checked by sky@chromium.org

[sky, 2016-05-05 15:39:26]

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/1954523002/#ps60001
(title: "merge 2 trunk")

[commit-bot: I haz the power, 2016-05-05 15:39:45]

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

[commit-bot: I haz the power, 2016-05-05 15:54:52]

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

[commit-bot: I haz the power, 2016-05-05 15:54:53]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sky, 2016-05-05 17:02:26]

The CQ bit was checked by sky@chromium.org

[sky, 2016-05-05 17:02:26]

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/1954523002/#ps80001
(title: "update gyp")

[commit-bot: I haz the power, 2016-05-05 17:02:48]

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

[commit-bot: I haz the power, 2016-05-05 18:25:06]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-05-05 18:26:53]

Description was changed from

==========
Refactors StackingController::GetDefaultParent to ash/wm/common

This way I can use it in mus too.

I'm not clear that the windowmanager should handle system modals or
lock screen stuff. But for the near term we likely need it for sys_ui.

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

to

==========
Refactors StackingController::GetDefaultParent to ash/wm/common

This way I can use it in mus too.

I'm not clear that the windowmanager should handle system modals or
lock screen stuff. But for the near term we likely need it for sys_ui.

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

Committed: https://crrev.com/cb51abf81b0437f5768d1b1de48f41d6e8bc1fab
Cr-Commit-Position: refs/heads/master@{#391847}
==========

[commit-bot: I haz the power, 2016-05-05 18:26:54]

Patchset 5 (id:??) landed as
https://crrev.com/cb51abf81b0437f5768d1b1de48f41d6e8bc1fab
Cr-Commit-Position: refs/heads/master@{#391847}