Expose worklet on Window through attribute renderWorklet.

Expose worklet on Window through attribute renderWorklet.

https://drafts.css-houdini.org/worklets/#dom-window-renderworklet

This may change in the future if HoudiniTF decides to split into separate
Worklets for each stage.

Additionally add kinuko as OWNERS to worklet directory.

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

[ikilpatrick, 2016-01-14 19:31:56]

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

[ikilpatrick, 2016-01-14 19:31:57]

https://codereview.chromium.org/1590583002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/worklet/WorkletGlobalScope.cpp (right):

https://codereview.chromium.org/1590583002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/worklet/WorkletGlobalScope.cpp:15:
workletGlobalScope->script()->initializeContextIfNeeded();
This was added here as within:
V8DOMWrapper::associateObjectWithWrapper (inside initializeContextIfNeeded())

there is an assert to check that the GlobalScope is adopted by something.

[kinuko, 2016-01-15 04:19:34]

lgtm

https://codereview.chromium.org/1590583002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/worklet/DOMWindowWorklet.cpp (right):

https://codereview.chromium.org/1590583002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/worklet/DOMWindowWorklet.cpp:39: return
DOMWindowWorklet::from(toLocalDOMWindow(window)).renderWorklet(executionContext);
nit: DOMWindowWorklet:: not needed

[ikilpatrick, 2016-01-15 16:22:44]

https://codereview.chromium.org/1590583002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/worklet/DOMWindowWorklet.cpp (right):

https://codereview.chromium.org/1590583002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/worklet/DOMWindowWorklet.cpp:39: return
DOMWindowWorklet::from(toLocalDOMWindow(window)).renderWorklet(executionContext);
On 2016/01/15 04:19:34, kinuko wrote:
> nit: DOMWindowWorklet:: not needed

Done.

[ikilpatrick, 2016-01-15 16:57:57]

The CQ bit was checked by ikilpatrick@chromium.org

[ikilpatrick, 2016-01-15 16:57:57]

The patchset sent to the CQ was uploaded after l-g-t-m from kinuko@chromium.org
Link to the patchset: https://codereview.chromium.org/1590583002/#ps40001
(title: "addressed comments.")

[commit-bot: I haz the power, 2016-01-15 16:58:36]

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

[commit-bot: I haz the power, 2016-01-15 17:25:25]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-01-15 17:26:20]

Description was changed from

==========
Expose worklet on Window through attribute renderWorklet.

https://drafts.css-houdini.org/worklets/#dom-window-renderworklet

This may change in the future if HoudiniTF decides to split into separate
Worklets for each stage.

Additionally add kinuko as OWNERS to worklet directory.

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

to

==========
Expose worklet on Window through attribute renderWorklet.

https://drafts.css-houdini.org/worklets/#dom-window-renderworklet

This may change in the future if HoudiniTF decides to split into separate
Worklets for each stage.

Additionally add kinuko as OWNERS to worklet directory.

BUG=567358

Committed: https://crrev.com/709d67b3876a36d7aa2a0a3cd744d6dd5ffb9e59
Cr-Commit-Position: refs/heads/master@{#369778}
==========

[commit-bot: I haz the power, 2016-01-15 17:26:21]

Patchset 3 (id:??) landed as
https://crrev.com/709d67b3876a36d7aa2a0a3cd744d6dd5ffb9e59
Cr-Commit-Position: refs/heads/master@{#369778}