1521094 - Make slow script prompt work

Status: RESOLVED FIXED

(GeckoView :: General, enhancement, P2)

Description

[James Willcox (:snorp) ([e-mail]) (he/him)]

We don't get slow script warnings in GeckoView when using e10s because it appears none of that is hooked up. We should be doing something like ProcessHangMonitor.jsm[0] in desktop.

[0] https://searchfox.org/mozilla-central/source/browser/modules/ProcessHangMonitor.jsm

Comment 1

[Chris Peterson [:cpeterson]]

We would need a new GV API so the app can show the slow script prompt. Fenix might want this.

Comment 2

[Elliot Stirling :estirling]

Attachment: Bug 1521094 - Basic API for handling nsIHangReport in GeckoView

Bug 1521094 - apiLint led changes.

Bug 1521094 - Add example alert dialog with wait/stop buttons

Bug 1521094 - Linter android-checkstyle change /reportId/mReportId/

Bug 1521094 - Cleanup unused message and logging/comments.

But 1521094 - Add preference assumption to test.

Bug 1521094 - Only handle nsIHangReports for our FrameLoader

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8fc1742afe2c
Basic API for handling nsIHangReport in GeckoView r=geckoview-reviewers,snorp

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/8fc1742afe2c

Comment 5

[Chris Peterson [:cpeterson]]

firefox69=wontfix because we don't need to uplift this slow script prompt to Beta.

Comment 6

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/releases/mozilla-beta/rev/8fc1742afe2c