Closed
Bug 1303800
Opened 9 years ago
Closed 9 years ago
Beta devtools tests failing with browser_net_cause.js | A promise chain failed to handle a rejection: - at chrome://mochitests/content/browser/devtools/client/netmonitor/test/browser_net_cause.js:118 - TypeError: stacktrace[j] is undefined
Categories
(DevTools :: Netmonitor, defect)
DevTools
Netmonitor
Tracking
(firefox50 fixed, firefox51 fixed, firefox52 fixed)
RESOLVED
FIXED
Firefox 52
People
(Reporter: intermittent-bug-filer, Assigned: jsnajdr)
Details
(Keywords: intermittent-failure)
Attachments
(2 files)
|
2.54 KB,
patch
|
ochameau
:
review+
|
Details | Diff | Splinter Review |
|
1.37 KB,
patch
|
ochameau
:
review+
|
Details | Diff | Splinter Review |
Filed by: wkocher [at] mozilla.com
https://treeherder.mozilla.org/logviewer.html#?job_id=1608145&repo=mozilla-beta
https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-beta-win32-debug/1474292431/mozilla-beta_win7_vm-debug_test-mochitest-e10s-devtools-chrome-4-bm128-tests1-windows-build7.txt.gz
status-firefox50:
--- → affected
status-firefox51:
--- → unaffected
| Assignee | ||
Comment 2•9 years ago
|
||
The test depends on javascript.options.asyncstack being set to "true". It's indeed "true" in central and aurora, but isn't when defined(RELEASE_BUILD). Is RELEASE_BUILD defined both in mozilla-release and mozilla-beta?
I'll post a simple fix soon.
Assignee: nobody → jsnajdr
Flags: needinfo?(jsnajdr)
Comment 3•9 years ago
|
||
Thanks. Yes, RELEASE_BUILD is true for beta and release.
| Comment hidden (Intermittent Failures Robot) |
Updated•9 years ago
|
Severity: normal → blocker
| Assignee | ||
Comment 5•9 years ago
|
||
Version of the patch intended for mozilla-central - patch browser_net_cause.js and browser_net_frame.js to turn on javascript.options.asyncstack.
The patch for mozilla-beta will be slightly different.
Attachment #8792766 -
Flags: review?(poirot.alex)
| Assignee | ||
Comment 6•9 years ago
|
||
Version of the patch for mozilla-beta. Fixes only browser_net_cause.js, as browser_net_frame.js in only in aurora now. Also, properly rebased for mozilla-beta.
Attachment #8792770 -
Flags: review?(poirot.alex)
Updated•9 years ago
|
Attachment #8792766 -
Flags: review?(poirot.alex) → review+
Updated•9 years ago
|
Attachment #8792770 -
Flags: review?(poirot.alex) → review+
| Assignee | ||
Comment 7•9 years ago
|
||
Please checkin. One patch is for mozilla-central and mozilla-aurora, the second one for mozilla-beta. No approvals needed, everything is testonly.
Keywords: checkin-needed
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ca56b87ab094
Turn on javascript.options.asyncstack pref for Netmonitor tests that need it. r=ochameau
Keywords: checkin-needed
Comment 9•9 years ago
|
||
Updated•9 years ago
|
Comment 10•9 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox52:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
| Comment hidden (Intermittent Failures Robot) |
Updated•7 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•