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

Unified Diff: chrome/browser/extensions/extension_updater_unittest.cc

Issue 9150016: Move creation and ownership of ResourceDispatcherHost and PluginService to content. This gives a ... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 8 years, 11 months 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/extension_updater_unittest.cc
===================================================================
--- chrome/browser/extensions/extension_updater_unittest.cc (revision 116798)
+++ chrome/browser/extensions/extension_updater_unittest.cc (working copy)
@@ -339,6 +339,7 @@
ExtensionUpdater updater(
&service, service.extension_prefs(), service.pref_service(),
service.profile(), 60*60*24);
+ updater.set_use_utility_process_for_testing(false);
updater.Start();
// Disable blacklist checks (tested elsewhere) so that we only see the
// update HTTP request.
@@ -391,6 +392,7 @@
ExtensionUpdater updater(
&service, service.extension_prefs(), service.pref_service(),
service.profile(), 60*60*24);
+ updater.set_use_utility_process_for_testing(false);
updater.Start();
// Tell the updater that it's time to do update checks.
@@ -503,6 +505,7 @@
ExtensionUpdater updater(
&service, service.extension_prefs(), service.pref_service(),
service.profile(), kUpdateFrequencySecs);
+ updater.set_use_utility_process_for_testing(false);
updater.Start();
// Check passing an empty list of parse results to DetermineUpdates
@@ -544,6 +547,7 @@
ExtensionUpdater updater(
&service, service.extension_prefs(), service.pref_service(),
service.profile(), kUpdateFrequencySecs);
+ updater.set_use_utility_process_for_testing(false);
updater.Start();
ManifestFetchData fetch_data(GURL("http://localhost/foo"));
@@ -582,6 +586,7 @@
service->pref_service(),
service->profile(),
kUpdateFrequencySecs);
+ updater.set_use_utility_process_for_testing(false);
updater.Start();
GURL url1("http://localhost/manifest1");
@@ -660,6 +665,7 @@
service->pref_service(),
service->profile(),
kUpdateFrequencySecs);
+ updater.set_use_utility_process_for_testing(false);
updater.Start();
GURL test_url("http://localhost/extension.crx");
@@ -724,6 +730,7 @@
ExtensionUpdater updater(
&service, service.extension_prefs(), service.pref_service(),
service.profile(), kUpdateFrequencySecs);
+ updater.set_use_utility_process_for_testing(false);
updater.Start();
GURL test_url("http://localhost/extension.crx");
@@ -775,6 +782,7 @@
ExtensionUpdater updater(
&service, service.extension_prefs(), service.pref_service(),
service.profile(), kUpdateFrequencySecs);
+ updater.set_use_utility_process_for_testing(false);
updater.Start();
EXPECT_FALSE(updater.crx_install_is_running_);
@@ -976,6 +984,7 @@
ExtensionUpdater updater(
&service, service.extension_prefs(), service.pref_service(),
service.profile(), kUpdateFrequencySecs);
+ updater.set_use_utility_process_for_testing(false);
updater.Start();
updater.set_blacklist_checks_enabled(false);
@@ -1063,6 +1072,7 @@
ExtensionUpdater updater(
&service, service.extension_prefs(), service.pref_service(),
service.profile(), kUpdateFrequencySecs);
+ updater.set_use_utility_process_for_testing(false);
updater.Start();
GURL update_url("http://www.google.com/manifest");
@@ -1219,6 +1229,7 @@
ExtensionUpdater updater(
&service, service.extension_prefs(), service.pref_service(),
service.profile(), kUpdateFrequencySecs);
+ updater.set_use_utility_process_for_testing(false);
updater.Start();
updater.StartUpdateCheck(new ManifestFetchData(GURL()));
// This should delete the newly-created ManifestFetchData.
@@ -1240,6 +1251,7 @@
ExtensionUpdater updater(
&service, service.extension_prefs(), service.pref_service(),
service.profile(), kUpdateFrequencySecs);
+ updater.set_use_utility_process_for_testing(false);
EXPECT_FALSE(updater.WillCheckSoon());
updater.Start();
EXPECT_FALSE(updater.WillCheckSoon());

Powered by Google App Engine
This is Rietveld 408576698