LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-01-08 12:19:37 +03:00
Code Issues Packages Projects Releases Wiki Activity

LibJS: Add fast path TimeZone conversion to PlainDate#toZonedDateTime

Browse Source 
This is a normative chane in the Temporal spec.

See: https://github.com/tc39/proposal-temporal/commit/fcab1af
This commit is contained in:
Luke Wilde 2022-10-16 00:43:13 +01:00 committed by Linus Groh
parent 707f12f927
commit 35c9e324b4
Notes: sideshowbarker 2024-07-17 05:43:02 +09:00 
Author: https://github.com/Lubrsi
Commit: https://github.com/SerenityOS/serenity/commit/35c9e324b4
Pull-request: https://github.com/SerenityOS/serenity/pull/15630
Reviewed-by: https://github.com/davidot ✅
Reviewed-by: https://github.com/linusg ✅
2 changed files with 42 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
Userland/Libraries/LibJS/Runtime/Temporal/PlainDatePrototype.cpp
View File

@ -525,23 +525,33 @@ JS_DEFINE_NATIVE_FUNCTION(PlainDatePrototype::to_zoned_date_time)
// 3. If Type(item) is Object, then
if (item.is_object()) {
// a. Let timeZoneLike be ? Get(item, "timeZone").
auto time_zone_like = TRY(item.as_object().get(vm.names.timeZone));
// b. If timeZoneLike is undefined, then
if (time_zone_like.is_undefined()) {
// i. Let timeZone be ? ToTemporalTimeZone(item).
time_zone = TRY(to_temporal_time_zone(vm, item));
// a. If item has an [[InitializedTemporalTimeZone]] internal slot, then
if (is<TimeZone>(item.as_object())) {
// i. Let timeZone be item.
time_zone = &item.as_object();
// ii. Let temporalTime be undefined.
}
// c. Else,
// b. Else,
else {
// i. Let timeZone be ? ToTemporalTimeZone(timeZoneLike).
time_zone = TRY(to_temporal_time_zone(vm, time_zone_like));
// i. Let timeZoneLike be ? Get(item, "timeZone").
auto time_zone_like = TRY(item.as_object().get(vm.names.timeZone));
// ii. Let temporalTime be ? Get(item, "plainTime").
temporal_time_value = TRY(item.as_object().get(vm.names.plainTime));
// ii. If timeZoneLike is undefined, then
if (time_zone_like.is_undefined()) {
// 1. Let timeZone be ? ToTemporalTimeZone(item).
time_zone = TRY(to_temporal_time_zone(vm, item));
// 2. Let temporalTime be undefined.
}
// iii. Else,
else {
// 1. Let timeZone be ? ToTemporalTimeZone(timeZoneLike).
time_zone = TRY(to_temporal_time_zone(vm, time_zone_like));
// 2. Let temporalTime be ? Get(item, "plainTime").
temporal_time_value = TRY(item.as_object().get(vm.names.plainTime));
}
}
}
// 4. Else,

20
Userland/Libraries/LibJS/Tests/builtins/Temporal/PlainDate/PlainDate.prototype.toZonedDateTime.js
View File

@ -74,6 +74,26 @@ describe("correct behavior", () => {
expect(zonedDateTime.calendar).toBe(plainDate.calendar);
expect(zonedDateTime.timeZone.id).toBe("UTC");
});
test("time zone fast path returns if it is passed a Temporal.TimeZone instance", () => {
const plainDate = new Temporal.PlainDate(2021, 7, 6);
// This is obseravble via there being no property lookups (avoiding a "timeZone" property lookup in this case)
let madeObservableHasPropertyLookup = false;
class TimeZone extends Temporal.TimeZone {
constructor() {
super("UTC");
}
get timeZone() {
madeObservableHasPropertyLookup = true;
return this;
}
}
const timeZone = new TimeZone();
plainDate.toZonedDateTime(timeZone);
expect(madeObservableHasPropertyLookup).toBeFalse();
});
});
describe("errors", () => {