MediaStream Image Capture (3): Adding mojo and browser impl for dummy takePhoto().

content/browser/media/capture/image_capture_context.cc

Index: content/browser/media/capture/image_capture_context.cc
diff --git a/content/browser/media/capture/image_capture_context.cc b/content/browser/media/capture/image_capture_context.cc
new file mode 100644
index 0000000000000000000000000000000000000000..6fb12b1e2ff8041a7ac8a437b6cedf7f759c2bde
--- /dev/null
+++ b/content/browser/media/capture/image_capture_context.cc
@@ -0,0 +1,21 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "content/browser/media/capture/image_capture_context.h"
+
+#include "content/browser/media/capture/image_capture_impl.h"
+
+namespace content {
+
+ImageCaptureContext::ImageCaptureContext() {}
+
+ImageCaptureContext::~ImageCaptureContext() {}
+
+void ImageCaptureContext::CreateService(
+    mojo::InterfaceRequest<blink::mojom::ImageCapture> request) {
+  services_.push_back(std::unique_ptr<ImageCaptureImpl>(

[Reilly Grant (use Gerrit), 2016/04/28 01:09:57]

Sorry, I should have been clearer. Now that ImageCaptureImpl has a StrongBinding
you don't need to maintain this vector. The ImageCaptureImpl instances are owned
by the StrongBinding they contain and will be destroyed when the pipe is closed
by the renderer.

[mcasas, 2016/04/28 19:35:49]

Ah OK, then I'm removing ImageCaptureContext altogether.

+      new ImageCaptureImpl(std::move(request))));
+}
+
+}  // namespace content