On 10 Feb 2014, at 21:44, Malini Das <[email protected]> wrote:

> On 2/10/2014, 9:39 AM, Chris Mills wrote:
>> Hi all,
>> 
>> Could I get someone to do me a quick technical review on the following 
>> article set?
>> 
>> https://developer.mozilla.org/en-US/Firefox_OS/Security/Debugging_and_security_testing
>> 
>> Particularly the following section needs some details added, as it’s still 
>> TBD.
>> 
>> https://developer.mozilla.org/en-US/Firefox_OS/Security/Testing_in_a_privileged_context#Windows
>> 
>> They are not very long, so looking through should be pretty quick.
>> 
>> Many thanks in advance!
>> 
>> Chris Mills
>>    Senior tech writer || Mozilla
>> developer.mozilla.org || MDN
>>    [email protected] || @chrisdavidmills
>> 
>> 
>> 
> At the time of reading this, the "flavor of nightly build (Linux, Mac OS, 
> Windows)" limk pointed to 
> http://ftp.mozilla.org/pub/mozilla.org/b2g/nightly/latest-mozilla-b2g18/ 
> which may not be what you want for *nightlies*. That would be 
> ftp://ftp.mozilla.org/pub/b2g/nightly/latest-mozilla-central/.
> 
> And to be pedantic, "Marionette: A JavaScript debugging shell for Firefox OS" 
> is inaccurate. Marionette is an automation driver, allowing you access to 
> automate actions in chrome and content and to inject javascript. The shell 
> you're using is built on top of marionette. I'd rewrite that as "Getting a 
> debugging shell for Firefox OS", then continue to describe the process of 
> setting up the shell.
> 
> I'd also change "Marionette is based on the Selenium/WebDriver API, and 
> provides a debugging shell to automate Firefox OS testing." to something like 
>  "Marionette is based on the Selenium/WebDriver API, and provides the basis 
> for the debugging shell we use to automate Firefox OS testing."
> 
> This will help mitigate any bugs, and help people get a better understanding 
> of why they're installing what they're installing:)
> 
> Furthermore, if these are nightly builds, they don't need to modify the 
> prefs.js stuff. It's set up to work when you start up b2g, so you can remove 
> those instructions.

Thanks for the help Malini - I’ve made your changes as suggested.

_______________________________________________
dev-b2g mailing list
[email protected]
https://lists.mozilla.org/listinfo/dev-b2g

Reply via email to