Pow __proto__, sock :unresolved, and clunk the created callback at once.

Source/core/dom/CustomElementRegistry.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 22 matching lines @@
 
 #include "HTMLNames.h"
 #include "SVGNames.h"
 #include "bindings/v8/CustomElementConstructorBuilder.h"
 #include "core/dom/CustomElementCallbackDispatcher.h"
 #include "core/dom/CustomElementDefinition.h"
 #include "core/dom/Document.h"
 #include "core/dom/Element.h"
 #include "core/html/HTMLElement.h"
 #include "core/svg/SVGElement.h"
+#include "wtf/Vector.h"
 
 namespace WebCore {
 
 void setTypeExtension(Element* element, const AtomicString& typeExtension)
 {
     ASSERT(element);
     if (!typeExtension.isEmpty())
         element->setAttribute(HTMLNames::isAttr, typeExtension);
 }
 
@@ skipping 71 matching lines @@
 
     RefPtr<CustomElementDefinition> definition = CustomElementDefinition::create(type, tagName.localName(), tagName.namespaceURI(), lifecycleCallbacks);
 
     if (!constructorBuilder->createConstructor(document(), definition.get())) {
         ec = NOT_SUPPORTED_ERR;
         return;
     }
 
     m_definitions.add(definition->type(), definition);
 
-    // Upgrade elements that were waiting for this definition.
-    CustomElementUpgradeCandidateMap::ElementSet upgradeCandidates = m_candidates.takeUpgradeCandidatesFor(definition.get());
-    if (!constructorBuilder->didRegisterDefinition(definition.get(), upgradeCandidates)) {
+    if (!constructorBuilder->didRegisterDefinition(definition.get())) {
         ec = NOT_SUPPORTED_ERR;
         return;
     }
 
-    for (CustomElementUpgradeCandidateMap::ElementSet::iterator it = upgradeCandidates.begin(); it != upgradeCandidates.end(); ++it) {
-        (*it)->setNeedsStyleRecalc(); // :unresolved has changed
+    // Upgrade elements that were waiting for this definition.
+    Vector<Element*> upgradeCandidates = m_candidates.takeUpgradeCandidatesFor(definition.get());

[esprehn, 2013/07/04 01:29:01]

I think you want a const reference to avoid the copy here?

 
+    for (Vector<Element*>::iterator it = upgradeCandidates.begin(); it != upgradeCandidates.end(); ++it) {
         CustomElementCallbackDispatcher::instance().enqueueCreatedCallback(lifecycleCallbacks.get(), *it);
     }
 }
 
-bool CustomElementRegistry::isUnresolved(Element* element) const
-{
-    return m_candidates.contains(element);
-}
-
 PassRefPtr<CustomElementDefinition> CustomElementRegistry::findFor(Element* element) const
 {
     ASSERT(element->document()->registry() == this);
 
     if (!element->isCustomElement())
         return 0;
 
     // When a custom tag and a type extension are provided as element
     // names at the same time, the custom tag takes precedence.
     if (isCustomTagName(element->localName())) {
@@ skipping 82 matching lines @@
     ASSERT(element->isCustomElement());
     m_candidates.remove(element);
 }
 
 inline Document* CustomElementRegistry::document() const
 {
     return toDocument(m_scriptExecutionContext);
 }
 
 }