1079336 - Suggest updating ADB Helper if summonRoot is missing

Status: RESOLVED FIXED

(DevTools Graveyard :: WebIDE, defect)

Description

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Runtime details will says root is "unknown" if you don't have a new enough to ADB Helper to check it.

We should suggest updating the add-on to be more clear about the issue.

Comment 1

[Paul Rouget [:paul]]

Isn't the addon supposed to be auto-updated?

Comment 2

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

(In reply to Paul Rouget [:paul] (slow to respond. Ping me on IRC) from comment #1)
> Isn't the addon supposed to be auto-updated?

Yes, but people can disable updates for all add-ons or specific add-ons.

:mdas, what was your situation?  Did you have to do a manual add-on update because you had disabled auto updates, or was something else going on?

Comment 3

[Malini Das [:mdas] - Away, not checking bugmail]

It didn't auto-update and I didn't really look into it much (maybe a pref was turned off?), I just removed it and installed the new one.

Comment 4

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Okay, so we can't know for sure.  Regardless, there is no harm is saying something more helpful than "unknown".

Comment 5

[Anirudh GP(:anirudhgp)]

Hey I'd like to work on this bug. How do i begin??

Comment 6

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

In WebIDE's Runtime Info panel[1], there is a string we show if the user does not have a version of ADB Helper that supports the right methods.

At the very least, we should extend that string to suggest that the user check that they have 0.4.0 or later of the add-on.  We could also go a bit further and inspect the add-on metadata to see what version they do have to present a more specific message.

[1]: http://dxr.mozilla.org/mozilla-central/source/browser/devtools/webide/content/runtimedetails.js#102
[2]: http://dxr.mozilla.org/mozilla-central/source/browser/locales/en-US/chrome/browser/devtools/webide.properties#65

Comment 7

[Anirudh GP(:anirudhgp)]

Ok so instead of unknown, we can make it such that it checks what version of the add-on is currently present on the user's system and displays an appropriate message.  Correct?

Comment 8

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Yes, that sounds right.  The simplest version would be to just change the string to state that 0.4.0 or greater is needed.

Comment 9

[Yussif Barcelos]

Hi! I'm interested to fix one bug, but I'm new no bugzilla. I want to work in this bug and try to fix it until 30th october. Could someone mentor me?

Comment 10

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Yussif, you are welcome to give it a try.

To get started with our code base, see the hacking[1] page.

I believe the problem is well-defined in previous comments, but feel free to flag me as "needinfo" below if you have further questions.

[1]: https://wiki.mozilla.org/DevTools/Hacking

Comment 11

[Anirudh GP(:anirudhgp)]

(In reply to J. Ryan Stinnett [:jryans] from comment #8)
> Yes, that sounds right.  The simplest version would be to just change the
> string to state that 0.4.0 or greater is needed.

So we change "unknown" to "requires version 0.4.0 or greater" ?

Comment 12

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

(In reply to anirudh.gp from comment #11)
> (In reply to J. Ryan Stinnett [:jryans] from comment #8)
> > Yes, that sounds right.  The simplest version would be to just change the
> > string to state that 0.4.0 or greater is needed.
> 
> So we change "unknown" to "requires version 0.4.0 or greater" ?

Yes, something like that.  Perhaps "unknown (requires ADB Helper 0.4.0 or later)".

Comment 13

[Anirudh GP(:anirudhgp)]

Thanks :) I'll get on it as soon as possible.

Comment 14

[Anirudh GP(:anirudhgp)]

Can i still work on this bug?

Comment 15

[Anirudh GP(:anirudhgp)]

Attachment: bug-1079336-fix.patch

Comment 16

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8539227 [details] [diff] [review]
bug-1079336-fix.patch

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

Cool, this seems like the right idea.  I've noted something to fix below.

::: browser/locales/en-US/chrome/browser/devtools/webide.properties
@@ +64,5 @@
>  addons_status_installing=installing
>  
>  runtimedetails_checkno=no
>  runtimedetails_checkyes=yes
> +runtimedetails_checkunkown=unknown (requires ADB Helper 0.4.0 or later)

When we change the string value of an existing string, we need to modify the string ID in some way[1].

Sometimes we do this by adding a number to the end of the ID.  In this case, it looks like "unkown" at the end of the ID is a misspelling of "unknown", so you can take care of this by fixing the misspelling (and also updating the page where this string ID is used).

[1]: https://developer.mozilla.org/en-US/docs/Mozilla/Localization/Localization_content_best_practices#Updating_Entity_Names

Comment 17

[Anirudh GP(:anirudhgp)]

Attachment: bug-1079336-fix-v2.patch

Comment 18

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8539674 [details] [diff] [review]
bug-1079336-fix-v2.patch

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

Great, this looks good to me!

Pushed to try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=b483f1579be4

Add checkin-needed assuming Try looks good.

Comment 19

[Anirudh GP(:anirudhgp)]

Great. Thanks!

Comment 20

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/fx-team/rev/1646139d1efa

Comment 21

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/1646139d1efa