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

Issue 3016533002: Disable power monitoring on all desktop platforms (Closed)

Created:
3 years, 3 months ago by charliea (OOO until 10-5)
Modified:
3 years, 3 months ago
Reviewers:
nednguyen
CC:
catapult-reviews_chromium.org, telemetry-reviews_chromium.org
Target Ref:
refs/heads/master
Project:
catapult
Visibility:
Public.

Description

Disable power monitoring on all desktop platforms BattOr is now the one true way to monitor power on desktop, and the old power monitoring code is causing redness on the waterfall. This CL shortcircuits all desktop power monitoring code until we can remove it, fixing the redness on the waterfall. BUG=chromium:765383 Review-Url: https://chromiumcodereview.appspot.com/3016533002 Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/c024bc03d2cc9057cb202899602bd26ed0afcd08

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -3 lines) Patch
M telemetry/telemetry/internal/platform/linux_platform_backend.py View 1 chunk +3 lines, -1 line 0 comments Download
M telemetry/telemetry/internal/platform/mac_platform_backend.py View 1 chunk +3 lines, -1 line 0 comments Download
M telemetry/telemetry/internal/platform/win_platform_backend.py View 1 chunk +3 lines, -1 line 0 comments Download

Messages

Total messages: 8 (4 generated)
charliea (OOO until 10-5)
3 years, 3 months ago (2017-09-15 19:27:21 UTC) #2
nednguyen
lgtm
3 years, 3 months ago (2017-09-15 19:28:26 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/3016533002/1
3 years, 3 months ago (2017-09-15 19:34:47 UTC) #5
commit-bot: I haz the power
3 years, 3 months ago (2017-09-15 20:07:07 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

Powered by Google App Engine
This is Rietveld 408576698