Revert of Moves definitions of RootWindowController into root_window_controller.cc

Revert of Moves definitions of RootWindowController into root_window_controller.cc (patchset #2 id:20001 of https://codereview.chromium.org/2628973002/ )

Reason for revert:
Attempting to resolve error in 
https://uberchromegw.corp.google.com/i/chromium.chromiumos/builders/ChromiumOS%20amd64-generic%20Compile/builds/28853

>On 2017/01/11 22:44:48, James Cook wrote:
>> FYI I have a CL that deletes this method. We'll see who lands first.
> Done.

It looks like jamescook@ committed a bit more than four minutes earlier.
-- sheriff.

Original issue's description:
> Moves definitions of RootWindowController into root_window_controller.cc
> 
> As part of this I made sure the definitions and declarations matched
> up. This resulted in reordering a couple of things. In addition the
> only other non-move I did was moving what was CreateLayoutManagers()
> into InitLayoutManagers() and CloseChildWindowsImpl() into
> CloseChildWindows().
> 
> BUG=671246
> TEST=none
> R=jamescook@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2628973002
> Cr-Commit-Position: refs/heads/master@{#443109}
> Committed: https://chromium.googlesource.com/chromium/src/+/d20a89e7b56ad53a58f19edf4775528d066577d5

TBR=jamescook@chromium.org,sky@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=671246
Review-Url: https://codereview.chromium.org/2623393002
Cr-Commit-Position: refs/heads/master@{#443117}
Committed: https://chromium.googlesource.com/chromium/src/+/07724bd484c9d39978b231f3e106689067a1e832

[dschuyler, 2017-01-12 02:24:10]

The CQ bit was checked by dschuyler@chromium.org

[dschuyler, 2017-01-12 02:24:11]

Created Revert of Moves definitions of RootWindowController into
root_window_controller.cc

[commit-bot: I haz the power, 2017-01-12 02:24:35]

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, 2017-01-12 02:25:14]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484187850870520, "parent_rev":
"52f0c6086c1a89e3a294855e4ced218cf74991bc", "commit_rev":
"07724bd484c9d39978b231f3e106689067a1e832"}

[commit-bot: I haz the power, 2017-01-12 02:25:46]

Description was changed from

==========
Revert of Moves definitions of RootWindowController into
root_window_controller.cc (patchset #2 id:20001 of
https://codereview.chromium.org/2628973002/ )

Reason for revert:
Attempting to resolve error in 
https://uberchromegw.corp.google.com/i/chromium.chromiumos/builders/ChromiumO...

>On 2017/01/11 22:44:48, James Cook wrote:
>> FYI I have a CL that deletes this method. We'll see who lands first.
> Done.

It looks like jamescook@ committed a bit more than four minutes earlier.
-- sheriff.

Original issue's description:
> Moves definitions of RootWindowController into root_window_controller.cc
> 
> As part of this I made sure the definitions and declarations matched
> up. This resulted in reordering a couple of things. In addition the
> only other non-move I did was moving what was CreateLayoutManagers()
> into InitLayoutManagers() and CloseChildWindowsImpl() into
> CloseChildWindows().
> 
> BUG=671246
> TEST=none
> R=jamescook@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2628973002
> Cr-Commit-Position: refs/heads/master@{#443109}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d20a89e7b56ad53a58f19edf4775...

TBR=jamescook@chromium.org,sky@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=671246
==========

to

==========
Revert of Moves definitions of RootWindowController into
root_window_controller.cc (patchset #2 id:20001 of
https://codereview.chromium.org/2628973002/ )

Reason for revert:
Attempting to resolve error in 
https://uberchromegw.corp.google.com/i/chromium.chromiumos/builders/ChromiumO...

>On 2017/01/11 22:44:48, James Cook wrote:
>> FYI I have a CL that deletes this method. We'll see who lands first.
> Done.

It looks like jamescook@ committed a bit more than four minutes earlier.
-- sheriff.

Original issue's description:
> Moves definitions of RootWindowController into root_window_controller.cc
> 
> As part of this I made sure the definitions and declarations matched
> up. This resulted in reordering a couple of things. In addition the
> only other non-move I did was moving what was CreateLayoutManagers()
> into InitLayoutManagers() and CloseChildWindowsImpl() into
> CloseChildWindows().
> 
> BUG=671246
> TEST=none
> R=jamescook@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2628973002
> Cr-Commit-Position: refs/heads/master@{#443109}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d20a89e7b56ad53a58f19edf4775...

TBR=jamescook@chromium.org,sky@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=671246

Review-Url: https://codereview.chromium.org/2623393002
Cr-Commit-Position: refs/heads/master@{#443117}
Committed:
https://chromium.googlesource.com/chromium/src/+/07724bd484c9d39978b231f3e106...
==========

[commit-bot: I haz the power, 2017-01-12 02:25:47]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/07724bd484c9d39978b231f3e106...