diff options
| author | tburkard@chromium.org <tburkard@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-12-05 00:11:08 +0000 |
|---|---|---|
| committer | tburkard@chromium.org <tburkard@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-12-05 00:11:08 +0000 |
| commit | 97414238180cbb0c914fd0250ec6a79df0741e90 (patch) | |
| tree | 8a1980f71fda3d0e649b9a030172b2fdc3e72322 /chrome | |
| parent | c5a1877d73540fce7ba4f8b8ad5ba553969334c8 (diff) | |
| download | chromium_src-97414238180cbb0c914fd0250ec6a79df0741e90.zip chromium_src-97414238180cbb0c914fd0250ec6a79df0741e90.tar.gz chromium_src-97414238180cbb0c914fd0250ec6a79df0741e90.tar.bz2 | |
Histogram first paint PLT's for Cache Sensitivity Analysis.
R=rvargas@chromium.org, jhawkins@chromium.org
Review URL: https://codereview.chromium.org/11419319
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171091 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
| -rw-r--r-- | chrome/renderer/page_load_histograms.cc | 19 |
1 files changed, 17 insertions, 2 deletions
diff --git a/chrome/renderer/page_load_histograms.cc b/chrome/renderer/page_load_histograms.cc index dc661f8..8d8775c 100644 --- a/chrome/renderer/page_load_histograms.cc +++ b/chrome/renderer/page_load_histograms.cc @@ -244,14 +244,19 @@ void PageLoadHistograms::Dump(WebFrame* frame) { PLT_HISTOGRAM("PLT.RequestToFinish", finish_all_loads - request); } PLT_HISTOGRAM("PLT.CommitToFinish", finish_all_loads - commit); + + scoped_ptr<TimeDelta> begin_to_first_paint; + scoped_ptr<TimeDelta> commit_to_first_paint; if (!first_paint.is_null()) { // 'first_paint' can be before 'begin' for an unknown reason. // See bug http://crbug.com/125273 for details. if (begin <= first_paint) { - PLT_HISTOGRAM("PLT.BeginToFirstPaint", first_paint - begin); + begin_to_first_paint.reset(new TimeDelta(first_paint - begin)); + PLT_HISTOGRAM("PLT.BeginToFirstPaint", *begin_to_first_paint); } DCHECK(commit <= first_paint); - PLT_HISTOGRAM("PLT.CommitToFirstPaint", first_paint - commit); + commit_to_first_paint.reset(new TimeDelta(first_paint - commit)); + PLT_HISTOGRAM("PLT.CommitToFirstPaint", *commit_to_first_paint); } if (!first_paint_after_load.is_null()) { // 'first_paint_after_load' can be before 'begin' for an unknown reason. @@ -809,6 +814,16 @@ void PageLoadHistograms::Dump(WebFrame* frame) { PLT_HISTOGRAM(base::FieldTrial::MakeName( "PLT.BeginToFinish_CacheSensitivity", "CacheSensitivityAnalysis"), begin_to_finish_all_loads); + if (begin_to_first_paint.get()) { + PLT_HISTOGRAM(base::FieldTrial::MakeName( + "PLT.BeginToFirstPaint_CacheSensitivity", "CacheSensitivityAnalysis"), + *begin_to_first_paint); + } + if (commit_to_first_paint.get()) { + PLT_HISTOGRAM(base::FieldTrial::MakeName( + "PLT.CommitToFirstPaint_CacheSensitivity", "CacheSensitivityAnalysis"), + *commit_to_first_paint); + } } // Since there are currently no guarantees that renderer histograms will be |