Closed Bug 1177730 Opened 5 years ago Closed 1 year ago

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

Categories

(DevTools :: Debugger, defect, P3)

defect

Tracking

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

RESOLVED FIXED
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)

No description provided.
This is hella-frequent. Any idea what landed late last week that might have caused it?
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) — 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) — 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: 5 years ago
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: RESOLVED → REOPENED
Resolution: FIXED → ---
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: 5 years ago5 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]