1215937 - Remove use of expression closure from js/xpconnect/.

Status: RESOLVED FIXED

(Core :: XPConnect, defect)

Description

[Tooru Fujisawa [:arai]]

Just noticed that use in js/xpconnect/ can be removed now.


Need to replace non-standard expression closure with one of:
  * function declaration
  * function expression
  * arrow function
before fixing bug 1083458.

converting rules are following:
  * function declaration
    add `return` and braces
  * standalone named function expression
    add `return` and braces
  * function expression, contains `arguments`
    add `return` and braces
  * standalone anonymous function expression contans and receives `this` (Array.filter, bind, etc)
    convert to arrow function, and remove code passing |this|
  * standalone anonymous function expression contans no `this`
    convert to arrow function
  * property with anonymous function expression, contains `this`
    add `return` and braces
  * property with anonymous function expression, contains no `this`, short body
    convert to arrow function
  * property with anonymous function expression, contains no `this`, long body
    add `return` and braces
  * property with named function expression
    add `return` and braces
  * getter property
    add `return` and braces
  * setter property
    add braces

I'll post a patch shortly.

Comment 1

[Tooru Fujisawa [:arai]]

Attachment: Remove use of expression closure from js/xpconnect/.

Comment 2

[Tooru Fujisawa [:arai]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/944ba8373890a00700ea0596a70267d36c7180a7
Bug 1215937 - Remove use of expression closure from js/xpconnect/. r=bholley

Comment 3

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/944ba8373890