Closed
Bug 1450170
Opened 6 years ago
Closed 6 years ago
Add a Nightly-only observer notification that fires every time a tab switch spinner is displayed
Categories
(Firefox :: Tabbed Browser, enhancement)
Firefox
Tabbed Browser
Tracking
()
RESOLVED
FIXED
Firefox 61
Tracking | Status | |
---|---|---|
firefox61 | --- | fixed |
People
(Reporter: mconley, Assigned: mconley)
References
Details
Attachments
(1 file)
I'm hoping to use that along with this WebExtension: https://github.com/mikeconley/spindoctor To get a handle on where some of the new spinners in bug 1449662 are coming from.
Comment hidden (mozreview-request) |
Comment 2•6 years ago
|
||
mozreview-review |
Comment on attachment 8963847 [details] Bug 1450170 - Add Nightly-only observer notification that fires anytime a tab switch spinner is displayed. https://reviewboard.mozilla.org/r/232712/#review238148 Code analysis found 1 defect in this patch: - 1 defect found by mozlint You can run this analysis locally with: - `./mach lint path/to/file` (JS/Python) If you see a problem in this automated review, please report it here: http://bit.ly/2y9N9Vx ::: browser/modules/AsyncTabSwitcher.jsm:1037 (Diff revision 1) > this.addMarker("AsyncTabSwitch:SpinnerShown"); > Services.telemetry > .getHistogramById("FX_TAB_SWITCH_SPINNER_VISIBLE_TRIGGER") > .add(this._loadTimerClearedBy); > + if (AppConstants.NIGHTLY_BUILD) { > + Services.obs.notifyObservers(null, "tabswitch-spinner", null); Error: Notifyobservers's third parameter can be omitted. [eslint: mozilla/no-useless-parameters]
Comment hidden (mozreview-request) |
Comment 4•6 years ago
|
||
mozreview-review |
Comment on attachment 8963847 [details] Bug 1450170 - Add Nightly-only observer notification that fires anytime a tab switch spinner is displayed. https://reviewboard.mozilla.org/r/232712/#review238156 LGTM
Attachment #8963847 -
Flags: review?(dothayer) → review+
Assignee | ||
Comment 5•6 years ago
|
||
Thanks!
Pushed by mconley@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/45ec22b83237 Add Nightly-only observer notification that fires anytime a tab switch spinner is displayed. r=dthayer
Comment 7•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/45ec22b83237
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
Updated•6 years ago
|
Assignee: nobody → mconley
You need to log in
before you can comment on or make changes to this bug.
Description
•