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

Issue 106973006: Add error handling and /proc/statm count in multi-process-rss.py. (Closed)

Created:
7 years ago by Dai Mikurube (NOT FULLTIME)
Modified:
7 years ago
Reviewers:
peria
CC:
chromium-reviews
Visibility:
Public.

Description

Add error handling and /proc/statm count in multi-process-rss.py. BUG=324194 TEST=None NOTRY=True Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=240119

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+29 lines, -2 lines) Patch
M tools/multi-process-rss.py View 3 chunks +29 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Dai Mikurube (NOT FULLTIME)
Hi, Could you take a look when you have time?
7 years ago (2013-12-06 10:06:10 UTC) #1
peria
lgtm
7 years ago (2013-12-09 04:51:01 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dmikurube@chromium.org/106973006/1
7 years ago (2013-12-11 16:25:13 UTC) #3
commit-bot: I haz the power
7 years ago (2013-12-11 16:25:22 UTC) #4
Message was sent while issue was closed.
Change committed as 240119

Powered by Google App Engine
This is Rietveld 408576698