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

Source/core/dom/CustomElementRegistry.cpp

Index: Source/core/dom/CustomElementRegistry.cpp
diff --git a/Source/core/dom/CustomElementRegistry.cpp b/Source/core/dom/CustomElementRegistry.cpp
index e89575d3664af0311c0b8fcfd902b2e9f1f33e95..b662d62942c72d3ccebd9ccba01b039e662227c3 100644
--- a/Source/core/dom/CustomElementRegistry.cpp
+++ b/Source/core/dom/CustomElementRegistry.cpp
@@ -131,25 +131,19 @@ void CustomElementRegistry::registerElement(CustomElementConstructorBuilder* con
 
     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.
+    CustomElementUpgradeCandidateMap::ElementSet upgradeCandidates = m_candidates.takeUpgradeCandidatesFor(definition.get());

[esprehn, 2013/07/04 00:54:58]

Why does this use an HashSet? takeUpgradeCandidatesFor is iterating over a set
already, so we should just return a Vector here and avoid having to do all the
hashing in there.

 
+    for (CustomElementUpgradeCandidateMap::ElementSet::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);