New issue
Advanced search Search tips

Issue 832364 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Make performance monitor report a single process memory again.

Project Member Reported by alph@chromium.org, Apr 12 2018

Issue description

Make performance monitor report a single process memory again.

 

Comment 1 by alph@chromium.org, Apr 12 2018

Components: Platform>DevTools>Performance
Labels: M-67
Project Member

Comment 3 by bugdroid1@chromium.org, Apr 17 2018

Comment 4 by alph@chromium.org, Apr 17 2018

Labels: Merge-Request-67
Status: Fixed (was: Assigned)

Comment 5 by alph@chromium.org, Apr 17 2018

Labels: OS-Linux OS-Mac OS-Windows
Project Member

Comment 6 by sheriffbot@chromium.org, Apr 18 2018

Labels: -Merge-Request-67 Merge-Approved-67 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M67. Please go ahead and merge the CL to branch 3396 manually. Please contact milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), kbleicher@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 7 by gov...@chromium.org, Apr 18 2018

Pls merge your change to M67 branch 3396 ASAP so we can pick it up for next M67 Dev/Beta release.

If already merged to M67 and nothing is pending, pls remove "Merge=Approved-67" label. Thank you.
Project Member

Comment 8 by bugdroid1@chromium.org, Apr 19 2018

Labels: -merge-approved-67 merge-merged-3396
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9db52305acb743789b7f736109a04b6659302e24

commit 9db52305acb743789b7f736109a04b6659302e24
Author: Alexei Filippov <alph@chromium.org>
Date: Thu Apr 19 02:36:05 2018

DevTools: Make performance monitor report a single process memory again.

BUG= 832364 
TBR=alph@chromium.org

(cherry picked from commit c96d64efeec6c741091a188895d24d84bea90268)

Change-Id: I4baf520742f740bc296eb12ce76d9e82796f9f9a
Reviewed-on: https://chromium-review.googlesource.com/1011586
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Commit-Queue: Alexei Filippov <alph@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#550704}
Reviewed-on: https://chromium-review.googlesource.com/1018589
Reviewed-by: Alexei Filippov <alph@chromium.org>
Cr-Commit-Position: refs/branch-heads/3396@{#117}
Cr-Branched-From: 9ef2aa869bc7bc0c089e255d698cca6e47d6b038-refs/heads/master@{#550428}
[modify] https://crrev.com/9db52305acb743789b7f736109a04b6659302e24/third_party/WebKit/LayoutTests/http/tests/devtools/oopif/oopif-performance-monitor-expected.txt
[modify] https://crrev.com/9db52305acb743789b7f736109a04b6659302e24/third_party/WebKit/LayoutTests/http/tests/devtools/oopif/oopif-performance-monitor.js
[modify] https://crrev.com/9db52305acb743789b7f736109a04b6659302e24/third_party/blink/renderer/devtools/front_end/sdk/PerformanceMetricsModel.js

Sign in to add a comment