Closed Bug 917331 Opened 6 years ago Closed 6 years ago

AdbController doesn't exists on b2g desktop and introduce exceptions

Categories

(Firefox OS Graveyard :: General, defect)

defect
Not set

Tracking

(firefox25 wontfix, firefox26 fixed, firefox27 fixed, b2g-v1.2 fixed)

RESOLVED FIXED
Tracking Status
firefox25 --- wontfix
firefox26 --- fixed
firefox27 --- fixed
b2g-v1.2 --- fixed

People

(Reporter: ochameau, Assigned: ochameau)

References

Details

(Whiteboard: [needs-coverage])

Attachments

(1 file)

AdbController is only defined on Gonk, whereas it is called from here, also on b2g desktop:
http://mxr.mozilla.org/mozilla-central/source/b2g/chrome/content/shell.js#1051
1051       DebuggerServer.onConnectionChange = function(what) {
1052         AdbController.updateState();
1053       }
1054     }

We shouldn't have this code on desktop or implement an empty AdbController implementation.
Assignee: nobody → poirot.alex
Blocks: 810092
Attachment #806020 - Flags: review?(fabrice)
Blocks: 915258
Comment on attachment 806020 [details] [diff] [review]
Fix exception on b2g desktop about undefined AdbController object r=fabrice

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

thanks, not sure why I didn't see that when testing it...
Attachment #806020 - Flags: review?(fabrice) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/c61c7d12e09a
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Comment on attachment 806020 [details] [diff] [review]
Fix exception on b2g desktop about undefined AdbController object r=fabrice

[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 810092
User impact if declined: b2g desktop remote debugging won't work. as b2g desktop is used for the simulator, we won't be able to debug/install apps on 1.2 simulations
Testing completed (on m-c, etc.): baked on m-c, tested with custom 1.2 builds for the simulator.
Risk to taking this patch (and alternatives if risky): 
String or IDL/UUID changes made by this patch: none
Attachment #806020 - Flags: approval-mozilla-aurora?
Attachment #806020 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.