1631802 - Intermittent browser/base/content/test/contextMenu/browser_contextmenu_loadblobinnewtab.js | Uncaught exception - TypeError: can't access property "href", content.document.getElementById(...) is null

Status: RESOLVED FIXED

(Firefox :: Menus, defect, P5)

Description

[Treeherder Bug Filer]

Filed by: cbrindusan [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=298627809&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/fTXext_PQsK3rrjUIcT1uw/runs/0/artifacts/public/logs/live_backing.log
Reftest URL: https://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/fTXext_PQsK3rrjUIcT1uw/runs/0/artifacts/public/logs/live_backing.log&only_show_unexpected=1

---

[task 2020-04-21T13:14:58.520Z] 13:14:58 INFO - TEST-START | browser/base/content/test/contextMenu/browser_contextmenu_loadblobinnewtab.js
[task 2020-04-21T13:14:58.568Z] 13:14:58 INFO - TEST-INFO | started process screencapture
[task 2020-04-21T13:14:58.725Z] 13:14:58 INFO - TEST-INFO | screencapture: exit 0
[task 2020-04-21T13:14:58.725Z] 13:14:58 INFO - Buffered messages logged at 13:14:58
[task 2020-04-21T13:14:58.726Z] 13:14:58 INFO - Entering test bound test_rightclick_open_bloburl_in_new_tab
[task 2020-04-21T13:14:58.726Z] 13:14:58 INFO - Buffered messages finished
[task 2020-04-21T13:14:58.726Z] 13:14:58 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/contextMenu/browser_contextmenu_loadblobinnewtab.js | Uncaught exception - TypeError: can't access property "href", content.document.getElementById(...) is null

Comment 1

[Cosmin Sabou [:CosminS]]

Moved onto tier 1: https://treeherder.mozilla.org/logviewer.html#?job_id=298660531&repo=autoland

Comment 2

[:Gijs (he/him)]

I expect https://hg.mozilla.org/integration/autoland/file/tip/browser/base/content/test/contextMenu/browser_contextmenu_loadblobinnewtab.js#l17 can fire when the page still has about:blank loaded, and the loaded promise should wait for the relevant document to have loaded by passing false as the second arg and the URL as the third arg to browserLoaded.

Subhamoy, can you update the test?

Comment 3

[:Gijs (he/him)]

(alternatively, use the withNewTab helper, which makes it easier to do this kind of thing)

Comment 4

[Subhamoy [:ssengupta][he/him]]

Attachment: Bug 1631802 - Browser mochitests revised to prevent intermittent failure r=baku,gijs

BrowserTestUtils.browserLoaded() sometimes resolves too early (already at about:blank) when used without explicit URL as 'wantLoad'.

The tests were added as a part of Bug 1626573.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/945dcd70abe9
Browser mochitests revised to prevent intermittent failure r=baku

Comment 6

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/945dcd70abe9

Comment 7

[Intermittent Failures Robot]

22 failures in 5582 pushes (0.004 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• autoland: 22

Platform breakdown:

• macosx1014-64: 10
• macosx1014-64-shippable: 3
• linux1804-64: 3
• windows10-64-qr: 3
• linux1804-64-asan: 2
• windows7-32-shippable: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1631802&startday=2020-04-20&endday=2020-04-26&tree=all