bindings: Introduces type-check for the internal pointers.

Source/bindings/core/v8/DOMWrapperMap.h

Index: Source/bindings/core/v8/DOMWrapperMap.h
diff --git a/Source/bindings/core/v8/DOMWrapperMap.h b/Source/bindings/core/v8/DOMWrapperMap.h
index 2b6e10b8ada260d241d9df0200876c463b3d41fa..1609d01409adbaaaf4667fcd823483589799faf2 100644
--- a/Source/bindings/core/v8/DOMWrapperMap.h
+++ b/Source/bindings/core/v8/DOMWrapperMap.h
@@ -70,7 +70,7 @@ public:
 
     void set(KeyType* key, v8::Handle<v8::Object> wrapper, const WrapperConfiguration& configuration)
     {
-        ASSERT(static_cast<KeyType*>(toNative(wrapper)) == key);
+        ASSERT(static_cast<KeyType*>(toInternalPointer(wrapper)) == key);
         RELEASE_ASSERT(!containsKey(key)); // See crbug.com/368095
         v8::UniquePersistent<v8::Object> unique(m_isolate, wrapper);
         configuration.configureWrapper(&unique);
@@ -142,7 +142,7 @@ private:
         static KeyType* KeyFromWeakCallbackData(
             const v8::WeakCallbackData<v8::Object, WeakCallbackDataType>& data)
         {
-            return static_cast<KeyType*>(toNative(data.GetValue()));
+            return reinterpret_cast<KeyType*>(toInternalPointer(data.GetValue()));

[Jens Widell, 2014/08/08 09:37:06]

Why use reinterpret_cast here? If KeyType is a ScriptWrapperBase sub-class,
static_cast would be the appropriate cast to use.

[Yuki, 2014/08/08 13:01:25]

We're using DOMWrapperMap with KeyType=NPObject, which is defined outside of
blink and does not inherit from ScriptWrappableBase.  So we need
reinterpret_cast here.

         }
 
         // Dispose traits: