This causes a crash if Object.keys is called on a plugin object, for example.
Created attachment 46126 [details] Proposed patch Patch needs to be adapted once the patch for https://bugs.webkit.org/show_bug.cgi?id=32242 is landed.
style-queue ran check-webkit-style on attachment 46126 [details] without any errors.
Comment on attachment 46126 [details] Proposed patch r=me, but this can't land until the other patch does.
Created attachment 46439 [details] Revised patch Yay, that other monster patch finally landed. :)
(In reply to comment #4) > Created an attachment (id=46439) [details] > Revised patch I didn't remove the unused mode parameter name. Style bot didn't complain (and check-webkit-style doesn't if I remove the name, either). Is it an unwritten style rule that the name should/should not be present if the argument is unused, or is it a dontcare?
Created attachment 46445 [details] Revised patch (remove unused parameter name) Ouch, with the unused parameter name, Safari didn't build for me (warnings treated as errors). Whereas on the Qt port, unused parameter warnings are just disabled. That's a bit odd.
Comment on attachment 46445 [details] Revised patch (remove unused parameter name) Since this patch includes a new regression test, it also needs to include the expected results for that test. The patch otherwise looks great!
(In reply to comment #7) > (From update of attachment 46445 [details]) > Since this patch includes a new regression test, it also needs to include the > expected results for that test. The patch otherwise looks great! The test will continue to say SUCCESS if all of it succeeds (including the new part), like before. But it now has a new potential FAILURE output. So the expected results don't need to be changed, unless you want me to change the test to have separate PASS output for each "part" of the test.
Comment on attachment 46445 [details] Revised patch (remove unused parameter name) Confused about Darin's comment (see my response in comment #8), setting review? again in hope of being advised before I do any more changes to the test.
(In reply to comment #8) > The test will continue to say SUCCESS if all of it succeeds (including the new > part), like before. But it now has a new potential FAILURE output. > So the expected results don't need to be changed, unless you want me to change > the test to have separate PASS output for each "part" of the test. OK. My mistake.
Comment on attachment 46445 [details] Revised patch (remove unused parameter name) Clearing flags on attachment: 46445 Committed r53285: <http://trac.webkit.org/changeset/53285>
All reviewed patches have been landed. Closing bug.