Title: [211510] branches/safari-603-branch/Source/WebCore
Revision
211510
Author
matthew_han...@apple.com
Date
2017-02-01 13:26:11 -0800 (Wed, 01 Feb 2017)

Log Message

Merge r210217. rdar://problem/30260068

Modified Paths

Diff

Modified: branches/safari-603-branch/Source/WebCore/ChangeLog (211509 => 211510)


--- branches/safari-603-branch/Source/WebCore/ChangeLog	2017-02-01 21:26:09 UTC (rev 211509)
+++ branches/safari-603-branch/Source/WebCore/ChangeLog	2017-02-01 21:26:11 UTC (rev 211510)
@@ -1,3 +1,13 @@
+2017-01-31  Matthew Hanson  <matthew_han...@apple.com>
+
+        Merge r210217. rdar://problem/30260068
+
+        Committed on behalf of Chris Dumez.
+
+        * bindings/js/JSStorageCustom.cpp:
+        (WebCore::JSStorage::nameGetter):
+        Early-return false if an item has an exception (after propagating it.)
+
 2017-01-27  Ryan Haddad  <ryanhad...@apple.com>
 
         Merge r211285.

Modified: branches/safari-603-branch/Source/WebCore/bindings/js/JSStorageCustom.cpp (211509 => 211510)


--- branches/safari-603-branch/Source/WebCore/bindings/js/JSStorageCustom.cpp	2017-02-01 21:26:09 UTC (rev 211509)
+++ branches/safari-603-branch/Source/WebCore/bindings/js/JSStorageCustom.cpp	2017-02-01 21:26:11 UTC (rev 211510)
@@ -41,8 +41,12 @@
         return false;
 
     auto item = wrapped().getItem(propertyNameToString(propertyName));
-    if (item.hasException())
-        propagateException(*state, item.releaseException());
+    if (item.hasException()) {
+        auto& vm = state->vm();
+        auto scope = DECLARE_THROW_SCOPE(vm);
+        propagateException(*state, scope, item.releaseException());
+        return false;
+    }
 
     auto string = item.releaseReturnValue();
     if (string.isNull())
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to