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

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

Issue 8525018: switch to DVLOG from VLOG, and move struct definition to cc file (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 9 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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/media_stream_manager.h" 5 #include "content/browser/renderer_host/media/media_stream_manager.h"
6 6
7 #include <list> 7 #include <list>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
42 MediaStreamType stream_type) { 42 MediaStreamType stream_type) {
43 if (stream_type == kVideoCapture && 43 if (stream_type == kVideoCapture &&
44 (options.video_option != StreamOptions::kNoCamera)) { 44 (options.video_option != StreamOptions::kNoCamera)) {
45 return true; 45 return true;
46 } else if (stream_type == kAudioCapture && options.audio == true) { 46 } else if (stream_type == kAudioCapture && options.audio == true) {
47 return true; 47 return true;
48 } 48 }
49 return false; 49 return false;
50 } 50 }
51 51
52 struct MediaStreamManager::DeviceRequest {
53 DeviceRequest()
54 : requester(NULL),
55 state(kNumMediaStreamTypes, kNotRequested) {
56 options.audio = false;
57 options.video_option = StreamOptions::kNoCamera;
58 }
59 DeviceRequest(
60 MediaStreamRequester* requester, const StreamOptions& request_options)
scherkus (not reviewing) 2011/11/11 01:44:43 nit: I'd move the args to previous line then align
wjia(left Chromium) 2011/11/11 01:51:00 Done.
61 : requester(requester),
62 options(request_options),
63 state(kNumMediaStreamTypes, kNotRequested) {
64 DCHECK(requester);
65 }
66 ~DeviceRequest() {}
67
68 enum RequestState {
69 kNotRequested = 0,
70 kRequested,
71 kOpening,
72 kDone,
73 kError
74 };
75
76 MediaStreamRequester* requester;
77 StreamOptions options;
78 std::vector<RequestState> state;
79 StreamDeviceInfoArray audio_devices;
80 StreamDeviceInfoArray video_devices;
81 };
82
52 MediaStreamManager::MediaStreamManager() 83 MediaStreamManager::MediaStreamManager()
53 : ALLOW_THIS_IN_INITIALIZER_LIST( 84 : ALLOW_THIS_IN_INITIALIZER_LIST(
54 device_settings_(new MediaStreamDeviceSettings(this))), 85 device_settings_(new MediaStreamDeviceSettings(this))),
55 enumeration_in_progress_(kNumMediaStreamTypes, false) { 86 enumeration_in_progress_(kNumMediaStreamTypes, false) {
56 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO)); 87 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
57 } 88 }
58 89
59 MediaStreamManager::~MediaStreamManager() { 90 MediaStreamManager::~MediaStreamManager() {
60 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO)); 91 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
61 } 92 }
(...skipping 344 matching lines...) Expand 10 before | Expand all | Expand 10 after
406 MediaStreamType stream_type) { 437 MediaStreamType stream_type) {
407 if (stream_type == kVideoCapture) { 438 if (stream_type == kVideoCapture) {
408 return video_capture_manager(); 439 return video_capture_manager();
409 } else if (stream_type == kAudioCapture) { 440 } else if (stream_type == kAudioCapture) {
410 return audio_input_device_manager(); 441 return audio_input_device_manager();
411 } 442 }
412 NOTREACHED(); 443 NOTREACHED();
413 return NULL; 444 return NULL;
414 } 445 }
415 446
416 MediaStreamManager::DeviceRequest::DeviceRequest()
417 : requester(NULL),
418 state(kNumMediaStreamTypes, kNotRequested) {
419 options.audio = false;
420 options.video_option = StreamOptions::kNoCamera;
421 }
422
423 MediaStreamManager::DeviceRequest::DeviceRequest(
424 MediaStreamRequester* requester, const StreamOptions& request_options)
425 : requester(requester),
426 options(request_options),
427 state(kNumMediaStreamTypes, kNotRequested) {
428 DCHECK(requester);
429 }
430
431 MediaStreamManager::DeviceRequest::~DeviceRequest() {}
432
433 } // namespace media_stream 447 } // namespace media_stream
OLDNEW
« no previous file with comments | « content/browser/renderer_host/media/media_stream_manager.h ('k') | content/renderer/media/media_stream_dispatcher.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698