Move public symbols to the root set.

src/ast-value-factory.cc

Index: src/ast-value-factory.cc
diff --git a/src/ast-value-factory.cc b/src/ast-value-factory.cc
index 518be23abcb2ab7646581778e1c4866eaf72f604..5ab2544ab26a5106e1ccc473e59d98d10d2de45d 100644
--- a/src/ast-value-factory.cc
+++ b/src/ast-value-factory.cc
@@ -182,9 +182,8 @@ void AstValue::Internalize(Isolate* isolate) {
       DCHECK(!string_->string().is_null());
       break;
     case SYMBOL:
-      value_ = Object::GetProperty(
-                   isolate, handle(isolate->native_context()->builtins()),
-                   symbol_name_).ToHandleChecked();
+      DCHECK_EQ(0, strcmp(symbol_name_, "iterator_symbol"));

[rossberg, 2014/11/12 16:45:25]

Why can this only happen for the iterator symbol?

[Yang, 2014/11/12 17:18:32]

right now this is only used for the iterator symbol. if we need other symbols
embedded in the future, we would have to multiplex.

+      value_ = isolate->factory()->iterator_symbol();
       break;
     case NUMBER:
       value_ = isolate->factory()->NewNumber(number_, TENURED);