Indonesian Bahasa Indonesia "id" builds have broken new tab pages

VERIFIED FIXED in Firefox 58



2 years ago
2 years ago


(Reporter: Mardak, Assigned: Mardak)



57 Branch
Firefox 59
Dependency tree / graph

Firefox Tracking Flags

(firefox-esr52 unaffected, firefox57 unaffected, firefox58blocking fixed, firefox59 verified)



(4 attachments)



2 years ago
[Tracking Requested - why for this release]: Sadness for "id". See attachment.

We added an exception to .gitignore in bug 1412930 but forgot to update it when files moved in bug 1413550. :(

The added files are those that would have been updated as part of bug 1415812.
Comment hidden (mozreview-request)

Comment 2

2 years ago
Putting some dates into context:

Bug 1413550 regressed this and made it to Nightly 58 mozilla-central 18 days ago on November 4th.
Beta 58.0b1 went to aurora channel November 7th.
Beta 58.0b3 went to beta channel less than a week ago November 15th.

We're looking through activity stream's telemetry to see if this drop in "id" usage can be detected, and hopefully if so, we can put up some monitoring / alerts for this type of failure. (In addition to the unit test added here.)
Assignee: nobody → edilee
Keywords: regression
Adding a few Indonesian localizers, so they are aware of the issue in case they see support requests.

Comment 4

2 years ago
Thanks for putting this. I just realized this a few nights ago when my Indonesian Nightly stopped working on every new tab opened, while my other languages (de, en-US) Nightly seems fine.

Comment 5

2 years ago
Comment on attachment 8930753 [details]
Bug 1419601 - Indonesian Bahasa Indonesia "id" builds have broken new tab pages.

Good idea with that test. Looks good
Attachment #8930753 - Flags: review?(khudson) → review+

Comment 6

2 years ago
Pushed by
Indonesian Bahasa Indonesia "id" builds have broken new tab pages. r=k88hudson


2 years ago
See Also: → 1419836
Sounds bad, marking as blocker for 58.  Gerry fyi.
Flags: needinfo?(gchang)
Severity: normal → critical
Priority: -- → P1

Comment 8

2 years ago
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 59

Comment 9

2 years ago
Romi, the latest Nightly 59 update should be available now (20171122220056). Could you try updating nightly to verify if this has been fixed?
Flags: needinfo?(romihardiyanto)

Comment 11

2 years ago
(In reply to Ed Lee :Mardak from comment #9)
> Romi, the latest Nightly 59 update should be available now (20171122220056).
> Could you try updating nightly to verify if this has been fixed?

20171122220056 has fixed this problem.
Flags: needinfo?(romihardiyanto)


2 years ago


2 years ago

Comment 12

2 years ago
Comment on attachment 8930753 [details]
Bug 1419601 - Indonesian Bahasa Indonesia "id" builds have broken new tab pages.

Approval Request Comment
[Feature/Bug causing the regression]: Activity Stream startup optimization bug 1413550
[User impact if declined]: Indonesian users see failed load page for new tabs. See attachment 8930749 [details]
[Is this code covered by automated tests?]: Yes
[Has the fix been verified in Nightly?]: Yes 20171122220056
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: Bug 1415812 implements the functionality used by the html "id" content in this patch.
[Is the change risky?]: Not really
[Why is the change risky/not risky?]: Adds back the missing locale files that behave the same as the other locales that were correctly packaged.
[String changes made/needed]: Changed yes, needed no. These are the "id" strings.
Attachment #8930753 - Flags: approval-mozilla-beta?
Hi :Mardak,
Are there errors ignorable per comment #13?
Flags: needinfo?(gchang) → needinfo?(edilee)

Comment 15

2 years ago
They should be. Do you see those errors in other uplift requests? In particular there's one that says "A failure in this test is considered a release blocker." But this change shouldn't be causing that.
Flags: needinfo?(edilee) → needinfo?(gchang)

Comment 16

2 years ago
andreio, can you answer comment 14?
Flags: needinfo?(andrei.br92)
Let's please remember that Indonesia is completely broken in 58 Beta.

For sure.

Are we asking if this patch breaks some edge case in our test cases, when we know for a fact that a country with millions of Firefox Beta users are experiencing a broken about:newtab and about:home experience?

Please, let's uplift this patch.
Flags: needinfo?(andrei.br92)
Comment on attachment 8933040 [details] [diff] [review]
rebased export for mozilla-beta

This is a critical blocker for Indonesia. Beta58+.
Flags: needinfo?(gchang)
Attachment #8933040 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Attachment #8930753 - Flags: approval-mozilla-beta? → approval-mozilla-beta-
NI :flod, not sure if the patch includes the string changes. Need :flod's help.
Flags: needinfo?(francesco.lodolo)
No impact of strings, they're localized outside of m-c. Good to uplift.
Flags: needinfo?(francesco.lodolo)
You need to log in before you can comment on or make changes to this bug.