Commit 573878b5 authored by skyostil@chromium.org's avatar skyostil@chromium.org

telemetry: Fix screen recording with multiple connected devices

Make it possible to use the Android screen recording profiler with
multiple connected devices. Only the screen on the device that is
actually running the telemetry test will get recorded.

BUG=331435
TEST=tools/perf/run_benchmark smoothness.key_mobile_sites \
--page-filter=linus --browser=android-content-shell \
--profiler=android-screen-recorder
NOTRY=true

Review URL: https://codereview.chromium.org/119323008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@243080 0039d316-1c4b-4281-b951-d872f2087c98
parent 0112caf9
......@@ -18,8 +18,11 @@ class AndroidScreenRecordingProfiler(profiler.Profiler):
self._output_path = output_path + '.mp4'
self._recorder = subprocess.Popen(
[os.path.join(util.GetChromiumSrcDir(), 'build', 'android',
'screenshot.py'), '--video', '--file',
self._output_path], stdin=subprocess.PIPE, stdout=subprocess.PIPE)
'screenshot.py'),
'--video',
'--file', self._output_path,
'--device', browser_backend.adb.device()],
stdin=subprocess.PIPE, stdout=subprocess.PIPE)
@classmethod
def name(cls):
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment