1243706 - Let PerfMonitoring ride the trains

Status: RESOLVED FIXED

(Toolkit :: Performance Monitoring, defect)

Description

[David Teller [:Yoric] - still alive but not very active]

Let's release the API. We can work on the UX later.

Comment 1

[David Teller [:Yoric] - still alive but not very active]

Attachment: MozReview Request: Bug 1243706 - Release PerformanceStats/PerformanceWatcher;r?felipe

Review commit: https://reviewboard.mozilla.org/r/33401/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/33401/

Comment 2

[:Felipe Gomes (needinfo for replies!)]

Comment on attachment 8716192 [details]
MozReview Request: Bug 1243706 - Release PerformanceStats/PerformanceWatcher;r?felipe

https://reviewboard.mozilla.org/r/33401/#review30609

May be worth adding relnote to the bug? Does this need an intent to ship e-mail? I'm guessing no because it isn't exposed to the web

::: browser/components/nsBrowserGlue.js:545
(Diff revision 1)
>      if (AppConstants.NIGHTLY_BUILD) {
> -      AddonWatcher.init();
>        os.addObserver(this, AddonWatcher.TOPIC_SLOW_ADDON_DETECTED, false);
>      }

shouldn't this entire `if (NIGHLTY_BUILD)` block be removed as well?

Comment 3

[David Teller [:Yoric] - still alive but not very active]

https://reviewboard.mozilla.org/r/33401/#review30609

I don't think we need an intent to ship, but yes, relnotes is a good idea.

> shouldn't this entire `if (NIGHLTY_BUILD)` block be removed as well?

Actually, the idea is to ship without the annoying yellow bar. That one will only ship when UX has turned it into something usable.

Comment 4

[David Teller [:Yoric] - still alive but not very active]

Release Note Request (optional, but appreciated)
[Why is this notable]: People have been asking for something like about:performance for years.
[Suggested wording]: Power users can now open about:performance to find out what is slowing down their Firefox. The underlying API is open to add-on developers who wish to implement their own performance tracking mechanism.
[Links (documentation, blog post, etc)]: TBD

Comment 5

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 8716192 [details]
MozReview Request: Bug 1243706 - Release PerformanceStats/PerformanceWatcher;r?felipe

I sent this to Try on top of m-c tip along with the Trunk-as-Aurora simulation patch and the browser-chrome mochitests look angry.

https://treeherder.mozilla.org/logviewer.html#?job_id=16616307&repo=try

TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_339445.js | A promise chain failed to handle a rejection: - at resource://gre/modules/XPCOMUtils.jsm:230 - TypeError: Cc[aContract] is undefined
TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_354894_perwindowpb.js | A promise chain failed to handle a rejection: - at resource://gre/modules/AddonWatcher.jsm:74 - ReferenceError: PerformanceWatcher is not defined 

etc.

Comment 6

[David Teller [:Yoric] - still alive but not very active]

Comment on attachment 8716192 [details]
MozReview Request: Bug 1243706 - Release PerformanceStats/PerformanceWatcher;r?felipe

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/33401/diff/1-2/

Comment 7

[David Teller [:Yoric] - still alive but not very active]

Ok, found the issue.
Ryan, do you wish to re-run the Aurora simulation?

Comment 8

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/02b9f8313561

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/02b9f8313561

Comment 10

[Ryan VanderMeulen [:RyanVM]]

(In reply to David Rajchenbach-Teller [:Yoric] (please use "needinfo") from comment #7)
> Ok, found the issue.
> Ryan, do you wish to re-run the Aurora simulation?

Still broken.
https://treeherder.mozilla.org/logviewer.html#?job_id=16844709&repo=try

Comment 11

[Ryan VanderMeulen [:RyanVM]]

(In reply to Ryan VanderMeulen [:RyanVM] from comment #10)

This was taken care of in bug 1246277.

Comment 12

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Added to Fx 47 (Aurora) release notes