Bug 158315 – Web Inspector: Modernize some function expressions that bind(this) to arrow functions

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Joseph Pecoraro]

* SUMMARY
Modernize some function expressions that bind(this) to arrow functions

The common pattern:

    setTimeout(function() {
        ...
    }.bind(this), delay);

Can use arrow functions:

    setTimeout(() => {
        ...
    }, delay);

Switching from bind to arrow functions has multiples benefits:

  - it is faster (microbenchmarks)
  - it uses less memory (~252bytes -> 128bytes)
  - the code reads better in most cases

Things to be careful of:

  - if the inner function uses `arguments`, then converting to an arrow function won't work.

[Joseph Pecoraro]

Created attachment 280361 [details]
[PATCH] Proposed Fix

[Blaze Burg]

Comment on attachment 280361 [details]
[PATCH] Proposed Fix

r=me

[WebKit Commit Bot]

Comment on attachment 280361 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 280361

Committed r201620: <http://trac.webkit.org/changeset/201620>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.