Revert of Revert of Reland factor out window proxy management portions of ScriptController.

Reland factor out window proxy management portions of ScriptController.

Right now, ScriptController is in charge of initializing window proxies,
clearing them for navigation/close, and maintaining a mapping of worlds
to window proxies. However, ScriptController is LocalFrame-specific, and
RemoteFrame also needs to expose a DOMWindow for scripting purposes. To
share code between the two, the window proxy management has been moved
out of ScriptController, leaving only LocalFrame-specific logic such as
wrapping NPObject for plugins or script evaluation.

Eventually, RemoteFrame will also embed a WindowProxyManager, and
WindowProxyManager will be extended to allow it be passed between frames
when transitioning between local and remote frames.

For integration with devtools, WindowProxyManager still contains some
LocalFrame-specific code.

BUG=425623
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=188574

[dcheng, 2015-01-16 03:29:13]

Created Revert of Revert of Reland factor out window proxy management portions
of ScriptController.

[haraken, 2015-01-16 03:41:20]

LGTM

[dcheng, 2015-01-16 03:42:01]

The CQ bit was checked by dcheng@chromium.org

[dcheng, 2015-01-16 03:42:15]

The CQ bit was unchecked by dcheng@chromium.org

[dcheng, 2015-01-16 04:32:29]

PTAL. I've rebased to ToT in Blink and deflaked the layout test. Since this
isn't really a revert of a revert anymore, I've updated the CL description to
the original patch's.

[haraken, 2015-01-16 04:55:01]

LGTM

https://codereview.chromium.org/854453003/diff/230001/Source/bindings/core/v8...
File Source/bindings/core/v8/WindowProxyManager.h (right):

https://codereview.chromium.org/854453003/diff/230001/Source/bindings/core/v8...
Source/bindings/core/v8/WindowProxyManager.h:1: // Copyright 2014 The Chromium
Authors. All rights reserved.

2015

https://codereview.chromium.org/854453003/diff/230001/Source/bindings/core/v8...
Source/bindings/core/v8/WindowProxyManager.h:21: class WindowProxyManager :
public NoBaseWillBeGarbageCollectedFinalized<WindowProxyManager> {

Add final.

[dcheng, 2015-01-17 00:31:13]

The CQ bit was checked by dcheng@chromium.org

[commit-bot: I haz the power, 2015-01-17 00:31:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/854453003/250001

[dcheng, 2015-01-17 00:33:05]

https://codereview.chromium.org/854453003/diff/230001/Source/bindings/core/v8...
File Source/bindings/core/v8/WindowProxyManager.h (right):

https://codereview.chromium.org/854453003/diff/230001/Source/bindings/core/v8...
Source/bindings/core/v8/WindowProxyManager.h:1: // Copyright 2014 The Chromium
Authors. All rights reserved.
On 2015/01/16 at 04:55:01, haraken wrote:
> 2015

Done.

https://codereview.chromium.org/854453003/diff/230001/Source/bindings/core/v8...
Source/bindings/core/v8/WindowProxyManager.h:21: class WindowProxyManager :
public NoBaseWillBeGarbageCollectedFinalized<WindowProxyManager> {
On 2015/01/16 at 04:55:01, haraken wrote:
> Add final.

Done.

[commit-bot: I haz the power, 2015-01-17 02:13:21]

Committed patchset #3 (id:250001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=188574

[dcheng, 2015-01-20 18:31:01]

A revert of this CL (patchset #3 id:250001) has been created in
https://codereview.chromium.org/834203003/ by dcheng@chromium.org.

The reason for reverting is: Needed data gathered, so reverting to reduce number
of crash dumps..