1133282 - Remove nonstandard expression closures from netwerk

Status: RESOLVED FIXED

(Core :: Networking, defect)

Description

[Chris Peterson [:cpeterson]]

Attachment: exclo-netwerk.patch

Expression closures (shorthand function syntax) are a nonstandard SpiderMonkey feature we would like to remove (in bug 1083459).

Comment 1

[Patrick McManus [:mcmanus]]

Comment on attachment 8564658 [details] [diff] [review]
exclo-netwerk.patch

Review of attachment 8564658 [details] [diff] [review]:
-----------------------------------------------------------------

thanks rs=mcmanus

Comment 2

[Dave Garrett]

Comment on attachment 8564658 [details] [diff] [review]
exclo-netwerk.patch

>+  _getParameters() {
>+    return {
>+      "%DATE%": function() { return Date.now().toString(); },
>+      "%PRODUCT%": function() { return gApp.name; },
>+      "%APP_ID%": function() { return gApp.ID; },
>+      "%APP_VERSION%": function() { return gApp.version; },
>+      "%BUILD_ID%": function() { return gApp.appBuildID; },
>+      "%OS%": function() { return gApp.OS; },
>+      "%CHANNEL%": function() { return UpdateChannel.get(); },
>+      "%DISTRIBUTION%": function() { return this._getPref(PREF_APP_DISTRIBUTION, ""); },
>+      "%DISTRIBUTION_VERSION%": function() { return this._getPref(PREF_APP_DISTRIBUTION_VERSION, ""); },
>+    };
>+  },
> 
>   _getUpdateURL: function() {
>     let url = this._getPref(PREF_UPDATES_URL, "");
>     let params = this._getParameters();
>     return url.replace(/%[A-Z_]+%/g, function(match) {
>       let param = params[match];
>       // preserve the %FOO% string (e.g. as an encoding) if it's not a valid parameter
>       return param ? encodeURIComponent(param()) : match;

_getParameters() is missing the colon & function keyword.

Comment 3

[Chris Peterson [:cpeterson]]

(In reply to Dave Garrett from comment #2)
> Comment on attachment 8564658 [details] [diff] [review]
> exclo-netwerk.patch
> 
> >+  _getParameters() {
> ...
> 
> _getParameters() is missing the colon & function keyword.

ES6 method syntax allows `: function` to be omitted in object literals:

https://github.com/google/traceur-compiler/wiki/LanguageFeatures#property-method-assignment

Comment 4

[Chris Peterson [:cpeterson]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/ad824f6cef5f

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/ad824f6cef5f