New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 899823 link

Starred by 1 user
Status: Fixed
Owner:
█ kvitekp@chromium.org
Last visit > 30 days ago
Closed: Nov 5
Cc:
caseq@chromium.org
dgozman@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task



Sign in to add a comment

Introduce DevTools SystemInfo.getProcessInfo API

Project Member Reported by kvitekp@chromium.org, Oct 29 
SystemInfo.getProcessInfo should return information about all the processes currently present in the system, including process id, type (browser, renderer, etc) and used CPU time.

We'll use used CPU timing info along with DevTools Performance metrics to track headless rendering performance metrics.
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 30 

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

commit f5ee8af906c44c35d85b2294ffd05cb26218b111
Author: Peter Kvitek <kvitekp@chromium.org>
Date: Tue Oct 30 20:45:02 2018

Introduced SystemInfo.getProcessInfo API.

Process handle and cumulative CPU usage is returned for browser and all
renderer processes.

BUG:  899823 

Change-Id: I6d44630df3ea9ef80bed17f30367dad929ecaa17
Reviewed-on: https://chromium-review.googlesource.com/c/1303074
Commit-Queue: Peter Kvitek <kvitekp@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Reviewed-by: Alexei Filippov <alph@chromium.org>
Reviewed-by: Andrey Kosyakov <caseq@chromium.org>
Cr-Commit-Position: refs/heads/master@{#604001}
[modify] https://crrev.com/f5ee8af906c44c35d85b2294ffd05cb26218b111/content/browser/devtools/protocol/system_info_handler.cc
[modify] https://crrev.com/f5ee8af906c44c35d85b2294ffd05cb26218b111/content/browser/devtools/protocol/system_info_handler.h
[modify] https://crrev.com/f5ee8af906c44c35d85b2294ffd05cb26218b111/content/shell/browser/layout_test/devtools_protocol_test_bindings.cc
[add] https://crrev.com/f5ee8af906c44c35d85b2294ffd05cb26218b111/third_party/WebKit/LayoutTests/inspector-protocol/system-info/process-info-expected.txt
[add] https://crrev.com/f5ee8af906c44c35d85b2294ffd05cb26218b111/third_party/WebKit/LayoutTests/inspector-protocol/system-info/process-info.js
[modify] https://crrev.com/f5ee8af906c44c35d85b2294ffd05cb26218b111/third_party/blink/renderer/core/inspector/browser_protocol.pdl

Comment 2 by maxlg@chromium.org, Oct 31 

Do you think this crash is related to the change? https://paste.googleplex.com/4583001883148288

Comment 3 by dgozman@chromium.org, Oct 31

Status: Assigned (was: Untriaged)
►

Sign in to add a comment