Bug 142420

Summary: Web Inspector: ES6: Improved Support for Iterator Objects
Product: WebKit Reporter: Joseph Pecoraro <joepeck>
Component: Web InspectorAssignee: Joseph Pecoraro <joepeck>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, ggaren, graouts, joepeck, jonowells, mattbaker, nvasilyev, timothy, webkit-bug-importer, ysuzuki
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
[PATCH] Proposed Fix
none
[IMAGE] Iterator Previews
none
[IMAGE] Iterator Internal Properties
none
[PATCH] Proposed Fix none

Joseph Pecoraro
Reported 2015-03-06 17:43:27 PST
* SUMMARY ES6: Improved Console Support for Iterator Objects. For iterators, it is useful to see previews of the next few objects in the iterator. js> var iter = "long string"[Symbol.iterator]() <- StringIterator ["l", "o", "n", "g", " ", ...] js> iter.next(); iter.next(); iter <- StringIterator ["n", "g", " ", "s", "t", ...] And also Internal Properties, such as the object being iterating and for some iterators the Kind of Iterator ("key", "value", "key+value").
Attachments
[PATCH] Proposed Fix (34.02 KB, patch)
2015-03-06 17:50 PST, Joseph Pecoraro
no flags
[IMAGE] Iterator Previews (142.09 KB, image/png)
2015-03-06 17:50 PST, Joseph Pecoraro
no flags
[IMAGE] Iterator Internal Properties (199.41 KB, image/png)
2015-03-06 17:51 PST, Joseph Pecoraro
no flags
[PATCH] Proposed Fix (54.60 KB, patch)
2015-03-06 18:32 PST, Joseph Pecoraro
no flags
Radar WebKit Bug Importer
Comment 1 2015-03-06 17:43:42 PST
Joseph Pecoraro
Comment 2 2015-03-06 17:50:03 PST
Created attachment 248121 [details] [PATCH] Proposed Fix
Joseph Pecoraro
Comment 3 2015-03-06 17:50:28 PST
Created attachment 248122 [details] [IMAGE] Iterator Previews
Joseph Pecoraro
Comment 4 2015-03-06 17:51:00 PST
Created attachment 248124 [details] [IMAGE] Iterator Internal Properties
WebKit Commit Bot
Comment 5 2015-03-06 17:53:08 PST
Attachment 248121 [details] did not pass style-queue: ERROR: Source/JavaScriptCore/inspector/JSInjectedScriptHost.cpp:424: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/inspector/JSInjectedScriptHost.cpp:432: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] Total errors found: 2 in 21 files If any of these errors are false positives, please file a bug against check-webkit-style.
Joseph Pecoraro
Comment 6 2015-03-06 18:32:55 PST
Created attachment 248129 [details] [PATCH] Proposed Fix
WebKit Commit Bot
Comment 7 2015-03-06 20:03:19 PST
Comment on attachment 248129 [details] [PATCH] Proposed Fix Clearing flags on attachment: 248129 Committed r181203: <http://trac.webkit.org/changeset/181203>
WebKit Commit Bot
Comment 8 2015-03-06 20:03:24 PST
All reviewed patches have been landed. Closing bug.
Joseph Pecoraro
Comment 9 2015-03-09 20:23:09 PDT
*** Bug 141041 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.