Implement system time zone change observers

NEW
Unassigned

Status

()

2 years ago
a year ago

People

(Reporter: anba, Unassigned)

Tracking

(Blocks: 5 bugs)

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox54 affected)

Details

(Reporter)

Description

2 years ago
Bug 1333197 was partly caused by missing support for system time zone change observers on all supported platforms [1]. We've added a workaround resp. requested a specification change for the ECMAScript Intl API, so the missing support doesn't lead to Talos regressions like bug 1314354, but ultimately it makes sense to get rid of this call [2], and instead use OS events to get notified when the system time zone was changed. 

[1] bug 1333197, comment #1, bug 1333197, comment #11
[2] http://searchfox.org/mozilla-central/rev/31b6089ce26fa76459642765115605d50a6c67b4/js/src/jscompartment.cpp#138-143
Forgot if I asked you already, but since we now have intl/locale/OSPreferences which is an intl related OS-specific code hooks that will have all the event listeners for reacting to system language changes, region preference changes etc.
Would it make sense to locate it there?
(Reporter)

Comment 2

2 years ago
I don't mind where this functionality is implemented, as long as it is available. ;-)
(Reporter)

Updated

2 years ago
Blocks: 1348916
(Reporter)

Updated

2 years ago
Blocks: 1346211
(Reporter)

Updated

2 years ago
Blocks: 895737
(Reporter)

Updated

2 years ago
Blocks: 1198543
(Reporter)

Updated

2 years ago
Blocks: 1300110
(Reporter)

Comment 3

a year ago
https://github.com/whatwg/html/pull/3047 will probably also require this functionality.
(Reporter)

Updated

a year ago
Blocks: 895742

Comment 4

a year ago
Any news on this issue?

It also blocks https://bugzilla.mozilla.org/show_bug.cgi?id=1158733
You need to log in before you can comment on or make changes to this bug.