Closed
Bug 952214
Opened 11 years ago
Closed 11 years ago
Covers DebuggerServer startup from shell.js with unit tests
Categories
(DevTools Graveyard :: WebIDE, defect)
DevTools Graveyard
WebIDE
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 29
People
(Reporter: ochameau, Assigned: ochameau)
Details
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
paul
:
review+
|
Details | Diff | Splinter Review |
We faced at least 2 major regressions this week that ended up breaking the app manager device support completely.
We used to face exception happening during actor instanciation due to some specific around actor loading on b2g and/or specific behavior of JSM on b2g.
It doesn't hit existing tests, as they all start DebuggerServer manually.
Ideally we would have a more functional test that would experience real world codepath for starting the debugger server.
We can easily do that by tweaking the existing webapps actor mochitest plain test script in order to trigger shell.js code instead of spawning our own server instance.
Assignee | ||
Comment 1•11 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=09172eea8c4d
First thing, this mochitest was disabled on b2g desktop...
Fix a small thing in specialpowers to get it working again.
Then, on b2g, I toggle the remote-enabled setting in order
to trigger b2g/settings.js, that itself calls shell.js:RemoteDebugger.start().
I have seen the test fail on styleeditor regression as my tree was outdated,
so it could have avoided such regression!
Now that would be worth looking if we can somehow test getAppActor request,
and see if we get working tab actors out of it!
Assignee | ||
Comment 2•11 years ago
|
||
The test fails on the emulator, but works fine locally on b2g desktop :(
14:08:19 INFO - 139041 INFO TEST-PASS | /tests/toolkit/devtools/apps/tests/test_webapps_actor.html | == TEST == Install packaged app
14:08:19 INFO - System JS : ERROR jar:file:///system/b2g/omni.ja!/components/SettingsService.js:79 - NS_ERROR_XPC_JS_THREW_STRING: 'DebuggerServer has not been initialized.' when calling method: [nsISettingsServiceCallback::handle]
14:08:19 INFO - 139042 ERROR TEST-UNEXPECTED-FAIL | /tests/toolkit/devtools/apps/tests/test_webapps_actor.html | Test timed out.
1
Sounds like a race between the chrome script and the setting toggle.
Assignee | ||
Comment 3•11 years ago
|
||
Attachment #8350201 -
Attachment is obsolete: true
Assignee | ||
Comment 4•11 years ago
|
||
Attachment #8356608 -
Attachment is obsolete: true
Assignee | ||
Comment 5•11 years ago
|
||
Comment on attachment 8357132 [details] [diff] [review]
Fixed exception on non-b2g envs
Try is now green!
Attachment #8357132 -
Flags: review?(paul)
Comment 6•11 years ago
|
||
Comment on attachment 8357132 [details] [diff] [review]
Fixed exception on non-b2g envs
This will need to be revisited once bug 942756 lands (I'll take care of that).
Attachment #8357132 -
Flags: review?(paul) → review+
Comment 7•11 years ago
|
||
checkin-needed?
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Comment 8•11 years ago
|
||
Comment 9•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 29
Updated•6 years ago
|
Product: Firefox → DevTools
Updated•5 years ago
|
Product: DevTools → DevTools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•