1177134 - ESLint cleanup, batch #1

Status: RESOLVED FIXED

(DevTools :: Console, defect)

Comment 1

[Jen Fong-Adwent [:ednapiranha]]

Attachment: Bug1177134.patch

First batch of tests. Let me know if anything should be adjusted.

Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=02acc9eacec5

Comment 2

[Patrick Brosset <:pbro>]

Comment on attachment 8626091 [details] [diff] [review]
Bug1177134.patch

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

Thanks for working on this Jen.
The changes look good to me. I haven't spent too much time looking at the details but they look mechanical enough, and try is green.
r=me

::: browser/devtools/webconsole/test/browser_bug_638949_copy_link_location.js
@@ +80,5 @@
>    info("expected clipboard value: " + message.url);
>  
>    let deferred = promise.defer();
>  
> +  waitForClipboard((aData) => {

2 minor remarks:
- while we're at it, let's rename aData to data (https://wiki.mozilla.org/DevTools/CodingStandards#Code_style)
- parenthesis can be removed around aData since it's only 1 argument.

::: browser/devtools/webconsole/test/browser_cached_messages.js
@@ +18,5 @@
>  
>    loadTab(TEST_URI).then(testOpenUI);
>  }
>  
> +function testOpenUI(aTestReopen) {

Same remark about aArgument here.

::: browser/devtools/webconsole/test/browser_console_error_source_click.js
@@ +17,5 @@
>    loadTab(TEST_URI).then(() => {
>      HUDService.toggleBrowserConsole().then(browserConsoleOpened);
>    });
>  
> +  function browserConsoleOpened(aHud) {

Since you're changing this line, it's a good opportunity to rename aHud to hud.
We probably shouldn't use this bug to get rid of *all* aArgument notations, but I guess if you're already changing lines that have some declarations in them, it's a good chance to do this for them.

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/b026f257192e

Comment 4

[Carsten Book [:Tomcat]]

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