Closed Bug 1517769 Opened 2 years ago Closed 2 years ago
History sidebar doesn't need to reference "this
47 bytes, text/x-phabricator-request
|Details | Review|
Picked up in a Coverity scan, historySidebar.js is currently referencing: this.TelemetryStopwatch.finish("HISTORY_LASTVISITED_TREE_QUERY_TIME_MS"); it should just be: TelemetryStopwatch.finish("HISTORY_LASTVISITED_TREE_QUERY_TIME_MS"); TelemetryStopwatch is in the global scope and we don't reference it with `this` anywhere else.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/2482a109b0e2 History sidebar shouldn't reference TelemetryStopwatch via 'this'. r=mak
You need to log in before you can comment on or make changes to this bug.