Rewrite base::JSONReader to be 35-40% faster, depending on the input string.

chrome/common/extensions/extension_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 "base/file_util.h"
 #include "base/path_service.h"
 #include "base/scoped_temp_dir.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
@@ skipping 85 matching lines @@
 // Crashes intermittently on Windows, see http://crbug.com/109738
 #if defined(OS_WIN)
 #define MAYBE_InvalidMessagesFile DISABLED_InvalidMessagesFile
 #else
 #define MAYBE_InvalidMessagesFile InvalidMessagesFile
 #endif
 TEST_F(ExtensionUnpackerTest, MAYBE_InvalidMessagesFile) {
   SetupUnpacker("invalid_messages_file.crx");
   EXPECT_FALSE(unpacker_->Run());
   EXPECT_TRUE(MatchPattern(unpacker_->error_message(),
-    ASCIIToUTF16("*_locales?en_US?messages.json: Line: 2, column: 3,"
-                " Dictionary keys must be quoted.")));
+    ASCIIToUTF16("*_locales?en_US?messages.json: Line: 2, column: 11,"
+        " Syntax error."))) << unpacker_->error_message();
 }
 
 // Crashes intermittently on Vista, see http://crbug.com/109238
 #if defined(OS_WIN)
 #define MAYBE_MissingDefaultData DISABLED_MissingDefaultData
 #else
 #define MAYBE_MissingDefaultData MissingDefaultData
 #endif
 TEST_F(ExtensionUnpackerTest, MAYBE_MissingDefaultData) {
   SetupUnpacker("missing_default_data.crx");
@@ skipping 62 matching lines @@
 #define MAYBE_NoL10n DISABLED_NoL10n
 #else
 #define MAYBE_NoL10n NoL10n
 #endif
 TEST_F(ExtensionUnpackerTest, MAYBE_NoL10n) {
   SetupUnpacker("no_l10n.crx");
   EXPECT_TRUE(unpacker_->Run());
   EXPECT_TRUE(unpacker_->error_message().empty());
   EXPECT_EQ(0U, unpacker_->parsed_catalogs()->size());
 }