Store Object.observe state per-isolate rather than per-context

src/objects-inl.h

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 708 matching lines @@
 bool Object::IsPolymorphicCodeCacheHashTable() {
   return IsHashTable();
 }
 
 
 bool Object::IsMapCache() {
   return IsHashTable();
 }
 
 
+bool Object::IsObjectHashTable() {
+  // TODO: Is this sufficient?

[rossberg, 2012/11/06 12:27:09]

Yeah, I don't think it's worth introducing the extra (sub)type in the heap_types
list when you cannot check for it anyways.

[adamk, 2012/11/06 12:59:13]

Okay, I've removed this and completely redone the functions in runtime.cc to
minimize the checks they do (also changed the safe ones to not use handles). Let
me know if I went overboard there.

+  return IsHashTable();
+}
+
+
 bool Object::IsPrimitive() {
   return IsOddball() || IsNumber() || IsString();
 }
 
 
 bool Object::IsJSGlobalProxy() {
   bool result = IsHeapObject() &&
                 (HeapObject::cast(this)->map()->instance_type() ==
                  JS_GLOBAL_PROXY_TYPE);
   ASSERT(!result || IsAccessCheckNeeded());
@@ skipping 4810 matching lines @@
 #undef WRITE_UINT32_FIELD
 #undef READ_SHORT_FIELD
 #undef WRITE_SHORT_FIELD
 #undef READ_BYTE_FIELD
 #undef WRITE_BYTE_FIELD
 
 
 } }  // namespace v8::internal
 
 #endif  // V8_OBJECTS_INL_H_