Skip to content

Commit 595da57

Browse files
legendecasRafaelGSS
authored andcommitted
deps: V8: cherry-pick 0f9ebbc672c7
Original commit message: [flags] Remove --harmony-string-is-well-formed The String.prototype.isWellFormed and toWellFormed have shipped since M111. Bug: v8:13557 Change-Id: I27e332d2fde0f9ea8ad649c016a84d2d3e0bf592 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4931269 Reviewed-by: Shu-yu Guo <[email protected]> Commit-Queue: Chengzhong Wu (legendecas) <[email protected]> Cr-Commit-Position: refs/heads/main@{#90398} Refs: v8/v8@0f9ebbc PR-URL: #50867 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Jiawen Geng <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]>
1 parent 3bf47d4 commit 595da57

File tree

3 files changed

+5
-14
lines changed

3 files changed

+5
-14
lines changed

common.gypi

+1-1
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@
3636

3737
# Reset this number to 0 on major V8 upgrades.
3838
# Increment by one for each non-official patch applied to deps/v8.
39-
'v8_embedder_string': '-node.16',
39+
'v8_embedder_string': '-node.17',
4040

4141
##### V8 defaults for Node.js #####
4242

deps/v8/src/flags/flag-definitions.h

-1
Original file line numberDiff line numberDiff line change
@@ -298,7 +298,6 @@ DEFINE_WEAK_IMPLICATION(harmony_rab_gsab_transfer, harmony_rab_gsab)
298298
#define HARMONY_SHIPPING_BASE(V) \
299299
V(harmony_import_assertions, "harmony import assertions") \
300300
V(harmony_change_array_by_copy, "harmony change-Array-by-copy") \
301-
V(harmony_string_is_well_formed, "harmony String#{is,to}WellFormed") \
302301
V(harmony_rab_gsab, \
303302
"harmony ResizableArrayBuffer / GrowableSharedArrayBuffer") \
304303
V(harmony_regexp_unicode_sets, "harmony RegExp Unicode Sets") \

deps/v8/src/init/bootstrapper.cc

+4-12
Original file line numberDiff line numberDiff line change
@@ -2158,6 +2158,8 @@ void Genesis::InitializeGlobal(Handle<JSGlobalObject> global_object,
21582158
Builtin::kStringPrototypeIncludes, 1, false);
21592159
SimpleInstallFunction(isolate_, prototype, "indexOf",
21602160
Builtin::kStringPrototypeIndexOf, 1, false);
2161+
SimpleInstallFunction(isolate(), prototype, "isWellFormed",
2162+
Builtin::kStringPrototypeIsWellFormed, 0, false);
21612163
SimpleInstallFunction(isolate_, prototype, "italics",
21622164
Builtin::kStringPrototypeItalics, 0, false);
21632165
SimpleInstallFunction(isolate_, prototype, "lastIndexOf",
@@ -2214,6 +2216,8 @@ void Genesis::InitializeGlobal(Handle<JSGlobalObject> global_object,
22142216
Builtin::kStringPrototypeStartsWith, 1, false);
22152217
SimpleInstallFunction(isolate_, prototype, "toString",
22162218
Builtin::kStringPrototypeToString, 0, true);
2219+
SimpleInstallFunction(isolate(), prototype, "toWellFormed",
2220+
Builtin::kStringPrototypeToWellFormed, 0, false);
22172221
SimpleInstallFunction(isolate_, prototype, "trim",
22182222
Builtin::kStringPrototypeTrim, 0, false);
22192223

@@ -5062,18 +5066,6 @@ void Genesis::InitializeGlobal_harmony_rab_gsab() {
50625066
Builtin::kSharedArrayBufferPrototypeGrow, 1, true);
50635067
}
50645068

5065-
void Genesis::InitializeGlobal_harmony_string_is_well_formed() {
5066-
if (!v8_flags.harmony_string_is_well_formed) return;
5067-
Handle<JSFunction> string_function(native_context()->string_function(),
5068-
isolate());
5069-
Handle<JSObject> string_prototype(
5070-
JSObject::cast(string_function->initial_map()->prototype()), isolate());
5071-
SimpleInstallFunction(isolate(), string_prototype, "isWellFormed",
5072-
Builtin::kStringPrototypeIsWellFormed, 0, false);
5073-
SimpleInstallFunction(isolate(), string_prototype, "toWellFormed",
5074-
Builtin::kStringPrototypeToWellFormed, 0, false);
5075-
}
5076-
50775069
void Genesis::InitializeGlobal_harmony_temporal() {
50785070
if (!v8_flags.harmony_temporal) return;
50795071
// -- T e m p o r a l

0 commit comments

Comments
 (0)