https://codereview.chromium.org/314953006/diff/40001/src/lookup.cc
File src/lookup.cc (right):

https://codereview.chromium.org/314953006/diff/40001/src/lookup.cc#newcode21
src/lookup.cc:21: if (holder_map_->has_named_interceptor()) {
I think "check_interceptor() && " is missing here.
BTW, it would be good to combine this method with LookupInHolder()
somehow to keep the logic in one place.

https://codereview.chromium.org/314953006/diff/40001/src/lookup.cc#newcode153
src/lookup.cc:153: return false;
"GlobalObject with property_details_.IsDeleted()" case should also be
treated as absent property.

https://codereview.chromium.org/314953006/

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to