Remove an unnecessary reinterpret_cast from a persistent to an handle

Source/bindings/core/v8/ScriptProfiler.cpp

 /*
  * Copyright (c) 2011, 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 252 matching lines @@
 {
     // visitNodeWrappers() should receive a ScriptState and retrieve an Isolate
     // from the ScriptState.
     v8::Isolate* isolate = v8::Isolate::GetCurrent();
     v8::HandleScope handleScope(isolate);
 
     class DOMNodeWrapperVisitor : public v8::PersistentHandleVisitor {
     public:
         DOMNodeWrapperVisitor(WrappedNodeVisitor* visitor, v8::Isolate* isolate)
             : m_visitor(visitor)
+#if ENABLE(ASSERT)
             , m_isolate(isolate)
+#endif
         {
         }
 
         virtual void VisitPersistentHandle(v8::Persistent<v8::Value>* value, uint16_t classId) override
         {
             if (classId != WrapperTypeInfo::NodeClassId)
                 return;
-            // Casting to Handle is safe here, since the Persistent cannot get
-            // GCd during visiting.
-            v8::Handle<v8::Object>* wrapper = reinterpret_cast<v8::Handle<v8::Object>*>(value);
-            ASSERT_UNUSED(m_isolate, V8Node::hasInstance(*wrapper, m_isolate));
-            ASSERT((*wrapper)->IsObject());
-            m_visitor->visitNode(V8Node::toImpl(*wrapper));
+
+#if ENABLE(ASSERT)
+            {
+                v8::HandleScope scope(m_isolate);
+                v8::Local<v8::Object> wrapper = v8::Local<v8::Object>::New(m_isolate, v8::Persistent<v8::Object>::Cast(*value));
+                ASSERT(V8Node::hasInstance(wrapper, m_isolate));
+                ASSERT(wrapper->IsObject());
+            }
+#endif
+            m_visitor->visitNode(toScriptWrappableBase(v8::Persistent<v8::Object>::Cast(*value))->toImpl<Node>());
         }
 
     private:
         WrappedNodeVisitor* m_visitor;
+#if ENABLE(ASSERT)
         v8::Isolate* m_isolate;
+#endif
     } wrapperVisitor(visitor, isolate);
 
     v8::V8::VisitHandlesWithClassIds(isolate, &wrapperVisitor);
 }
 
 ProfileNameIdleTimeMap* ScriptProfiler::currentProfileNameIdleTimeMap()
 {
     AtomicallyInitializedStatic(WTF::ThreadSpecific<ProfileNameIdleTimeMap>*, map = new WTF::ThreadSpecific<ProfileNameIdleTimeMap>);
     return *map;
 }
 
 void ScriptProfiler::setIdle(bool isIdle)
 {
     v8::Isolate* isolate = v8::Isolate::GetCurrent();
     if (v8::CpuProfiler* profiler = isolate->GetCpuProfiler())
         profiler->SetIdle(isIdle);
 }
 
 } // namespace blink