Update MockWebUserMediaClient for requestSources.

content/shell/renderer/test_runner/mock_web_user_media_client.cc

 // Copyright 2014 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/shell/renderer/test_runner/mock_web_user_media_client.h"
 
 #include "base/logging.h"
 #include "content/shell/renderer/test_runner/WebTestDelegate.h"
 #include "content/shell/renderer/test_runner/mock_constraints.h"
 #include "third_party/WebKit/public/platform/WebMediaConstraints.h"
 #include "third_party/WebKit/public/platform/WebMediaDeviceInfo.h"
 #include "third_party/WebKit/public/platform/WebMediaStream.h"
 #include "third_party/WebKit/public/platform/WebMediaStreamSource.h"
 #include "third_party/WebKit/public/platform/WebMediaStreamTrack.h"
+#include "third_party/WebKit/public/platform/WebMediaStreamTrackSourcesRequest.h"
+#include "third_party/WebKit/public/platform/WebSourceInfo.h"
 #include "third_party/WebKit/public/platform/WebVector.h"
 #include "third_party/WebKit/public/web/WebDocument.h"
 #include "third_party/WebKit/public/web/WebMediaDevicesRequest.h"
 #include "third_party/WebKit/public/web/WebMediaStreamRegistry.h"
 #include "third_party/WebKit/public/web/WebUserMediaRequest.h"
 
 using blink::WebMediaConstraints;
 using blink::WebMediaDeviceInfo;
 using blink::WebMediaDevicesRequest;
 using blink::WebMediaStream;
 using blink::WebMediaStreamSource;
 using blink::WebMediaStreamTrack;
+using blink::WebMediaStreamTrackSourcesRequest;
+using blink::WebSourceInfo;
 using blink::WebString;
 using blink::WebUserMediaRequest;
 using blink::WebVector;
 
 namespace content {
 
 class UserMediaRequestTask : public WebMethodTask<MockWebUserMediaClient> {
  public:
   UserMediaRequestTask(MockWebUserMediaClient* object,
                        const WebUserMediaRequest& request,
@@ skipping 53 matching lines @@
         request_(request),
         result_(result) {}
 
   virtual void RunIfValid() OVERRIDE { request_.requestSucceeded(result_); }
 
  private:
   WebMediaDevicesRequest request_;
   WebVector<WebMediaDeviceInfo> result_;
 };
 
+class SourcesRequestTask : public WebMethodTask<MockWebUserMediaClient> {
+ public:
+  SourcesRequestTask(MockWebUserMediaClient* object,
+                     const WebMediaStreamTrackSourcesRequest& request,
+                     const WebVector<WebSourceInfo>& result)
+      : WebMethodTask<MockWebUserMediaClient>(object),
+        request_(request),
+        result_(result) {}
+
+  virtual void RunIfValid() OVERRIDE { request_.requestSucceeded(result_); }
+
+ private:
+  WebMediaStreamTrackSourcesRequest request_;
+  WebVector<WebSourceInfo> result_;
+};

[jochen (gone - plz use gerrit), 2014/09/16 14:19:14]

nit. disallow_copy_and_assign

[Henrik Grunell, 2014/09/16 15:18:48]

Done.

+
 class MockExtraData : public WebMediaStream::ExtraData {
  public:
   int foo;
 };
 
 MockWebUserMediaClient::MockWebUserMediaClient(WebTestDelegate* delegate)
     : delegate_(delegate) {}
 
 void MockWebUserMediaClient::requestUserMedia(
     const WebUserMediaRequest& stream_request) {
@@ skipping 73 matching lines @@
                         "Build-in webcam",
                         "group2");
 
   delegate_->postTask(new MediaDevicesRequestTask(this, request, devices));
 }
 
 void MockWebUserMediaClient::cancelMediaDevicesRequest(
     const WebMediaDevicesRequest&) {
 }
 
+void MockWebUserMediaClient::requestSources(
+    const blink::WebMediaStreamTrackSourcesRequest& request) {
+  const size_t size = 2;

[Mike West, 2014/09/16 12:56:09]

I see that other bits of this file use the same pattern, but it feels pretty
hacky. I think it would be cleaner to adopt a struct-based pattern, such as
you'd find in
https://code.google.com/p/chromium/codesearch#chromium/src/net/base/net_util_....
That's less likely to get out of sync than this pattern.

[Henrik Grunell, 2014/09/16 13:22:35]

This will be used in layout tests, can it be controlled from there? The example
is a Chrome unit test.

[Mike West, 2014/09/16 13:36:29]

No. My point was that if you're going to hard-code things into the Mock, it'd be
better to do so in a way that doesn't force you to hard-code both the data and
the size of the data.

[Henrik Grunell, 2014/09/16 15:18:48]

Ah, of course. :) Done.

+  WebVector<WebSourceInfo> sources(size);
+
+  sources[0].initialize("device1",
+                        WebSourceInfo::SourceKindAudio,
+                        "Built-in microphone",
+                        WebSourceInfo::VideoFacingModeNone);
+  sources[1].initialize("device2",
+                        WebSourceInfo::SourceKindVideo,
+                        "Build-in webcam",
+                        WebSourceInfo::VideoFacingModeEnvironment);

[Mike West, 2014/09/16 12:56:09]

Is this enough flexibility for you? It might make sense (in a future CL) to add
some write capabilities to the mock so you can set up scenarios you care about
testing.

[Henrik Grunell, 2014/09/16 13:22:35]

Well, this will be used in layout tests, can it be controlled from there?
Despite that, note that the getSources JS API is going away so I think this is
enough.

[Mike West, 2014/09/16 13:36:29]

There are a number of mocks with setter methods that bind to TestRunner in order
to expose some bits and pieces to layout tests. Look at the various set* methods
bound in TestRunnerBindings::GetObjectTemplateBuilder for some examples. I think
that might be a route to explore for you in the future.
I'd accept that. :)

[Henrik Grunell, 2014/09/16 15:18:48]

OK, thanks for the pointer.
OK, I'll leave as is then.

+
+  delegate_->postTask(new SourcesRequestTask(this, request, sources));
+}
+
 }  // namespace content