1343826 - Implement system time zone change observers

Status: RESOLVED FIXED

(Core :: Hardware Abstraction Layer (HAL), enhancement)

Description

[André Bargull [:anba]]

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

Comment 1

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

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?

Comment 2

[André Bargull [:anba]]

I don't mind where this functionality is implemented, as long as it is available. ;-)

Comment 3

[André Bargull [:anba]]

https://github.com/whatwg/html/pull/3047 will probably also require this functionality.

Comment 4

[Sisim Biva]

Any news on this issue?

It also blocks https://bugzilla.mozilla.org/show_bug.cgi?id=1158733

Comment 5

[Sean Burke [:leftmostcat]]

Attachment: Bug 1343826 - Return updated timezone on system timezone change. r=smaug

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5ec59aaac2c4
Return updated timezone on system timezone change. r=smaug,mstange,stransky,handyman

Comment 7

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/5ec59aaac2c4