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

Side by Side Diff: chrome/browser/media/media_stream_devices_controller.h

Issue 929253004: Do not always store media device rejection. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Renamed SetPermission to StorePersmission. Created 5 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
« no previous file with comments | « no previous file | chrome/browser/media/media_stream_devices_controller.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 CHROME_BROWSER_MEDIA_MEDIA_STREAM_DEVICES_CONTROLLER_H_ 5 #ifndef CHROME_BROWSER_MEDIA_MEDIA_STREAM_DEVICES_CONTROLLER_H_
6 #define CHROME_BROWSER_MEDIA_MEDIA_STREAM_DEVICES_CONTROLLER_H_ 6 #define CHROME_BROWSER_MEDIA_MEDIA_STREAM_DEVICES_CONTROLLER_H_
7 7
8 #include <map> 8 #include <map>
9 #include <string> 9 #include <string>
10 10
(...skipping 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
95 // denied completely, else it still has to be partially fullfilled. 95 // denied completely, else it still has to be partially fullfilled.
96 int FilterBlockedByDefaultDevices(); 96 int FilterBlockedByDefaultDevices();
97 97
98 // Returns true if the media section in content settings is set to 98 // Returns true if the media section in content settings is set to
99 // |CONTENT_SETTING_BLOCK|, otherwise returns false. 99 // |CONTENT_SETTING_BLOCK|, otherwise returns false.
100 bool IsDefaultMediaAccessBlocked() const; 100 bool IsDefaultMediaAccessBlocked() const;
101 101
102 // Returns true if the origin is a secure scheme, otherwise returns false. 102 // Returns true if the origin is a secure scheme, otherwise returns false.
103 bool IsSchemeSecure() const; 103 bool IsSchemeSecure() const;
104 104
105 // Sets the permission of the origin of the request. This is triggered when 105 // Store the permission to use media devices for the origin of the request.
106 // the users deny the request or allow the request for https sites. 106 // This is triggered when the users deny the request or allow the request
107 void SetPermission(bool allowed) const; 107 // for https sites.
108 void StorePersmission(bool allowed) const;
mcasas 2015/02/24 17:58:10 s/Persmission/Permission/
perkj_chrome 2015/02/25 09:22:19 ooops... Thanks
108 109
109 // Notifies the content setting UI that the media stream access request or 110 // Notifies the content setting UI that the media stream access request or
110 // part of the request is accepted. 111 // part of the request is accepted.
111 void NotifyUIRequestAccepted() const; 112 void NotifyUIRequestAccepted() const;
112 113
113 // Notifies the content setting UI that the media stream access request or 114 // Notifies the content setting UI that the media stream access request or
114 // part of the request is denied. 115 // part of the request is denied.
115 void NotifyUIRequestDenied(); 116 void NotifyUIRequestDenied();
116 117
117 // Return true if the type has been requested and permission is currently set 118 // Return true if the type has been requested and permission is currently set
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
150 // passed to the tab specific content settings when the permissions have been 151 // passed to the tab specific content settings when the permissions have been
151 // resolved. Currently only used by MEDIA_DEVICE_AUDIO_CAPTURE and 152 // resolved. Currently only used by MEDIA_DEVICE_AUDIO_CAPTURE and
152 // MEDIA_DEVICE_VIDEO_CAPTURE since those are the only types that require 153 // MEDIA_DEVICE_VIDEO_CAPTURE since those are the only types that require
153 // updates in the settings. 154 // updates in the settings.
154 MediaStreamTypeSettingsMap request_permissions_; 155 MediaStreamTypeSettingsMap request_permissions_;
155 156
156 DISALLOW_COPY_AND_ASSIGN(MediaStreamDevicesController); 157 DISALLOW_COPY_AND_ASSIGN(MediaStreamDevicesController);
157 }; 158 };
158 159
159 #endif // CHROME_BROWSER_MEDIA_MEDIA_STREAM_DEVICES_CONTROLLER_H_ 160 #endif // CHROME_BROWSER_MEDIA_MEDIA_STREAM_DEVICES_CONTROLLER_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/media/media_stream_devices_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698