Fixing custom element type extensions to have correct native type

Source/bindings/dart/DartCustomElementWrapper.cpp

Index: Source/bindings/dart/DartCustomElementWrapper.cpp
diff --git a/Source/bindings/dart/DartCustomElementWrapper.cpp b/Source/bindings/dart/DartCustomElementWrapper.cpp
index 569c7bfd8ae584d4a3d4665fadeb69299708458b..9328bed08458a27e4b7fe8e70510d70961c92cb1 100644
--- a/Source/bindings/dart/DartCustomElementWrapper.cpp
+++ b/Source/bindings/dart/DartCustomElementWrapper.cpp
@@ -114,8 +114,7 @@ Dart_Handle DartCustomElementWrapper<HTMLElement>::upgradeDartWrapper(HTMLElemen
 
     Dart_Handle newInstance = Dart_Allocate(customType);
     ASSERT(!Dart_IsError(newInstance));
-    // TODO(14190): Use a more specific type instead of DartHTMLElement here.
-    DartDOMWrapper::writeNativePointer(newInstance, element, DartHTMLElement::dartClassId);
+    DartDOMWrapper::writeNativePointer(newInstance, element, binding->nativeClassId());
 
     Dart_Handle result = Dart_InvokeConstructor(newInstance, Dart_NewStringFromCString("created"), 0, 0);
 
@@ -134,7 +133,6 @@ Dart_Handle DartCustomElementWrapper<HTMLElement>::upgradeDartWrapper(HTMLElemen
         Dart_Handle fallbackWrapper = createUpgradeCandidateWrapper(element, createSpecificWrapper);
 
         DartDOMWrapper::associateWrapper<DartHTMLElement>(domData, element, fallbackWrapper);
-        // TODO(14190): Use a more specific type instead of DartHTMLElement here.
         DartDOMWrapper::writeNativePointer(fallbackWrapper, element, DartHTMLElement::dartClassId);

[vsm, 2013/10/18 21:18:15]

Shouldn't the fallback wrapper still be binding->nativeClassId()?  E.g., if we
fail to update a CanvasElement, we still return a base CanvasElement, right?

[blois, 2013/10/18 23:03:40]

Done.

         return fallbackWrapper;
     }