Move common features to new WindowOrWorkerGlobalScope mix-in

Move common features to new WindowOrWorkerGlobalScope mix-in

Move features common to the Window and WorkerGlobalScope interfaces to WindowOrWorkerGlobalScope mix-in.
https://html.spec.whatwg.org/multipage/webappapis.html#windoworworkerglobalscope-mixin

BUG=701457

[zakerinasab, 2017-04-04 15:48:47]

Patchset #2 (id:20001) has been deleted

[zakerinasab, 2017-04-04 16:27:02]

Description was changed from

==========
Move common features to new WindowOrWorkerGlobalScope mix-in

Move features common to the Window and WorkerGlobalScope interfaces to
WindowOrWorkerGlobalScope mix-in.
https://html.spec.whatwg.org/multipage/webappapis.html#windoworworkerglobalsc...

BUG=701457
==========

to

==========
Move common features to new WindowOrWorkerGlobalScope mix-in

Move features common to the Window and WorkerGlobalScope interfaces to
WindowOrWorkerGlobalScope mix-in.
https://html.spec.whatwg.org/multipage/webappapis.html#windoworworkerglobalsc...

BUG=701457
==========

[zakerinasab, 2017-04-04 16:27:02]

zakerinasab@chromium.org changed reviewers:
+ junov@chromium.org

[zakerinasab, 2017-04-04 19:09:38]

For now, it seems that we lack the needed support from the V8 engine. In
core_idl_files.gni, we have two sets: 1. core_idl_with_modules_dependency_files,
2. core_dependency_idl_files (which are at the right side of implements). The
issue is that the new WindowOrWorkerGlobalScope interface belongs to both, as it
has partial interfaces in modules/ (type 1) and is the right side of implements
for Window and WorkerGlobalScope (type 2).

[jsbell, 2017-04-04 19:13:34]

jsbell@chromium.org changed reviewers:
+ jsbell@chromium.org

[jsbell, 2017-04-04 19:13:35]

drive-by comment: Thanks for pursuing this! Fingers crossed you can get this
working.

https://codereview.chromium.org/2798433003/diff/80001/third_party/WebKit/Sour...
File
third_party/WebKit/Source/modules/indexeddb/WindowOrWorkerGlobalScopeIndexedDatabase.idl
(right):

https://codereview.chromium.org/2798433003/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/indexeddb/WindowOrWorkerGlobalScopeIndexedDatabase.idl:4:

Can you add a link to https://w3c.github.io/IndexedDB/#factory-interface while
you're here?

[zakerinasab, 2017-04-04 19:18:21]

On 2017/04/04 19:13:35, jsbell wrote:
> drive-by comment: Thanks for pursuing this! Fingers crossed you can get this
> working.
> 
>
https://codereview.chromium.org/2798433003/diff/80001/third_party/WebKit/Sour...
> File
>
third_party/WebKit/Source/modules/indexeddb/WindowOrWorkerGlobalScopeIndexedDatabase.idl
> (right):
> 
>
https://codereview.chromium.org/2798433003/diff/80001/third_party/WebKit/Sour...
>
third_party/WebKit/Source/modules/indexeddb/WindowOrWorkerGlobalScopeIndexedDatabase.idl:4:
> 
> Can you add a link to https://w3c.github.io/IndexedDB/#factory-interface while
> you're here?

My local branch is pretty dirty now, but I'll do it in the next CL if hopefully
there was one :)

[foolip, 2017-04-06 14:11:02]

foolip@chromium.org changed reviewers:
+ bashi@chromium.org, foolip@chromium.org

[foolip, 2017-04-06 14:11:03]

bashi@, can you advise on how to deal with issues in comment #4?

[bashi, 2017-04-07 00:04:53]

On 2017/04/06 14:11:03, foolip_UTC7 wrote:
> bashi@, can you advise on how to deal with issues in comment #4?

zakerinasab@ reached out to us already :)