Commit 7dadd85f authored by tonyg@chromium.org's avatar tonyg@chromium.org

[Telemetry] Improve Android cycle time.

There's no need to eagerly install the file_poller. This ensure we only do it
when we're running a measurement that needs it.

BUG=

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@272575 0039d316-1c4b-4281-b951-d872f2087c98
parent 1ea01750
...@@ -23,7 +23,6 @@ class DS2784PowerMonitor(power_monitor.PowerMonitor): ...@@ -23,7 +23,6 @@ class DS2784PowerMonitor(power_monitor.PowerMonitor):
super(DS2784PowerMonitor, self).__init__() super(DS2784PowerMonitor, self).__init__()
self._device = device self._device = device
self._powermonitor_process_port = None self._powermonitor_process_port = None
android_prebuilt_profiler_helper.InstallOnDevice(device, 'file_poller')
self._file_poller_binary = android_prebuilt_profiler_helper.GetDevicePath( self._file_poller_binary = android_prebuilt_profiler_helper.GetDevicePath(
'file_poller') 'file_poller')
...@@ -43,6 +42,8 @@ class DS2784PowerMonitor(power_monitor.PowerMonitor): ...@@ -43,6 +42,8 @@ class DS2784PowerMonitor(power_monitor.PowerMonitor):
def StartMonitoringPower(self, browser): def StartMonitoringPower(self, browser):
assert not self._powermonitor_process_port, ( assert not self._powermonitor_process_port, (
'Must call StopMonitoringPower().') 'Must call StopMonitoringPower().')
android_prebuilt_profiler_helper.InstallOnDevice(
self._device, 'file_poller')
self._powermonitor_process_port = int( self._powermonitor_process_port = int(
self._device.old_interface.RunShellCommand( self._device.old_interface.RunShellCommand(
'%s %d %s %s %s' % (self._file_poller_binary, SAMPLE_RATE_HZ, '%s %d %s %s %s' % (self._file_poller_binary, SAMPLE_RATE_HZ,
......
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