redesign and reimplement proxy config service and tracker, revise proxy ui on cros

chrome/browser/automation/testing_automation_provider.cc

Index: chrome/browser/automation/testing_automation_provider.cc
===================================================================
--- chrome/browser/automation/testing_automation_provider.cc	(revision 106059)
+++ chrome/browser/automation/testing_automation_provider.cc	(working copy)
@@ -2341,10 +2341,6 @@
   handler_map["NetworkScan"] = &TestingAutomationProvider::NetworkScan;
   handler_map["ToggleNetworkDevice"] =
       &TestingAutomationProvider::ToggleNetworkDevice;
-  handler_map["GetProxySettings"] =
-      &TestingAutomationProvider::GetProxySettings;
-  handler_map["SetProxySettings"] =
-      &TestingAutomationProvider::SetProxySettings;
   handler_map["ConnectToCellularNetwork"] =
       &TestingAutomationProvider::ConnectToCellularNetwork;
   handler_map["DisconnectFromCellularNetwork"] =
@@ -2553,6 +2549,10 @@
   browser_handler_map["CaptureProfilePhoto"] =
       &TestingAutomationProvider::CaptureProfilePhoto;
   browser_handler_map["GetTimeInfo"] = &TestingAutomationProvider::GetTimeInfo;
+  browser_handler_map["GetProxySettings"] =
+      &TestingAutomationProvider::GetProxySettings;
+  browser_handler_map["SetProxySettings"] =
+      &TestingAutomationProvider::SetProxySettings;

[Mattias Nissler (ping if slow), 2011/10/19 19:27:36]

Why did you move this down? It seems that previous order reflects declaration
order of the handler function.

[kuan, 2011/10/20 00:41:11]

dave, who wrote this code and reviewed this changelist, advises me this is the
way to pass in a Profile (via Browser object's Profile()) as construction
parameter to ProxyCrosSettingsProvider.

[Mattias Nissler (ping if slow), 2011/10/24 10:18:28]

Ah, I had missed the handler_map -> browser_handler_map change.

 #endif  // defined(OS_CHROMEOS)
 
   // Look for command in handlers that take a Browser handle.