Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1696)

Unified Diff: content/shell/renderer/layout_test/blink_test_runner.cc

Issue 1448203002: Add layout tests for the audio component of MediaStream Recording. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: format Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/shell/renderer/layout_test/blink_test_runner.cc
diff --git a/content/shell/renderer/layout_test/blink_test_runner.cc b/content/shell/renderer/layout_test/blink_test_runner.cc
index 45f129e085b9bd2431f7f9e91ee0b2f5f25a15ca..583745c10218aa406f0be712eb2bf52626808b97 100644
--- a/content/shell/renderer/layout_test/blink_test_runner.cc
+++ b/content/shell/renderer/layout_test/blink_test_runner.cc
@@ -47,6 +47,7 @@
#include "content/shell/renderer/layout_test/blink_test_helpers.h"
#include "content/shell/renderer/layout_test/layout_test_render_process_observer.h"
#include "content/shell/renderer/layout_test/leak_detector.h"
+#include "media/audio/audio_parameters.h"
#include "net/base/filename_util.h"
#include "net/base/net_errors.h"
#include "skia/ext/platform_canvas.h"
@@ -239,6 +240,22 @@ class MockVideoCapturerSource : public media::VideoCapturerSource {
void StopCapture() override {}
};
+class MockAudioCapturerSource : public media::AudioCapturerSource {
+ public:
+ MockAudioCapturerSource() = default;
+
+ void Initialize(const media::AudioParameters& params,
+ CaptureCallback* callback,
+ int session_id) override {}
+ void Start() override {}
+ void Stop() override {}
+ void SetVolume(double volume) override {}
+ void SetAutomaticGainControl(bool enable) override {}
+
+ protected:
+ ~MockAudioCapturerSource() override {}
+};
+
} // namespace
BlinkTestRunner::BlinkTestRunner(RenderView* render_view)
@@ -743,15 +760,25 @@ void BlinkTestRunner::OnWebTestProxyBaseDestroy(
test_runner::WebTestProxyBase* proxy) {
}
-bool BlinkTestRunner::AddMediaStreamSourceAndTrack(
+bool BlinkTestRunner::AddMediaStreamVideoSourceAndTrack(
blink::WebMediaStream* stream) {
DCHECK(stream);
#if defined(ENABLE_WEBRTC)
return AddVideoTrackToMediaStream(
make_scoped_ptr(new MockVideoCapturerSource()),
- false /* is_remote */,
- false /* is_readonly */,
- stream);
+ false /* is_remote */, false /* is_readonly */, stream);
+#else
+ return false;
+#endif
+}
+
+bool BlinkTestRunner::AddMediaStreamAudioSourceAndTrack(
+ blink::WebMediaStream* stream) {
+ DCHECK(stream);
+#if defined(ENABLE_WEBRTC)
+ return AddAudioTrackToMediaStream(
+ make_scoped_refptr(new MockAudioCapturerSource()),
+ false /* is_remote */, false /* is_readonly */, stream);
#else
return false;
#endif

Powered by Google App Engine
This is Rietveld 408576698