[inspector] store stack frame in struct instead of JSObject

src/runtime/runtime-wasm.cc

 // Copyright 2016 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/runtime/runtime-utils.h"
 
 #include "src/arguments.h"
 #include "src/assembler.h"
 #include "src/compiler/wasm-compiler.h"
 #include "src/conversions.h"
@@ skipping 85 matching lines @@
   }
 
   // Patch the detailed stack trace (array of JSObjects with various
   // properties).
   Handle<Object> detailed_stack_trace_obj = JSReceiver::GetDataProperty(
       error, isolate->factory()->detailed_stack_trace_symbol());
   if (detailed_stack_trace_obj->IsJSArray()) {
     Handle<FixedArray> stack_elements(
         FixedArray::cast(JSArray::cast(*detailed_stack_trace_obj)->elements()));
     DCHECK_GE(stack_elements->length(), 1);
-    Handle<JSObject> top_frame(JSObject::cast(stack_elements->get(0)));
-    Handle<String> wasm_offset_key =
-        isolate->factory()->InternalizeOneByteString(
-            STATIC_CHAR_VECTOR("column"));
-    LookupIterator it(top_frame, wasm_offset_key, top_frame,
-                      LookupIterator::PROTOTYPE_CHAIN_SKIP_INTERCEPTOR);
-    if (it.IsFound()) {
-      DCHECK(JSReceiver::GetDataProperty(&it)->IsSmi());
-      // Make column number 1-based here.
-      Maybe<bool> data_set = JSReceiver::SetDataProperty(
-          &it, handle(Smi::FromInt(byte_offset + 1), isolate));
-      DCHECK(data_set.IsJust() && data_set.FromJust() == true);
-      USE(data_set);
+    Handle<StackFrameInfo> top_frame(
+        StackFrameInfo::cast(stack_elements->get(0)));
+    if (top_frame->column_number()) {
+      top_frame->set_column_number(byte_offset + 1);
     }
   }
 
   return isolate->Throw(*error_obj);
 }
 
 RUNTIME_FUNCTION(Runtime_ThrowWasmErrorFromTrapIf) {
   DCHECK_EQ(1, args.length());
   CONVERT_SMI_ARG_CHECKED(message_id, 0);
   return ThrowRuntimeError(isolate, message_id, 0, false);
@@ skipping 124 matching lines @@
 
 RUNTIME_FUNCTION(Runtime_WasmCompileLazy) {
   DCHECK(args.length() == 0);
   HandleScope scope(isolate);
 
   return *wasm::CompileLazy(isolate);
 }
 
 }  // namespace internal
 }  // namespace v8