Worklets - Change inheritance heirarchy of WorkletGlobalScope

Previously the inheritance for WorkletGlobalScopes was:

core/workers/WorkerOrWorkletGlobalScope
  -> core/workers/MainThreadWorkletGlobalScope
  -> modules/worklet/WorkletGlobalScope

With ThreadedWorkletGlobalScope, not all global scopes will exist on the main thread.

MainThreadWorkletGlobalScope exists in core/workers as the MainThreadDebugger & friends need to access it's LocalFrame from core/

The new inheritance is:
core/workers/WorkerOrWorkletGlobalScope
  -> core/workers/WorkletGlobalScope
  -> core/workers/MainThreadWorkletGlobalScope

This moves WorkletGlobalScope into core and a few other classes. ThreadedWorkletGlobalScope will exist in modules and inherit from WorkletGlobalScope.

Implementation Doc:
https://docs.google.com/document/d/14hEO_BqQRBne3tBD2xghEEo02nz2csLqY4SS6x6rmjA/edit

BUG=567358
Committed: https://crrev.com/b3a781795afd30e7c931ecf0a2276f76b411d58b
Cr-Commit-Position: refs/heads/master@{#397722}

[ikilpatrick, 2016-06-01 21:51:46]

Description was changed from

==========
Introduce WorkletGlobalScopeProxy interface.

Fix inheritance of mainthreadworkletglobalscope

BUG=
==========

to

==========
Previously the inheritance for WorkletGlobalScopes was:

core/workers/WorkerOrWorkletGlobalScope 
  -> core/workers/MainThreadWorkletGlobalScope
  -> modules/worklet/WorkletGlobalScope

With ThreadedWorkletGlobalScope, not all global scopes will exist on the main
thread.

MainThreadWorkletGlobalScope exists in core/workers as the MainThreadDebugger &
friends need to access it's LocalFrame from core/

The new inheritance is:
core/workers/WorkerOrWorkletGlobalScope 
  -> core/workers/WorkletGlobalScope
  -> core/workers/MainThreadWorkletGlobalScope

This moves WorkletGlobalScope into core and a few other classes.
ThreadedWorkletGlobalScope will exist in modules and inherit from
WorkletGlobalScope.

BUG=567358
==========

[ikilpatrick, 2016-06-01 21:56:13]

ikilpatrick@chromium.org changed reviewers:
+ kinuko@chromium.org, yhirano@chromium.org

[ikilpatrick, 2016-06-02 04:22:07]

Description was changed from

==========
Previously the inheritance for WorkletGlobalScopes was:

core/workers/WorkerOrWorkletGlobalScope 
  -> core/workers/MainThreadWorkletGlobalScope
  -> modules/worklet/WorkletGlobalScope

With ThreadedWorkletGlobalScope, not all global scopes will exist on the main
thread.

MainThreadWorkletGlobalScope exists in core/workers as the MainThreadDebugger &
friends need to access it's LocalFrame from core/

The new inheritance is:
core/workers/WorkerOrWorkletGlobalScope 
  -> core/workers/WorkletGlobalScope
  -> core/workers/MainThreadWorkletGlobalScope

This moves WorkletGlobalScope into core and a few other classes.
ThreadedWorkletGlobalScope will exist in modules and inherit from
WorkletGlobalScope.

BUG=567358
==========

to

==========
Previously the inheritance for WorkletGlobalScopes was:

core/workers/WorkerOrWorkletGlobalScope 
  -> core/workers/MainThreadWorkletGlobalScope
  -> modules/worklet/WorkletGlobalScope

With ThreadedWorkletGlobalScope, not all global scopes will exist on the main
thread.

MainThreadWorkletGlobalScope exists in core/workers as the MainThreadDebugger &
friends need to access it's LocalFrame from core/

The new inheritance is:
core/workers/WorkerOrWorkletGlobalScope 
  -> core/workers/WorkletGlobalScope
  -> core/workers/MainThreadWorkletGlobalScope

This moves WorkletGlobalScope into core and a few other classes.
ThreadedWorkletGlobalScope will exist in modules and inherit from
WorkletGlobalScope.

Implementation Doc:
https://docs.google.com/document/d/14hEO_BqQRBne3tBD2xghEEo02nz2csLqY4SS6x6rm...

BUG=567358
==========

[ikilpatrick, 2016-06-02 04:24:19]

Moving WorkletGlobalScope to core prevents adding a worklet only method to
WorkerOrWorkletGlobalScope (for accessing frame() inside the debugger).

[kinuko, 2016-06-02 07:42:03]

lgtm

(win bot seems to be having compile error)

[haraken, 2016-06-02 07:50:25]

LGTM

[yhirano, 2016-06-02 08:14:42]

https://codereview.chromium.org/2029163002/diff/100001/third_party/WebKit/Sou...
File third_party/WebKit/Source/bindings/core/v8/V8Binding.cpp (right):

https://codereview.chromium.org/2029163002/diff/100001/third_party/WebKit/Sou...
third_party/WebKit/Source/bindings/core/v8/V8Binding.cpp:720: return
V8WorkerGlobalScope::toImpl(workletWrapper)->getExecutionContext();
Shouldn't this be V8WorkletGlobalScope?

https://codereview.chromium.org/2029163002/diff/100001/third_party/WebKit/Sou...
File third_party/WebKit/Source/modules/csspaint/PaintWorklet.h (right):

https://codereview.chromium.org/2029163002/diff/100001/third_party/WebKit/Sou...
third_party/WebKit/Source/modules/csspaint/PaintWorklet.h:24:
WorkletGlobalScopeProxy* workletGlobalScopeProxy() const final;
Why don't you return PaingWorkletGlobalScope* and delete
globalScopeForTesting()?

[ikilpatrick, 2016-06-02 18:26:00]

Thanks Yutaka! win compile now works as well.

https://codereview.chromium.org/2029163002/diff/100001/third_party/WebKit/Sou...
File third_party/WebKit/Source/bindings/core/v8/V8Binding.cpp (right):

https://codereview.chromium.org/2029163002/diff/100001/third_party/WebKit/Sou...
third_party/WebKit/Source/bindings/core/v8/V8Binding.cpp:720: return
V8WorkerGlobalScope::toImpl(workletWrapper)->getExecutionContext();
On 2016/06/02 08:14:42, yhirano wrote:
> Shouldn't this be V8WorkletGlobalScope?

Yup, done.

https://codereview.chromium.org/2029163002/diff/100001/third_party/WebKit/Sou...
File third_party/WebKit/Source/modules/csspaint/PaintWorklet.h (right):

https://codereview.chromium.org/2029163002/diff/100001/third_party/WebKit/Sou...
third_party/WebKit/Source/modules/csspaint/PaintWorklet.h:24:
WorkletGlobalScopeProxy* workletGlobalScopeProxy() const final;
On 2016/06/02 08:14:42, yhirano wrote:
> Why don't you return PaingWorkletGlobalScope* and delete
> globalScopeForTesting()?

Done.

[yhirano, 2016-06-03 05:39:07]

lgtm

[ikilpatrick, 2016-06-03 15:38:38]

The CQ bit was checked by ikilpatrick@chromium.org

[ikilpatrick, 2016-06-03 15:38:39]

The patchset sent to the CQ was uploaded after l-g-t-m from kinuko@chromium.org,
haraken@chromium.org, yhirano@chromium.org
Link to the patchset: https://codereview.chromium.org/2029163002/#ps160001
(title: "rebase.")

[commit-bot: I haz the power, 2016-06-03 15:39:10]

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

[commit-bot: I haz the power, 2016-06-03 16:21:37]

Description was changed from

==========
Previously the inheritance for WorkletGlobalScopes was:

core/workers/WorkerOrWorkletGlobalScope 
  -> core/workers/MainThreadWorkletGlobalScope
  -> modules/worklet/WorkletGlobalScope

With ThreadedWorkletGlobalScope, not all global scopes will exist on the main
thread.

MainThreadWorkletGlobalScope exists in core/workers as the MainThreadDebugger &
friends need to access it's LocalFrame from core/

The new inheritance is:
core/workers/WorkerOrWorkletGlobalScope 
  -> core/workers/WorkletGlobalScope
  -> core/workers/MainThreadWorkletGlobalScope

This moves WorkletGlobalScope into core and a few other classes.
ThreadedWorkletGlobalScope will exist in modules and inherit from
WorkletGlobalScope.

Implementation Doc:
https://docs.google.com/document/d/14hEO_BqQRBne3tBD2xghEEo02nz2csLqY4SS6x6rm...

BUG=567358
==========

to

==========
Previously the inheritance for WorkletGlobalScopes was:

core/workers/WorkerOrWorkletGlobalScope 
  -> core/workers/MainThreadWorkletGlobalScope
  -> modules/worklet/WorkletGlobalScope

With ThreadedWorkletGlobalScope, not all global scopes will exist on the main
thread.

MainThreadWorkletGlobalScope exists in core/workers as the MainThreadDebugger &
friends need to access it's LocalFrame from core/

The new inheritance is:
core/workers/WorkerOrWorkletGlobalScope 
  -> core/workers/WorkletGlobalScope
  -> core/workers/MainThreadWorkletGlobalScope

This moves WorkletGlobalScope into core and a few other classes.
ThreadedWorkletGlobalScope will exist in modules and inherit from
WorkletGlobalScope.

Implementation Doc:
https://docs.google.com/document/d/14hEO_BqQRBne3tBD2xghEEo02nz2csLqY4SS6x6rm...

BUG=567358
==========

[commit-bot: I haz the power, 2016-06-03 16:21:38]

Committed patchset #9 (id:160001)

[commit-bot: I haz the power, 2016-06-03 16:24:15]

Description was changed from

==========
Previously the inheritance for WorkletGlobalScopes was:

core/workers/WorkerOrWorkletGlobalScope 
  -> core/workers/MainThreadWorkletGlobalScope
  -> modules/worklet/WorkletGlobalScope

With ThreadedWorkletGlobalScope, not all global scopes will exist on the main
thread.

MainThreadWorkletGlobalScope exists in core/workers as the MainThreadDebugger &
friends need to access it's LocalFrame from core/

The new inheritance is:
core/workers/WorkerOrWorkletGlobalScope 
  -> core/workers/WorkletGlobalScope
  -> core/workers/MainThreadWorkletGlobalScope

This moves WorkletGlobalScope into core and a few other classes.
ThreadedWorkletGlobalScope will exist in modules and inherit from
WorkletGlobalScope.

Implementation Doc:
https://docs.google.com/document/d/14hEO_BqQRBne3tBD2xghEEo02nz2csLqY4SS6x6rm...

BUG=567358
==========

to

==========
Previously the inheritance for WorkletGlobalScopes was:

core/workers/WorkerOrWorkletGlobalScope
  -> core/workers/MainThreadWorkletGlobalScope
  -> modules/worklet/WorkletGlobalScope

With ThreadedWorkletGlobalScope, not all global scopes will exist on the main
thread.

MainThreadWorkletGlobalScope exists in core/workers as the MainThreadDebugger &
friends need to access it's LocalFrame from core/

The new inheritance is:
core/workers/WorkerOrWorkletGlobalScope
  -> core/workers/WorkletGlobalScope
  -> core/workers/MainThreadWorkletGlobalScope

This moves WorkletGlobalScope into core and a few other classes.
ThreadedWorkletGlobalScope will exist in modules and inherit from
WorkletGlobalScope.

Implementation Doc:
https://docs.google.com/document/d/14hEO_BqQRBne3tBD2xghEEo02nz2csLqY4SS6x6rm...

BUG=567358

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

[commit-bot: I haz the power, 2016-06-03 16:24:16]

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