Closed Bug 1517769 Opened 8 months ago Closed 8 months 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 email@example.com: 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.