Closed Bug 1521094 Opened Last year Closed 5 months ago

Make slow script prompt work

Categories

(GeckoView :: General, enhancement, P2)

Unspecified
Android
enhancement

Tracking

(geckoview66 wontfix, firefox-esr60 wontfix, firefox66 wontfix, firefox67 wontfix, firefox68 wontfix, firefox69 wontfix, firefox70 fixed)

RESOLVED FIXED
mozilla70
Tracking Status
geckoview66 --- wontfix
firefox-esr60 --- wontfix
firefox66 --- wontfix
firefox67 --- wontfix
firefox68 --- wontfix
firefox69 --- wontfix
firefox70 --- fixed

People

(Reporter: snorp, Assigned: estirling)

References

(Blocks 1 open bug)

Details

(Whiteboard: [geckoview:fenix:p2])

Attachments

(1 file)

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

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

OS: All → Android
Priority: -- → P2
Whiteboard: [geckoview:fenix:p3]
Whiteboard: [geckoview:fenix:p3] → [geckoview:fenix:p2]
Assignee: nobody → estirling

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

Attachment #9085470 - Attachment description: Bug 1521094 - Basic API for handling nsIHangReport → Bug 1521094 - Basic API for handling nsIHangReport in GeckoView
Pushed by jwillcox@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8fc1742afe2c
Basic API for handling nsIHangReport in GeckoView r=geckoview-reviewers,snorp
Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70

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

You need to log in before you can comment on or make changes to this bug.