Fix potential buffer over-read errors for un-terminated JSON strings and comments.

extensions/utility/unpacker_unittest.cc

 // Copyright (c) 2012 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 <memory>
 
 #include "base/bind.h"
 #include "base/files/file_util.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/path_service.h"
@@ skipping 71 matching lines @@
   EXPECT_FALSE(unpacker_->Run());
   EXPECT_EQ(ASCIIToUTF16(errors::kInvalidDefaultLocale),
             unpacker_->error_message());
 }
 
 TEST_F(UnpackerTest, InvalidMessagesFile) {
   SetupUnpacker("invalid_messages_file.crx");
   EXPECT_FALSE(unpacker_->Run());
   EXPECT_TRUE(base::MatchPattern(
       unpacker_->error_message(),
-      ASCIIToUTF16(
-          "*_locales?en_US?messages.json: Line: 2, column: 11,"
-          " Syntax error.")))
+      ASCIIToUTF16("*_locales?en_US?messages.json: Line: 2, column: 10,"
+                   " Syntax error.")))
       << unpacker_->error_message();
 }
 
 TEST_F(UnpackerTest, MissingDefaultData) {
   SetupUnpacker("missing_default_data.crx");
   EXPECT_FALSE(unpacker_->Run());
   EXPECT_EQ(ASCIIToUTF16(errors::kLocalesNoDefaultMessages),
             unpacker_->error_message());
 }
 
@@ skipping 149 matching lines @@
       {FILE_PATH_LITERAL("test/manifest.json"), false},
       {FILE_PATH_LITERAL("manifest.json/test"), false},
       {FILE_PATH_LITERAL("other.file"), false},
   };
   base::Callback<bool(const base::FilePath&)> filter =
       base::Bind(&Unpacker::IsManifestFile);
   RunZipFileFilterTest(cases, filter);
 }
 
 }  // namespace extensions