LibJS: Add StringOrSymbol constructor that takes a FlyString

This avoids refcount churn from implicit conversion in some places.

1 files changed, 8 insertions, 0 deletions

diff --git a/Libraries/LibJS/Runtime/StringOrSymbol.h b/Libraries/LibJS/Runtime/StringOrSymbol.h
index 6939194183..bb4aba3e7a 100644
--- a/Libraries/LibJS/Runtime/StringOrSymbol.h
+++ b/Libraries/LibJS/Runtime/StringOrSymbol.h
@@ -26,6 +26,7 @@
 
 #pragma once
 
+#include <AK/FlyString.h>
 #include <LibJS/Runtime/PrimitiveString.h>
 #include <LibJS/Runtime/Symbol.h>
 #include <LibJS/Runtime/Value.h>
@@ -58,6 +59,13 @@ public:
         static_cast<const StringImpl*>(m_ptr)->ref();
     }
 
+    StringOrSymbol(const FlyString& string)
+        : m_ptr(string.impl())
+    {
+        ASSERT(!string.is_null());
+        static_cast<const StringImpl*>(m_ptr)->ref();
+    }
+
     ~StringOrSymbol()
     {
         if (is_string())