Changeset 130664 in webkit

Timestamp:

Oct 8, 2012, 11:46:20 AM (14 years ago)

Author:

msaboff@apple.com

Message:

After r130344, OpaqueJSString() creates an empty string which should be a null string
​https://bugs.webkit.org/show_bug.cgi?id=98417

Reviewed by Sam Weinig.

Changed create() of a null string to return 0. This is the same behavior as before r130344.

• API/OpaqueJSString.cpp:

(OpaqueJSString::create):

Location:

trunk/Source/JavaScriptCore

Files:

• API/OpaqueJSString.cpp (modified) (1 diff)
• ChangeLog (modified) (1 diff)

trunk/Source/JavaScriptCore/API/OpaqueJSString.cpp

@@ -37 +37 @@
     if (!string.isNull())
         return adoptRef(new OpaqueJSString(string));
-    return adoptRef(new OpaqueJSString());
+    return 0;
 }
 

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2012-10-08  Michael Saboff  <msaboff@apple.com>
+
+        After r130344, OpaqueJSString() creates an empty string which should be a null string
+        https://bugs.webkit.org/show_bug.cgi?id=98417
+
+        Reviewed by Sam Weinig.
+
+        Changed create() of a null string to return 0. This is the same behavior as before r130344.
+
+        * API/OpaqueJSString.cpp:
+        (OpaqueJSString::create):
+
 2012-10-07  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>