[oilpan] Fix NPObject bindings by not using HandleScopes in WebBindings for non-webcore specific me…

Source/WebKit/chromium/src/WebBindings.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 45 matching lines @@
 namespace WebKit {
 
 bool WebBindings::construct(NPP npp, NPObject* object, const NPVariant* args, uint32_t argCount, NPVariant* result)
 {
     HandleScope scope;
     return _NPN_Construct(npp, object, args, argCount, result);
 }
 
 NPObject* WebBindings::createObject(NPP npp, NPClass* npClass)
 {
-    HandleScope scope;
+    // WebCore independent method that is called from non-WebCore related
+    // threads. Therefore there is no HandleScope nor NoHandleScope here.
     return _NPN_CreateObject(npp, npClass);
 }
 
 bool WebBindings::enumerate(NPP npp, NPObject* object, NPIdentifier** identifier, uint32_t* identifierCount)
 {
     HandleScope scope;
     return _NPN_Enumerate(npp, object, identifier, identifierCount);
 }
 
 bool WebBindings::evaluate(NPP npp, NPObject* object, NPString* script, NPVariant* result)
 {
     HandleScope scope;
     return _NPN_Evaluate(npp, object, script, result);
 }
 
 bool WebBindings::evaluateHelper(NPP npp, bool popupsAllowed, NPObject* object, NPString* script, NPVariant* result)
 {
     HandleScope scope;
     return _NPN_EvaluateHelper(npp, popupsAllowed, object, script, result);
 }
 
 NPIdentifier WebBindings::getIntIdentifier(int32_t number)
 {
-    HandleScope scope;
+    // WebCore independent method that is called from non-WebCore related
+    // threads. Therefore there is no HandleScope nor NoHandleScope here.
     return _NPN_GetIntIdentifier(number);
 }
 
 bool WebBindings::getProperty(NPP npp, NPObject* object, NPIdentifier property, NPVariant* result)
 {
     HandleScope scope;
     return _NPN_GetProperty(npp, object, property, result);
 }
 
 NPIdentifier WebBindings::getStringIdentifier(const NPUTF8* string)
 {
-    HandleScope scope;
+    // WebCore independent method that is called from non-WebCore related
+    // threads. Therefore there is no HandleScope nor NoHandleScope here.
     return _NPN_GetStringIdentifier(string);
 }
 
 void WebBindings::getStringIdentifiers(const NPUTF8** names, int32_t nameCount, NPIdentifier* identifiers)
 {
-    HandleScope scope;
+    // WebCore independent method that is called from non-WebCore related
+    // threads. Therefore there is no HandleScope nor NoHandleScope here.
     _NPN_GetStringIdentifiers(names, nameCount, identifiers);
 }
 
 bool WebBindings::hasMethod(NPP npp, NPObject* object, NPIdentifier method)
 {
     HandleScope scope;
     return _NPN_HasMethod(npp, object, method);
 }
 
 bool WebBindings::hasProperty(NPP npp, NPObject* object, NPIdentifier property)
 {
     HandleScope scope;
     return _NPN_HasProperty(npp, object, property);
 }
 
 bool WebBindings::identifierIsString(NPIdentifier identifier)
 {
-    HandleScope scope;
+    // WebCore independent method that is called from non-WebCore related
+    // threads. Therefore there is no HandleScope nor NoHandleScope here.
     return _NPN_IdentifierIsString(identifier);
 }
 
 int32_t WebBindings::intFromIdentifier(NPIdentifier identifier)
 {
-    HandleScope scope;
+    // WebCore independent method that is called from non-WebCore related
+    // threads. Therefore there is no HandleScope nor NoHandleScope here.
     return _NPN_IntFromIdentifier(identifier);
 }
 
 void WebBindings::initializeVariantWithStringCopy(NPVariant* variant, const NPString* value)
 {
-    HandleScope scope;
+    // WebCore independent method that is called from non-WebCore related
+    // threads. Therefore there is no HandleScope nor NoHandleScope here.
     _NPN_InitializeVariantWithStringCopy(variant, value);
 }
 
 bool WebBindings::invoke(NPP npp, NPObject* object, NPIdentifier method, const NPVariant* args, uint32_t argCount, NPVariant* result)
 {
     HandleScope scope;
     return _NPN_Invoke(npp, object, method, args, argCount, result);
 }
 
 bool WebBindings::invokeDefault(NPP npp, NPObject* object, const NPVariant* args, uint32_t argCount, NPVariant* result)
 {
     HandleScope scope;
     return _NPN_InvokeDefault(npp, object, args, argCount, result);
 }
 
 void WebBindings::releaseObject(NPObject* object)
 {
-    HandleScope scope;
+    // WebCore independent method that is called from non-WebCore related
+    // threads. Therefore there is no HandleScope nor NoHandleScope here.
     return _NPN_ReleaseObject(object);
 }
 
 void WebBindings::releaseVariantValue(NPVariant* variant)
 {
-    HandleScope scope;
+    // WebCore independent method that is called from non-WebCore related
+    // threads. Therefore there is no HandleScope nor NoHandleScope here.
     _NPN_ReleaseVariantValue(variant);
 }
 
 bool WebBindings::removeProperty(NPP npp, NPObject* object, NPIdentifier identifier)
 {
     HandleScope scope;
     return _NPN_RemoveProperty(npp, object, identifier);
 }
 
 NPObject* WebBindings::retainObject(NPObject* object)
 {
-    HandleScope scope;
+    // WebCore independent method that is called from non-WebCore related
+    // threads. Therefore there is no HandleScope nor NoHandleScope here.
     return _NPN_RetainObject(object);
 }
 
 void WebBindings::setException(NPObject* object, const NPUTF8* message)
 {
     HandleScope scope;
     _NPN_SetException(object, message);
 }
 
 bool WebBindings::setProperty(NPP npp, NPObject* object, NPIdentifier identifier, const NPVariant* value)
 {
     HandleScope scope;
     return _NPN_SetProperty(npp, object, identifier, value);
 }
 
 void WebBindings::unregisterObject(NPObject* object)
 {
-    HandleScope scope;
+    // WebCore independent method that is called from non-WebCore related
+    // threads. Therefore there is no HandleScope nor NoHandleScope here.
     _NPN_UnregisterObject(object);
 }
 
 NPUTF8* WebBindings::utf8FromIdentifier(NPIdentifier identifier)
 {
-    HandleScope scope;
+    // WebCore independent method that is called from non-WebCore related
+    // threads. Therefore there is no HandleScope nor NoHandleScope here.
     return _NPN_UTF8FromIdentifier(identifier);
 }
 
 void WebBindings::extractIdentifierData(const NPIdentifier& identifier, const NPUTF8*& string, int32_t& number, bool& isString)
 {
-    HandleScope scope;
+    // WebCore independent method that is called from non-WebCore related
+    // threads. Therefore there is no HandleScope nor NoHandleScope here.
     PrivateIdentifier* data = static_cast<PrivateIdentifier*>(identifier);
     if (!data) {
         isString = false;
         number = 0;
         return;
     }
 
     isString = data->isString;
     if (isString)
         string = data->value.string;
@@ skipping 176 matching lines @@
             return v8::Undefined();
         V8NPObject* v8Object = reinterpret_cast<V8NPObject*>(object);
         return convertNPVariantToV8Object(variant, v8Object->rootObject->frame()->script()->windowScriptNPObject(), isolate);
     }
     // Safe to pass 0 since we have checked the script object class to make sure the
     // argument is a primitive v8 type.
     return convertNPVariantToV8Object(variant, 0, isolate);
 }
 
 } // namespace WebKit