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

Issue 1547423002: Fix webview perf test crash in sysfs_power_monitor.py (Closed)

Created:
4 years, 11 months ago by Xianzhu
Modified:
4 years, 11 months ago
CC:
chromium-reviews, telemetry-reviews_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Fix webview perf test crash in sysfs_power_monitor.py BUG=561190 TBR=mikecase@chromium.org

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M tools/telemetry/telemetry/internal/platform/power_monitor/sysfs_power_monitor.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 10 (4 generated)
Xianzhu
4 years, 11 months ago (2015-12-29 18:27:41 UTC) #2
Xianzhu
Sample crash: https://uberchromegw.corp.google.com/i/internal.client.clank/builders/occam-svelte-memory-health-plan/builds/1751/steps/power.power_health_webview/logs/stdio
4 years, 11 months ago (2015-12-29 18:28:59 UTC) #3
Xianzhu
TBR to fix downstream red bots. Feel free to revert if the CL is not ...
4 years, 11 months ago (2015-12-29 19:32:31 UTC) #5
Yoland Yan(Google)
On 2015/12/29 18:28:59, Xianzhu wrote: > Sample crash: > https://uberchromegw.corp.google.com/i/internal.client.clank/builders/occam-svelte-memory-health-plan/builds/1751/steps/power.power_health_webview/logs/stdio Sorry this is actually my ...
4 years, 11 months ago (2015-12-29 19:32:36 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1547423002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1547423002/1
4 years, 11 months ago (2015-12-29 19:32:49 UTC) #8
Xianzhu
4 years, 11 months ago (2015-12-29 19:34:19 UTC) #10
Message was sent while issue was closed.
Thanks yolandyan@ for fixing. Closing this one.

Powered by Google App Engine
This is Rietveld 408576698