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

Issue 3050040: Reland 54771 (and 54795) To enable TCP Preconnection by default... (Closed)

Created:
10 years, 4 months ago by jar (doing other things)
Modified:
9 years, 7 months ago
Reviewers:
Mike Belshe
CC:
chromium-reviews, cbentzel+watch_chromium.org, brettw-cc_chromium.org, darin-cc_chromium.org, ben+cc_chromium.org
Visibility:
Public.

Description

Reland 54771 (and 54795) To enable TCP Preconnection by default I pulled out the code to update the socket connectivity stats. I added defensive code which should preclude the crash that was reported on the stability bot. I added a second call to update the stats from ~ClientSocketHandle to ensure that we updated the related ClientSocket when we are torn down. As noted in the original checkin: Enable speculative preconnection by default Added histogram to track preconnection utilization (vs waste). BUG=42694 r=mbelshe Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=55197

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Patch Set 4 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+225 lines, -38 lines) Patch
M chrome/browser/browser_main.cc View 1 2 3 1 chunk +6 lines, -1 line 0 comments Download
M chrome/browser/net/preconnect.h View 3 chunks +19 lines, -5 lines 0 comments Download
M chrome/browser/net/preconnect.cc View 6 chunks +43 lines, -21 lines 0 comments Download
M chrome/browser/net/predictor.h View 3 chunks +12 lines, -4 lines 0 comments Download
M chrome/browser/net/predictor.cc View 1 chunk +14 lines, -0 lines 0 comments Download
M chrome/common/chrome_switches.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/common/chrome_switches.cc View 1 chunk +3 lines, -0 lines 0 comments Download
M net/net.gyp View 1 chunk +1 line, -0 lines 0 comments Download
M net/socket/client_socket.h View 2 chunks +38 lines, -0 lines 0 comments Download
A net/socket/client_socket.cc View 1 2 3 1 chunk +69 lines, -0 lines 0 comments Download
M net/socket/client_socket_handle.h View 2 chunks +13 lines, -1 line 0 comments Download
M net/socket/client_socket_handle.cc View 1 chunk +1 line, -0 lines 0 comments Download
M net/socket/client_socket_pool_base.h View 3 chunks +5 lines, -6 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
jar (doing other things)
Hopefully, three times is the charm. The changes from the last attempted landing are, as ...
10 years, 4 months ago (2010-08-05 23:20:16 UTC) #1
Mike Belshe
10 years, 4 months ago (2010-08-06 06:53:36 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698