Closed Bug 1024910 Opened 9 years ago Closed 9 years ago

Errors "A promise chain failed to handle a rejection" output in browser console when hovering over the rule-view


(DevTools :: Inspector, defect)

Not set


(Not tracked)

Firefox 37


(Reporter: pbro, Assigned: sjakthol)




(1 file, 1 obsolete file)

I don't have exact steps to reproduce and this issue seems rather random. It might even also occur on the computed-view, not sure.

- open the inspector on a page
- open the browser console (cmd+shift+J)
- move your mouse over the rule-view
- wait, repeat ...

Sometimes, not really as you move the mouse, but after a while, a series of "A promise chain failed to handle a rejection" errors will be output to the browser console.
They tend to come in groups, sometimes 3 at the same time, sometimes a lot more.
And, they tend to come completely asynchronously to any client-side action you might be doing in the rule-view.

However, they don't seem to occur with other panels, or if the devtools are closed, so there is definitely a link with the rule-view.

The full error is:

"A promise chain failed to handle a rejection.

Date: Fri Jun 13 2014 10:32:31 GMT+0200 (CEST)
Full Message: false"
I saw similar errors when using Network Monitor and I tracked the problem down to Tooltip._onBaseNodeMouseMove which does not catch rejections .isValidHoverTarget throws[1]. isValidHoverTarget rejects every time mouse moves over an element that should not trigger the tooltip.

I think these are same problems as tooltips are used in the rule-view (and few other places) too. STR with Network Monitor:
1. Open network monitor and browser console.
3. Navigate to a page that performs a few requests.
4. Move mouse over netmonitor items and wait a while.
5. Bunch of errors should appear to browser console.

Here's a try run:

Attachment #8534920 - Flags: review?(pbrosset)
Comment on attachment 8534920 [details] [diff] [review]

Review of attachment 8534920 [details] [diff] [review]:

Wow, good catch Sami! Thanks for fixing this, this has been really annoying for a long time.
The code changes look good to me, and try is green, so let's land this.
Attachment #8534920 - Flags: review?(pbrosset) → review+
Keywords: checkin-needed
Sami, can you change the commit message to end with "; r=pbrosset" ? Thanks!
Assignee: nobody → sjakthol
Keywords: checkin-needed
Attachment #8534920 - Attachment is obsolete: true
Attachment #8534935 - Flags: review+
Keywords: checkin-needed
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 37
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.