Closed Bug 1132713 Opened 5 years ago Closed 5 years ago

Addon Debugger failures from new performance tool

Categories

(DevTools :: Performance Tools (Profiler/Timeline), defect)

37 Branch
x86
macOS
defect
Not set

Tracking

(firefox38 fixed)

RESOLVED FIXED
Firefox 38
Tracking Status
firefox38 --- fixed

People

(Reporter: jsantell, Assigned: jsantell)

References

Details

Attachments

(1 file)

Whole bunch of 'em on gum, related to lacking an actorID.

 252 INFO TEST-UNEXPECTED-FAIL | browser/devtools/debugger/test/browser_dbg_addon-console.js | A promise chain failed to handle a rejection: - at resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1123 - Error: Can't manage front without an actor ID.

    1005274 Intermittent browser_dbg_addon-console.js | Test timed out followed by 30+ more failures

312 INFO TEST-UNEXPECTED-FAIL | browser/devtools/debugger/test/browser_dbg_addon-modules-unpacked.js | A promise chain failed to handle a rejection: - at resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1123 - Error: Can't manage front without an actor ID.

    1128315 Intermittent browser_dbg_addon-modules-unpacked.js | application crashed [@ nsPurpleBuffer::Block::VisitEntries<RemoveSkippableVisitor>(nsPurpleBuffer&, RemoveSkippableVisitor&)]

379 INFO TEST-UNEXPECTED-FAIL | browser/devtools/debugger/test/browser_dbg_addon-modules.js | A promise chain failed to handle a rejection: - at resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1123 - Error: Can't manage front without an actor ID.
414 INFO TEST-UNEXPECTED-FAIL | browser/devtools/debugger/test/browser_dbg_addon-panels.js | A promise chain failed to handle a rejection: - at resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1123 - Error: Can't manage front without an actor ID.
450 INFO TEST-UNEXPECTED-FAIL | browser/devtools/debugger/test/browser_dbg_addon-sources.js | A promise chain failed to handle a rejection: - at resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1123 - Error: Can't manage front without an actor ID. 
 252 INFO TEST-UNEXPECTED-FAIL | browser/devtools/debugger/test/browser_dbg_addon-console.js | A promise chain failed to handle a rejection: - at resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1123 - Error: Can't manage front without an actor ID.

    1005274 Intermittent browser_dbg_addon-console.js | Test timed out followed by 30+ more failures

312 INFO TEST-UNEXPECTED-FAIL | browser/devtools/debugger/test/browser_dbg_addon-modules-unpacked.js | A promise chain failed to handle a rejection: - at resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1123 - Error: Can't manage front without an actor ID.

    1128315 Intermittent browser_dbg_addon-modules-unpacked.js | application crashed [@ nsPurpleBuffer::Block::VisitEntries<RemoveSkippableVisitor>(nsPurpleBuffer&, RemoveSkippableVisitor&)]

379 INFO TEST-UNEXPECTED-FAIL | browser/devtools/debugger/test/browser_dbg_addon-modules.js | A promise chain failed to handle a rejection: - at resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1123 - Error: Can't manage front without an actor ID.
414 INFO TEST-UNEXPECTED-FAIL | browser/devtools/debugger/test/browser_dbg_addon-panels.js | A promise chain failed to handle a rejection: - at resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1123 - Error: Can't manage front without an actor ID.
450 INFO TEST-UNEXPECTED-FAIL | browser/devtools/debugger/test/browser_dbg_addon-sources.js | A promise chain failed to handle a rejection: - at resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js:1123 - Error: Can't manage front without an actor ID.
Gonna depend this on bug 1130202.

We need to disable profiler for addons here -- the global profiler is kicking in, running the SharedPerformanceFront, attempting to connect to memory actor which does not exist in this target, but the faulty compatibility test is passing? Either way, looking at this more after bug 1130202 lands.
Depends on: 1130202
Also only fails if we run the browser_dbg_aaa_leak_test before any of the addon tests.
This was due to the global profiler kicking in on the addon debugger context, where no profiler/timeline/memory actors exist. While the performance tool does not show for the addon debugger (because profiler actor does not exist), we still attempted to connect to the timeline/memory actors from the global profiler (in gDevTools). The changes in compatibility now works, as we test for both Gecko 38+ features, AND the memory actor existing.

 https://treeherder.mozilla.org/#/jobs?repo=try&revision=bfadeb605a78
Assignee: nobody → jsantell
Status: NEW → ASSIGNED
Attachment #8564321 - Flags: review?(vporof)
FYI tests were failing locally on my gum-ified branch, and now are passing with this patch, so should fix all those debugger issues on gum
Comment on attachment 8564321 [details] [diff] [review]
1132713-addon-debugger-fix.patch

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

Yay less oranges
Attachment #8564321 - Flags: review?(vporof) → review+
https://hg.mozilla.org/mozilla-central/rev/82abcbb019d7
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 38
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.