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

Unified Diff: chrome/browser/chromeos/chrome_browser_main_chromeos.cc

Issue 2880083002: chromeos: Remove LibCrosService's ResolveNetworkProxy method (Closed)
Patch Set: Created 3 years, 7 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
« no previous file with comments | « no previous file | chromeos/dbus/services/proxy_resolution_service_provider.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/chrome_browser_main_chromeos.cc
diff --git a/chrome/browser/chromeos/chrome_browser_main_chromeos.cc b/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
index 998ce5db10de0e4555ba72d2c9fe3e031663b749..fb179adbef40fe64eb4fc945e9a52ce6345de49a 100644
--- a/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
+++ b/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
@@ -277,12 +277,6 @@ class DBusServices {
DBusThreadManager::Get()->GetPowerManagerClient());
CrosDBusService::ServiceProviderList service_providers;
- // TODO(derat): Remove this provider once all callers are using
- // |proxy_resolution_service_| instead: http://crbug.com/703217
- service_providers.push_back(
- base::MakeUnique<ProxyResolutionServiceProvider>(
- kLibCrosServiceInterface, kResolveNetworkProxy,
- base::MakeUnique<ChromeProxyResolutionServiceProviderDelegate>()));
if (GetAshConfig() == ash::Config::CLASSIC) {
// TODO(crbug.com/629707): revisit this with mustash dbus work.
service_providers.push_back(base::MakeUnique<DisplayPowerServiceProvider>(
@@ -312,8 +306,6 @@ class DBusServices {
kNetworkProxyServiceName, dbus::ObjectPath(kNetworkProxyServicePath),
CrosDBusService::CreateServiceProviderList(
base::MakeUnique<ProxyResolutionServiceProvider>(
- kNetworkProxyServiceInterface,
- kNetworkProxyServiceResolveProxyMethod,
base::MakeUnique<
ChromeProxyResolutionServiceProviderDelegate>())));
« no previous file with comments | « no previous file | chromeos/dbus/services/proxy_resolution_service_provider.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698