Closed Bug 1025726 Opened 10 years ago Closed 10 years ago

[Dolphin] adb unavailable on user build

Categories

(Firefox OS Graveyard :: General, defect)

All
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: pehrsons, Unassigned)

References

Details

I have just flashed a VARIANT=user build on Dolphin and enabled adb in the developer menu but adb is unavailable on the Dolphin:

$ adb devices
List of devices attached 
0123456789ABCDEF	unauthorized

$ adb shell
error: device unauthorized. Please check the confirmation dialog on your device.


Seems like adb is waiting for user authorization of the adb session. On Android you'd get a dialogue on the device to authorize adb, but Firefox OS doesn't provide the same functionality.

I believe we are controlling adb access by turning it off when a lock screen is active instead.
Blocks: dolphin
Kai-Zhen, what is the desired UX regarding remote debugging or adb on a user build? I.e., is it supposed to be available? Is it supposed to engage the user with a dialog that needs approval?

I take back what I said about Firefox OS not providing the dialog functionality in the description. I have seen it on other devices :)
Flags: needinfo?(kli)
This might actually be fixed for the dolphin already, but it'd be good to know how the UX is supposed to be anyway.
I think we should have a design to let the user know and confirm when remote debugging or adb on a user build. But I don't have a good idea now. I'll ni our UX for some suggestion.

Jenny, do you have any idea? How should we handle this requirement in our setting?
Flags: needinfo?(kli) → needinfo?(jelee)
Hi Kai-Zhen, it seems we need a dialog for confirmation. Do you have a reference screenshot on this? Thanks!
Flags: needinfo?(jelee)
I think there is a confirmation dialog already, but it's been patched away here for some reason. I was just curious on the current UX behavior, so I can verify that I see the correct thing.
This has now been fixed on dolphin. adb is available on user builds.
Status: UNCONFIRMED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.