Reporting top cpu and memory consumers via rappor on chromeos

chrome/browser/task_management/providers/web_contents/renderer_task.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/task_management/providers/web_contents/renderer_task.h"
 
 #include "base/i18n/rtl.h"
 #include "base/numerics/safe_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
@@ skipping 41 matching lines @@
   if (index != std::string::npos)
     return cache.GetNameOfProfileAtIndex(index);
 
   return base::string16();
 }
 
 inline bool IsRendererResourceSamplingDisabled(int64 flags) {
   return (flags & (REFRESH_TYPE_V8_MEMORY | REFRESH_TYPE_WEBCACHE_STATS)) == 0;
 }
 
+std::string GetRapporSampleName(content::WebContents* web_contents) {
+  return web_contents->GetVisibleURL().host();

[ncarter (slow), 2015/11/10 23:29:57]

I recommend capturing the origin (GURL::GetOrigin()), not just the host. It'll
make blaming extensions easier, for instance.

Are there any URLs you want to exclude from sampling (e.g. localhost
equivalents, about:blank, file:// scheme URIs) ?

[afakhry, 2015/11/12 00:21:28]

Done.

+}
+
 }  // namespace
 
 RendererTask::RendererTask(const base::string16& title,
                            const gfx::ImageSkia* icon,
                            content::WebContents* web_contents,
                            content::RenderProcessHost* render_process_host)
-    : Task(title, icon, render_process_host->GetHandle()),
+    : Task(title,
+           GetRapporSampleName(web_contents),
+           icon,
+           render_process_host->GetHandle()),
       web_contents_(web_contents),
       render_process_host_(render_process_host),
       renderer_resources_sampler_(
           CreateRendererResourcesSampler(render_process_host_)),
       render_process_id_(render_process_host_->GetID()),
       v8_memory_allocated_(0),
       v8_memory_used_(0),
       webcache_stats_(),
       profile_name_(GetRendererProfileName(render_process_host_)) {
   // All renderer tasks are capable of reporting network usage, so the default
   // invalid value of -1 doesn't apply here.
   OnNetworkBytesRead(0);
 
   // Tag the web_contents with a |ContentFaviconDriver| (if needed) so that
   // we can use it to observe favicons changes.
   favicon::CreateContentFaviconDriverForWebContents(web_contents);
   favicon::ContentFaviconDriver::FromWebContents(web_contents)->AddObserver(
       this);
 }
 
 RendererTask::~RendererTask() {
   favicon::ContentFaviconDriver::FromWebContents(web_contents())->
       RemoveObserver(this);
 }
 
+void RendererTask::UpdateRapporSampleName() {
+  set_rappor_sample_name(GetRapporSampleName(web_contents()));
+}
+
 void RendererTask::Activate() {
   if (!web_contents_->GetDelegate())
     return;
 
   web_contents_->GetDelegate()->ActivateContents(web_contents_);
 }
 
 void RendererTask::Refresh(const base::TimeDelta& update_interval,
                            int64 refresh_flags) {
   Task::Refresh(update_interval, refresh_flags);
@@ skipping 114 matching lines @@
     if (is_incognito)
       message_id = IDS_TASK_MANAGER_EXTENSION_INCOGNITO_PREFIX;
     else
       message_id = IDS_TASK_MANAGER_EXTENSION_PREFIX;
   }
 
   return l10n_util::GetStringFUTF16(message_id, title);
 }
 
 }  // namespace task_management