Reland [Mojo Video Capture] Simplify media::VideoCaptureDevice::Client:Buffer to a struct

media/capture/video/video_capture_buffer_tracker.h

Index: media/capture/video/video_capture_buffer_tracker.h
diff --git a/media/capture/video/video_capture_buffer_tracker.h b/media/capture/video/video_capture_buffer_tracker.h
index 3e3d7bf888a2ff37aacdd53963c80476ea1f3c98..7ef13ecfc490a67c12103a974ad84f3473e78c76 100644
--- a/media/capture/video/video_capture_buffer_tracker.h
+++ b/media/capture/video/video_capture_buffer_tracker.h
@@ -50,10 +50,10 @@ class CAPTURE_EXPORT VideoCaptureBufferTracker {
   void set_frame_feedback_id(int value) { frame_feedback_id_ = value; }
   int frame_feedback_id() { return frame_feedback_id_; }
 
-  // Returns a scoped handle to the underlying storage.
-  virtual std::unique_ptr<VideoCaptureBufferHandle> GetBufferHandle() = 0;
-
+  virtual std::unique_ptr<VideoCaptureBufferHandle> GetMemoryMappedAccess() = 0;
   virtual mojo::ScopedSharedBufferHandle GetHandleForTransit() = 0;
+  virtual base::SharedMemoryHandle
+  GetNonOwnedSharedMemoryHandleForLegacyIPC() = 0;

[miu, 2017/01/13 20:34:38]

If you haven't, consider running `git cl format`. It seems like this second line
of the method declaration should be indented 4 spaces. (But I could be wrong.)

[chfremer, 2017/01/17 23:35:58]

Tried, but seems the auto-formatting tool wants it this way.

 
  private:
   // |dimensions_| may change as a VideoCaptureBufferTracker is re-used, but