Can GC during timed portion of TPS test

RESOLVED FIXED in Firefox 48

Status

RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: billm, Assigned: billm)

Tracking

Version 3
mozilla48
Points:
---

Firefox Tracking Flags

(firefox48 fixed)

Details

Attachments

(1 attachment)

Created attachment 8727204 [details] [diff] [review]
patch

Here's what happens:
- We get multiple notifications that a few pages, like aljazeera and the guardian, have loaded. The first notification is actually an error (not sure why this happens) while the second is real.
- We load the frame script into these tabs twice.
- When we ask these tabs to GC, they do it twice and send back two acknowledgements.
- After the first acknowledgement, we start the timed portion of the test even though the second GC is still happening.

This patch ignores the notifications that have non-successful status.
Attachment #8727204 - Flags: review?(mconley)
Comment on attachment 8727204 [details] [diff] [review]
patch

Thank you for figuring that out!
Attachment #8727204 - Flags: review?(mconley) → review+
Because of signing requirements, we'll need to repack and sign the tabswitcher addon. Ni'ing myself for instructions.
Flags: needinfo?(mconley)
So tabswitcher will need to be signed in order to run in automation:

https://wiki.mozilla.org/EngineeringProductivity/HowTo/SignExtensions

You'll need jpm 1.0.6 to do this. Let me know if you have any questions or hit any roadblocks.
Flags: needinfo?(mconley)
Looks like this never landed. I'll do that now.
Blocks: 1186585
https://hg.mozilla.org/integration/fx-team/rev/ef7006351314c812fe04d08b5ab49c66e49cfcc8
Bug 1253961 - Load talos-content.js framescript only once in tabswitch browsers to avoid GC's during timed portion of Talos test. r=mconley

Comment 6

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/ef7006351314
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox48: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in before you can comment on or make changes to this bug.