Remove HolderIsNonGlobalHiddenPrototype since OWN* now also checks globals

src/lookup-inl.h

Index: src/lookup-inl.h
diff --git a/src/lookup-inl.h b/src/lookup-inl.h
index ae4b66df01d74c662bd58dc1a9c18427bd6dafc8..d87221e7c913d60be0ebbc8938e47f13e87482fe 100644
--- a/src/lookup-inl.h
+++ b/src/lookup-inl.h
@@ -23,7 +23,7 @@ JSReceiver* LookupIterator::NextHolder(Map* map) {
       !(check_hidden() && next->map()->is_hidden_prototype()) &&
       // Always lookup behind the JSGlobalProxy into the JSGlobalObject, even
       // when not checking other hidden prototypes.

[mvstanton, 2014/08/25 11:10:02]

Could you update this comment?

[Toon Verwaest, 2014/08/25 11:36:27]

The comment is still valid. We just don't need to check whether it is the global
object.

[mvstanton, 2014/08/25 11:48:24]

Yes, okay. If you have a JSGlobalProxy, then you know there is a JSGlobalObject
behind, and you return it down on line 30.

-      !(map->IsJSGlobalProxyMap() && next->IsJSGlobalObject())) {
+      !map->IsJSGlobalProxyMap()) {
     return NULL;
   }