Status
People
(Reporter: nthomas, Assigned: rstrong)
Tracking
Firefox Tracking Flags
(Not tracked)
Details
Attachments
(1 attachment)
4.47 KB,
patch
|
mossop
:
review+
|
Details | Diff | Splinter Review |
I happened to inspect a windows 7 mini doing a mochitest-chrome run and it had the software update dialog up, wanting to restart Minefield to apply the update. Seems like this could be source of focus errors for tests that run in the browser, eg reftest. Could we disable the updater ? Eg at http://mxr.mozilla.org/mozilla-central/source/build/automation.py.in#292
![]() |
(Assignee) | |
Comment 1•8 years ago
|
||
There are a bunch of mochitest-chrome app update tests which is probably why you saw the UI. Also note that these tests have been very consistently passing and I highly doubt they would cause problems since the UI will only show when there is an update available which is extremely unlikely to happen on the tinderbox. Having said that I don't have a problem with disabling app update for test automation but I need to verify that the app update tests work properly with app update disabled just in case.
Assignee: nobody → robert.bugzilla
![]() |
(Assignee) | |
Comment 2•8 years ago
|
||
Created attachment 464344 [details] [diff] [review] patch
Attachment #464344 -
Flags: review?(dtownsend)
(Reporter) | ||
Comment 3•8 years ago
|
||
Hmm. I think I was looking at test jobs which had been running for several days, so they would have gotten offered a more recent nightly at some point. And normally that wouldn't happen with dep builds more recent than the nightly.
Updated•8 years ago
|
Attachment #464344 -
Flags: review?(dtownsend) → review+
Comment 4•8 years ago
|
||
Also a=me for landing
![]() |
(Assignee) | |
Comment 5•8 years ago
|
||
Pushed to mozilla-central http://hg.mozilla.org/mozilla-central/rev/c727df97c28c
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•