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

Issue 6544004: Add field trial for A/B testing of SSL False Start. (Closed)

Created:
9 years, 10 months ago by Mike Belshe
Modified:
9 years, 5 months ago
CC:
chromium-reviews, wtc
Visibility:
Public.

Description

Add field trial for A/B testing of SSL False Start. Enable two histograms for measurement: - measure PLT differences - measure SSL_Connect_Latency conditioned on the test. BUG=none TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=75402

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+61 lines, -0 lines) Patch
M chrome/browser/browser_main.h View 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/browser/browser_main.cc View 1 2 2 chunks +23 lines, -0 lines 0 comments Download
M chrome/renderer/page_load_histograms.cc View 1 chunk +21 lines, -0 lines 0 comments Download
M net/socket/ssl_client_socket_pool.cc View 2 chunks +14 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Mike Belshe
9 years, 10 months ago (2011-02-18 00:18:37 UTC) #1
jar (doing other things)
9 years, 10 months ago (2011-02-18 01:49:38 UTC) #2
LGTM

caveat: Please run the code, and be sure you can sometimes see each of the
histograms.  I'm always nervous about subtle typos that my eyes can't see.

Please also update the XML description file when you have a chance.

Powered by Google App Engine
This is Rietveld 408576698