Closed Bug 1804227 Opened 3 years ago Closed 3 years ago

Enable interaction measurements in nightly only

Categories

(Toolkit :: Places, task)

task

Tracking

()

RESOLVED FIXED
110 Branch
Tracking Status
firefox110 --- fixed

People

(Reporter: mossop, Assigned: mossop)

References

Details

Attachments

(1 file)

Turn on interaction measurements in nightly only (browser.places.interactions.enabled) for Firefox 110

Pushed by dtownsend@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/61dad1ad2a2d Enable interaction measurements in nightly only. r=Standard8

Backed out for causing mochitest failures in browser/base/content/test/performance/browser_startup_content.js

Backout link: https://hg.mozilla.org/integration/autoland/rev/ea43e6f742608b63b5ef8ba730c6e0108b55e168

Push with failures

Failure log

INFO - Buffered messages finished
[task 2022-12-20T09:34:33.507Z] 09:34:33     INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/performance/browser_startup_content.js | should have no unexpected modules loaded on content process startup - Got 2, expected +0
[task 2022-12-20T09:34:33.507Z] 09:34:33     INFO - Stack trace:
Flags: needinfo?(dtownsend)

This failure https://treeherder.mozilla.org/logviewer?job_id=400205710&repo=autoland has also turned perma from the above changes, TH link.

(In reply to Sandor Molnar from comment #3)

Backed out for causing mochitest failures in browser/base/content/test/performance/browser_startup_content.js

(In reply to Cosmin Sabou [:CosminS] from comment #4)

This failure https://treeherder.mozilla.org/logviewer?job_id=400205710&repo=autoland has also turned perma from the above changes, TH link.

Mark is there any chance you have a few moments to look at these? They look fairly straightforward. It would be really good to get this landed but unfortunately it looks like I'm going to be out sick the rest of the week.

Flags: needinfo?(dtownsend) → needinfo?(standard8)

I've hopefully fixed the test issues. I had a deep look at seeing if we can avoid loading the interactions modules in the content process early on, but I can't see a way to do that - the test is loading an empty page via http protocol, and that's a case where we do need the interactions modules loaded to potentially process what's happening. Hence, I don't think there's a way around it.

The other case was to handle browser being closed really quickly before they open.

Flags: needinfo?(standard8)
Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/70a7556819cb Enable interaction measurements in nightly only. r=Standard8
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 110 Branch
Regressions: 1807002

(In reply to Mark Banner (:standard8) from comment #6)

I've hopefully fixed the test issues. I had a deep look at seeing if we can avoid loading the interactions modules in the content process early on, but I can't see a way to do that - the test is loading an empty page via http protocol, and that's a case where we do need the interactions modules loaded to potentially process what's happening. Hence, I don't think there's a way around it.

The other case was to handle browser being closed really quickly before they open.

Thanks Mark!

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

Attachment

General

Created:
Updated:
Size: