CustomElements: createCustomElementSync accepts a definition

CustomElement::createCustomElementSync can accept a CustomElementDefinition.

BUG=648828
Committed: https://crrev.com/930362923fecec7e947400e44a06983fffa75f24
Cr-Commit-Position: refs/heads/master@{#433169}

[yurak, 2016-11-15 12:43:40]

Description was changed from

==========
createCustomElementSync accepts a definition


More tests, fix case conversion of 'is' attribute


V1 definition check in document::createElement


New tests


Made changes


createElement - sync

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

to

==========
CustomElement::createCustomElementSync can accept a CustomElementDefinition.

BUG=648828
==========

[yurak, 2016-11-15 12:43:40]

yurak@google.com changed reviewers:
+ dominicc@chromium.org

[yurak, 2016-11-15 12:45:59]

PTL

[dominicc (has gone to gerrit), 2016-11-17 01:39:21]

LGTM, one optional comment inline.

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

https://codereview.chromium.org/2505573002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/dom/custom/CustomElement.h:67: static bool
shouldCreateCustomizedBuiltinElement(const AtomicString&);
I think you *do* want to name this parameter. It's a local name, right? To
contrast it to shouldCreateCustomElement (that one's a custom element name, not
necessarily a tag name.)

[yurak, 2016-11-18 08:14:40]

The CQ bit was checked by yurak@google.com

[yurak, 2016-11-18 08:14:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
dominicc@chromium.org
Link to the patchset: https://codereview.chromium.org/2505573002/#ps60001
(title: "createCustomElementSync accepts a definition")

[commit-bot: I haz the power, 2016-11-18 08:14:58]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-18 10:41:43]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-11-18 10:43:28]

Description was changed from

==========
CustomElement::createCustomElementSync can accept a CustomElementDefinition.

BUG=648828
==========

to

==========
CustomElement::createCustomElementSync can accept a CustomElementDefinition.

BUG=648828

Committed: https://crrev.com/930362923fecec7e947400e44a06983fffa75f24
Cr-Commit-Position: refs/heads/master@{#433169}
==========

[commit-bot: I haz the power, 2016-11-18 10:43:29]

Patchset 4 (id:??) landed as
https://crrev.com/930362923fecec7e947400e44a06983fffa75f24
Cr-Commit-Position: refs/heads/master@{#433169}