1713902 - Tests skipped in 'browser/extensions/formautofill/test/browser/browser.ini' for new Fission platform triage

Status: RESOLVED FIXED

(Toolkit :: Form Autofill, task)

Description

[Andrew Halberstadt [:ahal]]

The following tests were disabled in browser/extensions/formautofill/test/browser/browser.ini due to new Fission platform triage:
browser_remoteiframe.js

Full diff:

diff --git a/browser/extensions/formautofill/test/browser/browser.ini b/browser/extensions/formautofill/test/browser/browser.ini
--- a/browser/extensions/formautofill/test/browser/browser.ini
+++ b/browser/extensions/formautofill/test/browser/browser.ini
@@ -21,12 +21,13 @@ skip-if = verify || (!debug && os == "ma
 skip-if = verify || (!debug && os == "mac") # perma-fail see Bug 1600059
 [browser_manageAddressesDialog.js]
 skip-if = !debug && os == "mac" # perma-fail see Bug 1600059
 [browser_remoteiframe.js]
 skip-if =
   !debug && os == "mac" # perma-fail see Bug 1600059
   os == 'linux' && (debug || asan || tsan) #Bug 1701078
   os == 'win' && bits == 64 && asan #Bug 1701078
+  fission && os == "mac" && debug  # New platform triage
 [browser_submission_in_private_mode.js]
 skip-if = !debug && os == "mac" # perma-fail see Bug 1600059
 [browser_update_doorhanger.js]
 skip-if = true # bug 1426981 # Bug 1445538

See the following try push for failures:
https://treeherder.mozilla.org/jobs?repo=try&revision=ad7a18fdad3b8e35b44152c16d94babb80387cd8

To run these failures in your own try push, first revert the skip-if
annotations, then run:

$ ./mach try fuzzy --rebuild 3 browser/extensions/formautofill/test/browser/browser.ini

Then use the fuzzy interface to select the task(s) which are relevant to the
removed skip-if expression(s). Please note the failure may be intermittent
and/or been fixed since the triage took place.

Comment 1

[Chris Peterson [:cpeterson]]

We need to determine whether this browser-chrome test failure should block shipping Fission. Tracking for Fission Milestone M7a for the initial audit. Then we can decide whether to move this bug to Fission M8 or post-MVP (Fission Future).

Here is a recent Try run for Fission browser-chrome:

https://treeherder.mozilla.org/jobs?repo=try&revision=ad7a18fdad3b8e35b44152c16d94babb80387cd8

Comment 2

[Nika Layzell [:nika] (ni? for response)]

This test failure appears to be very flakey already based on the other situations it's skipped in. ni? Neil as this does seem to be a test specifically written to test Fission.

Comment 3

[Neil Deakin]

Hopefully the patch in bug 1600059, which landed today,will help here.

Comment 4

[Neha Kochar [:neha]]

Neil, is this ok to be re-enabled now? Can you do it, please?

Comment 5

[Neil Deakin]

We could probably re-enable browser_remoteiframe.js. This failure bug for this is bug 1701078 which I think is fixed now.

Comment 6

[Neha Kochar [:neha]]

Re-enabled in bug 1701078.