1415733 - Migrate the "Search" section of Preferences to the new Localization API

Status: RESOLVED FIXED

(Firefox :: Settings UI, defect, P3)

Description

[Zibi Braniecki [:zbraniecki][:gandalf]]

This bug will cover the migration work required to move the "Search" section of Preferences to Fluent.

URL: http://searchfox.org/mozilla-central/source/browser/components/preferences/in-content/sync.xul

Comment 1

[Zibi Braniecki [:zbraniecki][:gandalf]]

Attachment: Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

Review commit: https://reviewboard.mozilla.org/r/197978/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/197978/

Comment 2

[Zibi Braniecki [:zbraniecki][:gandalf]]

Comment on attachment 8926739 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/197978/diff/1-2/

Comment 3

[Francesco Lodolo [:flod]]

Comment on attachment 8926739 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

https://reviewboard.mozilla.org/r/197978/#review203234

::: browser/locales/en-US/browser/preferences/search.ftl:1
(Diff revision 2)
> +pane-search-title = Search

This string is defined in preferences.dtd. Defining it here means that we need to duplicate it later? I think it should remain in a shared file (not sure if main.ftl, since this seems to be built on top of other code)

http://searchfox.org/mozilla-central/rev/c99d035f00dd894feff38e4ad28a73fb679c63a6/browser/components/preferences/in-content/preferences.xul#142

::: browser/locales/en-US/browser/preferences/search.ftl:37
(Diff revision 2)
> +engine-keyword-column
> +    .label = Keyword
> +
> +restore-default-se
> +    .label = Restore Default Search Engines
> +    .accesskey = d

D

::: browser/locales/en-US/browser/preferences/search.ftl:41
(Diff revision 2)
> +    .label = Restore Default Search Engines
> +    .accesskey = d
> +
> +remove-engine
> +    .label = Remove
> +    .accesskey = r

R

Comment 4

[Francesco Lodolo [:flod]]

Comment on attachment 8926739 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

https://reviewboard.mozilla.org/r/197978/#review203266

::: browser/locales/en-US/browser/preferences/search.ftl:54
(Diff revision 2)
> +duplicate-engine-msg =
> +    You have chosen a keyword that is currently in use by “{ $name }”.
> +    Please select another.
> +
> +duplicate-bookmark-msg =
> +    You have chosen a keyword that is currently in use by a bookmark.

Did you go on a new line here for formatting reasons, or do you have a new line because the second part of the string should be displayed on a new line?

I would assume the former, because that's what the current code does.

Isn't Fluent going to display this on two lines, i.e. "You have chosen a keyword that is currently in use by a bookmark.\nPlease select another."?

Comment 5

[Axel Hecht [:Pike]]

Comment on attachment 8926739 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

https://reviewboard.mozilla.org/r/197978/#review203400

::: browser/components/preferences/in-content/search.js:305
(Diff revision 2)
> -        let emsg = strings.getFormattedString("duplicateEngineMsg", [dupName]);
> +        ] = await document.l10n.formatValues([
> +          ["duplicate-title"],
> +          eduplicate ?
> +            ["duplicate-engine-msg", { name: dupName }] :
> +            ["duplicate-bookmark-msg"],
> +        ]);

I'd make this three statements:

let msgids = ['duplicate-title'];
if (eduplicate) {
  msgids.push([..{}]);
}
else {
}
let [dtitle, msg] = await document.l10n.formatValues(msgids);

::: browser/locales/en-US/browser/preferences/search.ftl:22
(Diff revision 2)
> +    .label = Show search suggestions in address bar results
> +    .accesskey = l
> +
> +url-bar-suggestions-permanent-pb =
> +    Search suggestions will not be shown in location bar results
> +    because you have configured &brandShortName; to never remember history.

DTD entity reference should be branding?

Comment 6

[Zibi Braniecki [:zbraniecki][:gandalf]]

Comment on attachment 8926739 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/197978/diff/2-3/

Comment 7

[Axel Hecht [:Pike]]

Comment on attachment 8926739 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

https://reviewboard.mozilla.org/r/197978/#review204174

The ftl file should closely map what a migration can do, in particular keep in-string whitespace.

The two cases below actually end up in Services.prompt.alert(), which is whitespace-sensitive.

::: browser/locales/en-US/browser/preferences/search.ftl:51
(Diff revision 3)
> +
> +// Variable
> +//   $name (String) - name of the engine using selected keyword
> +duplicate-engine-msg =
> +    You have chosen a keyword that is currently in use by “{ $name }”.
> +    Please select another.

This is not what we'll have with migrations, we should stick to a 1-1 mapping here so that our en-US testing reflects what migrations will do for localizations.

::: browser/locales/en-US/browser/preferences/search.ftl:55
(Diff revision 3)
> +    You have chosen a keyword that is currently in use by “{ $name }”.
> +    Please select another.
> +
> +duplicate-bookmark-msg =
> +    You have chosen a keyword that is currently in use by a bookmark.
> +    Please select another.

... here, too.

Comment 8

[Zibi Braniecki [:zbraniecki][:gandalf]]

Comment on attachment 8926739 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/197978/diff/3-4/

Comment 9

[Zibi Braniecki [:zbraniecki][:gandalf]]

Comment on attachment 8926739 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

Thanks for all the feedback so far!

I updated the patch to it and am left with one remaining issue - the jar.mn entries for branding.
The only way I got it to work for en-US at least is to hardcode the en-US, but I suspect that it's not the way it should be in official/aurora/nightly.

Pike - can you help me here?

(I'll still retest that migration produced FTL and the one in en-US match)

Comment 10

[Zibi Braniecki [:zbraniecki][:gandalf]]

Comment on attachment 8926739 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/197978/diff/4-5/

Comment 11

[Zibi Braniecki [:zbraniecki][:gandalf]]

Comment on attachment 8926739 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

Pike - actually, I think I only need a non-en-US brand for ./official/. Still, mot sure how to get it.

Stas - can you look at the migration file? Does it look reasonable?

Comment 12

[Staś Małolepszy :stas]

Comment on attachment 8926739 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

https://reviewboard.mozilla.org/r/197978/#review207904

::: python/l10n/fluent_migrations/bug_1415733.py:10
(Diff revision 5)
> +
> +
> +def migrate(ctx):
> +    """Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API, part {index}"""
> +
> +    ctx.maybe_add_localization(

I realized that perhaps we don't need `maybe_add_localization` at all and filed bug 1420225 about it.

::: python/l10n/fluent_migrations/bug_1415733.py:20
(Diff revision 5)
> +
> +    ctx.add_transforms(
> +        'browser/branding/brand.ftl',
> +        'browser/branding/locales/en-US/brand.ftl',
> +        [
> +            FTL.Message(

This will need to be a private message in Fluent Syntax 0.5. I wonder if it would be easier to wait for 0.5 to land in fluent.js and python-fluent before running this migration.

::: python/l10n/fluent_migrations/bug_1415733.py:228
(Diff revision 5)
> +                value=COPY(
> +                    REPLACE(
> +                        'browser/chrome/browser/engineManager.properties',
> +                        'duplicateEngineMsg',
> +                        {
> +                            '%S': EXTERNAL_ARGUMENT(

In theory, we need bug 1317336 to land first before we can reliable support %S placeables in properties files.

In practice, in this particular case all existing translations use the literal %S which is well-supported by the migration code:

https://transvision.mozfr.org/string/?entity=browser/chrome/browser/engineManager.properties:duplicateEngineMsg&repo=gecko_strings

Comment 13

[Zibi Braniecki [:zbraniecki][:gandalf]]

Comment on attachment 8926739 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/197978/diff/5-6/

Comment 14

[Zibi Braniecki [:zbraniecki][:gandalf]]

Thanks :stas!

I got the migration complete and was able to get the branding work, so I think this is ready for a more complete review even if we'll want to wait for some migration code updates before landing it.

Comment 15

[Zibi Braniecki [:zbraniecki][:gandalf]]

Comment on attachment 8926739 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/197978/diff/6-7/

Comment 16

[Axel Hecht [:Pike]]

Comment on attachment 8926739 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

https://reviewboard.mozilla.org/r/197978/#review211422

I think this should be two patches, one for the pane controlling search engines, and one for the search-in-preferences feature.

As I mentioned in bug 1411012, too, please don't use abbreviations in IDs, that's just making code harder to read.

Also, I notice that we're using '-' both for English and for hiearchy. Like `show-url-bar-suggestions`. Is that a bug or a feature?

Lastly, now that we'll have grammatical data in branding, should we expose all pre-release-channel branding files to localizers? Or are we OK with pre-release channels having funky grammar? Also fine to be discussed in a dedicated bug.

::: browser/branding/aurora/locales/en-US/brand.ftl:1
(Diff revision 7)
> +brand-short-name = Firefox

Needs a license header

::: browser/branding/nightly/locales/en-US/brand.ftl:1
(Diff revision 7)
> +brand-short-name = Nightly

Also, license header.

::: browser/locales/en-US/browser/preferences/search.ftl:10
(Diff revision 7)
> +    .label = Use the address bar for search and navigation
> +search-bar-shown
> +    .label = Add search bar in toolbar
> +
> +default-search-engine = Default Search Engine
> +default-search-engine-cta = Choose the default search engine to use in the address bar and search bar.

no `-cta`

::: browser/locales/en-US/browser/preferences/search.ftl:20
(Diff revision 7)
> +
> +show-url-bar-suggestions
> +    .label = Show search suggestions in address bar results
> +    .accesskey = l
> +
> +url-bar-suggestions-permanent-pb = Search suggestions will not be shown in location bar results because you have configured { brand-short-name } to never remember history.

no `-pb`

::: browser/locales/en-US/browser/preferences/search.ftl:24
(Diff revision 7)
> +
> +url-bar-suggestions-permanent-pb = Search suggestions will not be shown in location bar results because you have configured { brand-short-name } to never remember history.
> +
> +one-click-search-engines = One-Click Search Engines
> +
> +one-click-search-engines-cta = Choose the alternative search engines that appear below the address bar and search bar when you start to enter a keyword.

`-cta`

::: browser/locales/en-US/browser/preferences/search.ftl:31
(Diff revision 7)
> +engine-name-column
> +    .label = Search Engine
> +engine-keyword-column
> +    .label = Keyword
> +
> +restore-default-se

`-se` should be spelled out.

::: browser/locales/en-US/browser/preferences/search.ftl:39
(Diff revision 7)
> +
> +remove-engine
> +    .label = Remove
> +    .accesskey = R
> +
> +find-more-se = Find more search engines

`-se`

::: python/l10n/fluent_migrations/bug_1415733.py:1
(Diff revision 7)
> +# coding=utf8

License header.

::: python/l10n/fluent_migrations/bug_1415733.py:17
(Diff revision 7)
> +    ctx.maybe_add_localization(
> +        'browser/chrome/browser/preferences/search.dtd')
> +    ctx.maybe_add_localization(
> +        'browser/chrome/browser/preferences/preferences.dtd')
> +    ctx.maybe_add_localization(
> +        'browser/chrome/browser/engineManager.properties')

We don't need these anymore?

Comment 17

[Zibi Braniecki [:zbraniecki][:gandalf]]

Attachment: Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

Review commit: https://reviewboard.mozilla.org/r/227248/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/227248/

Comment 18

[Francesco Lodolo [:flod]]

Comment on attachment 8958314 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

https://reviewboard.mozilla.org/r/227248/#review233080

::: browser/components/preferences/in-content/search.js:361
(Diff revision 1)
>  
>        // Notify the user if they have chosen an existing engine/bookmark keyword
>        if (eduplicate || bduplicate) {
> -        let strings = document.getElementById("engineManagerBundle");
> -        let dtitle = strings.getString("duplicateTitle");
> -        let bmsg = strings.getString("duplicateBookmarkMsg");
> +        let msgids = [["duplicate-title"]];
> +        if (eduplicate) {
> +          msgids.push(["duplicate-engine-msg", { name: dupName }]);

I think you have the wrong l10n-ids here, unless I'm misunderstanding the code?

::: browser/locales/en-US/browser/preferences/preferences.ftl:408
(Diff revision 1)
> +
> +search-suggestions-cant-show = Search suggestions will not be shown in location bar results because you have configured { -brand-short-name } to never remember history.
> +
> +search-one-click-header = One-Click Search Engines
> +
> +search-choose-alternative-engines = Choose the alternative search engines that appear below the address bar and search bar when you start to enter a keyword.

search-one-click-desc for consistency (it's the description of this section)

::: browser/locales/en-US/browser/preferences/preferences.ftl:425
(Diff revision 1)
> +    .label = Remove
> +    .accesskey = R
> +
> +search-find-more-link = Find more search engines
> +
> +search-keyword-warning-title = Duplicate Keyword

Add a comment here to avoid possible confusion:

# This warning is displayed when the chosen keyword is already in use
# ('Duplicate' is an adjective)

::: browser/locales/en-US/browser/preferences/preferences.ftl:426
(Diff revision 1)
> +    .accesskey = R
> +
> +search-find-more-link = Find more search engines
> +
> +search-keyword-warning-title = Duplicate Keyword
> +search-keyword-warning-engine = You have chosen a keyword that is currently in use by “{ $name }”. Please select another.

Worth adding a note that $name is the name of a search engine.

::: python/l10n/fluent_migrations/bug_1415733_preferences_search.py:122
(Diff revision 1)
> +                value=REPLACE(
> +                    'browser/chrome/browser/preferences/search.dtd',
> +                    'urlBarSuggestionsPermanentPB.label',
> +                    {
> +                        '&brandShortName;': MESSAGE_REFERENCE(
> +                            'brand-short-name'

Missing leading -

Comment 19

[Zibi Braniecki [:zbraniecki][:gandalf]]

Comment on attachment 8958314 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/227248/diff/1-2/

Comment 20

[Zibi Braniecki [:zbraniecki][:gandalf]]

Comment on attachment 8958314 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/227248/diff/2-3/

Comment 21

[Francesco Lodolo [:flod]]

Not sure why from the diff, but this patch fails to apply on central for me. Can you check if it needs a rebase?

Comment 22

[Zibi Braniecki [:zbraniecki][:gandalf]]

It's based on the search results patch. Sorry for that!

Comment 23

[Francesco Lodolo [:flod]]

Comment on attachment 8958314 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

https://reviewboard.mozilla.org/r/227248/#review233382

Looks good to me.

Comment 24

[Zibi Braniecki [:zbraniecki][:gandalf]]

Attachment: Screenshot of the area being refactored

Comment 25

[:Gijs (he/him)]

Comment on attachment 8958314 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

https://reviewboard.mozilla.org/r/227248/#review233668

LGTM. FWIW, I noticed android and various other things running on the trypush. Those don't use the prefs. For future reference, something like this:

try: -b od -p win64,macosx64,linux64,linux64-asan -u mochitest-bc,firefox-ui-functional,marionette,marionette-e10s -t none --artifact

will get you all the test coverage the prefs have, I think, with artifact builds to make it go more quickly. :-)

(the tier-2 firefox ui en-US tests have a single high-frequency-intermittent-orange to do with search keyword attribute for artifact builds, but otherwise it should Just Work.)

::: browser/components/preferences/in-content/findInPage.js:296
(Diff revision 3)
>          document.l10n.setAttributes(msgElem, "search-results-sorry-message", {
>            query: this.query
>          });
>          let helpUrl = Services.urlFormatter.formatURLPref("app.support.baseURL") + "preferences";
>          let helpContainer = document.getElementById("need-help");
> -        let link = helpContainer.querySelector("a").setAttribute("href", helpUrl);
> +        helpContainer.querySelector("a").setAttribute("href", helpUrl);

Stray hunk from the other bug?

Comment 26

[Zibi Braniecki [:zbraniecki][:gandalf]]

Comment on attachment 8958314 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/227248/diff/3-4/

Comment 27

[Zibi Braniecki [:zbraniecki][:gandalf]]

Comment on attachment 8958314 [details]
Bug 1415733 - Migrate the "Search" section of Preferences to the new Localization API.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/227248/diff/4-5/

Comment 28

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/647154f30332
Migrate the "Search" section of Preferences to the new Localization API. r=flod,Gijs

Comment 29

[Tiberius Oros[:tiberius_oros]]

https://hg.mozilla.org/mozilla-central/rev/647154f30332

Comment 30

[Natalia Csoregi [:nataliaCs]]

Backed out for frequently failing browser-chrome's browser_search_within_preferences_1.js (bug 1446186)
backout: https://hg.mozilla.org/mozilla-central/rev/db044a29f878

Failure log example: https://treeherder.mozilla.org/logviewer.html#?repo=autoland&job_id=168314661&lineNumber=2138
[task 2018-03-15T20:51:54.125Z] 20:51:54     INFO - Leaving test bound 
[task 2018-03-15T20:51:54.126Z] 20:51:54     INFO - Entering test bound 
[task 2018-03-15T20:51:54.127Z] 20:51:54     INFO - Buffered messages logged at 20:51:52
[task 2018-03-15T20:51:54.127Z] 20:51:54     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Element should not be null, when checking visibility - 
[task 2018-03-15T20:51:54.128Z] 20:51:54     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Should not be in search results yet - 
[task 2018-03-15T20:51:54.128Z] 20:51:54     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Search input should be focused when visiting preferences - 
[task 2018-03-15T20:51:54.129Z] 20:51:54     INFO - Buffered messages logged at 20:51:53
[task 2018-03-15T20:51:54.130Z] 20:51:54     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Element should not be null, when checking visibility - 
[task 2018-03-15T20:51:54.130Z] 20:51:54     INFO - Buffered messages finished
[task 2018-03-15T20:51:54.132Z] 20:51:54     INFO - TEST-UNEXPECTED-FAIL | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Should be in search results - 
[task 2018-03-15T20:51:54.132Z] 20:51:54     INFO - Stack trace:
[task 2018-03-15T20:51:54.133Z] 20:51:54     INFO - chrome://mochitests/content/browser/browser/components/preferences/in-content/tests/head.js:is_element_visible:24
[task 2018-03-15T20:51:54.133Z] 20:51:54     INFO - chrome://mochitests/content/browser/browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js:null:164
[task 2018-03-15T20:51:55.011Z] 20:51:55     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Element should not be null, when checking visibility - 
[task 2018-03-15T20:51:55.013Z] 20:51:55     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Should not be in search results - 
[task 2018-03-15T20:51:55.196Z] 20:51:55     INFO - Leaving test bound 
[task 2018-03-15T20:51:55.198Z] 20:51:55     INFO - Entering test bound 
[task 2018-03-15T20:51:56.987Z] 20:51:56     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Element should not be null, when checking visibility - 
[task 2018-03-15T20:51:57.003Z] 20:51:57     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Should not be in general - 
[task 2018-03-15T20:51:57.004Z] 20:51:57     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Search input should be focused when visiting preferences - 
[task 2018-03-15T20:51:59.994Z] 20:51:59     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Element should not be null, when checking visibility - 
[task 2018-03-15T20:51:59.996Z] 20:51:59     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Should be in generalPane - 
[task 2018-03-15T20:52:00.197Z] 20:52:00     INFO - Leaving test bound 
[task 2018-03-15T20:52:00.197Z] 20:52:00     INFO - Entering test bound 
[task 2018-03-15T20:52:02.340Z] 20:52:02     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Element should not be null, when checking visibility - 
[task 2018-03-15T20:52:02.357Z] 20:52:02     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Should not be in general - 
[task 2018-03-15T20:52:02.359Z] 20:52:02     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Search input should be focused when visiting preferences - 
[task 2018-03-15T20:52:03.774Z] 20:52:03     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Element should not be null, when checking visibility - 
[task 2018-03-15T20:52:03.776Z] 20:52:03     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Should be hidden in search results - 
[task 2018-03-15T20:52:05.132Z] 20:52:05     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Element should not be null, when checking visibility - 
[task 2018-03-15T20:52:05.134Z] 20:52:05     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Should be in generalPane - 
[task 2018-03-15T20:52:05.310Z] 20:52:05     INFO - Leaving test bound 
[task 2018-03-15T20:52:05.312Z] 20:52:05     INFO - Entering test bound 
[task 2018-03-15T20:52:07.715Z] 20:52:07     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Search input should be focused when visiting preferences - 
[task 2018-03-15T20:52:09.278Z] 20:52:09     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | search input should be empty - 
[task 2018-03-15T20:52:09.280Z] 20:52:09     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | No other panel should be selected - 
[task 2018-03-15T20:52:09.281Z] 20:52:09     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | No other panel should be selected - 
[task 2018-03-15T20:52:09.282Z] 20:52:09     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | No other panel should be selected - 
[task 2018-03-15T20:52:09.284Z] 20:52:09     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Privacy panel should be selected - 
[task 2018-03-15T20:52:09.285Z] 20:52:09     INFO - TEST-PASS | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | No other panel should be selected - 
[task 2018-03-15T20:52:09.440Z] 20:52:09     INFO - Leaving test bound 
[task 2018-03-15T20:52:09.459Z] 20:52:09     INFO - GECKO(1053) | MEMORY STAT | vsize 20974301MB | residentFast 1323MB
[task 2018-03-15T20:52:09.461Z] 20:52:09     INFO - TEST-OK | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | took 33790ms

Comment 31

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/mozilla-central/rev/c87a57081a24
Migrate the "Search" section of Preferences to the new Localization API. r=flod,Gijs a=reland