1315887 - Intermittent browser/base/content/test/urlbar/browser_locationBarExternalLoad.js | Test timed out -

Status: RESOLVED FIXED

Description

[Treeherder Bug Filer]

Filed by: philringnalda [at] gmail.com

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

https://queue.taskcluster.net/v1/task/J_xd_CM0QnGlaK9g1xL18w/runs/0/artifacts/public%2Flogs%2Flive_backing.log

Comment 1

[Intermittent Failures Robot]

1 failures in 889 pushes (0.001 failures/push) were associated with this bug in the last 7 days.    

Repository breakdown:
* mozilla-inbound: 1

Platform breakdown:
* osx-10-10: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1315887&startday=2017-12-04&endday=2017-12-10&tree=all

Comment 2

[Intermittent Failures Robot]

1 failures in 702 pushes (0.001 failures/push) were associated with this bug in the last 7 days.    

Repository breakdown:
* try: 1

Platform breakdown:
* windows10-64: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1315887&startday=2018-02-05&endday=2018-02-11&tree=all

Comment 4

[Kris Maglione [:kmag]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/8133cfb7904cb6ffda4ad3af0fa23000742bdc3a
Bug 1315887: Temporarilly disable browser_ext_locationBarExternalLoad for surge in intermittents. r=me

Comment 5

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/8133cfb7904c

Comment 6

[Kris Maglione [:kmag]]

Unfortunately, it looks like this test is a mess. I can't reproduce this issue locally without rr chaos mode, but when I can, we wind up never finishing the tab switch to the newly-created tab. We wind up waiting indefinitely to receive its layers, and the tab remains blank. (Well, the tab is supposed to remain blank, since the test loads about:blank. But I changed it to load a data: URL instead, and that had the same problem.)

Presumably this started happening after the removal of shims because the event shims send sync RPC messages to propagate events from the content window to the parent <browser>, which affects timings of a lot of things. But since those shims are only ever enabled for tests these days, it means that the timing issues may have been masked in test runs, but still show up in the wild.

I'm going to look into this some more to try to figure out what exactly is going on.

Comment 7

[Intermittent Failures Robot]

6 failures in 747 pushes (0.008 failures/push) were associated with this bug in the last 7 days.    

Repository breakdown:
* mozilla-inbound: 6

Platform breakdown:
* linux64: 3
* linux32: 2
* linux64-stylo-disabled: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1315887&startday=2018-03-05&endday=2018-03-11&tree=all

Comment 8

[Marco Bonardo [::mak]]

This is the only remaining skip-if = true test in urlbar, it would be nice to figure it out

Comment 9

[David Durst [:ddurst]]

Unassigning kmaglione. Mconley, are you a good candidate for this?

Comment 10

[Mike Conley (:mconley) (:⚙️)]

I looked at this yesterday using rr. I too was able to reproduce this using rr chaos mode.

I think I see what's going wrong. Here:

https://searchfox.org/mozilla-central/rev/adcc169dcf58c2e45ba65c4ed5661d666fc3ac74/browser/base/content/test/urlbar/browser_locationBarExternalLoad.js#48-50

We're calling loadFunc(), which might send messages to the content process to load a data: URI (in the same content process). Then, we use BrowserTestUtils.waitForContentEvent to send a message to add an event listener in the content process for pageshow.

That's race-y, because the loadFunc() messages might cause the content process to finish loading the data URI and fire pageshow before the pageshow event listener gets sent.

The fastest solution here, I think, is to make sure the pageshow event listener is set before we call loadFunc(). I've got a try push cooking with that solution here:

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

Comment 11

[Mike Conley (:mconley) (:⚙️)]

Attachment: Bug 1315887 - Re-enable browser_locationBarExternalLoad.js by making it less race-y. r?mak

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1b3f44ff852b
Re-enable browser_locationBarExternalLoad.js by making it less race-y. r=mak

Comment 13

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/1b3f44ff852b

Comment 14

[Mike Conley (:mconley) (:⚙️)]

Whoops - should have removed the leave-open.

Comment 15

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/355458563690