694644 - Reftest should turn off locale picker when running on mobile

Status: RESOLVED FIXED

(Testing :: Reftest, defect)

Description

[cmtalbert]

Attachment: the patch

This is probably not causing us any issues, but we turn this off for mochitests, so it seems like we should turn it off for reftests too.

Note that even with this off, you don't get the reftests foregrounded in the UI.  What you get is the reftest window opening up and then the fennec UI opening up on top of it.  So you're looking at the fennec UI and it doesn't appear that anything is happening, but the reftests themselves are running in the window behind the fennec ui.

Attaching a patch to deal with the locale picker issue.

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 567159 [details] [diff] [review]
the patch

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

::: layout/tools/reftest/remotereftest.py
@@ +336,5 @@
>            fhandle = open(os.path.join(profileDir, "user.js"), 'a')
>            fhandle.write("""
>  user_pref("capability.principal.codebase.p2.granted", "UniversalPreferencesWrite UniversalXPConnect UniversalBrowserWrite UniversalPreferencesRead UniversalBrowserRead");
>  user_pref("capability.principal.codebase.p2.id", "http://%s:%s");
> +user_pref("browser.firstrun.show.localepicker", false);

can we move this out of the options.enablePrivilege block?  That will require us to do another fhandle open/write/close.

Comment 2

[cmtalbert]

Attachment: A better patch

Good point. I didn't realize I was inside that if statement.  Been coding too much javascript.

Here's a version that does the locale picker preference no matter what, and only adds the enable privilege stuff if we're running jsreftest.  It also only opens the file once.

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 567171 [details] [diff] [review]
A better patch

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

looks great, thanks!

Comment 4

[cmtalbert]

landed on m-i: https://hg.mozilla.org/integration/mozilla-inbound/rev/5321159aac61

Comment 5

[Marco Bonardo [:mak]]

https://hg.mozilla.org/mozilla-central/rev/5321159aac61