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

Side by Side Diff: content/browser/renderer_host/media/audio_input_renderer_host_unittest.cc

Issue 2761793002: Fix null argument to base::GetProcId in RenderProcessHostImpl::CreateMessageFilters. (Closed)
Patch Set: Updated unit test. Created 3 years, 9 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/renderer_host/media/audio_input_renderer_host.h" 5 #include "content/browser/renderer_host/media/audio_input_renderer_host.h"
6 6
7 #include <utility> 7 #include <utility>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
92 public: 92 public:
93 AudioInputRendererHostWithInterception( 93 AudioInputRendererHostWithInterception(
94 int render_process_id, 94 int render_process_id,
95 int32_t renderer_pid, 95 int32_t renderer_pid,
96 media::AudioManager* audio_manager, 96 media::AudioManager* audio_manager,
97 MediaStreamManager* media_stream_manager, 97 MediaStreamManager* media_stream_manager,
98 AudioMirroringManager* audio_mirroring_manager, 98 AudioMirroringManager* audio_mirroring_manager,
99 media::UserInputMonitor* user_input_monitor, 99 media::UserInputMonitor* user_input_monitor,
100 MockRenderer* renderer) 100 MockRenderer* renderer)
101 : AudioInputRendererHost(render_process_id, 101 : AudioInputRendererHost(render_process_id,
102 renderer_pid,
103 audio_manager, 102 audio_manager,
104 media_stream_manager, 103 media_stream_manager,
105 audio_mirroring_manager, 104 audio_mirroring_manager,
106 user_input_monitor), 105 user_input_monitor),
107 renderer_(renderer) { 106 renderer_(renderer) {
108 set_peer_process_for_testing(base::Process::Current()); 107 set_peer_process_for_testing(base::Process::Current());
108 set_renderer_pid(renderer_pid);
109 } 109 }
110 110
111 protected: 111 protected:
112 ~AudioInputRendererHostWithInterception() override = default; 112 ~AudioInputRendererHostWithInterception() override = default;
113 113
114 private: 114 private:
115 bool Send(IPC::Message* message) override { 115 bool Send(IPC::Message* message) override {
116 DCHECK_CURRENTLY_ON(BrowserThread::IO); 116 DCHECK_CURRENTLY_ON(BrowserThread::IO);
117 bool handled = true; 117 bool handled = true;
118 118
(...skipping 439 matching lines...) Expand 10 before | Expand all | Expand 10 after
558 EXPECT_CALL(controller_factory_, ControllerCreated()); 558 EXPECT_CALL(controller_factory_, ControllerCreated());
559 559
560 airh_->OnMessageReceived(AudioInputHostMsg_CreateStream( 560 airh_->OnMessageReceived(AudioInputHostMsg_CreateStream(
561 kStreamId, kRenderFrameId, session_id, DefaultConfig())); 561 kStreamId, kRenderFrameId, session_id, DefaultConfig()));
562 base::RunLoop().RunUntilIdle(); 562 base::RunLoop().RunUntilIdle();
563 563
564 EXPECT_CALL(*controller_factory_.controller(0), Close(_)); 564 EXPECT_CALL(*controller_factory_.controller(0), Close(_));
565 } 565 }
566 566
567 } // namespace content 567 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698