Closed Bug 1026396 Opened 10 years ago Closed 8 years ago

browser_ruleview_livepreview.js | A promise chain failed to handle a rejection at resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/shared/widgets/Tooltip.js:259 - A promise chain failed to handle a rejection

Tracking

(Not tracked)

Status:

RESOLVED WORKSFORME

People

(Reporter: pbro, Assigned: pbro)

References

(Blocks 1 open bug)

Details

Patrick Brosset <:pbro>

Assignee

Description

•

10 years ago

See the logs: https://tbpl.mozilla.org/php/getParsedLog.php?id=40626296&tree=Try&full=1#error81

Part of bug 1018184 that aims at removing all errors caused by unhandled rejected promises.

Full stack trace:

03:57:09  WARNING -  TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/styleinspector/test/browser_ruleview_livepreview.js | A promise chain failed to handle a rejection at resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/shared/widgets/Tooltip.js:259 - A promise chain failed to handle a rejection: TypeError: this.panel is null
03:57:09     INFO -  Stack trace:
03:57:09     INFO -      Tooltip.prototype.isShown@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/shared/widgets/Tooltip.js:259:5
03:57:09     INFO -  TextPropertyEditor.prototype.editing@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/styleinspector/rule-view.js:2004:1
03:57:09     INFO -  TextPropertyEditor.prototype.update@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/styleinspector/rule-view.js:2203:5
03:57:09     INFO -  TextProperty.prototype.updateEditor@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/styleinspector/rule-view.js:952:7
03:57:09     INFO -  ElementStyle.prototype.markOverridden@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/styleinspector/rule-view.js:361:9
03:57:09     INFO -  ElementStyle.prototype.markOverriddenAll@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/styleinspector/rule-view.js:284:5
03:57:09     INFO -  Rule.prototype.applyProperties/promise<@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/styleinspector/rule-view.js:621:7
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:40:11
03:57:09     INFO -  then@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:20:43
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:72:11
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:40:11
03:57:09     INFO -  then@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:20:43
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:72:11
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:40:11
03:57:09     INFO -  then@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:20:43
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:72:11
03:57:09     INFO -  Front<.onPacket@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1112:7
03:57:09     INFO -  DebuggerClient.prototype.onPacket/<@resource://gre/modules/devtools/dbg-client.jsm:863:9
03:57:09     INFO -  resolve@resource://gre/modules/devtools/deprecated-sync-thenables.js:40:11
03:57:09     INFO -  then@resource://gre/modules/devtools/deprecated-sync-thenables.js:20:43
03:57:09     INFO -  then@resource://gre/modules/devtools/deprecated-sync-thenables.js:58:9
03:57:09     INFO -  DebuggerClient.prototype.onPacket@resource://gre/modules/devtools/dbg-client.jsm:905:1
03:57:09     INFO -  LocalDebuggerTransport.prototype.send/<@resource://gre/modules/devtools/dbg-client.jsm -> resource://gre/modules/devtools/transport/transport.js:540:11
03:57:09     INFO -  makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:84:7
03:57:09     INFO -  console.error:
03:57:09     INFO -    Message: Error: Connection closed
03:57:09     INFO -    Stack:
03:57:09     INFO -      Front<.destroy@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1040:7
03:57:09     INFO -  exports.WalkerFront<.destroy@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/inspector.js:2134:5
03:57:09     INFO -  frontProto/</proto[name]/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1223:11
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:40:11
03:57:09     INFO -  then@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:20:43
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:72:11
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:40:11
03:57:09     INFO -  then@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:20:43
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:72:11
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:40:11
03:57:09     INFO -  then@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:20:43
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:72:11
03:57:09     INFO -  Front<.onPacket@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1112:7
03:57:09     INFO -  DebuggerClient.prototype.onPacket/<@resource://gre/modules/devtools/dbg-client.jsm:863:9
03:57:09     INFO -  resolve@resource://gre/modules/devtools/deprecated-sync-thenables.js:40:11
03:57:09     INFO -  then@resource://gre/modules/devtools/deprecated-sync-thenables.js:20:43
03:57:09     INFO -  then@resource://gre/modules/devtools/deprecated-sync-thenables.js:58:9
03:57:09     INFO -  DebuggerClient.prototype.onPacket@resource://gre/modules/devtools/dbg-client.jsm:905:1
03:57:09     INFO -  LocalDebuggerTransport.prototype.send/<@resource://gre/modules/devtools/dbg-client.jsm -> resource://gre/modules/devtools/transport/transport.js:540:11
03:57:09     INFO -  makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:84:7
03:57:09     INFO -  console.error:
03:57:09     INFO -    Message: Error: Connection closed
03:57:09     INFO -    Stack:
03:57:09     INFO -      Front<.destroy@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1040:7
03:57:09     INFO -  exports.WalkerFront<.destroy@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/inspector.js:2134:5
03:57:09     INFO -  frontProto/</proto[name]/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1223:11
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:40:11
03:57:09     INFO -  then@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:20:43
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:72:11
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:40:11
03:57:09     INFO -  then@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:20:43
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:72:11
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:40:11
03:57:09     INFO -  then@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:20:43
03:57:09     INFO -  resolve@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/deprecated-sync-thenables.js:72:11
03:57:09     INFO -  Front<.onPacket@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1112:7
03:57:09     INFO -  DebuggerClient.prototype.onPacket/<@resource://gre/modules/devtools/dbg-client.jsm:863:9
03:57:09     INFO -  resolve@resource://gre/modules/devtools/deprecated-sync-thenables.js:40:11
03:57:09     INFO -  then@resource://gre/modules/devtools/deprecated-sync-thenables.js:20:43
03:57:09     INFO -  then@resource://gre/modules/devtools/deprecated-sync-thenables.js:58:9
03:57:09     INFO -  DebuggerClient.prototype.onPacket@resource://gre/modules/devtools/dbg-client.jsm:905:1
03:57:09     INFO -  LocalDebuggerTransport.prototype.send/<@resource://gre/modules/devtools/dbg-client.jsm -> resource://gre/modules/devtools/transport/transport.js:540:11

Patrick Brosset <:pbro>

Assignee

Updated

•

10 years ago

Assignee: nobody → pbrosset

Blocks: 1018184

Nicolas Chevobbe [:nchevobbe]

Comment 1

•

8 years ago

devtools/styleinspector/test/browser_ruleview_livepreview.js does not exist anymore, but executing devtools/client/inspector/rules/test/browser_rules_livepreview.js does not output the unhandled promise error.

Do you still see this Patrick ?

Flags: needinfo?(pbrosset)

Priority: -- → P3

Patrick Brosset <:pbro>

Assignee

Comment 2

•

8 years ago

This probably got fixed when I worked on bug 1247263 and other related rule-view promise bugs in Q1.
Safe to close because we have something that detects unhandled rejections in tests now anyway,

Status: NEW → RESOLVED

Closed: 8 years ago

Flags: needinfo?(pbrosset)

Resolution: --- → WORKSFORME

BMO Automation

Updated

•

6 years ago

Product: Firefox → DevTools

You need to log in before you can comment on or make changes to this bug.

Bugzilla

Quick Search

browser_ruleview_livepreview.js | A promise chain failed to handle a rejection at resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/shared/widgets/Tooltip.js:259 - A promise chain failed to handle a rejection

Categories

(DevTools :: Inspector, defect, P3)

Tracking

(Not tracked)

People

(Reporter: pbro, Assigned: pbro)

References

(Blocks 1 open bug)

Details

Crash Data

Security

(public)

User Story

Description

Updated

Comment 1

Comment 2

Updated