1362155 - Intermittent Linux "reftest | application timed out after 330 seconds with no output" due to default browser screen taking focus

Status: NEW

(Testing :: Reftest, defect)

Description

[Treeherder Bug Filer]

Filed by: rvandermeulen [at] mozilla.com

https://treeherder.mozilla.org/logviewer.html#?job_id=96683367&repo=mozilla-beta

https://queue.taskcluster.net/v1/task/ClEYMRfPTOePNOenLUibIg/runs/0/artifacts/public/logs/live_backing.log

https://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://queue.taskcluster.net/v1/task/ClEYMRfPTOePNOenLUibIg/runs/0/artifacts/public/logs/live_backing.log&only_show_unexpected=1

Comment 1

[Ryan VanderMeulen [:RyanVM]]

https://public-artifacts.taskcluster.net/ClEYMRfPTOePNOenLUibIg/0/public/test_info//mozilla-test-fail-screenshot_L8N_cK.png

I'm surprised because I thought we had a pref set for disabling this.

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

I see:
https://dxr.mozilla.org/mozilla-central/source/browser/components/nsBrowserGlue.js#1071

has this check:
      const skipDefaultBrowserCheck =
        Services.prefs.getBoolPref("browser.shell.skipDefaultBrowserCheckOnFirstRun") &&
        Services.prefs.getBoolPref("browser.shell.skipDefaultBrowserCheck");

and for release or beta we will be false:
https://dxr.mozilla.org/mozilla-central/source/browser/app/profile/firefox.js#237

I see this was modified in bug 1225648, lets see if :jaws might know more about this pref and what we should set in automation.

Comment 3

[Jared Wein [:jaws] (please needinfo? me)]

This was actually last changed by bug 1322723. Redirecting to dao since he worked on that bug.

Comment 4

[Dão Gottwald [:dao]]

The pref to set in automation is browser.shell.checkDefaultBrowser = false.