Limit the usage of SharedPersistent to ScriptValue only

Source/bindings/v8/custom/V8HTMLPlugInElementCustom.cpp

 /*
 * Copyright (C) 2009 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 15 matching lines @@
 * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 */
 
 #include "config.h"
 
 #include "V8HTMLAppletElement.h"
 #include "V8HTMLEmbedElement.h"
 #include "V8HTMLObjectElement.h"
-#include "bindings/v8/SharedPersistent.h"
+#include "bindings/v8/ScriptValue.h"
 #include "bindings/v8/V8Binding.h"
 #include "bindings/v8/V8NPObject.h"
 #include "core/frame/UseCounter.h"
 
 namespace WebCore {
 
 // FIXME: Consider moving getter/setter helpers to V8NPObject and renaming this file to V8PluginElementFunctions
 // to match JSC bindings naming convention.
 
 template <class C>
 static void npObjectNamedGetter(v8::Local<v8::String> name, const v8::PropertyCallbackInfo<v8::Value>& info)
 {
     HTMLPlugInElement* imp = C::toNative(info.Holder());
-    RefPtr<SharedPersistent<v8::Object> > wrapper = imp->pluginWrapper();
-    if (!wrapper)
-        return;
-
-    v8::Local<v8::Object> instanceTemplate = wrapper->newLocal(info.GetIsolate());
+    v8::Handle<v8::Object> instanceTemplate = v8::Handle<v8::Object>::Cast(imp->pluginWrapper().v8Value());
     if (instanceTemplate.IsEmpty())
         return;
 
     npObjectGetNamedProperty(instanceTemplate, name, info);
 }
 
 template <class C>
 static void npObjectNamedSetter(v8::Local<v8::String> name, v8::Local<v8::Value> value, const v8::PropertyCallbackInfo<v8::Value>& info)
 {
     HTMLPlugInElement* imp = C::toNative(info.Holder());
-    RefPtr<SharedPersistent<v8::Object> > wrapper = imp->pluginWrapper();
-    if (!wrapper)
-        return;
-
-    v8::Local<v8::Object> instanceTemplate = wrapper->newLocal(info.GetIsolate());
+    v8::Handle<v8::Object> instanceTemplate = v8::Handle<v8::Object>::Cast(imp->pluginWrapper().v8Value());
     if (instanceTemplate.IsEmpty())
         return;
 
     npObjectSetNamedProperty(instanceTemplate, name, value, info);
 }
 
 void V8HTMLAppletElement::namedPropertyGetterCustom(v8::Local<v8::String> name, const v8::PropertyCallbackInfo<v8::Value>& info)
 {
     npObjectNamedGetter<V8HTMLAppletElement>(name, info);
 }
@@ skipping 41 matching lines @@
 {
     HTMLPlugInElement* imp = V8HTMLObjectElement::toNative(info.Holder());
     UseCounter::count(imp->document(), UseCounter::HTMLObjectElementLegacyCall);
     npObjectInvokeDefaultHandler(info);
 }
 
 template <class C>
 void npObjectIndexedGetter(uint32_t index, const v8::PropertyCallbackInfo<v8::Value>& info)
 {
     HTMLPlugInElement* imp = C::toNative(info.Holder());
-    RefPtr<SharedPersistent<v8::Object> > wrapper = imp->pluginWrapper();
-    if (!wrapper)
-        return;
-
-    v8::Local<v8::Object> instanceTemplate = wrapper->newLocal(info.GetIsolate());
+    v8::Handle<v8::Object> instanceTemplate = v8::Handle<v8::Object>::Cast(imp->pluginWrapper().v8Value());
     if (instanceTemplate.IsEmpty())
         return;
 
     npObjectGetIndexedProperty(instanceTemplate, index, info);
 }
 
 template <class C>
 void npObjectIndexedSetter(uint32_t index, v8::Local<v8::Value> value, const v8::PropertyCallbackInfo<v8::Value>& info)
 {
     HTMLPlugInElement* imp = C::toNative(info.Holder());
-    RefPtr<SharedPersistent<v8::Object> > wrapper = imp->pluginWrapper();
-    if (!wrapper)
-        return;
-
-    v8::Local<v8::Object> instanceTemplate = wrapper->newLocal(info.GetIsolate());
+    v8::Handle<v8::Object> instanceTemplate = v8::Handle<v8::Object>::Cast(imp->pluginWrapper().v8Value());
     if (instanceTemplate.IsEmpty())
         return;
 
     npObjectSetIndexedProperty(instanceTemplate, index, value, info);
 }
 
 void V8HTMLAppletElement::indexedPropertyGetterCustom(uint32_t index, const v8::PropertyCallbackInfo<v8::Value>& info)
 {
     npObjectIndexedGetter<V8HTMLAppletElement>(index, info);
 }
@@ skipping 17 matching lines @@
 {
     npObjectIndexedSetter<V8HTMLEmbedElement>(index, value, info);
 }
 
 void V8HTMLObjectElement::indexedPropertySetterCustom(uint32_t index, v8::Local<v8::Value> value, const v8::PropertyCallbackInfo<v8::Value>& info)
 {
     npObjectIndexedSetter<V8HTMLObjectElement>(index, value, info);
 }
 
 } // namespace WebCore