From 53da8893acb8b67e4d0daf05af71278e9cf73255 Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Sun, 16 Mar 2025 21:25:29 -0500 Subject: [PATCH] LibJS: Replace PropertyKey(char[]) with PropertyKey(FlyString) ...and deal with the fallout. --- Libraries/LibJS/Bytecode/Interpreter.cpp | 15 ++-- Libraries/LibJS/Contrib/Test262/262Object.cpp | 16 ++--- .../LibJS/Contrib/Test262/AgentObject.cpp | 4 +- .../LibJS/Contrib/Test262/GlobalObject.cpp | 4 +- Libraries/LibJS/CyclicModule.cpp | 10 +-- .../LibJS/Runtime/AbstractOperations.cpp | 4 +- .../Runtime/AsyncDisposableStackPrototype.cpp | 2 +- .../AsyncFromSyncIteratorPrototype.cpp | 2 +- .../Runtime/AsyncFunctionDriverWrapper.cpp | 4 +- Libraries/LibJS/Runtime/AsyncGenerator.cpp | 18 ++--- Libraries/LibJS/Runtime/CommonPropertyNames.h | 3 + Libraries/LibJS/Runtime/Completion.cpp | 4 +- .../Runtime/DisposableStackPrototype.cpp | 2 +- Libraries/LibJS/Runtime/GeneratorObject.cpp | 6 +- Libraries/LibJS/Runtime/Intrinsics.cpp | 2 +- Libraries/LibJS/Runtime/NativeFunction.h | 2 +- Libraries/LibJS/Runtime/PromiseCapability.cpp | 2 +- Libraries/LibJS/Runtime/PromisePrototype.cpp | 8 +-- Libraries/LibJS/Runtime/PropertyKey.h | 5 +- Libraries/LibJS/Runtime/ProxyConstructor.cpp | 2 +- Libraries/LibJS/Runtime/ShadowRealm.cpp | 2 +- Libraries/LibJS/Runtime/StringPrototype.cpp | 2 +- Libraries/LibTest/JavaScriptTestRunner.h | 6 +- .../LibWeb/Animations/KeyframeEffect.cpp | 6 +- Libraries/LibWeb/Bindings/MainThreadVM.cpp | 4 +- Libraries/LibWeb/Crypto/CryptoAlgorithms.cpp | 72 +++++++++---------- Libraries/LibWeb/Crypto/CryptoAlgorithms.h | 2 +- Libraries/LibWeb/Crypto/CryptoBindings.cpp | 34 ++++----- Libraries/LibWeb/Crypto/CryptoKey.cpp | 6 +- Libraries/LibWeb/Crypto/KeyAlgorithms.cpp | 16 ++--- Libraries/LibWeb/Crypto/SubtleCrypto.cpp | 4 +- Libraries/LibWeb/DOM/EventTarget.cpp | 2 +- .../HTML/CrossOrigin/AbstractOperations.cpp | 6 +- Libraries/LibWeb/HTML/HTMLDialogElement.cpp | 4 +- Libraries/LibWeb/HTML/HTMLElement.cpp | 2 +- Libraries/LibWeb/HTML/HTMLInputElement.cpp | 24 +++---- Libraries/LibWeb/HTML/Scripting/ImportMap.cpp | 12 ++-- .../LibWeb/HTML/UniversalGlobalScope.cpp | 4 +- Libraries/LibWeb/HTML/Window.cpp | 2 +- .../LibWeb/HTML/WindowOrWorkerGlobalScope.cpp | 2 +- Libraries/LibWeb/IndexedDB/IDBFactory.cpp | 4 +- Libraries/LibWeb/Internals/Internals.cpp | 4 +- .../MediaCapabilities.cpp | 6 +- Libraries/LibWeb/Streams/Transformer.cpp | 16 ++--- Libraries/LibWeb/Streams/UnderlyingSink.cpp | 12 ++-- Libraries/LibWeb/Streams/UnderlyingSource.cpp | 12 ++-- Libraries/LibWeb/WebAssembly/WebAssembly.cpp | 4 +- .../LibWeb/WebGL/WebGLContextAttributes.cpp | 18 ++--- Libraries/LibWeb/WebIDL/Promise.cpp | 8 +-- .../BindingsGenerator/IDLGenerators.cpp | 36 +++++----- .../GenerateWindowOrWorkerInterfaces.cpp | 12 ++-- .../ConsoleGlobalEnvironmentExtensions.cpp | 8 +-- Tests/LibJS/test262-runner.cpp | 8 +-- Tests/LibWasm/test-wasm.cpp | 6 +- Utilities/js.cpp | 24 +++---- 55 files changed, 254 insertions(+), 251 deletions(-) diff --git a/Libraries/LibJS/Bytecode/Interpreter.cpp b/Libraries/LibJS/Bytecode/Interpreter.cpp index 1d40f18586e..347c3cd7869 100644 --- a/Libraries/LibJS/Bytecode/Interpreter.cpp +++ b/Libraries/LibJS/Bytecode/Interpreter.cpp @@ -193,16 +193,16 @@ ALWAYS_INLINE void Interpreter::set(Operand op, Value value) ALWAYS_INLINE Value Interpreter::do_yield(Value value, Optional