Use originalGetTag on wrapped object

pkg/web_components/lib/dart_support.js

Index: pkg/web_components/lib/dart_support.js
diff --git a/pkg/web_components/lib/dart_support.js b/pkg/web_components/lib/dart_support.js
index 44b5a38cad1912c75ad8e8b5be586d05e845aa86..503faeeec9b3010c332c0bc153d112cc8d2c1885 100644
--- a/pkg/web_components/lib/dart_support.js
+++ b/pkg/web_components/lib/dart_support.js
@@ -15,11 +15,18 @@
   var needsConstructorFix = window.constructor === window.Window;
 
   // TODO(jmesserly): we need to wrap document somehow (a dart:html hook?)
-  window.dartExperimentalFixupGetTag = function(originalGetTag) {
+
+  // dartNativeDispatchHooksTransformer is described on initHooks() in
+  // sdk/lib/_internal/lib/native_helper.dart.
+  if (typeof window.dartNativeDispatchHooksTransformer == 'undefined')
+    window.dartNativeDispatchHooksTransformer = [];
+
+  window.dartNativeDispatchHooksTransformer.push(function(hooks) {
     var NodeList = ShadowDOMPolyfill.wrappers.NodeList;
     var ShadowRoot = ShadowDOMPolyfill.wrappers.ShadowRoot;
     var unwrapIfNeeded = ShadowDOMPolyfill.unwrapIfNeeded;
-    function getTag(obj) {
+    var originalGetTag = hooks.getTag;
+    hooks.getTag = function getTag(obj) {
       // TODO(jmesserly): do we still need these?
       if (obj instanceof NodeList) return 'NodeList';
       if (obj instanceof ShadowRoot) return 'ShadowRoot';
@@ -44,8 +51,7 @@
         if (ctor === unwrapped.constructor) {
           var name = ctor._ShadowDOMPolyfill$cacheTag_;
           if (!name) {
-            name = Object.prototype.toString.call(unwrapped);
-            name = name.substring(8, name.length - 1);
+            name = originalGetTag(unwrapped);

[Jennifer Messerly, 2014/05/13 21:01:31]

Interesting! Do we still need ctor._ShadowDOMPolyfill$cacheTag_ ? It seems like
we're willing to `return originalGetTag(obj);` below so I'm guessing it's not
needed. We could replace everything starting from line 52 to 57 (in the new
code) with just `return originalGetTag(unwrapped);`

I'm not sure if that then allows even more code folding with line 62 or not...

anyway, cool to see the Firefox code needs less workarounds now!

[Siggi Cherem (dart-lang), 2014/05/13 21:06:58]

+1, lgtm, if we don't need the caching, I'm happy to get rid of it too :)

[Siggi Cherem (dart-lang), 2014/05/13 21:55:21]

FYI - we chatted more about it offline and decided to keep the cashing for now
(even though we removed the toString here, it is still used underneath, so
caching it might not be a bad idea, we can measure and revisit if we want
later).

[Jennifer Messerly, 2014/05/13 22:12:36]

interesting. yeah, seems okay, but makes me wonder why we don't cache for
everything then?

FWIW -- I thought in the old dart2js code, "originalGetTag" did cache. At least,
that was my understanding when I reviewed the dart2js code at the time, and
reason it does not attempt caching below.

             ctor._ShadowDOMPolyfill$cacheTag_ = name;
           }
           return name;
@@ -55,7 +61,5 @@
       }
       return originalGetTag(obj);
     }
-
-    return getTag;
-  };
+  });
 })();