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

Unified Diff: chrome/browser/media/media_stream_devices_controller.h

Issue 2258763002: Add a feature-controlled persistence checkbox to geolocation prompts on desktop Views platforms. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@permission-infobardelegate-clean
Patch Set: Fix for multiple requests Created 4 years, 4 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/media/media_stream_devices_controller.h
diff --git a/chrome/browser/media/media_stream_devices_controller.h b/chrome/browser/media/media_stream_devices_controller.h
index 3becc1d19c7bd93ea4fe977c4a6045446f1f766e..eca57a8eecabfcdef83439a7c6193bfa5632112e 100644
--- a/chrome/browser/media/media_stream_devices_controller.h
+++ b/chrome/browser/media/media_stream_devices_controller.h
@@ -57,6 +57,8 @@ class MediaStreamDevicesController : public PermissionRequest {
// PermissionRequest instead of a custom infobar delegate.
void GroupedRequestFinished(bool audio_accepted, bool video_accepted);
+ bool ShouldShowPersistenceToggle() const override;
+
// PermissionRequest:
int GetIconId() const override;
base::string16 GetMessageTextFragment() const override;
@@ -131,9 +133,6 @@ class MediaStreamDevicesController : public PermissionRequest {
// audio/video devices was granted or not.
content::MediaResponseCallback callback_;
- // Whether the permissions granted or denied by the user should be persisted.
- bool persist_permission_changes_;
-
DISALLOW_COPY_AND_ASSIGN(MediaStreamDevicesController);
};

Powered by Google App Engine
This is Rietveld 408576698