Fixes use after free caused by delete in RootWindowController (2)

Fixes use after free caused by delete in RootWindowController (2)

RootWindowController::CloseChildWindows() was explicitly deleting
windows. It should only do that for windows that are owned by the
parent, otherwise the window should be removed.

BUG=297028
TEST=covered by test now.
R=oshima@chromium.org
TBR=oshima@chromium.org

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

[sky, 2013-10-02 19:15:59]

Shutdown

[sky, 2013-10-02 19:16:53]

Patchset 1 is the one you reviewed, 3 the fix. I needed to invoke Shutdown() to
NULL out a member in the test. TBRing since it's a trivial change.

[sky, 2013-10-02 19:25:27]

Committed patchset #3 manually as r226524 (presubmit successful).

[oshima, 2013-10-02 20:58:34]

lgtm