Revert "Parser & internalization fix: ensure no heap allocs during GetString(Handle<String>)."

src/ast-value-factory.h

Index: src/ast-value-factory.h
diff --git a/src/ast-value-factory.h b/src/ast-value-factory.h
index 09a41400fb3ec9b4b89f97b3ebe57e097f374e07..071ca9ce9cc18abf23e17f09c31e69ca69a8ccce 100644
--- a/src/ast-value-factory.h
+++ b/src/ast-value-factory.h
@@ -287,16 +287,12 @@ class AstValueFactory {
 
   Zone* zone() const { return zone_; }
 
-  const AstRawString* GetOneByteString(Vector<const uint8_t> literal) {
-    return GetOneByteStringInternal(literal);
-  }
+  const AstRawString* GetOneByteString(Vector<const uint8_t> literal);
   const AstRawString* GetOneByteString(const char* string) {
     return GetOneByteString(Vector<const uint8_t>(
         reinterpret_cast<const uint8_t*>(string), StrLength(string)));
   }
-  const AstRawString* GetTwoByteString(Vector<const uint16_t> literal) {
-    return GetTwoByteStringInternal(literal);
-  }
+  const AstRawString* GetTwoByteString(Vector<const uint16_t> literal);
   const AstRawString* GetString(Handle<String> literal);
   const AstConsString* NewConsString(const AstString* left,
                                      const AstString* right);
@@ -331,10 +327,8 @@ class AstValueFactory {
   const AstValue* NewTheHole();
 
  private:
-  AstRawString* GetOneByteStringInternal(Vector<const uint8_t> literal);
-  AstRawString* GetTwoByteStringInternal(Vector<const uint16_t> literal);
-  AstRawString* GetString(uint32_t hash, bool is_one_byte,
-                          Vector<const byte> literal_bytes);
+  const AstRawString* GetString(uint32_t hash, bool is_one_byte,
+                                Vector<const byte> literal_bytes);
 
   // All strings are copied here, one after another (no NULLs inbetween).
   HashMap string_table_;