v8::Private::ForApi should be context-independent.

src/heap/heap.h

Index: src/heap/heap.h
diff --git a/src/heap/heap.h b/src/heap/heap.h
index 9e63c5dc35e7497a88ae8c8c4daa4a075ba46e5d..cfcb29dfcec6e28ab33bcd326687768045c70565 100644
--- a/src/heap/heap.h
+++ b/src/heap/heap.h
@@ -192,7 +192,6 @@ using v8::MemoryPressureLevel;
     ExperimentalExtraNativesSourceCache)                                       \
   /* Lists and dictionaries */                                                 \
   V(NameDictionary, empty_properties_dictionary, EmptyPropertiesDictionary)    \
-  V(Object, symbol_registry, SymbolRegistry)                                   \
   V(Object, script_list, ScriptList)                                           \
   V(UnseededNumberDictionary, code_stubs, CodeStubs)                           \
   V(FixedArray, materialized_objects, MaterializedObjects)                     \
@@ -200,6 +199,9 @@ using v8::MemoryPressureLevel;
   V(FixedArray, detached_contexts, DetachedContexts)                           \
   V(ArrayList, retained_maps, RetainedMaps)                                    \
   V(WeakHashTable, weak_object_to_code_table, WeakObjectToCodeTable)           \
+  V(NameDictionary, public_symbol_table, PublicSymbolTable)                    \

[peria, 2016/12/06 05:12:36]

optional;
I don't know the ordering rule, but isn't it better to
move these new macros in between #194-195?
We have another V(NameDictionary, ...) on #194.

[Yang, 2016/12/06 12:53:07]

Done.

+  V(NameDictionary, api_symbol_table, ApiSymbolTable)                          \
+  V(NameDictionary, api_private_symbol_table, ApiPrivateSymbolTable)           \
   /* weak_new_space_object_to_code_list is an array of weak cells, where */    \
   /* slots with even indices refer to the weak object, and the subsequent */   \
   /* slots refer to the code with the reference to the weak object. */         \