Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(101)

Side by Side Diff: chrome/renderer/extensions/extension_api_json_validity_unittest.cc

Issue 5581008: Add a new GetInstance() method for singleton classes, take 2. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 10 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/file_path.h" 5 #include "base/file_path.h"
6 #include "base/file_util.h" 6 #include "base/file_util.h"
7 #include "base/path_service.h" 7 #include "base/path_service.h"
8 #include "base/scoped_ptr.h" 8 #include "base/scoped_ptr.h"
9 #include "chrome/common/json_value_serializer.h" 9 #include "chrome/common/json_value_serializer.h"
10 #include "chrome/common/chrome_paths.h" 10 #include "chrome/common/chrome_paths.h"
(...skipping 112 matching lines...) Expand 10 before | Expand all | Expand 10 after
123 #define MAYBE_WithV8 DISABLED_WithV8 123 #define MAYBE_WithV8 DISABLED_WithV8
124 #else 124 #else
125 #define MAYBE_WithV8 WithV8 125 #define MAYBE_WithV8 WithV8
126 #endif 126 #endif
127 127
128 // Use V8 to load the string resource version of extension_api.json . 128 // Use V8 to load the string resource version of extension_api.json .
129 // This test mimics the method extension_api.json is loaded in 129 // This test mimics the method extension_api.json is loaded in
130 // chrome/renderer/resources/extension_process_bindings.js . 130 // chrome/renderer/resources/extension_process_bindings.js .
131 TEST_F(ExtensionApiJsonValidityTest, MAYBE_WithV8) { 131 TEST_F(ExtensionApiJsonValidityTest, MAYBE_WithV8) {
132 std::string ext_api_string = 132 std::string ext_api_string =
133 bindings_utils::GetStringResource<IDR_EXTENSION_API_JSON>(); 133 bindings_utils::GetStringResource(IDR_EXTENSION_API_JSON);
134 134
135 // Create a global variable holding the text of extension_api.json . 135 // Create a global variable holding the text of extension_api.json .
136 SetGlobalStringVar("ext_api_json_text", ext_api_string); 136 SetGlobalStringVar("ext_api_json_text", ext_api_string);
137 137
138 // Parse the text of extension_api.json . If there is a parse error, 138 // Parse the text of extension_api.json . If there is a parse error,
139 // an exception will be printed that includes a line number. 139 // an exception will be printed that includes a line number.
140 std::string test_js = "var extension_api = JSON.parse(ext_api_json_text);"; 140 std::string test_js = "var extension_api = JSON.parse(ext_api_json_text);";
141 ExecuteScriptInContext(test_js, "ParseExtensionApiJson"); 141 ExecuteScriptInContext(test_js, "ParseExtensionApiJson");
142 } 142 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698