1288830 - Start running ESLint on some of the performance tool

Status: RESOLVED FIXED

(DevTools :: Performance Tools (Profiler/Timeline), defect, P1)

Description

[Greg Tatum [:gregtatum]]

Begin the process of converting the performance tool over to using ESLint.

Comment 1

[Greg Tatum [:gregtatum]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=06fd3fce2c3e

Comment 2

[Greg Tatum [:gregtatum]]

Attachment: Start to lint parts of the performance tool

This should be fairly straight forward. The only scary thing I did was remove
`const { Cc, Ci, Cu, Cr } = require("chrome");` from `devtools/client/performance/panel.js`
as it didn't appear to be used, or in the same global scope as the other files.

For the most part I kept all of the lint comments at the beginning of the files, except
for performance controller which had a large amount. I kept those near to the declarations
so they'll be easier to (eventually) clean-up.

Comment 3

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8773939 [details] [diff] [review]
Start to lint parts of the performance tool

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

::: devtools/client/performance/performance-controller.js
@@ +2,5 @@
>   * License, v. 2.0. If a copy of the MPL was not distributed with this file,
>   * You can obtain one at http://mozilla.org/MPL/2.0/. */
>  "use strict";
>  
> +/* globals document, PerformanceView, ToolbarView, RecordingsView, DetailsView */

You may find it simpler to use the import-globals-from <FILE> syntax, such as this[1] example:

/* import-globals-from ../../debugger-controller.js */

[1]: https://dxr.mozilla.org/mozilla-central/rev/4c05938a64a7fde3ac2d7f4493aee1c5f2ad8a0a/devtools/client/debugger/content/views/event-listeners-view.js#7

Comment 4

[Jordan Santell [:jsantell] [@jsantell]]

Comment on attachment 8773939 [details] [diff] [review]
Start to lint parts of the performance tool

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

lgtm

Comment 5

[Greg Tatum [:gregtatum]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=0a8dffdd6e15

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/fx-team/rev/2d23f317a203
Start to lint parts of the performance tool. r=jsantell

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/2d23f317a203