837001 - Disable remote debugger by default

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[Dave Hylands [:dhylands]]

So it turns out that Marionette needs remote-debugging to be disabled and adb to be enabled.

So when bug 836103 coupled them together, it wound up breaking marionette.

Marionette is enabled for non-user builds.
We want remote-debugging disabled by default for user builds.

So that means that remote-debugging should be disabled by default.

Bug 836973 will change the adb coupling so that adb is only coupled to the remote debugger setting in user builds, and adb will be always on when Marionette is enabled.

This bug was filed separately from bug 836973 because it affects the gaia component. Bug 836973 affects the gecko component.

Comment 1

[Dave Hylands [:dhylands]]

We should land bug 836973 to the gecko component and branches before landing this bug to gaia.

If this bug lands before 836973 then adb will wind up being permanently off.

Comment 2

[Dave Hylands [:dhylands]]

Attachment: Disables remote debugging by default

NOTE: If blocking-basecamp+ is set, just land it for now.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: 
Testing completed: 
Risk to taking this patch (and alternatives if risky):

We want remote debugging disabled for user builds.
We want remote debugging disabled for marionette builds (non-user builds).

So disable remote debugging by default.

Comment 3

[Lukas Blakk [:lsblakk] use ?needinfo]

Blocking, so go ahead and uplift to v1-train and v1.0.0 when ready.

Comment 4

[Jonathan Griffin (:jgriffin)]

Marionette works again with this patch (along with other related patches which have already landed).

Comment 5

[Dave Hylands [:dhylands]]

https://github.com/mozilla-b2g/gaia/commit/b8b3de7cec0c3975066bdae067c861d1863b032a

Comment 6

[John Ford [:jhford] CET/CEST Berlin Time]

v1-train: 3c91c12c977753b2f6e936a41e2e8f7576103283
v1.0.0: b9b9c734744c2ca2143e8255c538bac6c1d31846

Comment 7

[Mila Davydova]

Verified as fixed.
Remote debugger is disabled by default.
Unagi device, (nightly) build 20130215070202
Dec 5th Kernel
Gecko  http://hg.mozilla.org/releases/mozilla-b2g18_v1_0_1/rev/a9e4f8912607
Gaia   21ba59d933c66024cb351c2379315301d5352e0c