[Mojo Video Capture] Add test coverage for accelerated jpeg decoding

content/browser/renderer_host/media/video_capture_controller.h

Index: content/browser/renderer_host/media/video_capture_controller.h
diff --git a/content/browser/renderer_host/media/video_capture_controller.h b/content/browser/renderer_host/media/video_capture_controller.h
index 634fb169316dadc1adf9aab789e803b493819c11..49b9361b4c45f5f90f96c3fe06bde5d004e7b682 100644
--- a/content/browser/renderer_host/media/video_capture_controller.h
+++ b/content/browser/renderer_host/media/video_capture_controller.h
@@ -102,6 +102,8 @@ class CONTENT_EXPORT VideoCaptureController : public media::VideoFrameReceiver {
   void OnError() override;
   void OnLog(const std::string& message) override;
   void OnStarted() override;
+  void OnStartedUsingGpuDecode() override;
+  void OnStoppedUsingGpuDecode() override;

[mcasas, 2017/03/07 19:39:37]

I don't see the need for OnStoppedUsingGpuDecode():
it's not ending up anywhere in this CL and, once
OnStartedUsingGpuDecode() has been received,
it's fair assumption to think it's running until
OnError or capture stops.

[chfremer, 2017/03/07 22:30:38]

True that it is not strictly needed. I added it for symmetry reasons and for
indicating to the reader that it actually _can_ happen that Gpu decode stops
after it has started. Looking at [1], when the Gpu decoder errors out, the
VCDClient just switches back to software decoding. It does not call OnError() in
that case.

Let me know if you still want me to remove it, and I'll make the change.

[1]
https://cs.chromium.org/chromium/src/media/capture/video/video_capture_device...

 
  private:
   struct ControllerClient;