New issue
Advanced search Search tips

Issue 720541 link

Starred by 1 user

Issue metadata

Status: Started
Owner:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Remove temporary estimates for private memory footprint once memory infra service emits the metric.

Project Member Reported by erikc...@chromium.org, May 10 2017

Issue description

As title indicates.
 

Comment 2 by ssid@chromium.org, Dec 20 2017

Do we still need this metric?
Status: Started (was: Assigned)
https://chromium-review.googlesource.com/c/chromium/src/+/837656
Project Member

Comment 4 by bugdroid1@chromium.org, Jan 3 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8f7bf1829627fc0e4be6c6be17517a9eee32eb12

commit 8f7bf1829627fc0e4be6c6be17517a9eee32eb12
Author: Erik Chen <erikchen@chromium.org>
Date: Wed Jan 03 19:13:12 2018

Remove temporary estimates for private memory footprint

The equivalent metrics are now uploaded by ProcessMemoryMetricsEmitter, except
for all platforms rather than just macOS.

Bug: 720541
Change-Id: I36ed3a888bb0021744e12df84587c796cfd80865
Reviewed-on: https://chromium-review.googlesource.com/837656
Commit-Queue: Erik Chen <erikchen@chromium.org>
Reviewed-by: Siddhartha S <ssid@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526759}
[modify] https://crrev.com/8f7bf1829627fc0e4be6c6be17517a9eee32eb12/chrome/browser/memory_details.cc
[modify] https://crrev.com/8f7bf1829627fc0e4be6c6be17517a9eee32eb12/chrome/browser/memory_details.h
[modify] https://crrev.com/8f7bf1829627fc0e4be6c6be17517a9eee32eb12/chrome/browser/memory_details_mac.cc
[modify] https://crrev.com/8f7bf1829627fc0e4be6c6be17517a9eee32eb12/chrome/browser/metrics/metrics_memory_details.cc
[modify] https://crrev.com/8f7bf1829627fc0e4be6c6be17517a9eee32eb12/tools/metrics/histograms/histograms.xml

Sign in to add a comment