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

Unified Diff: content/browser/gpu/gpu_process_host.cc

Issue 2310623002: Remove GPU memory UMAs (Closed)
Patch Set: Rebase Created 4 years, 3 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 | « content/browser/gpu/gpu_process_host.h ('k') | content/common/gpu_host_messages.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/gpu/gpu_process_host.cc
diff --git a/content/browser/gpu/gpu_process_host.cc b/content/browser/gpu/gpu_process_host.cc
index e26831e93c061eef68c887f79b950573d8e4d2fc..4319a1c64b651bb9657962f9e4d53e79289fa138 100644
--- a/content/browser/gpu/gpu_process_host.cc
+++ b/content/browser/gpu/gpu_process_host.cc
@@ -441,8 +441,7 @@ GpuProcessHost::GpuProcessHost(int host_id, GpuProcessKind kind)
swiftshader_rendering_(false),
kind_(kind),
process_launched_(false),
- initialized_(false),
- uma_memory_stats_received_(false) {
+ initialized_(false) {
if (base::CommandLine::ForCurrentProcess()->HasSwitch(
switches::kSingleProcess) ||
base::CommandLine::ForCurrentProcess()->HasSwitch(
@@ -490,19 +489,6 @@ GpuProcessHost::~GpuProcessHost() {
UMA_HISTOGRAM_COUNTS_100("GPU.AtExitSurfaceCount",
GpuSurfaceTracker::Get()->GetSurfaceCount());
#endif
- UMA_HISTOGRAM_BOOLEAN("GPU.AtExitReceivedMemoryStats",
- uma_memory_stats_received_);
-
- if (uma_memory_stats_received_) {
- UMA_HISTOGRAM_COUNTS_100("GPU.AtExitContextGroupCount",
- uma_memory_stats_.context_group_count);
- UMA_HISTOGRAM_CUSTOM_COUNTS(
- "GPU.AtExitMBytesAllocated",
- uma_memory_stats_.bytes_allocated_current / 1024 / 1024, 1, 2000, 50);
- UMA_HISTOGRAM_CUSTOM_COUNTS(
- "GPU.AtExitMBytesAllocatedMax",
- uma_memory_stats_.bytes_allocated_max / 1024 / 1024, 1, 2000, 50);
- }
std::string message;
bool block_offscreen_contexts = true;
@@ -658,8 +644,6 @@ bool GpuProcessHost::OnMessageReceived(const IPC::Message& message) {
IPC_MESSAGE_HANDLER(GpuHostMsg_DidLoseContext, OnDidLoseContext)
IPC_MESSAGE_HANDLER(GpuHostMsg_DidDestroyOffscreenContext,
OnDidDestroyOffscreenContext)
- IPC_MESSAGE_HANDLER(GpuHostMsg_GpuMemoryUmaStats,
- OnGpuMemoryUmaStatsReceived)
IPC_MESSAGE_HANDLER(GpuHostMsg_DestroyChannel, OnDestroyChannel)
IPC_MESSAGE_HANDLER(GpuHostMsg_CacheShader, OnCacheShader)
#if defined(OS_WIN)
@@ -920,13 +904,6 @@ void GpuProcessHost::OnDidDestroyOffscreenContext(const GURL& url) {
urls_with_live_offscreen_contexts_.erase(url);
}
-void GpuProcessHost::OnGpuMemoryUmaStatsReceived(
- const gpu::GPUMemoryUmaStats& stats) {
- TRACE_EVENT0("gpu", "GpuProcessHost::OnGpuMemoryUmaStatsReceived");
- uma_memory_stats_received_ = true;
- uma_memory_stats_ = stats;
-}
-
void GpuProcessHost::OnFieldTrialActivated(const std::string& trial_name) {
// Activate the trial in the browser process to match its state in the
// GPU process. This is done by calling FindFullName which finalizes the group
« no previous file with comments | « content/browser/gpu/gpu_process_host.h ('k') | content/common/gpu_host_messages.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698