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

Side by Side Diff: chrome/browser/media/media_capture_devices_dispatcher.cc

Issue 12079097: Introduce PrefRegistrySyncable, simplifying PrefServiceSyncable. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge to head again; base::File changes conflicted. Created 7 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 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 #include "chrome/browser/media/media_capture_devices_dispatcher.h" 5 #include "chrome/browser/media/media_capture_devices_dispatcher.h"
6 6
7 #include "chrome/browser/media/media_stream_capture_indicator.h" 7 #include "chrome/browser/media/media_stream_capture_indicator.h"
8 #include "chrome/browser/prefs/pref_registry_syncable.h"
9 #include "chrome/browser/prefs/pref_service.h"
8 #include "chrome/browser/prefs/scoped_user_pref_update.h" 10 #include "chrome/browser/prefs/scoped_user_pref_update.h"
9 #include "chrome/browser/profiles/profile.h" 11 #include "chrome/browser/profiles/profile.h"
10 #include "chrome/common/pref_names.h" 12 #include "chrome/common/pref_names.h"
11 #include "content/public/browser/browser_thread.h" 13 #include "content/public/browser/browser_thread.h"
12 #include "content/public/browser/media_devices_monitor.h" 14 #include "content/public/browser/media_devices_monitor.h"
13 #include "content/public/common/media_stream_request.h" 15 #include "content/public/common/media_stream_request.h"
14 16
15 using content::BrowserThread; 17 using content::BrowserThread;
16 using content::MediaStreamDevices; 18 using content::MediaStreamDevices;
17 19
(...skipping 22 matching lines...) Expand all
40 return Singleton<MediaCaptureDevicesDispatcher>::get(); 42 return Singleton<MediaCaptureDevicesDispatcher>::get();
41 } 43 }
42 44
43 MediaCaptureDevicesDispatcher::MediaCaptureDevicesDispatcher() 45 MediaCaptureDevicesDispatcher::MediaCaptureDevicesDispatcher()
44 : devices_enumerated_(false), 46 : devices_enumerated_(false),
45 media_stream_capture_indicator_(new MediaStreamCaptureIndicator()) {} 47 media_stream_capture_indicator_(new MediaStreamCaptureIndicator()) {}
46 48
47 MediaCaptureDevicesDispatcher::~MediaCaptureDevicesDispatcher() {} 49 MediaCaptureDevicesDispatcher::~MediaCaptureDevicesDispatcher() {}
48 50
49 void MediaCaptureDevicesDispatcher::RegisterUserPrefs( 51 void MediaCaptureDevicesDispatcher::RegisterUserPrefs(
50 PrefServiceSyncable* user_prefs) { 52 PrefService* user_prefs,
53 PrefRegistrySyncable* registry) {
54 // TODO(joi): Get rid of the need for PrefService param above.
51 if (!user_prefs->FindPreference(prefs::kDefaultAudioCaptureDevice)) { 55 if (!user_prefs->FindPreference(prefs::kDefaultAudioCaptureDevice)) {
52 user_prefs->RegisterStringPref(prefs::kDefaultAudioCaptureDevice, 56 registry->RegisterStringPref(prefs::kDefaultAudioCaptureDevice,
53 std::string(), 57 std::string(),
54 PrefServiceSyncable::UNSYNCABLE_PREF); 58 PrefRegistrySyncable::UNSYNCABLE_PREF);
55 } 59 }
56 if (!user_prefs->FindPreference(prefs::kDefaultVideoCaptureDevice)) { 60 if (!user_prefs->FindPreference(prefs::kDefaultVideoCaptureDevice)) {
57 user_prefs->RegisterStringPref(prefs::kDefaultVideoCaptureDevice, 61 registry->RegisterStringPref(prefs::kDefaultVideoCaptureDevice,
58 std::string(), 62 std::string(),
59 PrefServiceSyncable::UNSYNCABLE_PREF); 63 PrefRegistrySyncable::UNSYNCABLE_PREF);
60 } 64 }
61 } 65 }
62 66
63 void MediaCaptureDevicesDispatcher::AddObserver(Observer* observer) { 67 void MediaCaptureDevicesDispatcher::AddObserver(Observer* observer) {
64 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 68 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
65 if (!observers_.HasObserver(observer)) 69 if (!observers_.HasObserver(observer))
66 observers_.AddObserver(observer); 70 observers_.AddObserver(observer);
67 } 71 }
68 72
69 void MediaCaptureDevicesDispatcher::RemoveObserver(Observer* observer) { 73 void MediaCaptureDevicesDispatcher::RemoveObserver(Observer* observer) {
(...skipping 150 matching lines...) Expand 10 before | Expand all | Expand 10 after
220 int render_process_id, 224 int render_process_id,
221 int render_view_id, 225 int render_view_id,
222 const content::MediaStreamDevice& device, 226 const content::MediaStreamDevice& device,
223 content::MediaRequestState state) { 227 content::MediaRequestState state) {
224 FOR_EACH_OBSERVER(Observer, observers_, 228 FOR_EACH_OBSERVER(Observer, observers_,
225 OnRequestUpdate(render_process_id, 229 OnRequestUpdate(render_process_id,
226 render_view_id, 230 render_view_id,
227 device, 231 device,
228 state)); 232 state));
229 } 233 }
OLDNEW
« no previous file with comments | « chrome/browser/media/media_capture_devices_dispatcher.h ('k') | chrome/browser/media/media_stream_devices_controller.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698