Closed Bug 1053164 Opened 10 years ago Closed 10 years ago

Exceptions are trapped during actors loading

Categories

(DevTools :: Framework, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 34

People

(Reporter: ochameau, Assigned: ochameau)

Details

Attachments

(1 file)

While I was working on making the actors being lazy loaded, I realized a pretty significant codepath like DebuggerServer.addBrowserActors() have its exception being trapped and ignored by promises. No output in stdout nor the js console.
Attached patch patchSplinter Review
Assignee: nobody → poirot.alex
Attachment #8472291 - Flags: review?(jryans)
Comment on attachment 8472291 [details] [diff] [review]
patch

Review of attachment 8472291 [details] [diff] [review]:
-----------------------------------------------------------------

Good catch!  I bet you could find many more issues just like this one if you look hard enough... :/
Attachment #8472291 - Flags: review?(jryans) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/a855508f7cbc
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 34
QA Whiteboard: [qa-]
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: