Intermittent devtools/client/netmonitor/test/browser_net_html-preview.js | Uncaught exception - at chrome://mochitests/content/browser/devtools/client/netmonitor/test/browser_net_html-preview.js:37 - TypeError: document.querySelector(...) is null

RESOLVED FIXED in Firefox 55

Status

()

Firefox
Developer Tools: Netmonitor
P1
normal
RESOLVED FIXED
7 months ago
7 months ago

People

(Reporter: Treeherder Bug Filer, Assigned: Locke Chen, Mentored)

Tracking

unspecified
Firefox 55
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

(Reporter)

Description

7 months ago
treeherder
Filed by: cbook [at] mozilla.com

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

https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-inbound-win64/1490867362/mozilla-inbound_win8_64_test-mochitest-e10s-devtools-chrome-5-bm110-tests1-windows-build212.txt.gz
@Locke: this seems to be related to bug 1350229 you have been working on.
Looks like we can remove the test.

Honza
Flags: needinfo?(locke12456)
Priority: -- → P1
(Assignee)

Comment 2

7 months ago
@Honza:Ok,I'll fix it. :)
Can you assign this issue to me?
Flags: needinfo?(locke12456) → needinfo?(odvarko)
Done!

Honza
Assignee: nobody → locke12456
Mentor: odvarko@gmail.com
Status: NEW → ASSIGNED
Flags: needinfo?(odvarko)
(Assignee)

Updated

7 months ago
Blocks: 1350229
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 6

7 months ago
mozreview-review
Comment on attachment 8853232 [details]
Bug 1352035 - Removed test case for preview-panel.

https://reviewboard.mozilla.org/r/125298/#review127902

You need to remove also:

* html_content-type-test-page.html
* const CONTENT_TYPE_URL = ... (in head.js)

Honza
Attachment #8853232 - Flags: review?(odvarko)
(Assignee)

Comment 7

7 months ago
Ok, I got it.
Thank you,@honza
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 10

7 months ago
mozreview-review
Comment on attachment 8853370 [details]
Bug 1352035 - Removed the dependency on the test case preview-panel settings.

https://reviewboard.mozilla.org/r/125488/#review128044

Thanks!

Honza
Attachment #8853370 - Flags: review?(odvarko) → review+
(Assignee)

Updated

7 months ago
Keywords: intermittent-failure → checkin-needed
Autoland can't push this because the first patch doesn't have sufficient review for landing.
Keywords: checkin-needed
(Assignee)

Comment 12

7 months ago
Sorry, I have no idea how to operate it. should I reset these commit and push again?
many thanks!
Flags: needinfo?(odvarko)

Comment 13

7 months ago
mozreview-review
Comment on attachment 8853232 [details]
Bug 1352035 - Removed test case for preview-panel.

https://reviewboard.mozilla.org/r/125298/#review128542
Attachment #8853232 - Flags: review+
(In reply to Locke Chen from comment #12)
> Sorry, I have no idea how to operate it. should I reset these commit and
> push again?
> many thanks!
No need to push again, I just R+ the first patch.

Honza
Flags: needinfo?(odvarko)
Keywords: checkin-needed

Comment 15

7 months ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/d5c4cf3eeb84
Removed test case for preview-panel. r=Honza
https://hg.mozilla.org/integration/autoland/rev/9cb3f1aee967
Removed the dependency on the test case preview-panel settings. r=Honza
Keywords: checkin-needed
(Assignee)

Comment 16

7 months ago
(In reply to Jan Honza Odvarko [:Honza] from comment #14)
> (In reply to Locke Chen from comment #12)
> > Sorry, I have no idea how to operate it. should I reset these commit and
> > push again?
> > many thanks!
> No need to push again, I just R+ the first patch.
> 
> Honza


So that's how it is. I understand.
Thank you.

Comment 17

7 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/d5c4cf3eeb84
https://hg.mozilla.org/mozilla-central/rev/9cb3f1aee967
Status: ASSIGNED → RESOLVED
Last Resolved: 7 months ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
You need to log in before you can comment on or make changes to this bug.