Bug 142420 - Web Inspector: ES6: Improved Support for Iterator Objects
Summary: Web Inspector: ES6: Improved Support for Iterator Objects
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Joseph Pecoraro
URL:
Keywords: InRadar
: 141041 (view as bug list)
Depends on:
Blocks:
 
Reported: 2015-03-06 17:43 PST by Joseph Pecoraro
Modified: 2015-03-09 20:23 PDT (History)
10 users (show)

See Also:


Attachments
[PATCH] Proposed Fix (34.02 KB, patch)
2015-03-06 17:50 PST, Joseph Pecoraro
no flags Details | Formatted Diff | Diff
[IMAGE] Iterator Previews (142.09 KB, image/png)
2015-03-06 17:50 PST, Joseph Pecoraro
no flags Details
[IMAGE] Iterator Internal Properties (199.41 KB, image/png)
2015-03-06 17:51 PST, Joseph Pecoraro
no flags Details
[PATCH] Proposed Fix (54.60 KB, patch)
2015-03-06 18:32 PST, Joseph Pecoraro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Joseph Pecoraro 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").
Comment 1 Radar WebKit Bug Importer 2015-03-06 17:43:42 PST
<rdar://problem/20080363>
Comment 2 Joseph Pecoraro 2015-03-06 17:50:03 PST
Created attachment 248121 [details]
[PATCH] Proposed Fix
Comment 3 Joseph Pecoraro 2015-03-06 17:50:28 PST
Created attachment 248122 [details]
[IMAGE] Iterator Previews
Comment 4 Joseph Pecoraro 2015-03-06 17:51:00 PST
Created attachment 248124 [details]
[IMAGE] Iterator Internal Properties
Comment 5 WebKit Commit Bot 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.
Comment 6 Joseph Pecoraro 2015-03-06 18:32:55 PST
Created attachment 248129 [details]
[PATCH] Proposed Fix
Comment 7 WebKit Commit Bot 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>
Comment 8 WebKit Commit Bot 2015-03-06 20:03:24 PST
All reviewed patches have been landed.  Closing bug.
Comment 9 Joseph Pecoraro 2015-03-09 20:23:09 PDT
*** Bug 141041 has been marked as a duplicate of this bug. ***