Title: [191397] trunk/Source/_javascript_Core
Revision
191397
Author
[email protected]
Date
2015-10-21 12:56:50 -0700 (Wed, 21 Oct 2015)

Log Message

Web Inspector: Array previews with Symbol objects have too few preview values
https://bugs.webkit.org/show_bug.cgi?id=150404

Patch by Joseph Pecoraro <[email protected]> on 2015-10-21
Reviewed by Timothy Hatcher.

* inspector/InjectedScriptSource.js:
(InjectedScript.RemoteObject.prototype._appendPropertyPreviews):
We should be continuing inside this loop not returning.

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (191396 => 191397)


--- trunk/Source/_javascript_Core/ChangeLog	2015-10-21 18:53:17 UTC (rev 191396)
+++ trunk/Source/_javascript_Core/ChangeLog	2015-10-21 19:56:50 UTC (rev 191397)
@@ -1,3 +1,14 @@
+2015-10-21  Joseph Pecoraro  <[email protected]>
+
+        Web Inspector: Array previews with Symbol objects have too few preview values
+        https://bugs.webkit.org/show_bug.cgi?id=150404
+
+        Reviewed by Timothy Hatcher.
+
+        * inspector/InjectedScriptSource.js:
+        (InjectedScript.RemoteObject.prototype._appendPropertyPreviews):
+        We should be continuing inside this loop not returning.
+
 2015-10-21  Filip Pizlo  <[email protected]>
 
         Failures in PutStackSinkingPhase should be less severe

Modified: trunk/Source/_javascript_Core/inspector/InjectedScriptSource.js (191396 => 191397)


--- trunk/Source/_javascript_Core/inspector/InjectedScriptSource.js	2015-10-21 18:53:17 UTC (rev 191396)
+++ trunk/Source/_javascript_Core/inspector/InjectedScriptSource.js	2015-10-21 19:56:50 UTC (rev 191397)
@@ -1185,7 +1185,7 @@
                     preview.lossless = false;
                 }
                 this._appendPropertyPreview(preview, internal, {name, type, value: symbolString}, propertiesThreshold);
-                return;
+                continue;
             }
 
             // Object.
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to