WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
138563
The third parameter of a Map.prototype.forEach callback function is undefined
https://bugs.webkit.org/show_bug.cgi?id=138563
Summary
The third parameter of a Map.prototype.forEach callback function is undefined
Steve South
Reported
2014-11-10 05:09:12 PST
Codepen here:
http://codepen.io/stevesouth/pen/NPKzYz/
According to MDN the callback function of a Map.forEach should get called with three arguments callback is invoked with three arguments: the element value the element key the Map object being traversed
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Map/forEach
However in Webkit the third parameter is undefined.
Attachments
Patch
(6.55 KB, patch)
2014-12-07 07:23 PST
,
Diego Pino
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2014-11-11 11:52:34 PST
<
rdar://problem/18943661
>
Diego Pino
Comment 2
2014-12-07 07:23:15 PST
Created
attachment 242741
[details]
Patch
Diego Pino
Comment 3
2014-12-07 07:26:01 PST
The same change is necessary for Set. According to the ES6 spec [1]: "callbackfn is called with three arguments: the first two arguments are a value contained in the Set. The same value of passed for both arguments. The Set object being traversed is passed as the third argument." [1]
http://people.mozilla.org/~jorendorff/es6-draft.html#sec-set.prototype.foreach
Matthew Mirman
Comment 4
2015-06-09 12:56:08 PDT
Comment on
attachment 242741
[details]
Patch The map part was updated in
http://trac.webkit.org/changeset/183374
Matthew Mirman
Comment 5
2015-06-09 12:57:49 PDT
And the set part in
https://bugs.webkit.org/show_bug.cgi?id=144188
Geoffrey Garen
Comment 6
2015-06-09 13:05:31 PDT
Comment on
attachment 242741
[details]
Patch r=me
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug