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

Unified Diff: content/browser/renderer_host/media/audio_output_delegate_impl_unittest.cc

Issue 2655413004: Strip out stream counting from AudioRendererHost. (Closed)
Patch Set: Fix all teh tests. Created 3 years, 10 months 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/browser/renderer_host/media/audio_output_delegate_impl_unittest.cc
diff --git a/content/browser/renderer_host/media/audio_output_delegate_impl_unittest.cc b/content/browser/renderer_host/media/audio_output_delegate_impl_unittest.cc
index 0d93a7d50cba4e162ecceb580cba6e2842eb3599..255c5aa669aa79888d71deb7c17d9d9bf0e2f6ad 100644
--- a/content/browser/renderer_host/media/audio_output_delegate_impl_unittest.cc
+++ b/content/browser/renderer_host/media/audio_output_delegate_impl_unittest.cc
@@ -78,7 +78,6 @@ class MockObserver : public content::MediaObserver {
class MockEventHandler : public AudioOutputDelegate::EventHandler {
public:
- MOCK_METHOD1(OnStreamStateChanged, void(bool playing));
MOCK_METHOD3(OnStreamCreated,
void(int stream_id,
base::SharedMemory* shared_memory,
@@ -153,11 +152,6 @@ class AudioOutputDelegateTest : public testing::Test {
OnStreamCreated(kStreamId, NotNull(), NotNull()));
EXPECT_CALL(mirroring_manager_,
AddDiverter(kRenderProcessId, kRenderFrameId, NotNull()));
- {
- InSequence s;
- EXPECT_CALL(event_handler_, OnStreamStateChanged(true));
- EXPECT_CALL(event_handler_, OnStreamStateChanged(false));
- }
{
AudioOutputDelegateImpl delegate(
@@ -212,13 +206,6 @@ class AudioOutputDelegateTest : public testing::Test {
OnStreamCreated(kStreamId, NotNull(), NotNull()));
EXPECT_CALL(mirroring_manager_,
AddDiverter(kRenderProcessId, kRenderFrameId, NotNull()));
- {
- InSequence s;
- EXPECT_CALL(event_handler_, OnStreamStateChanged(true));
- EXPECT_CALL(event_handler_, OnStreamStateChanged(false));
- EXPECT_CALL(event_handler_, OnStreamStateChanged(true));
- EXPECT_CALL(event_handler_, OnStreamStateChanged(false));
- }
{
AudioOutputDelegateImpl delegate(
@@ -248,11 +235,6 @@ class AudioOutputDelegateTest : public testing::Test {
OnStreamCreated(kStreamId, NotNull(), NotNull()));
EXPECT_CALL(mirroring_manager_,
AddDiverter(kRenderProcessId, kRenderFrameId, NotNull()));
- {
- InSequence s;
- EXPECT_CALL(event_handler_, OnStreamStateChanged(true));
- EXPECT_CALL(event_handler_, OnStreamStateChanged(false));
- }
{
AudioOutputDelegateImpl delegate(
@@ -340,11 +322,6 @@ class AudioOutputDelegateTest : public testing::Test {
OnStreamCreated(kStreamId, NotNull(), NotNull()));
EXPECT_CALL(mirroring_manager_,
AddDiverter(kRenderProcessId, kRenderFrameId, NotNull()));
- {
- InSequence s;
- EXPECT_CALL(event_handler_, OnStreamStateChanged(true));
- EXPECT_CALL(event_handler_, OnStreamStateChanged(false));
- }
DummyAudioOutputStream stream;
{

Powered by Google App Engine
This is Rietveld 408576698