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

Side by Side Diff: content/renderer/media/media_stream_dependency_factory.h

Issue 133903004: Cleaned up the WebRtcAudioCapturer a bit. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebased and fixed the comment. Created 6 years, 11 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #ifndef CONTENT_RENDERER_MEDIA_MEDIA_STREAM_DEPENDENCY_FACTORY_H_ 5 #ifndef CONTENT_RENDERER_MEDIA_MEDIA_STREAM_DEPENDENCY_FACTORY_H_
6 #define CONTENT_RENDERER_MEDIA_MEDIA_STREAM_DEPENDENCY_FACTORY_H_ 6 #define CONTENT_RENDERER_MEDIA_MEDIA_STREAM_DEPENDENCY_FACTORY_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 200 matching lines...) Expand 10 before | Expand all | Expand 10 after
211 virtual scoped_refptr<webrtc::VideoTrackInterface> 211 virtual scoped_refptr<webrtc::VideoTrackInterface>
212 CreateLocalVideoTrack(const std::string& id, 212 CreateLocalVideoTrack(const std::string& id,
213 cricket::VideoCapturer* capturer); 213 cricket::VideoCapturer* capturer);
214 214
215 virtual bool EnsurePeerConnectionFactory(); 215 virtual bool EnsurePeerConnectionFactory();
216 virtual bool PeerConnectionFactoryCreated(); 216 virtual bool PeerConnectionFactoryCreated();
217 217
218 // Returns a new capturer or existing capturer based on the |render_view_id| 218 // Returns a new capturer or existing capturer based on the |render_view_id|
219 // and |device_info|. When the |render_view_id| and |device_info| are valid, 219 // and |device_info|. When the |render_view_id| and |device_info| are valid,
220 // it reuses existing capture if any; otherwise it creates a new capturer. 220 // it reuses existing capture if any; otherwise it creates a new capturer.
221 virtual scoped_refptr<WebRtcAudioCapturer> MaybeCreateAudioCapturer( 221 virtual scoped_refptr<WebRtcAudioCapturer> CreateAudioCapturer(
222 int render_view_id, const StreamDeviceInfo& device_info, 222 int render_view_id, const StreamDeviceInfo& device_info,
223 const blink::WebMediaConstraints& constraints); 223 const blink::WebMediaConstraints& constraints);
224 224
225 private: 225 private:
226 // Creates and deletes |pc_factory_|, which in turn is used for 226 // Creates and deletes |pc_factory_|, which in turn is used for
227 // creating PeerConnection objects. 227 // creating PeerConnection objects.
228 bool CreatePeerConnectionFactory(); 228 bool CreatePeerConnectionFactory();
229 229
230 void InitializeWorkerThread(talk_base::Thread** thread, 230 void InitializeWorkerThread(talk_base::Thread** thread,
231 base::WaitableEvent* event); 231 base::WaitableEvent* event);
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
271 base::Thread chrome_worker_thread_; 271 base::Thread chrome_worker_thread_;
272 272
273 base::PlatformFile aec_dump_file_; 273 base::PlatformFile aec_dump_file_;
274 274
275 DISALLOW_COPY_AND_ASSIGN(MediaStreamDependencyFactory); 275 DISALLOW_COPY_AND_ASSIGN(MediaStreamDependencyFactory);
276 }; 276 };
277 277
278 } // namespace content 278 } // namespace content
279 279
280 #endif // CONTENT_RENDERER_MEDIA_MEDIA_STREAM_DEPENDENCY_FACTORY_H_ 280 #endif // CONTENT_RENDERER_MEDIA_MEDIA_STREAM_DEPENDENCY_FACTORY_H_
OLDNEW
« no previous file with comments | « content/browser/media/webrtc_browsertest.cc ('k') | content/renderer/media/media_stream_dependency_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698