Introduce DocumentOrShadowRoot Mixin

Introduce DocumentOrShadowRoot Mixin

The spec is:
- https://dom.spec.whatwg.org/#documentorshadowroot
- http://w3c.github.io/webcomponents/spec/shadow/#extensions-to-the-documentorshadowroot-mixin

BUG=595600
Committed: https://crrev.com/6e97f9b878e58270e4a8d719800d16650c95459e
Cr-Commit-Position: refs/heads/master@{#382515}

[hayato, 2016-03-18 09:02:57]

The CQ bit was checked by hayato@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-03-18 09:03:04]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1815643002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1815643002/1

[hayato, 2016-03-18 09:04:47]

Description was changed from

==========
Introduce DocumenOrShadowRootMixin

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

to

==========
Introduce DocumenOrShadowRoot Mixin

The spec is:
- https://dom.spec.whatwg.org/#documentorshadowroot
-
http://w3c.github.io/webcomponents/spec/shadow/#extensions-to-the-documentors...

BUG=595600
==========

[commit-bot: I haz the power, 2016-03-18 09:06:40]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-18 09:06:40]

Dry run: Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  cast_shell_linux on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[hayato, 2016-03-18 09:25:04]

The CQ bit was checked by hayato@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-03-18 09:25:16]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1815643002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1815643002/1

[commit-bot: I haz the power, 2016-03-18 09:28:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-18 09:29:00]

Dry run: Try jobs failed on following builders:
  android_chromium_gn_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  linux_chromium_clobber_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)

[hayato, 2016-03-18 09:47:45]

The CQ bit was checked by hayato@chromium.org to run a CQ dry run

[hayato, 2016-03-18 09:47:46]

fix typo

[commit-bot: I haz the power, 2016-03-18 09:47:55]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1815643002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1815643002/20001

[commit-bot: I haz the power, 2016-03-18 10:59:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-18 10:59:45]

Dry run: This issue passed the CQ dry run.

[hayato, 2016-03-22 03:07:13]

hayato@chromium.org changed reviewers:
+ kochi@chromium.org, tkent@chromium.org

[hayato, 2016-03-22 03:07:13]

PTAL

[kochi, 2016-03-22 03:43:56]

Could you also fix typo in the title and the description?
s/DocumenOrShadowRoot/DocumentOrShadowRoot/

https://codereview.chromium.org/1815643002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/dom/DocumentOrShadowRoot.h (right):

https://codereview.chromium.org/1815643002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/dom/DocumentOrShadowRoot.h:17: return
document.activeElement();
Can activeElement() and styleSheets() be a pure virtual function for
TreeScope(),
and override in Document and ShadowRoot, then call it as
treeScope.activeElement()
like getSelection() below?

[hayato, 2016-03-22 03:47:59]

Description was changed from

==========
Introduce DocumenOrShadowRoot Mixin

The spec is:
- https://dom.spec.whatwg.org/#documentorshadowroot
-
http://w3c.github.io/webcomponents/spec/shadow/#extensions-to-the-documentors...

BUG=595600
==========

to

==========
Introduce DocumentOrShadowRoot Mixin

The spec is:
- https://dom.spec.whatwg.org/#documentorshadowroot
-
http://w3c.github.io/webcomponents/spec/shadow/#extensions-to-the-documentors...

BUG=595600
==========

[hayato, 2016-03-22 03:50:08]

Fixed typo in the subject.

https://codereview.chromium.org/1815643002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/dom/DocumentOrShadowRoot.h (right):

https://codereview.chromium.org/1815643002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/dom/DocumentOrShadowRoot.h:17: return
document.activeElement();
On 2016/03/22 at 03:43:56, kochi wrote:
> Can activeElement() and styleSheets() be a pure virtual function for
TreeScope(),
> and override in Document and ShadowRoot, then call it as
treeScope.activeElement()
> like getSelection() below?

I did not do it intentionally in this CL. That should be done in another CL.

[kochi, 2016-03-22 03:53:43]

On 2016/03/22 03:50:08, hayato wrote:
> Fixed typo in the subject.
> 
>
https://codereview.chromium.org/1815643002/diff/20001/third_party/WebKit/Sour...
> File third_party/WebKit/Source/core/dom/DocumentOrShadowRoot.h (right):
> 
>
https://codereview.chromium.org/1815643002/diff/20001/third_party/WebKit/Sour...
> third_party/WebKit/Source/core/dom/DocumentOrShadowRoot.h:17: return
> document.activeElement();
> On 2016/03/22 at 03:43:56, kochi wrote:
> > Can activeElement() and styleSheets() be a pure virtual function for
> TreeScope(),
> > and override in Document and ShadowRoot, then call it as
> treeScope.activeElement()
> > like getSelection() below?
> 
> I did not do it intentionally in this CL. That should be done in another CL.

okay then, lgtm.

[hayato, 2016-03-22 04:07:19]

The CQ bit was checked by hayato@chromium.org

[hayato, 2016-03-22 04:07:27]

The CQ bit was checked by hayato@chromium.org

[commit-bot: I haz the power, 2016-03-22 04:07:31]

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

[commit-bot: I haz the power, 2016-03-22 05:39:05]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-22 05:41:54]

Description was changed from

==========
Introduce DocumentOrShadowRoot Mixin

The spec is:
- https://dom.spec.whatwg.org/#documentorshadowroot
-
http://w3c.github.io/webcomponents/spec/shadow/#extensions-to-the-documentors...

BUG=595600
==========

to

==========
Introduce DocumentOrShadowRoot Mixin

The spec is:
- https://dom.spec.whatwg.org/#documentorshadowroot
-
http://w3c.github.io/webcomponents/spec/shadow/#extensions-to-the-documentors...

BUG=595600

Committed: https://crrev.com/6e97f9b878e58270e4a8d719800d16650c95459e
Cr-Commit-Position: refs/heads/master@{#382515}
==========

[commit-bot: I haz the power, 2016-03-22 05:41:55]

Patchset 2 (id:??) landed as
https://crrev.com/6e97f9b878e58270e4a8d719800d16650c95459e
Cr-Commit-Position: refs/heads/master@{#382515}