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

Unified Diff: chrome/renderer/chrome_content_renderer_client.cc

Issue 1656983002: Remove dependency from content_renderer on startup_metric_utils_common. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/renderer/chrome_content_renderer_client.h ('k') | content/content_renderer.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/chrome_content_renderer_client.cc
diff --git a/chrome/renderer/chrome_content_renderer_client.cc b/chrome/renderer/chrome_content_renderer_client.cc
index c0be001511a02ae388779f4c725c961d97662e03..5602bc9b30e88b5f43eda6cac18c3267098fa9ee 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -16,6 +16,7 @@
#include "base/strings/string_number_conversions.h"
#include "base/strings/string_util.h"
#include "base/strings/utf_string_conversions.h"
+#include "base/time/time.h"
#include "base/values.h"
#include "build/build_config.h"
#include "chrome/common/channel_info.h"
@@ -74,6 +75,7 @@
#include "components/pdf/renderer/pepper_pdf_host.h"
#include "components/plugins/renderer/mobile_youtube_plugin.h"
#include "components/signin/core/common/profile_management_switches.h"
+#include "components/startup_metric_utils/common/startup_metric_messages.h"
#include "components/version_info/version_info.h"
#include "components/visitedlink/renderer/visitedlink_slave.h"
#include "components/web_cache/renderer/web_cache_render_process_observer.h"
@@ -319,9 +321,20 @@ ChromeContentRendererClient::ChromeContentRendererClient() {
ChromeContentRendererClient::~ChromeContentRendererClient() {
}
+void ChromeContentRendererClient::RendererMainEntered() {
+ // Record the main entry time. It will be sent to the browser process via an
+ // IPC once the RenderThread has been created.
+ DCHECK(renderer_main_entry_time_.is_null());
+ renderer_main_entry_time_ = base::TimeTicks::Now();
jam 2016/02/01 23:28:02 why not just do this in the constructor to avoid a
fdoray 2016/02/02 01:32:49 Done.
+}
+
void ChromeContentRendererClient::RenderThreadStarted() {
RenderThread* thread = RenderThread::Get();
+ DCHECK(!renderer_main_entry_time_.is_null());
+ thread->Send(new StartupMetricHostMsg_RecordRendererMainEntryTime(
+ renderer_main_entry_time_));
+
chrome_observer_.reset(new ChromeRenderProcessObserver());
web_cache_observer_.reset(new web_cache::WebCacheRenderProcessObserver());
« no previous file with comments | « chrome/renderer/chrome_content_renderer_client.h ('k') | content/content_renderer.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698