summaryrefslogtreecommitdiffstats
path: root/net
Commit message (Collapse)AuthorAgeFilesLines
* Make sure we flush ssl pool when CloseCurrentConnections is called.lzheng@chromium.org2010-08-191-0/+1
| | | | | | | | | BUG=none TEST=none (this is a method for testing/debugging) Review URL: http://codereview.chromium.org/3171016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56698 0039d316-1c4b-4281-b951-d872f2087c98
* SpdySessionPool closes down sessions accurately now.erikchen@google.com2010-08-196-33/+53
| | | | | | | | | | This may or may not fix 50265. Regardless, the previous behavior was incorrect. TEST=none BUG=none Review URL: http://codereview.chromium.org/3150023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56696 0039d316-1c4b-4281-b951-d872f2087c98
* Don't run some more HttpNetworkTransactionTest under ThreadSanitizertimurrrr@chromium.org2010-08-193-8/+6
| | | | | | | | BUG=51145 TBR=glider Review URL: http://codereview.chromium.org/3174018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56678 0039d316-1c4b-4281-b951-d872f2087c98
* Don't run ↵timurrrr@chromium.org2010-08-191-0/+4
| | | | | | | | | | HttpNetworkTransactionTest.UseAlternateProtocolForNpnSpdyWithExistingSpdySession under ThreadSanitizer on Windows TBR=glider BUG=52699 Review URL: http://codereview.chromium.org/3126019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56658 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Remove unnecessary STL #includes from src/net.viettrungluu@chromium.org2010-08-1929-59/+16
| | | | | | | | | | | | This removes obviously unneeded inclusions of <map>, <set>, <string>, <vector>, etc. from src/net. BUG=none TEST=builds Review URL: http://codereview.chromium.org/3132022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56654 0039d316-1c4b-4281-b951-d872f2087c98
* A WebKit layout test failed with the HttpNetworkTransaction refactoring.mbelshe@chromium.org2010-08-191-1/+6
| | | | | | | | | | | | | | | The problem is that upload progress continues after the socket has been returned to the pool, but because we had zeroed out the stream, we could no longer report a value. (We'd go from 100% done, to 0 progress because GetUploadProgress() would return 0) BUG=none TEST=none TBR=vandebo@chromium.org Review URL: http://codereview.chromium.org/3189011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56650 0039d316-1c4b-4281-b951-d872f2087c98
* Extract connection logic from HttpNetworkTransaction into a newmbelshe@chromium.org2010-08-1928-1150/+2227
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | HttpStreamFactory. The HttpNetworkTransaction now deals exclusively with streams rather than connections directly. This cut the size of HTN roughly in in half. The HttpNetworkTransaction is still responsible for all proxy and server authentication functions. This is because the streams may come and go - we could create a stream, have the server declare auth is needed, and then the next attempt would be on a different stream. So Auth belongs on the HNT. The HNT no longer has direct access to the connection itself; instead, it only knows of an HttpStream. The StreamRequest, however, is responsible for determining whether the connection needs to use a proxy, whether AlternateProtocols are available, and whether the connection should be SPDY or HTTP. Other changes: - moved some static configuration methods from HNT to HttpStreamFactory. - added some methods to the HttpStream. BUG=none TEST=all Review URL: http://codereview.chromium.org/3171002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56646 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Remove unneeded headers from base/ (part 6)thestig@chromium.org2010-08-194-12/+8
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3093013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56641 0039d316-1c4b-4281-b951-d872f2087c98
* Add a command-line flag to disable SSL/TLS False Startdavidben@chromium.org2010-08-186-2/+49
| | | | | | | | | | | | | | Some servers are not compatible with False Start. Adding a command-line flag will make it easier to test and verify such cases. Also, blacklist www.picnik.com as incompatible with False Start. BUG=50650 TEST=see bug Review URL: http://codereview.chromium.org/3167015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56622 0039d316-1c4b-4281-b951-d872f2087c98
* Send user agent with CONNECT requests.vandebo@chromium.org2010-08-188-6/+61
| | | | | | | | | BUG=52591 TEST=HttpNetworkTransactionTest.BuildTunnelRequest_UserAgent Review URL: http://codereview.chromium.org/3164024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56621 0039d316-1c4b-4281-b951-d872f2087c98
* Don't resolve IP literals.vandebo@chromium.org2010-08-186-85/+139
| | | | | | | | | | | | | For each resolution request this checks to see if this 'host' is a literal ip address. If so, it synthesises a struct addrinfo and returns it without adding it to the cache. BUG=39830 TEST=unit tests, new and old Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=56384 Review URL: http://codereview.chromium.org/3023048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56615 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Convert FormatBytes to string16."evan@chromium.org2010-08-181-2/+1
| | | | | | This reverts commit r56603. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56608 0039d316-1c4b-4281-b951-d872f2087c98
* Convert FormatBytes to string16.evan@chromium.org2010-08-181-1/+2
| | | | | | | | | | I considered ASCII, but it's producing what is intended to be a human-readable string. For example, in theory the "bytes/s" annotation could be localized into a language where the "b" is non-ASCII. Review URL: http://codereview.chromium.org/3107021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56603 0039d316-1c4b-4281-b951-d872f2087c98
* DnsRRResolver: hoist inner class and callback on MessageLoop.agl@chromium.org2010-08-183-125/+191
| | | | | | | | | | | | | | | | | This change moves the Response class to the top level to save other files from having to #include the header in order to get a pointer to a Response object. This change also modifies the semantics of resolution: the callback will now be made of the origin MessageLoop, rather than a random thread. TEST=net_unittests BUG=none http://codereview.chromium.org/3110015/show git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56585 0039d316-1c4b-4281-b951-d872f2087c98
* Remove CRLF.cbentzel@chromium.org2010-08-181-1/+1
| | | | | | | | | | | The svn:eol-style properties are correct on this file. When using svn, the CRLF does not show up due to munging. Using git it does show up. BUG=None TEST=None Review URL: http://codereview.chromium.org/3194006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56569 0039d316-1c4b-4281-b951-d872f2087c98
* Added a new MockSocket that enforces ordering of reads/writes.erikchen@google.com2010-08-188-40/+530
| | | | | | | | | | Re-enabled two broken SPDY tests, using new MockSocket. All SPDY tests should eventually convert to using this new object. TEST=none BUG=none Review URL: http://codereview.chromium.org/3179016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56560 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 56539 - Added HostResolveImpl Requests and Jobs to log.mmenke@chromium.org2010-08-1818-370/+96
| | | | | | | | | | | | | | | | | | ConnectJobs point to the requests, Requests point back to ConnectJobs and to the DNS lookup they were attached to, if any. Also CONNECT_JOBs are now identified by their host/port on the Requests list. BUG= 46844 TEST= Look at the net-internals screen. Review URL: http://codereview.chromium.org/3080034 TBR=eroman@chromium.org Review URL: http://codereview.chromium.org/3137022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56556 0039d316-1c4b-4281-b951-d872f2087c98
* Added HostResolveImpl Requests and Jobs to log.mmenke@chromium.org2010-08-1818-96/+370
| | | | | | | | | | | | | | | ConnectJobs point to the requests, Requests point back to ConnectJobs and to the DNS lookup they were attached to, if any. Also CONNECT_JOBs are now identified by their host/port on the Requests list. BUG= 46844 TEST= Look at the net-internals screen. Review URL: http://codereview.chromium.org/3080034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56539 0039d316-1c4b-4281-b951-d872f2087c98
* Syncing url_to_filename_encoder with internal version.mbelshe@chromium.org2010-08-1711-420/+510
| | | | | | | | | | | | | | | I had to implement UrlUtilities::Unescape and ported over the tests as well. Deleted redundant copy of url_to_filename_encoder.h and pointed the one link to that at the new location: net/tools/dump_cache/url_to_filename_encoder.h This patch is on behalf of sligocki@google.com BUG=none TEST=url_to_filename_encoder_unittest. Review URL: http://codereview.chromium.org/3117019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56454 0039d316-1c4b-4281-b951-d872f2087c98
* Test server cleanup patch of death:phajdan.jr@chromium.org2010-08-1711-1006/+821
| | | | | | | | | | | | | | - reduce the number of different classes - clean up the internal code - modify the interface to expose less internal details (this will allow more flexibility with port numbers) TEST=many BUG=49680 Review URL: http://codereview.chromium.org/3080029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56405 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up the client certificate filtering code on Linuxdavidben@chromium.org2010-08-171-29/+26
| | | | | | | | | | | | | | | | | | | | | Instead of filtering over all certificates and then picking the ones suitable for client auth, NSS provides a function that does this in one go. In addition, delay checking for the private key until we actually send the certificate. While this means we may potentially offer a client certificate with no matching private key, such a store should not exist. This avoids requiring us authenticate to every certificate offered. In addition, it delays the password dialog, when we implement it, until after the user has designated that they are interested in a certificate. This mirrors Firefox's use of NSS. R=wtc BUG=none TEST=linux ssl client auth works Review URL: http://codereview.chromium.org/3177018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56404 0039d316-1c4b-4281-b951-d872f2087c98
* Re-land part of r55416, minus actually moving the test_suite.h contents to a ↵brettw@chromium.org2010-08-172-1/+2
| | | | | | | | | | | | | | | | | | | | | | | .cc file. Cleanup in base. This moves the implementation (and a bunch of header file dependencies) from the multiprocess test and the test_suite headers to .cc files. Moves multiprocess_test to the test directory, and all of this stuff to the existing base_test_support project. I also used the base namespace. Previously other projects included this functionality just by #include because it was all inline, so I had to add dependencies on base_test_support in a few places. Moves and renames the command line switch this was using to base_switches. Move the base switch for process type to chrome switches. TEST=none BUG=none Review URL: http://codereview.chromium.org/3026055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56399 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 56376 - This CL will introduce a new way to do exponential back-off ↵joi@chromium.org2010-08-1713-856/+9
| | | | | | | | | | | | | | | | | | | | | | on failure within Chromium. It is a network level implementation and should constitute a good enough bottleneck to manage every outgoing http request. Committing for malavv@google.com. Original review: http://codereview.chromium.org/2487001/show R=phajdan-jr BUG=none TEST=unit tests Review URL: http://codereview.chromium.org/3005049 TBR=joi@chromium.org Review URL: http://codereview.chromium.org/3165029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56393 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 56384 - Don't resolve IP literals.vandebo@chromium.org2010-08-177-130/+91
| | | | | | | | | | | | | | For each resolution request this checks to see if this 'host' is a literal ip address. If so, it synthesises a struct addrinfo and returns it without adding it to the cache. BUG=39830 TEST=unit tests, new and old Review URL: http://codereview.chromium.org/3023048 TBR=vandebo@chromium.org Review URL: http://codereview.chromium.org/3115014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56385 0039d316-1c4b-4281-b951-d872f2087c98
* Don't resolve IP literals.vandebo@chromium.org2010-08-177-91/+130
| | | | | | | | | | | For each resolution request this checks to see if this 'host' is a literal ip address. If so, it synthesises a struct addrinfo and returns it without adding it to the cache. BUG=39830 TEST=unit tests, new and old Review URL: http://codereview.chromium.org/3023048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56384 0039d316-1c4b-4281-b951-d872f2087c98
* This CL will introduce a new way to do exponential back-off on failure within joi@chromium.org2010-08-1713-9/+856
| | | | | | | | | | | | | | | | Chromium. It is a network level implementation and should constitute a good enough bottleneck to manage every outgoing http request. Committing for malavv@google.com. Original review: http://codereview.chromium.org/2487001/show BUG=none TEST=unit tests Review URL: http://codereview.chromium.org/3005049 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56376 0039d316-1c4b-4281-b951-d872f2087c98
* SPDY flow control: fix for WINDOW_UPDATEs arriving while request is being sent.agayev@chromium.org2010-08-178-76/+93
| | | | | | | | | BUG=48100 TEST=net_unittests Review URL: http://codereview.chromium.org/3048058 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56355 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "DnsRRResolver: hoist inner class and callback on MessageLoop."agl@chromium.org2010-08-173-111/+49
| | | | | | (Everything goes to hell when the try servers don't work.) git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56344 0039d316-1c4b-4281-b951-d872f2087c98
* DnsRRResolver: hoist inner class and callback on MessageLoop.agl@chromium.org2010-08-173-49/+111
| | | | | | | | | | | | | | | | | This change moves the Response class to the top level to save other files from having to #include the header in order to get a pointer to a Response object. This change also modifies the semantics of resolution: the callback will now be made of the origin MessageLoop, rather than a random thread. TEST=net_unittests BUG=none http://codereview.chromium.org/3110015/show git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56342 0039d316-1c4b-4281-b951-d872f2087c98
* More uninitialized class member vars.finnur@chromium.org2010-08-173-5/+8
| | | | | | | | | | BUG=None TEST=None CID=1473, 5546, 11622 Review URL: http://codereview.chromium.org/3123009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56322 0039d316-1c4b-4281-b951-d872f2087c98
* Watch the Internet Explorer registry keys so we can detect proxy settings ↵eroman@chromium.org2010-08-174-18/+191
| | | | | | | | | | | | | | | | | changes faster. Previously we would poll on a background thread every 5 seconds (triggered by network activity) to check for changes. This meant there was a 5 second window during which new requests might still be using the old settings. With this CL, the settings are noticed almost immediately and the continuous polling is de-emphasized. (We still poll, but less often. This is a precaution since we are relying on implementation details of a winhttp function, and can't be sure it won't change in the future, or is even completely accurate right now). BUG=52120 TEST=Open chrome://net-internals/#requests. Request some webpage. Now make changes to the Internet explorer proxy settinsg. You should see a PROXY_CONFIG_CHANGED event emitted immediately after saving each change in the dialog box. Review URL: http://codereview.chromium.org/3161016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56305 0039d316-1c4b-4281-b951-d872f2087c98
* Revert the workaround for servers that use tiny DH keys.wtc@chromium.org2010-08-178-68/+69
| | | | | | | | | | | | | | | | | | Add the new error code ERR_SSL_WEAK_SERVER_KEY for these broken servers. Use the new SSL_RENEGOTIATE_TRANSITIONAL option. On the client side it is equivalent to SSL_RENEGOTIATE_UNRESTRICTED. R=agl BUG=51694 TEST=Visit https://portal-plumprod.cgc.enbridge.com and https://www.citylink.com.au. The network error page should display the error message: Error 129 (net::ERR_SSL_WEAK_SERVER_KEY): Unknown error. Review URL: http://codereview.chromium.org/3149012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56283 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Header cleanup in renderer_host.jhawkins@chromium.org2010-08-162-5/+5
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3164011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56237 0039d316-1c4b-4281-b951-d872f2087c98
* Remove test isolation enforcement code.phajdan.jr@chromium.org2010-08-161-1/+0
| | | | | | | | | | | | | | | | | | | I think it was a failed experiment, and it only ever worked for base_unittests. This removes some potentially risky code from LazyInstance that was only needed for the test isolation. The lesson from this experiment is that removing singletons works better than trying to make tests work fine even when singletons are overused. TEST=none BUG=12710 Review URL: http://codereview.chromium.org/3127012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56229 0039d316-1c4b-4281-b951-d872f2087c98
* Fix a bug in the mock sockets. For SSL sockets, when they werembelshe@chromium.org2010-08-161-2/+2
| | | | | | | | | | | | | | | completing the connect asynchronously, we never set the connected_ flag to true. This is usually benign, because we don't often check the flag. BUG=none TEST=none Review URL: http://codereview.chromium.org/3107013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56219 0039d316-1c4b-4281-b951-d872f2087c98
* Re-land just the project file changes from r55416. This makes projects that usebrettw@chromium.org2010-08-151-0/+1
| | | | | | | | | base test files dependent on test_support_base. TEST=none BUG=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56172 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup Registry API: part 3.tfarina@chromium.org2010-08-141-4/+6
| | | | | | | | | | | | - Remove the default arguments from the constructor. - Add a empty constructor to handle the case of the default arguments were used. BUG=44644 TEST=trybots Review URL: http://codereview.chromium.org/3172009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56123 0039d316-1c4b-4281-b951-d872f2087c98
* CommandLine: eliminate wstring-accepting AppendLooseValueevan@chromium.org2010-08-132-2/+2
| | | | | | | | Instead use AppendArg variants which accept a FilePath or an ASCII string. Review URL: http://codereview.chromium.org/3134008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56100 0039d316-1c4b-4281-b951-d872f2087c98
* Simple offline startup integration test.skrul@chromium.org2010-08-132-1/+38
| | | | | | | | | | This is a simple offline startup integration test. This change also includes the groundwork for more complex offline startup test cases, including the ability to enable and disable network connectivity as well as configure the mock sync server with a username. BUG=47918 Review URL: http://codereview.chromium.org/3078031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56087 0039d316-1c4b-4281-b951-d872f2087c98
* Added check for null delegate when a socket stream closes between ↵cdn@chromium.org2010-08-131-1/+1
| | | | | | | | | | DoResolveHost and DoResolveHostDone. BUG=51846 TEST=None Review URL: http://codereview.chromium.org/3145010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56076 0039d316-1c4b-4281-b951-d872f2087c98
* Initialize all data members in HTTPResponseInfo's new ctor and remove the ↵michaeln@chromium.org2010-08-131-0/+2
| | | | | | | | | | | related valgrind suppression. BUG=51940 TEST=n/a Review URL: http://codereview.chromium.org/3163010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56068 0039d316-1c4b-4281-b951-d872f2087c98
* Fix check for whether we need to consider client auth as partmbelshe@chromium.org2010-08-132-2/+2
| | | | | | | | | | | | | | of SSLClientSocket construction. Before we checked for if there was a client_cert present, but we really should check the boolean for SSLConfig.send_client_cert. BUG=none TEST=none Review URL: http://codereview.chromium.org/3155016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56056 0039d316-1c4b-4281-b951-d872f2087c98
* Update to NSS 3.12.7.wtc@chromium.org2010-08-1314-95/+100
| | | | | | | | | R=agl BUG=51694 TEST=No build errors or test failures. Review URL: http://codereview.chromium.org/3177012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56042 0039d316-1c4b-4281-b951-d872f2087c98
* Adding ability to sort by date to directory_lister.cc. Adding support for ↵dhg@chromium.org2010-08-132-8/+61
| | | | | | | | | | title and making so long filenames are ellipsis. BUG=chromium-os:5565 chromium-os:5566 TEST=manual Review URL: http://codereview.chromium.org/3177010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56036 0039d316-1c4b-4281-b951-d872f2087c98
* Redirects NetLog output to the log file, when the command linemmenke@chromium.org2010-08-132-0/+84
| | | | | | | | | | | | option "--log-net-log" is used. BUG= 50987 TEST= Run "chrome --log-net-log --enable-logging --log-level=0" And then check to make sure the log contains the NetLog events. Review URL: http://codereview.chromium.org/3040051 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56021 0039d316-1c4b-4281-b951-d872f2087c98
* Emit an event to NetLog whenever the proxy settings change.eroman@chromium.org2010-08-135-121/+190
| | | | | | | | | | Also removes the operator<< on ProxyConfig. BUG=52004 Review URL: http://codereview.chromium.org/3144008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55999 0039d316-1c4b-4281-b951-d872f2087c98
* sync: Add sessions to protocol and chromiumsync python test server.jerrica@google.com2010-08-131-6/+6
| | | | | | | BUG=30519 Review URL: http://codereview.chromium.org/3107008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55973 0039d316-1c4b-4281-b951-d872f2087c98
* Skip CertDatabaseNSSTest.ImportFromPKCS12 on heapcheck, since the leaks ↵mattm@chromium.org2010-08-121-1/+6
| | | | | | | | | | | can't be suppressed cleanly. BUG=51988,51332,51330,51328 TEST=heapcheck bot should go green Review URL: http://codereview.chromium.org/3112007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55955 0039d316-1c4b-4281-b951-d872f2087c98
* Add NSS PKCS12 import/export functions to CertDatabase.mattm@chromium.org2010-08-127-1/+640
| | | | | | | | | | | Use sql: prefix when opening NSS UserDB (this will affect existing Chromeos installations, which had been using the old berkelydb format.) BUG=19991,51327,51328,51330,51332 TEST=net/base/cert_database_nss_unittest.cc Review URL: http://codereview.chromium.org/3018038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55916 0039d316-1c4b-4281-b951-d872f2087c98
* Net-internal's Javascript code no longer requires all enties have a unique ↵mmenke@chromium.org2010-08-121-6/+1
| | | | | | | | | | | | | | | | | | | | | | source id. It will assign entries with a source type of NONE their own unique negative id, which will never be displayed. Also, the id column in the Requests tab is now right-aligned. BUG= 48806 TEST= Run chromium while your ip address changes. Alternatively, add the following code to ConnectJob::LogConnectStart(), or somewhere else that looks promising: net_log().net_log()->AddEntry(net::NetLog::TYPE_NETWORK_IP_ADDRESSSES_CHANGED, base::TimeTicks::Now(), net::NetLog::Source(), net::NetLog::PHASE_NONE, NULL); Review URL: http://codereview.chromium.org/2883061 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55900 0039d316-1c4b-4281-b951-d872f2087c98