Skip to content

Commit

Permalink
[gin] Remove array change-by-copy flag
Browse files Browse the repository at this point in the history
The feature has shipped since M110

Bug: v8:12764
Change-Id: I759456825e61c4cf6630cad264e5f3fc2ac171d8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5035293
Commit-Queue: Camillo Bruni <cbruni@chromium.org>
Auto-Submit: Shu-yu Guo <syg@chromium.org>
Reviewed-by: Camillo Bruni <cbruni@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1225364}
  • Loading branch information
syg authored and Chromium LUCI CQ committed Nov 16, 2023
1 parent 9b877f8 commit d39be82
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 9 deletions.
5 changes: 0 additions & 5 deletions gin/gin_features.cc
Original file line number Diff line number Diff line change
Expand Up @@ -197,11 +197,6 @@ BASE_FEATURE(kJavaScriptSymbolAsWeakMapKey,
"JavaScriptSymbolAsWeakMapKey",
base::FEATURE_ENABLED_BY_DEFAULT);

// Enables the change-Array-by-copy proposal.
BASE_FEATURE(kJavaScriptChangeArrayByCopy,
"JavaScriptChangeArrayByCopy",
base::FEATURE_ENABLED_BY_DEFAULT);

// Enables the Resizable ArrayBuffer proposal.
BASE_FEATURE(kJavaScriptRabGsab,
"JavaScriptRabGsab",
Expand Down
1 change: 0 additions & 1 deletion gin/gin_features.h
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,6 @@ GIN_EXPORT BASE_DECLARE_FEATURE(kV8TurboFastApiCalls);
GIN_EXPORT BASE_DECLARE_FEATURE(kV8UseLibmTrigFunctions);
GIN_EXPORT extern const base::FeatureParam<base::TimeDelta>
kV8MemoryReducerStartDelay;
GIN_EXPORT BASE_DECLARE_FEATURE(kJavaScriptChangeArrayByCopy);
GIN_EXPORT BASE_DECLARE_FEATURE(kJavaScriptRabGsab);
GIN_EXPORT BASE_DECLARE_FEATURE(kJavaScriptRegExpUnicodeSets);
GIN_EXPORT BASE_DECLARE_FEATURE(kJavaScriptSymbolAsWeakMapKey);
Expand Down
3 changes: 0 additions & 3 deletions gin/v8_initializer.cc
Original file line number Diff line number Diff line change
Expand Up @@ -365,9 +365,6 @@ void SetFlags(IsolateHolder::ScriptMode mode,
SetV8FlagsIfOverridden(features::kJavaScriptSymbolAsWeakMapKey,
"--harmony-symbol-as-weakmap-key",
"--no-harmony-symbol-as-weakmap-key");
SetV8FlagsIfOverridden(features::kJavaScriptChangeArrayByCopy,
"--harmony-change-array-by-copy",
"--no-harmony-change-array-by-copy");
if (base::FeatureList::IsEnabled(features::kJavaScriptRabGsab)) {
SetV8Flags("--harmony-rab-gsab");
} else {
Expand Down

0 comments on commit d39be82

Please sign in to comment.