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

Issue 2997603002: Try fixing telemetry on CrOS (Closed)

Created:
3 years, 4 months ago by pfeldman
Modified:
3 years, 4 months ago
Reviewers:
achuithb
CC:
catapult-reviews_chromium.org, telemetry-reviews_chromium.org
Target Ref:
refs/heads/master
Project:
catapult
Visibility:
Public.

Description

Try fixing telemetry on CrOS BUG=chromium:753116

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+31 lines, -124 lines) Patch
M telemetry/telemetry/core/cros_interface.py View 2 chunks +0 lines, -22 lines 0 comments Download
M telemetry/telemetry/core/cros_interface_unittest.py View 2 chunks +0 lines, -48 lines 0 comments Download
M telemetry/telemetry/core/local_server.py View 1 chunk +0 lines, -3 lines 0 comments Download
M telemetry/telemetry/core/util.py View 1 chunk +0 lines, -14 lines 0 comments Download
M telemetry/telemetry/internal/backends/chrome/cros_browser_backend.py View 6 chunks +31 lines, -19 lines 0 comments Download
M telemetry/telemetry/internal/platform/android_platform_backend.py View 2 chunks +0 lines, -10 lines 0 comments Download
M telemetry/telemetry/internal/platform/cros_platform_backend.py View 1 chunk +0 lines, -5 lines 0 comments Download
M telemetry/telemetry/internal/platform/platform_backend.py View 1 chunk +0 lines, -3 lines 0 comments Download

Messages

Total messages: 3 (2 generated)
pfeldman
3 years, 4 months ago (2017-08-08 17:59:36 UTC) #2
Could you take a look at this? I failed to set up vm environment, but this is
something that could serve as a conceptual fix. Could you look at it / give it a
try and tell me what you think?

Powered by Google App Engine
This is Rietveld 408576698