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

Unified Diff: chrome/browser/component_updater/component_updater_configurator.cc

Issue 80963004: Change the protocol for component updates from http to https. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 1 month 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/component_updater/component_updater_configurator.cc
diff --git a/chrome/browser/component_updater/component_updater_configurator.cc b/chrome/browser/component_updater/component_updater_configurator.cc
index c325810b16ebcd1f313a094b5f8b8f317afbab30..80f7a262eecec00d4cf857dc96413813214697b8 100644
--- a/chrome/browser/component_updater/component_updater_configurator.cc
+++ b/chrome/browser/component_updater/component_updater_configurator.cc
@@ -40,10 +40,15 @@ extern const char kSwitchDisablePings[] = "disable-pings";
// Sets the URL for updates.
const char kSwitchUrlSource[] = "url-source";
+#define COMPONENT_UPDATER_SERVICE_ENDPOINT \
+ "//clients2.google.com/service/update2"
+
// The default url for the v3 protocol service endpoint. Can be
// overridden with --component-updater=url-source=someurl.
-const char kDefaultUrlSource[] =
- "http://clients2.google.com/service/update2";
+const char kDefaultUrlSource[] = "https:" COMPONENT_UPDATER_SERVICE_ENDPOINT;
+
+// The url to send the pings to.
+const char kPingUrl[] = "http:" COMPONENT_UPDATER_SERVICE_ENDPOINT;
#if defined(OS_WIN)
// Disables differential updates.
@@ -176,7 +181,7 @@ GURL ChromeConfigurator::UpdateUrl() {
}
GURL ChromeConfigurator::PingUrl() {
- return pings_enabled_ ? UpdateUrl() : GURL();
+ return pings_enabled_ ? GURL(kPingUrl) : GURL();
}
const char* ChromeConfigurator::ExtraRequestParams() {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698