[wasm] Make reported "column number" 1-based

src/runtime/runtime-internal.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/runtime/runtime-utils.h"
 
 #include "src/arguments.h"
 #include "src/ast/prettyprinter.h"
 #include "src/bootstrapper.h"
 #include "src/conversions.h"
@@ skipping 123 matching lines @@
         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), isolate));
+          &it, handle(Smi::FromInt(byte_offset + 1), isolate));
       DCHECK(data_set.IsJust() && data_set.FromJust() == true);
       USE(data_set);
     }
   }
 
   return isolate->Throw(*error_obj);
 }
 
 RUNTIME_FUNCTION(Runtime_UnwindAndFindExceptionHandler) {
   SealHandleScope shs(isolate);
@@ skipping 463 matching lines @@
   HandleScope scope(isolate);
   DCHECK_EQ(1, args.length());
   CONVERT_ARG_HANDLE_CHECKED(Object, object, 0);
   bool is_wasm_object = object->IsJSObject() &&
                         wasm::IsWasmObject(Handle<JSObject>::cast(object));
   return *isolate->factory()->ToBoolean(is_wasm_object);
 }
 
 }  // namespace internal
 }  // namespace v8