LibJS: Store symbols in a Handle inside PropertyKey

If the stored symbol is custom (non-global), it is allocated on the
heap, and may not be visited by anyone else, so we must register it in
order to keep it alive.

1 files changed, 2 insertions, 1 deletions

diff --git a/Userland/Libraries/LibJS/Runtime/PropertyKey.h b/Userland/Libraries/LibJS/Runtime/PropertyKey.h
index 3dfb6a0779..5be70863fa 100644
--- a/Userland/Libraries/LibJS/Runtime/PropertyKey.h
+++ b/Userland/Libraries/LibJS/Runtime/PropertyKey.h
@@ -7,6 +7,7 @@
 #pragma once
 
 #include <AK/FlyString.h>
+#include <LibJS/Heap/Handle.h>
 #include <LibJS/Runtime/Completion.h>
 #include <LibJS/Runtime/StringOrSymbol.h>
 
@@ -186,7 +187,7 @@ private:
     Type m_type { Type::Invalid };
     u32 m_number { 0 };
     FlyString m_string;
-    Symbol* m_symbol { nullptr };
+    Handle<Symbol> m_symbol;
 };
 
 }