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

Side by Side Diff: chrome/browser/extensions/all_urls_apitest.cc

Issue 5730004: Rename ExtensionsService to ExtensionService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix typo 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 "chrome/browser/extensions/extension_apitest.h" 5 #include "chrome/browser/extensions/extension_apitest.h"
6 #include "chrome/browser/extensions/extensions_service.h" 6 #include "chrome/browser/extensions/extension_service.h"
7 #include "chrome/browser/extensions/extension_test_message_listener.h" 7 #include "chrome/browser/extensions/extension_test_message_listener.h"
8 #include "chrome/browser/profiles/profile.h" 8 #include "chrome/browser/profiles/profile.h"
9 #include "chrome/browser/ui/browser.h" 9 #include "chrome/browser/ui/browser.h"
10 #include "chrome/common/extensions/extension.h" 10 #include "chrome/common/extensions/extension.h"
11 #include "chrome/test/ui_test_utils.h" 11 #include "chrome/test/ui_test_utils.h"
12 12
13 const std::string kAllUrlsTarget = 13 const std::string kAllUrlsTarget =
14 "files/extensions/api_test/all_urls/index.html"; 14 "files/extensions/api_test/all_urls/index.html";
15 15
16 typedef ExtensionApiTest AllUrlsApiTest; 16 typedef ExtensionApiTest AllUrlsApiTest;
(...skipping 20 matching lines...) Expand all
37 37
38 // Then add the two extensions to the whitelist. 38 // Then add the two extensions to the whitelist.
39 Checkpoint("Setting up whitelist", start_time); 39 Checkpoint("Setting up whitelist", start_time);
40 Extension::ScriptingWhitelist whitelist; 40 Extension::ScriptingWhitelist whitelist;
41 whitelist.push_back(Extension::GenerateIdForPath(extension_dir1)); 41 whitelist.push_back(Extension::GenerateIdForPath(extension_dir1));
42 whitelist.push_back(Extension::GenerateIdForPath(extension_dir2)); 42 whitelist.push_back(Extension::GenerateIdForPath(extension_dir2));
43 Extension::SetScriptingWhitelist(whitelist); 43 Extension::SetScriptingWhitelist(whitelist);
44 44
45 // Then load extensions. 45 // Then load extensions.
46 Checkpoint("LoadExtension1", start_time); 46 Checkpoint("LoadExtension1", start_time);
47 ExtensionsService* service = browser()->profile()->GetExtensionsService(); 47 ExtensionService* service = browser()->profile()->GetExtensionService();
48 const size_t size_before = service->extensions()->size(); 48 const size_t size_before = service->extensions()->size();
49 ASSERT_TRUE(LoadExtension(extension_dir1)); 49 ASSERT_TRUE(LoadExtension(extension_dir1));
50 Checkpoint("LoadExtension2", start_time); 50 Checkpoint("LoadExtension2", start_time);
51 ASSERT_TRUE(LoadExtension(extension_dir2)); 51 ASSERT_TRUE(LoadExtension(extension_dir2));
52 EXPECT_EQ(size_before + 2, service->extensions()->size()); 52 EXPECT_EQ(size_before + 2, service->extensions()->size());
53 53
54 std::string url; 54 std::string url;
55 55
56 // Now verify we run content scripts on chrome://newtab/. 56 // Now verify we run content scripts on chrome://newtab/.
57 Checkpoint("Verify content scripts", start_time); 57 Checkpoint("Verify content scripts", start_time);
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
116 // Test that an extension NOT whitelisted for scripting can ask for <all_urls> 116 // Test that an extension NOT whitelisted for scripting can ask for <all_urls>
117 // and run scripts on non-restricted all pages. 117 // and run scripts on non-restricted all pages.
118 // Disabled, http://crbug.com/64304. 118 // Disabled, http://crbug.com/64304.
119 IN_PROC_BROWSER_TEST_F(AllUrlsApiTest, DISABLED_RegularExtensions) { 119 IN_PROC_BROWSER_TEST_F(AllUrlsApiTest, DISABLED_RegularExtensions) {
120 // First load the two extension. 120 // First load the two extension.
121 FilePath extension_dir1 = test_data_dir_.AppendASCII("all_urls") 121 FilePath extension_dir1 = test_data_dir_.AppendASCII("all_urls")
122 .AppendASCII("content_script"); 122 .AppendASCII("content_script");
123 FilePath extension_dir2 = test_data_dir_.AppendASCII("all_urls") 123 FilePath extension_dir2 = test_data_dir_.AppendASCII("all_urls")
124 .AppendASCII("execute_script"); 124 .AppendASCII("execute_script");
125 125
126 ExtensionsService* service = browser()->profile()->GetExtensionsService(); 126 ExtensionService* service = browser()->profile()->GetExtensionService();
127 const size_t size_before = service->extensions()->size(); 127 const size_t size_before = service->extensions()->size();
128 ASSERT_TRUE(LoadExtension(extension_dir1)); 128 ASSERT_TRUE(LoadExtension(extension_dir1));
129 ASSERT_TRUE(LoadExtension(extension_dir2)); 129 ASSERT_TRUE(LoadExtension(extension_dir2));
130 EXPECT_EQ(size_before + 2, service->extensions()->size()); 130 EXPECT_EQ(size_before + 2, service->extensions()->size());
131 131
132 // Now verify we can script a regular http page. 132 // Now verify we can script a regular http page.
133 ASSERT_TRUE(test_server()->Start()); 133 ASSERT_TRUE(test_server()->Start());
134 GURL page_url = test_server()->GetURL(kAllUrlsTarget); 134 GURL page_url = test_server()->GetURL(kAllUrlsTarget);
135 ExtensionTestMessageListener listener1a("content script: " + page_url.spec(), 135 ExtensionTestMessageListener listener1a("content script: " + page_url.spec(),
136 false); 136 false);
137 ExtensionTestMessageListener listener1b("execute: " + page_url.spec(), false); 137 ExtensionTestMessageListener listener1b("execute: " + page_url.spec(), false);
138 ui_test_utils::NavigateToURL(browser(), page_url); 138 ui_test_utils::NavigateToURL(browser(), page_url);
139 ASSERT_TRUE(listener1a.WaitUntilSatisfied()); 139 ASSERT_TRUE(listener1a.WaitUntilSatisfied());
140 ASSERT_TRUE(listener1b.WaitUntilSatisfied()); 140 ASSERT_TRUE(listener1b.WaitUntilSatisfied());
141 } 141 }
OLDNEW
« no previous file with comments | « chrome/browser/download/download_util.cc ('k') | chrome/browser/extensions/browser_action_apitest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698