Closed Bug 1177730 Opened 9 years ago Closed 6 years ago

Intermittent browser_dbg_promises-chrome-allocation-stack.js | application timed out after 330 seconds with no output

Categories

(DevTools :: Debugger, defect, P3)

Product:
DevTools
Component:
Debugger
Type:
defect

Tracking

(firefox40 unaffected, firefox41 disabled, firefox42 disabled, firefox43 disabled, firefox-esr31 unaffected, firefox-esr38 unaffected)

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
firefox40 --- unaffected
firefox41 --- disabled
firefox42 --- disabled
firefox43 --- disabled
firefox-esr31 --- unaffected
firefox-esr38 --- unaffected

People

(Reporter: philor, Unassigned)

References

Details

(Keywords: intermittent-failure, Whiteboard: [test disabled][leave open])

Attachments

(1 file, 3 obsolete files)

1177730-skipall.patch
(deleted), patch
Details | Diff | Splinter Review
No description provided.
This is hella-frequent. Any idea what landed late last week that might have caused it?
status-firefox40: --- → unaffected
status-firefox41: --- → affected
status-firefox-esr31: --- → unaffected
status-firefox-esr38: --- → unaffected
Flags: needinfo?(nfitzgerald)
Oh, derp.
Blocks: 1084525
Flags: needinfo?(nfitzgerald) → needinfo?(gabriel.luong)
Looking into this.
Assignee: nobody → gabriel.luong
Status: NEW → ASSIGNED
Flags: needinfo?(gabriel.luong)
(In reply to Gabriel Luong [:gl] from comment #31) > Looking into this. Maybe try doing requestLongerTimeout(2) (or even 4?) just to see if the test happens to be slow, and isn't timing out due to correctness failures.
Possible stack of interest: console.error: Error sending event: promises-settled console.error: Message: TypeError: this._navigationLifetimePool is null Stack: PromisesActor<._createObjectActorForPromise@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/promises.js:149:1 PromisesActor<.objectGrip@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/promises.js:181:12 createValueGrip@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/object.js:1836:1 PromisesActor<._createObjectActorForPromise/actor<.createValueGrip@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/promises.js:158:9 ObjectActor.prototype._createPromiseState@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/object.js:144:28 ObjectActor.prototype.grip@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/object.js:92:26 .write@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/promises.js:19:19 types.addArrayType/<.write/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:199:40 types.addArrayType/<.write@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:199:24 Arg<.write@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:450:12 Request<.write/str<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:619:1 Request<.write@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:617:15 Actor<._sendEvent@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:882:16 Actor<.initialize/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:866:11 emitOnObject@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:112:9 emit@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:89:38 PromisesActor<._makePromiseEventHandler/</<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/promises.js:227:11 makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:83:14 EventLoop.prototype.enter@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/script.js:339:5 ThreadActor.prototype._pushThreadPause@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/script.js:534:5 ThreadActor.prototype.onAttach@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/script.js:647:7 DSC_onPacket@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/main.js:1598:15 LocalDebuggerTransport.prototype.send/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/transport/transport.js:569:11 makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:83:14 makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:83:14
Attachment #8630187 - Flags: review?(nfitzgerald)
Comment on attachment 8630187 [details] [diff] [review] 1177730.patch Review of attachment 8630187 [details] [diff] [review]: ----------------------------------------------------------------- Are both of these changes required? It would be a shame to requestLongerTimeout(10) if just setting those hooks to undefined does the trick...
Attachment #8630187 - Flags: review?(nfitzgerald) → review+
(In reply to Nick Fitzgerald [:fitzgen][:nf] from comment #114) > Comment on attachment 8630187 [details] [diff] [review] > 1177730.patch > > Review of attachment 8630187 [details] [diff] [review]: > ----------------------------------------------------------------- > > Are both of these changes required? It would be a shame to > requestLongerTimeout(10) if just setting those hooks to undefined does the > trick... Try still doesn't look good.
Can we disable the test for now? This is a top failure at the moment.
Flags: needinfo?(gabriel.luong)
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #186) > Can we disable the test for now? This is a top failure at the moment. Yes let's do that
Flags: needinfo?(gabriel.luong)
Attached patch 1177730-skip.patch (obsolete) (deleted) — — Splinter Review
Let's just skip the test on linux for now.
Attachment #8630187 - Attachment is obsolete: true
Attachment #8631810 - Flags: review?(nfitzgerald)
Attached patch 1177730-skip.patch (obsolete) (deleted) — — Splinter Review
Attachment #8631810 - Attachment is obsolete: true
Attachment #8631810 - Flags: review?(nfitzgerald)
Attachment #8631813 - Flags: review?(nfitzgerald)
Attachment #8631813 - Flags: review?(nfitzgerald) → review+
https://hg.mozilla.org/mozilla-central/rev/6ca1e31ef5d3
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox42: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
In the future, please use commit messages that summarize what the patch is actually doing rather than just restating the problem being fixed. https://developer.mozilla.org/en-US/docs/Mercurial_FAQ#Commit_Message_Conventions https://hg.mozilla.org/releases/mozilla-aurora/rev/59b3423361bf
status-firefox41: affected → disabled
status-firefox42: fixed → disabled
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attached patch 1177730-skipall.patch (deleted) — — Splinter Review
Skip running browser_dbg_promises-chrome-allocation-stack.js on linux
Attachment #8631813 - Attachment is obsolete: true
Attachment #8633134 - Flags: review?(nfitzgerald)
See comment 239. I already landed the follow-up.
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #259) > See comment 239. I already landed the follow-up. Ah thanks!
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED
Attachment #8633134 - Flags: review?(nfitzgerald)
Skipped on Windows too. https://hg.mozilla.org/integration/fx-team/rev/c4acce49322e https://hg.mozilla.org/releases/mozilla-aurora/rev/17f7a59c1f4a
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Whiteboard: [test disabled on Linux and Windows][leave open]
ni? myself to skip this on OSX when I get into the office later
Flags: needinfo?(ryanvm)
status-firefox43: --- → disabled
Flags: needinfo?(ryanvm)
Whiteboard: [test disabled on Linux and Windows][leave open] → [test disabled][leave open]
Target Milestone: Firefox 42 → ---
Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3
Assignee: gl → nobody
Product: Firefox → DevTools
Status: REOPENED → RESOLVED
Closed: 9 years ago → 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: