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

Issue 186143002: Don't set the max current on the monsoon device by default (Closed)

Created:
6 years, 9 months ago by atreat
Modified:
6 years, 9 months ago
Reviewers:
dtu, tonyg
CC:
chromium-reviews, chrome-speed-team+watch_google.com, telemetry+watch_chromium.org
Base URL:
https://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

Don't set the max current on the monsoon device by default Don't set the max current on the monsoon device by default. Not all devices will run with this setting. BUG=None Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=254790

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -2 lines) Patch
M tools/telemetry/telemetry/core/platform/power_monitor/monsoon_power_monitor.py View 1 chunk +0 lines, -1 line 0 comments Download
M tools/telemetry/telemetry/core/platform/profiler/monsoon_profiler.py View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 13 (0 generated)
atreat
Please have a look. This allows the samsung note hardware to run with the monsoon ...
6 years, 9 months ago (2014-03-03 22:03:12 UTC) #1
tonyg
lgtm
6 years, 9 months ago (2014-03-03 22:39:18 UTC) #2
atreat
The CQ bit was checked by adam.treat@samsung.com
6 years, 9 months ago (2014-03-03 22:42:05 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/adam.treat@samsung.com/186143002/1
6 years, 9 months ago (2014-03-03 22:44:37 UTC) #4
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 9 months ago (2014-03-03 22:53:17 UTC) #5
commit-bot: I haz the power
Retried try job too often on win_rel for step(s) app_list_unittests, ash_unittests, aura_unittests, cacheinvalidation_unittests, cc_unittests, check_deps, ...
6 years, 9 months ago (2014-03-03 22:53:18 UTC) #6
atreat
The CQ bit was checked by adam.treat@samsung.com
6 years, 9 months ago (2014-03-04 15:33:29 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/adam.treat@samsung.com/186143002/1
6 years, 9 months ago (2014-03-04 15:33:41 UTC) #8
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 9 months ago (2014-03-04 16:44:49 UTC) #9
commit-bot: I haz the power
Retried try job too often on win_rel for step(s) net_unittests http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&number=274010
6 years, 9 months ago (2014-03-04 16:44:49 UTC) #10
tonyg
The CQ bit was checked by tonyg@chromium.org
6 years, 9 months ago (2014-03-04 16:47:09 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/adam.treat@samsung.com/186143002/1
6 years, 9 months ago (2014-03-04 16:48:43 UTC) #12
commit-bot: I haz the power
6 years, 9 months ago (2014-03-04 18:52:18 UTC) #13
Message was sent while issue was closed.
Change committed as 254790

Powered by Google App Engine
This is Rietveld 408576698