1354855 - Intermittent browser/base/content/test/appUpdate/browser_updatesPartialPatchWithBadPartialSize.js | The right notification showed up. - Got update-manual, expected update-available

Status: RESOLVED FIXED

Description

[Treeherder Bug Filer]

Filed by: philringnalda [at] gmail.com

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

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

Comment 1

[Robert Strong (Robert he/him) [:rstrong] (use needinfo to contact me)]

Created attachment 8857343 [details]
relevant part of the log file

Comment 2

[Doug Thayer [:dthayer]]

Created attachment 8857656 [details]
Bug 1354855 - Set update timer prefs in new update tests

In the log file attached to this bug, we see multiple requests for
background updates come in at the same time. I can't see a way for
this to happen other than notify being called while the test is
running.

Review commit: https://reviewboard.mozilla.org/r/129606/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/129606/

Comment 3

[Robert Strong (Robert he/him) [:rstrong] (use needinfo to contact me)]

Comment on attachment 8857656 [details]
Bug 1354855 - Set update timer prefs in new update tests

https://reviewboard.mozilla.org/r/129606/#review132198

Comment 4

[Robert Strong (Robert he/him) [:rstrong] (use needinfo to contact me)]

Comment on attachment 8857656 [details]
Bug 1354855 - Set update timer prefs in new update tests

https://reviewboard.mozilla.org/r/129606/#review132200

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/27cee08fb420
Set update timer prefs in new update tests r=rstrong

Comment 6

[Wes Kocher (:KWierso)]

https://hg.mozilla.org/mozilla-central/rev/27cee08fb420