Add the NonElementParentNode interface

Add the NonElementParentNode interface

As a side effect, DocumentFragment.getElementById now also has per-world
bindings. This makes the generated code bigger, and probably won't help.

Use [LegacyTreatAsPartialInterface] like the other *Node interfaces.

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

[philipj_slow, 2015-03-01 03:04:55]

philipj@opera.com changed reviewers:
+ haraken@chromium.org

[philipj_slow, 2015-03-01 03:04:55]

PTAL

[haraken, 2015-03-01 08:53:07]

LGTM

https://codereview.chromium.org/967753003/diff/1/Source/core/dom/NonElementPa...
File Source/core/dom/NonElementParentNode.h (right):

https://codereview.chromium.org/967753003/diff/1/Source/core/dom/NonElementPa...
Source/core/dom/NonElementParentNode.h:13: class NonElementParentNode {

Add final.

[philipj_slow, 2015-03-01 15:32:14]

https://codereview.chromium.org/967753003/diff/1/Source/core/dom/NonElementPa...
File Source/core/dom/NonElementParentNode.h (right):

https://codereview.chromium.org/967753003/diff/1/Source/core/dom/NonElementPa...
Source/core/dom/NonElementParentNode.h:13: class NonElementParentNode {
On 2015/03/01 08:53:07, haraken wrote:
> 
> Add final.

Hmm, does that do anything on a class that inherits nothing and has no virtual
members? Note that NonDocumentTypeChildNode does not have final, so whatever the
rule is I'd like to apply it consistently. I'm not really sure what to take away
from the blink-dev discussion:
https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/VTNZzizN0zo

[haraken, 2015-03-01 16:30:27]

On 2015/03/01 15:32:14, philipj_UTC7 wrote:
>
https://codereview.chromium.org/967753003/diff/1/Source/core/dom/NonElementPa...
> File Source/core/dom/NonElementParentNode.h (right):
> 
>
https://codereview.chromium.org/967753003/diff/1/Source/core/dom/NonElementPa...
> Source/core/dom/NonElementParentNode.h:13: class NonElementParentNode {
> On 2015/03/01 08:53:07, haraken wrote:
> > 
> > Add final.
> 
> Hmm, does that do anything on a class that inherits nothing and has no virtual
> members? Note that NonDocumentTypeChildNode does not have final, so whatever
the
> rule is I'd like to apply it consistently. I'm not really sure what to take
away
> from the blink-dev discussion:
>
https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/VTNZzizN0zo

Yeah, I'm not sure either... (Either way the CL looks good.)

[philipj_slow, 2015-03-01 16:34:07]

Thanks, landing this as-is then :)

[philipj_slow, 2015-03-01 16:34:12]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2015-03-01 16:35:28]

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

[commit-bot: I haz the power, 2015-03-01 16:50:20]

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

[commit-bot: I haz the power, 2015-03-01 16:50:21]

Try jobs failed on following builders:
  blink_presubmit on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/2...)

[philipj_slow, 2015-03-01 17:03:04]

rebase

[philipj_slow, 2015-03-01 17:03:30]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2015-03-01 17:03:31]

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

[commit-bot: I haz the power, 2015-03-01 17:04:03]

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

[commit-bot: I haz the power, 2015-03-01 18:28:15]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=191066