Age | Commit message (Collapse) | Author |
|
BUG=chromium:893748
TEST=precq
CQ-DEPEND=CL:1409763
Change-Id: I661c527c4a53864595d02f3302dbc9047692a042
Reviewed-on: https://chromium-review.googlesource.com/1407709
Commit-Ready: Alex Klein <saklein@chromium.org>
Tested-by: Alex Klein <saklein@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
|
|
This is the only repo that uses these funcs anymore, so move them
here so we can delete the source file entirely.
BUG=None
TEST=precq passes
Change-Id: Ie3cfa2168783999f3a02df20957755ae4c6c46af
Reviewed-on: https://chromium-review.googlesource.com/1163163
Commit-Ready: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Caroline Tice <cmtice@chromium.org>
|
|
In addition to fixing the lint errors, this also fixes the Python
formatting issues (ran tc_pyformat on nearly all the files).
BUG=chromium:570450
TEST=Ran all crosperf & bisect tool unit tests.
Ran afe_lock_machine.py (check machine status)
Ran full crosperf test (octane, speedometer, BootPerf) on alex.
Change-Id: Ic86f9192801ac67769f3de30f1c5f0d203ce0831
Reviewed-on: https://chromium-review.googlesource.com/471886
Commit-Ready: Caroline Tice <cmtice@chromium.org>
Tested-by: Caroline Tice <cmtice@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
|
|
The input rgb2short can reach to 255. When the value exceeds 250,
it causes out of bound error. This CL fixes that.
BUG=chromium:649374
TEST=The out of bound error is fixed.
Change-Id: Iee31337e82907e1803a84b5837542733bb224e8f
Reviewed-on: https://chrome-internal-review.googlesource.com/289536
Commit-Ready: Yunlian Jiang <yunlian@google.com>
Tested-by: Yunlian Jiang <yunlian@google.com>
Reviewed-by: Caroline Tice <cmtice@google.com>
|
|
This removes colortrans.py from cros_utils. We add a single
function inside misc.py instead.
BUG=chromium:647308
TEST=crosperf still works, the color text is shown on terminal.
Change-Id: I1871bd2c57608f59ded6a8cf720c52cb2c07c3ff
Reviewed-on: https://chrome-internal-review.googlesource.com/287538
Commit-Ready: Yunlian Jiang <yunlian@google.com>
Tested-by: Yunlian Jiang <yunlian@google.com>
Reviewed-by: Luis Lozano <llozano@chromium.org>
|
|
Found all instances of 'raise Exception' and changed them to raise
something slightly more specific. Changed a few 'except Exception'
statements appropriately as well.
BUG=chromium:566255
TEST=None
Change-Id: If2666bd55838342bc71431e5e6ab2157c2c9e125
Reviewed-on: https://chrome-internal-review.googlesource.com/270731
Commit-Ready: Caroline Tice <cmtice@google.com>
Tested-by: Caroline Tice <cmtice@google.com>
Reviewed-by: Yunlian Jiang <yunlian@google.com>
|
|
This CL finishes switching the subdirectory from 'utils' to
'cros_utils'. It changes all the remaining import statements to
use 'cros_utils'; it removes the 'cros_utils' symlink, and it
renames the 'utils' subdirectory to 'cros_utils'.
BUG=chromium:568195
TEST=ran crosperf & binary search tool unittests.
Change-Id: I7427f8bfb2ddac3a4b6108e46782039059684382
Reviewed-on: https://chrome-internal-review.googlesource.com/270396
Commit-Ready: Caroline Tice <cmtice@google.com>
Tested-by: Caroline Tice <cmtice@google.com>
Reviewed-by: Cassidy Burden <cburden@google.com>
Reviewed-by: Luis Lozano <llozano@chromium.org>
|