Support headless mode on Windows

RESOLVED FIXED in Firefox 56

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: bdahl, Assigned: bdahl)

Tracking

({dev-doc-complete})

unspecified
Firefox 56
dev-doc-complete
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox56 fixed)

Details

Attachments

(6 attachments)

Comment hidden (empty)
Component: General → Headless
This is the first part of the actual work. Overall, most of the patches are really small as most of the work was done already for linux.
Attachment #8876945 - Flags: review?(jmuizelaar)
This test is inconsistent across platforms, so I'm going to disable for now on headless. I filed bug 1356681 to re-enable and also see https://bugzilla.mozilla.org/show_bug.cgi?id=1335457#c1
Attachment #8876949 - Flags: review?(dburns)
Attachment #8876943 - Flags: review?(dburns) → review+
Attachment #8876949 - Flags: review?(dburns) → review+
Attachment #8876945 - Flags: review?(jmuizelaar) → review+
Attachment #8876946 - Flags: review?(jmuizelaar) → review+
Attachment #8876947 - Flags: review?(jmuizelaar) → review+
Attachment #8876948 - Flags: review?(jmuizelaar) → review+

Comment 7

2 years ago
Pushed by bdahl@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/8cd5e6e5d0f7
Enable headless mode on windows. r=automatedtester, r=jrmuizel

Comment 8

2 years ago
Pushed by kwierso@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/37f2e9416b73
Followup to disable headless tests on windows asan a=bustage r=bdahl
(In reply to Pulsebot from comment #8)
> Pushed by kwierso@gmail.com:
> https://hg.mozilla.org/integration/mozilla-inbound/rev/37f2e9416b73
> Followup to disable headless tests on windows asan a=bustage r=bdahl

17:22:38 <KWierso> bdahl: looks like your patch has failing tests
17:24:12 <bdahl> KWierso: windows asan?
17:24:17 <KWierso> yeah
17:26:27 <bdahl> KWierso: dang, forgot that one in my try run
17:26:43 <KWierso> it's tier2 so I could just file a bug for it
17:26:50 <KWierso> maybe hide it for now until you can sort it out
17:26:55 <bdahl> KWierso: actually that shouldn't be enabled, i don't think regular marionette tests are running on windows asan
17:27:10 <KWierso> I don't see any
17:27:39 <KWierso> so... drop https://hg.mozilla.org/integration/mozilla-inbound/rev/8cd5e6e5d0f71c18d928f76cf4312961145bb5af#l1.65 ?
17:29:47 <KWierso> bdahl: I can push that right now
17:31:48 <bdahl> KWierso: actually looks like marionette should be running on asan, but we could turn off headless for now on asan
17:32:24 <KWierso> bdahl: r=you? to drop it for now?
17:32:49 <bdahl> KWierso: yeah

Comment 10

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/8cd5e6e5d0f7
https://hg.mozilla.org/mozilla-central/rev/37f2e9416b73
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Keywords: dev-doc-needed
Assignee: nobody → bdahl
See https://bugzilla.mozilla.org/show_bug.cgi?id=1338004#c39 for docs.
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.