[Telemetry] Refactoring in preparation for video-based Speed Index support.

tools/telemetry/telemetry/core/platform/platform_backend.py

Index: tools/telemetry/telemetry/core/platform/platform_backend.py
diff --git a/tools/telemetry/telemetry/core/platform/platform_backend.py b/tools/telemetry/telemetry/core/platform/platform_backend.py
index 7f74deabf1978ca302bea45219da4acb30157d51..cc86b857e937e93aaa60f1353703dd8b139f4b47 100644
--- a/tools/telemetry/telemetry/core/platform/platform_backend.py
+++ b/tools/telemetry/telemetry/core/platform/platform_backend.py
@@ -75,3 +75,12 @@ class PlatformBackend(object):
 
   def InstallApplication(self, application):
     raise NotImplementedError()
+
+  def CanCaptureVideo(self):
+    return False
+
+  def StartVideoCapture(self, min_bitrate_mbps):
+    raise NotImplementedError()
+
+  def StopVideoCapture(self):

[slamm, 2013/12/02 22:51:15]

Should this have a doc string?

BTW, it surprised me that StopVideoCapture returns the frames. The name made me
think it would only stop it (instead of collecting the frames and returning
them).

[tonyg, 2013/12/02 23:18:36]

The docs all live in platform/__init__.py
Yeah, I think you are right about this, but we currently use this paradigm of
Stop*() returning the results all over. My vote would be to stick with
consistency for now and clean up in the future.

+    raise NotImplementedError()