Add remote add-ons to the list of targets on a remote device

VERIFIED FIXED in Firefox 31

Status

defect
VERIFIED FIXED
5 years ago
Last year

People

(Reporter: mossop, Assigned: mossop)

Tracking

(Blocks 1 bug)

unspecified
Firefox 31
Dependency tree / graph

Firefox Tracking Flags

(firefox31 verified)

Details

Attachments

(1 attachment)

The connection screen should list remote add-ons that can be debugged
wow. this would be something !!
Very simple but it's going to depend on bug 993029 landing first
Assignee: nobody → dtownsend+bugmail
Depends on: 993029
I was waiting for this one to be filed, it was just a matter of time :-) Thanks!
Posted patch patchSplinter Review
Fairly straightforward. Are there any tests for the connection screen?
Attachment #8405489 - Flags: review?(nfitzgerald)
Comment on attachment 8405489 [details] [diff] [review]
patch

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

Moving this along to Panos since I think he wrote the connection stuff. I haven't ever even looked at this code :-/
Attachment #8405489 - Flags: review?(nfitzgerald) → review?(past)
Comment on attachment 8405489 [details] [diff] [review]
patch

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

I didn't test Fennec, but I assume you have. Desktop works fine, except for the missing check for non-debuggable add-ons.

We don't have any tests for the remote connection screen, sadly. It also might go away at some point in the (distant) future, if we can turn the App Manager into a viable replacement for all use cases.

::: browser/devtools/framework/connect/connect.js
@@ +88,5 @@
> +  if (!response.error && response.addons.length > 0) {
> +    // Add one entry for each add-on.
> +    for (let addon of response.addons) {
> +      //if (!addon.isDebuggable)
> +      //  continue;

You probably meant to uncomment this and test for !addon.debuggable?
Attachment #8405489 - Flags: review?(past) → review+
https://hg.mozilla.org/mozilla-central/rev/67c1405cd110
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 31
Dave, what are the steps to do in order to verify this bug as fixed?

For Desktop, Past helped me to open "Console - about:addons" window by using two FF instances, starting the debugger server (with listen 6000 in Developer Toolbar) and then selecting Connect in chrome://browser/content/devtools/connect.xhtml. In this window I can debug the about:addons page, but not an individual add-on installed. How could I do this?

For Fennec, I can see the "Available remote add-ons:" option on latest Firefox beta available on Play Store. Selecting the add-on that I installed on device, opens a new window (Web Console like) containing only Toolbox Options and Debugger tabs. Should I check anything else here?

Thank you!
Flags: needinfo?(dtownsend+bugmail)
You have just verified it(In reply to Petruta Rasa [QA] [:petruta] from comment #9)
> Dave, what are the steps to do in order to verify this bug as fixed?
> 
> For Desktop, Past helped me to open "Console - about:addons" window by using
> two FF instances, starting the debugger server (with listen 6000 in
> Developer Toolbar) and then selecting Connect in
> chrome://browser/content/devtools/connect.xhtml. In this window I can debug
> the about:addons page, but not an individual add-on installed. How could I
> do this?

Did you have chrome debugging enabled in the Firefox you were connecting to?

> For Fennec, I can see the "Available remote add-ons:" option on latest
> Firefox beta available on Play Store. Selecting the add-on that I installed
> on device, opens a new window (Web Console like) containing only Toolbox
> Options and Debugger tabs. Should I check anything else here?

That's fine
Flags: needinfo?(dtownsend+bugmail)
Thanks! I managed to verify it on desktop side too using Firefox 31 RC under Win 7 64-bit, Ubuntu 13.04 32-bit and Mac OSX 10.9.4. The same window as on device was opened on the second Firefox instance containing the debugger of the add-on.
Status: RESOLVED → VERIFIED
Keywords: verifyme
QA Contact: petruta.rasa

Updated

Last year
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.