Comment #17 on issue 14478 by bugdro...@chromium.org: Add profiling data  
for recent network requests
http://code.google.com/p/chromium/issues/detail?id=14478

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=25584

------------------------------------------------------------------------
r25584 | ero...@chromium.org | 2009-09-05 17:48:37 -0700 (Sat, 05 Sep 2009)  
| 43 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/net/proxy/proxy_config.cc?r1=25584&r2=25583
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/net/proxy/proxy_config_unittest.cc?r1=25584&r2=25583
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/net/proxy/proxy_service.cc?r1=25584&r2=25583

Aesthetic changes to the proxy configuration log format.

Also in this change, I stop brodacasting the proxy settings changes to  
LOG(INFO) --- This is because navigating  
to "view-net-internal:proxyservice.config" offers a better solution, so no  
point polluting the log file.

Note that the format change is intended to make the string more succint and  
human readable. For example, here is a before and after comparison:

------------
New version:
------------

Automatic settings:
   Auto-detect: No
   Custom PAC script: [None]
Manual settings:
   Proxy server: [None]
   Bypass list: [None]
   Bypass local names: No

------------
Old version:
------------

{
   auto_detect: 0
   pac_url:
   proxy_rules:
   {
     type: TYPE_NO_RULES
     single_proxy:
     proxy_for_http:
     proxy_for_https:
     proxy_for_ftp:
     socks_proxy:
   }
   proxy_bypass_local_names: 0
   proxy_bypass_list:
   id: 1
}

BUG=http://crbug.com/14478
TEST=ProxyConfigTest.ToString

Review URL: http://codereview.chromium.org/198039
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to