Chromium Code Reviews| Index: chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.cc |
| diff --git a/chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.cc b/chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.cc |
| index 4eca8affbe670aa975dd0cff50dc0c88cf67b553..4f6f2f3344a1821bdadb94ccc60931441ebf0c82 100644 |
| --- a/chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.cc |
| +++ b/chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.cc |
| @@ -265,6 +265,8 @@ void ChromeBrowserMainExtraPartsMetrics::PostBrowserStart() { |
| first_web_contents_profiler_ = |
| FirstWebContentsProfiler::CreateProfilerForFirstWebContents(this).Pass(); |
| #endif // !defined(OS_ANDROID) |
| + if (!first_web_contents_profiler_) |
| + chrome::SetBrowserStartupIsComplete(); |
| } |
| void ChromeBrowserMainExtraPartsMetrics::OnDisplayAdded( |
| @@ -284,6 +286,10 @@ void ChromeBrowserMainExtraPartsMetrics::OnDisplayMetricsChanged( |
| void ChromeBrowserMainExtraPartsMetrics::ProfilerFinishedCollectingMetrics() { |
| first_web_contents_profiler_.reset(); |
| + |
| + // TODO(michaeln): Revisit this in light of session restore, for now |
| + // claim complete after the first page loads. |
| + chrome::SetBrowserStartupIsComplete(); |
|
gab
2015/03/10 15:00:32
Doesn't feel like ExtraPartsMetrics is the right p
michaeln
2015/03/10 19:46:26
Ditto any guidance.
I picked this location becaus
gab
2015/03/10 19:55:27
Then perhaps put it directly in FirstWebContentsPr
michaeln
2015/03/20 00:21:27
I put it directly into ChromeBrowserMainParts, not
|
| } |
| void ChromeBrowserMainExtraPartsMetrics::EmitDisplaysChangedMetric() { |