Remove OpenWindow from the WindowManager interface.

Remove OpenWindow from WindowManager API. This time much
simpler than before. Instead the test just waits for the
embed to happen (since it's all URL fakery anyway) and
returns the id of the root view.

Also adds a simple window manager shell implementation
for tests, needed to keep the window manager tests from
crashing.

R=sky@chromium.org
BUG=none

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=289467

[sky, 2014-08-13 14:41:53]

Nice indeed, LGTM

https://codereview.chromium.org/466273002/diff/1/mojo/services/window_manager...
File mojo/services/window_manager/main.cc (right):

https://codereview.chromium.org/466273002/diff/1/mojo/services/window_manager...
mojo/services/window_manager/main.cc:25: view_manager_(NULL) {}
root_(NULL) too.

[Ben Goodger (Google), 2014-08-13 21:21:21]

Plz take another look. Had to make some additional changes to fix some uafs &
shutdown races in the tests.

[sky, 2014-08-13 21:35:47]

LGTM

https://codereview.chromium.org/466273002/diff/60001/mojo/services/window_man...
File mojo/services/window_manager/main.cc (right):

https://codereview.chromium.org/466273002/diff/60001/mojo/services/window_man...
mojo/services/window_manager/main.cc:25: view_manager_(NULL) {}
Add root_(NULL)

[Ben Goodger (Google), 2014-08-13 22:18:43]

The CQ bit was checked by ben@chromium.org

[commit-bot: I haz the power, 2014-08-13 22:20:47]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ben@chromium.org/466273002/80001

[Ben Goodger (Google), 2014-08-14 05:19:16]

Committed patchset #5 manually as 289467 (presubmit successful).