2009-03-20 Anders Carlsson <andersca@apple.com>

        Reviewed by Darin Adler.

        Only release the result NPVariant if the call to NPN_Invoke was successful.
        
        * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp:
        (testNPRuntime):



git-svn-id: http://svn.webkit.org/repository/webkit/trunk@41869 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKitTools/ChangeLog

+2009-03-20  Anders Carlsson  <andersca@apple.com>
+
+        Reviewed by Darin Adler.
+
+        Only release the result NPVariant if the call to NPN_Invoke was successful.
+        
+        * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp:
+        (testNPRuntime):
+
 2009-03-19  Gustavo Noronha Silva  <gns@gnome.org>
         Reviewed by Mark Rowe.

WebKitTools/DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp

@@ -715,8 +715,8 @@ void testNPRuntime(NPP npp)
     OBJECT_TO_NPVARIANT(windowScriptObject, args[6]);
     
     NPVariant result;
-    browser->invoke(npp, windowScriptObject, testNPInvoke, args, 7, &result);
+    if (browser->invoke(npp, windowScriptObject, testNPInvoke, args, 7, &result))
+        browser->releasevariantvalue(&result);
     
-    browser->releasevariantvalue(&result);
     browser->releaseobject(windowScriptObject);
 }