Rename the configuration flags of the LookupIterator

src/lookup.h

 // Copyright 2014 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_LOOKUP_H_
 #define V8_LOOKUP_H_
 
 #include "src/factory.h"
 #include "src/isolate.h"
 #include "src/objects.h"
 
 namespace v8 {
 namespace internal {
 
 class LookupIterator V8_FINAL BASE_EMBEDDED {
  public:
   enum Configuration {
-    CHECK_OWN_REAL = 0,
-    CHECK_HIDDEN = 1 << 0,
-    CHECK_DERIVED = 1 << 1,
+    // Configuration bits.
+    CHECK_HIDDEN_PROPERTY = 1 << 0,
+    CHECK_DERIVED_PROPERTY = 1 << 1,
     CHECK_INTERCEPTOR = 1 << 2,
     CHECK_ACCESS_CHECK = 1 << 3,
-    CHECK_HIDDEN_ACCESS = CHECK_HIDDEN | CHECK_ACCESS_CHECK,
-    SKIP_INTERCEPTOR = CHECK_HIDDEN_ACCESS | CHECK_DERIVED,
-    CHECK_ALL = SKIP_INTERCEPTOR | CHECK_INTERCEPTOR,
-    CHECK_OWN = CHECK_HIDDEN_ACCESS | CHECK_INTERCEPTOR
+
+    // Convience combinations of bits.
+    CHECK_PROPERTY = 0,
+    CHECK_HIDDEN_SKIP_INTERCEPTOR = CHECK_HIDDEN_PROPERTY | CHECK_ACCESS_CHECK,
+    CHECK_DERIVED_SKIP_INTERCEPTOR =
+        CHECK_HIDDEN_SKIP_INTERCEPTOR | CHECK_DERIVED_PROPERTY,
+    CHECK_DERIVED = CHECK_DERIVED_SKIP_INTERCEPTOR | CHECK_INTERCEPTOR,
+    CHECK_HIDDEN = CHECK_HIDDEN_SKIP_INTERCEPTOR | CHECK_INTERCEPTOR
   };
 
   enum State {
     NOT_FOUND,
     PROPERTY,
     INTERCEPTOR,
     ACCESS_CHECK,
     JSPROXY
   };
 
   enum PropertyKind {
     DATA,
     ACCESSOR
   };
 
   enum PropertyEncoding {
     DICTIONARY,
     DESCRIPTOR
   };
 
-  LookupIterator(Handle<Object> receiver,
-                 Handle<Name> name,
-                 Configuration configuration = CHECK_ALL)
+  LookupIterator(Handle<Object> receiver, Handle<Name> name,
+                 Configuration configuration = CHECK_DERIVED)
       : configuration_(ComputeConfiguration(configuration, name)),
         state_(NOT_FOUND),
         property_kind_(DATA),
         property_encoding_(DESCRIPTOR),
         property_details_(NONE, NONEXISTENT, Representation::None()),
         isolate_(name->GetIsolate()),
         name_(name),
         maybe_receiver_(receiver),
         number_(DescriptorArray::kNotFound) {
     Handle<JSReceiver> root = GetRoot();
     holder_map_ = handle(root->map());
     maybe_holder_ = root;
     Next();
   }
 
-  LookupIterator(Handle<Object> receiver,
-                 Handle<Name> name,
+  LookupIterator(Handle<Object> receiver, Handle<Name> name,
                  Handle<JSReceiver> holder,
-                 Configuration configuration = CHECK_ALL)
+                 Configuration configuration = CHECK_DERIVED)
       : configuration_(ComputeConfiguration(configuration, name)),
         state_(NOT_FOUND),
         property_kind_(DATA),
         property_encoding_(DESCRIPTOR),
         property_details_(NONE, NONEXISTENT, Representation::None()),
         isolate_(name->GetIsolate()),
         name_(name),
         holder_map_(holder->map()),
         maybe_receiver_(receiver),
         maybe_holder_(holder),
@@ skipping 75 matching lines @@
   // This means the receiver needs to be present as opposed to just the receiver
   // map. Other objects in the prototype chain are transitively guaranteed to be
   // present via the receiver map.
   bool is_guaranteed_to_have_holder() const {
     return !maybe_receiver_.is_null();
   }
   bool check_interceptor() const {
     return !IsBootstrapping() && (configuration_ & CHECK_INTERCEPTOR) != 0;
   }
   bool check_derived() const {
-    return (configuration_ & CHECK_DERIVED) != 0;
+    return (configuration_ & CHECK_DERIVED_PROPERTY) != 0;
   }
   bool check_hidden() const {
-    return (configuration_ & CHECK_HIDDEN) != 0;
+    return (configuration_ & CHECK_HIDDEN_PROPERTY) != 0;
   }
   bool check_access_check() const {
     return (configuration_ & CHECK_ACCESS_CHECK) != 0;
   }
   int descriptor_number() const {
     DCHECK(has_property_);
     DCHECK_EQ(DESCRIPTOR, property_encoding_);
     return number_;
   }
   int dictionary_entry() const {
     DCHECK(has_property_);
     DCHECK_EQ(DICTIONARY, property_encoding_);
     return number_;
   }
 
   static Configuration ComputeConfiguration(
       Configuration configuration, Handle<Name> name) {
     if (name->IsOwn()) {
-      return static_cast<Configuration>(configuration & CHECK_OWN);
+      return static_cast<Configuration>(configuration & CHECK_HIDDEN);
     } else {
       return configuration;
     }
   }
 
   // If configuration_ becomes mutable, update
   // HolderIsReceiverOrHiddenPrototype.
   Configuration configuration_;
   State state_;
   bool has_property_;
   PropertyKind property_kind_;
   PropertyEncoding property_encoding_;
   PropertyDetails property_details_;
   Isolate* isolate_;
   Handle<Name> name_;
   Handle<Map> holder_map_;
   MaybeHandle<Object> maybe_receiver_;
   MaybeHandle<JSReceiver> maybe_holder_;
 
   int number_;
 };
 
 
 } }  // namespace v8::internal
 
 #endif  // V8_LOOKUP_H_