correctly invalidate global cells

src/runtime/runtime-debug.cc

 // 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.
 
 #include "src/v8.h"
 
 #include "src/accessors.h"
 #include "src/arguments.h"
 #include "src/compiler.h"
 #include "src/debug.h"
@@ skipping 120 matching lines @@
   // Check if the name is trivially convertible to an index and get the element
   // if so.
   uint32_t index;
   if (name->AsArrayIndex(&index)) {
     Handle<FixedArray> details = isolate->factory()->NewFixedArray(2);
     Handle<Object> element_or_char;
     ASSIGN_RETURN_FAILURE_ON_EXCEPTION(
         isolate, element_or_char,
         Runtime::GetElementOrCharAt(isolate, obj, index));
     details->set(0, *element_or_char);
-    details->set(1, PropertyDetails(NONE, DATA, 0).AsSmi());
+    details->set(
+        1, PropertyDetails(NONE, DATA, 0, PropertyCellType::kInvalid).AsSmi());
     return *isolate->factory()->NewJSArrayWithElements(details);
   }
 
   LookupIterator it(obj, name, LookupIterator::HIDDEN);
   bool has_caught = false;
   Handle<Object> value = DebugGetProperty(&it, &has_caught);
   if (!it.IsFound()) return isolate->heap()->undefined_value();
 
   Handle<Object> maybe_pair;
   if (it.state() == LookupIterator::ACCESSOR) {
     maybe_pair = it.GetAccessors();
   }
 
   // If the callback object is a fixed array then it contains JavaScript
   // getter and/or setter.
   bool has_js_accessors = !maybe_pair.is_null() && maybe_pair->IsAccessorPair();
   Handle<FixedArray> details =
       isolate->factory()->NewFixedArray(has_js_accessors ? 6 : 3);
   details->set(0, *value);
   // TODO(verwaest): Get rid of this random way of handling interceptors.
-  PropertyDetails d = it.state() == LookupIterator::INTERCEPTOR
-                          ? PropertyDetails(NONE, DATA, 0)
-                          : it.property_details();
+  PropertyDetails d =
+      it.state() == LookupIterator::INTERCEPTOR
+          ? PropertyDetails(NONE, DATA, 0, PropertyCellType::kInvalid)
+          : it.property_details();
   details->set(1, d.AsSmi());
   details->set(
       2, isolate->heap()->ToBoolean(it.state() == LookupIterator::INTERCEPTOR));
   if (has_js_accessors) {
     AccessorPair* accessors = AccessorPair::cast(*maybe_pair);
     details->set(3, isolate->heap()->ToBoolean(has_caught));
     details->set(4, accessors->GetComponent(ACCESSOR_GETTER));
     details->set(5, accessors->GetComponent(ACCESSOR_SETTER));
   }
 
@@ skipping 2645 matching lines @@
   return Smi::FromInt(isolate->debug()->is_active());
 }
 
 
 RUNTIME_FUNCTION(Runtime_DebugBreakInOptimizedCode) {
   UNIMPLEMENTED();
   return NULL;
 }
 }
 }  // namespace v8::internal