tools/gn: rename char_offset to column_number

tools/gn/string_utils.cc

 // Copyright (c) 2013 The Chromium 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 "tools/gn/string_utils.h"
 
 #include "tools/gn/err.h"
 #include "tools/gn/input_file.h"
 #include "tools/gn/parser.h"
 #include "tools/gn/scope.h"
 #include "tools/gn/token.h"
 #include "tools/gn/tokenizer.h"
 #include "tools/gn/value.h"
 
 namespace {
 
 // Constructs an Err indicating a range inside a string. We assume that the
 // token has quotes around it that are not counted by the offset.
 Err ErrInsideStringToken(const Token& token, size_t offset, size_t size,
                          const std::string& msg,
                          const std::string& help = std::string()) {
   // The "+1" is skipping over the " at the beginning of the token.
   int int_offset = static_cast<int>(offset);
   Location begin_loc(token.location().file(),
                      token.location().line_number(),
-                     token.location().char_offset() + int_offset + 1,
+                     token.location().column_number() + int_offset + 1,
                      token.location().byte() + int_offset + 1);
   Location end_loc(
       token.location().file(),
       token.location().line_number(),
-      token.location().char_offset() + int_offset + 1 + static_cast<int>(size),
+      token.location().column_number() + int_offset + 1 +
+          static_cast<int>(size),
       token.location().byte() + int_offset + 1 + static_cast<int>(size));
   return Err(LocationRange(begin_loc, end_loc), msg, help);
 }
 
 // Notes about expression interpolation. This is based loosly on Dart but is
 // slightly less flexible. In Dart, seeing the ${ in a string is something
 // the toplevel parser knows about, and it will recurse into the block
 // treating it as a first-class {...} block. So even things like this work:
 //   "hello ${"foo}"*2+"bar"}"  =>  "hello foo}foo}bar"
 // (you can see it did not get confused by the nested strings or the nested "}"
@@ skipping 193 matching lines @@
     } else if (input[i] == '$') {
       if (!AppendStringInterpolation(scope, literal, input, size, &i,
                                      &output, err))
         return false;
     } else {
       output.push_back(input[i]);
     }
   }
   return true;
 }