First run page is missing some text

[Ross]

Environment

ABP 2.99.0.1889beta
Firefox 57 / Windows 10

Also

Firefox 57 Android

Also

Firefox 56.0, Debian Buster, Adblock Plus debuild #1895

How to reproduce

1. Ensure previous extension data is cleared so first run page appears.
2. Install build.

Observed behaviour

The first run page is missing some (but not all) texts. For example the text of the donate button. Specifically these strings are missing:

firstRun_share_headline
firstRun_donate
firstRun_donate_label
firstRun_share
firstRun_contributor_credits

Expected behaviour

The first run page to display correctly / not be missing text

Notes

• The locale files seem to have a different content in the development builds which are online, than when they are generated manually. If you first ​download the development build based on ​abd390e823bd, then build the extension yourself with ./build.py -t gecko build, extract both xpi files and ​compare the differences you will notice quite a few.
• This is apparently a regression ​caused by this commit d4ca9bfb82b5.

[sebastian]

I can reproduce this, both on Firefox and Chrome, when installing the development build from AMO or the CWS. However, I cannot reproduce it when building the extension myself, which is weird and makes it difficult to debug.

[saroyanm]

So far I only can say that it was broken in between Version 2.99.0.1879beta and Version 2.99.0.1881beta builds. Trying to find out which exact change caused this..

In between October 10-12

[sebastian]

Well, Ross and I can also reproduce it with *.1889 from AMO and the CWS.

[saroyanm]

Replying to sebastian:

Well, Ross and I can also reproduce it with *.1889 from AMO and the CWS.

I meant to say that build *.1879 is not broken(FRP strings are there), while ​*.1880 and next builds look to be broken.

[saroyanm]

I think this issue somehow connected with buildtools, I can confirm that the strings that are missing do exist in the adblockplusui firstRun.json dependency version.

Note: Also the project built locally and the one available online are missing different strings (This is very odd to me).

@Buildtool team can you please have a look, because I'm having hard time of finding what exactly cause the issue. Also the issue started to appear after the ​current change.

I think we should fix this before the release.

[kzar]

(Could possibly be related to #5916, but I've not looked into it yet.)

[tlucas]

I was able to recreate this using the official dev builds, but i could not recreate this using packages generated locally - just like Sebastian.

However, i managed to create a diff between the two packages - There are only additions, only affecting messages.json files, all additions appear to affect stringIDs with the prefix "firstRun", some locales have more additions then others.

(here's a full list, couldn't think of anything better to distribute this: ​https://codereview.adblockplus.org/29587789/ )

Replying to kzar:

(Could possibly be related to #5916, but I've not looked into it yet.)

This might be true

[saroyanm]

Replying to tlucas:

I was able to recreate this using the official dev builds, but i could not recreate this using packages generated locally - just like Sebastian.

As I mentioned above, package generated locally misses different strings. While the packaged that's available online missed for example string from ​Donation button. The package generated locally has the string for "Donation" button, but misses Adblock Browser promotion strings, like ​Get Adblock Browser here.

[kzar]

I can also reproduce this, Tristan and I are investigating.

[kzar]

(here's a full list, couldn't think of anything better to distribute this: ​​https://codereview.adblockplus.org/29587789/ )

I also also reproduce those differences, so the question is what caused them.

[abpbot]

A commit referencing this issue has landed:
​Issue 5877 - Account for random locale processing order

[Ross]

Fixed. All strings now appear correctly on the first run page.

ABP 1.13.4.1903
Chrome 49 / 62 / Windows 10
Opera 36 / 48 / Windows 10

ABP 2.99.0.1902beta
Firefox 50 / 57 / Windows 10