Closed Bug 1513934 Opened 4 years ago Closed 4 years ago

Import patch for ICU-20302


(Core :: JavaScript: Internationalization API, defect)

Not set



Tracking Status
firefox64 --- unaffected
firefox65 + fixed
firefox66 + fixed


(Reporter: anba, Assigned: anba)





(1 file) (<>) fixes a time zone detection issue on Windows 7. We should import the patch in our local ICU copy.

Hat tip to Jeff Genovy for pinging us in the ICU bug report!
This issue only affects ICU 63, so I guess we want to backport the patch to Beta, given that Beta uses ICU 63 (bug 1499026).
Blocks: 1499026
Attached patch bug1513934.patchSplinter Review
Imported the two PRs via
  wget -O bug-1513934-timezone-detection-win7-part1.diff
  wget -O bug-1513934-timezone-detection-win7-part2.diff

and then reimported the ICU-63 maintenance branch. That means we also get <> to fix a performance regression in UnicodeSet <>.
Attachment #9031432 - Flags: review?(jwalden+bmo)
Waldo, can you please prioritize this review as it's a patch targeting Fx65 and we're already halfway through the Beta cycle?
Flags: needinfo?(jwalden)
Attachment #9031432 - Flags: review?(jwalden) → review+
Comment on attachment 9031432 [details] [diff] [review]

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: None

User impact if declined: Potential problems with time zone detection on Windows 7 -- exactly what problems, seems to depend on the state of the program and other things.

Is this code covered by automated tests?: No

Has the fix been verified in Nightly?: No

Needs manual test from QE?: No

If yes, steps to reproduce: 

List of other uplifts needed: None

Risk to taking this patch: Medium

Why is the change risky/not risky? (and alternatives if risky): This is importing fixes from upstream, so that probably mitigates the risk a bit.  But this does touch a bit of code (mostly for the fixes that are *not* time zone-related, that were already on the maintenance branch), so it's not the targeted patch it could be.

String changes made/needed: N/A
Flags: needinfo?(jwalden)
Attachment #9031432 - Flags: approval-mozilla-beta?
Pushed by
Import ICU patch to fix possible time zone misdetection on Windows 7. r=Waldo
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
Comment on attachment 9031432 [details] [diff] [review]

[Triage Comment]
Fixes problems with time zone detection on Windows 7 caused by the most recent ICU update. Approved for 65.0b8.
Attachment #9031432 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.