Revision: 12411
Author: [email protected]
Date: Wed Aug 29 23:45:45 2012
Log: Fixed comment and simplified the related code a bit.
[email protected]
Review URL: https://chromiumcodereview.appspot.com/10911003
http://code.google.com/p/v8/source/detail?r=12411
Modified:
/branches/bleeding_edge/src/ic.cc
=======================================
--- /branches/bleeding_edge/src/ic.cc Tue Aug 28 23:12:46 2012
+++ /branches/bleeding_edge/src/ic.cc Wed Aug 29 23:45:45 2012
@@ -1323,15 +1323,9 @@
receiver->map()->LookupTransition(*receiver, *name, lookup);
}
if (!StoreICableLookup(lookup)) {
- // 2nd chance: There can be accessors somewhere in the prototype
chain. Note
- // that we explicitly exclude native accessors for now, because the
stubs
- // are not yet prepared for this scenario.
+ // 2nd chance: There can be accessors somewhere in the prototype chain.
receiver->Lookup(*name, lookup);
- if (!lookup->IsPropertyCallbacks()) {
- return false;
- }
- Handle<Object> callback(lookup->GetCallbackObject());
- return StoreICableLookup(lookup);
+ return lookup->IsPropertyCallbacks() && StoreICableLookup(lookup);
}
if (lookup->IsInterceptor() &&
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
