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

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

Issue 1311783007: refactor to Introduce AUDIO_CAPTURE and VIDEO_CAPTURE permissions. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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_device_permission_context.h
diff --git a/chrome/browser/media/media_stream_device_permission_context.h b/chrome/browser/media/media_stream_device_permission_context.h
new file mode 100644
index 0000000000000000000000000000000000000000..6e6b3f5b6aa6b3ee3380e8f584add87fee543344
--- /dev/null
+++ b/chrome/browser/media/media_stream_device_permission_context.h
@@ -0,0 +1,51 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_MEDIA_MEDIA_STREAM_DEVICE_PERMISSION_CONTEXT_H_
+#define CHROME_BROWSER_MEDIA_MEDIA_STREAM_DEVICE_PERMISSION_CONTEXT_H_
+
+#include "base/macros.h"
+#include "chrome/browser/permissions/permission_context_base.h"
+#include "components/content_settings/core/common/content_settings_types.h"
+
+// Common class which handles the mic and camera permissions.
+// MediaStreamMicPermissionContextFactory and
+// MediaStreamCameraPermissionContextFactory will instantiate this with
+// corresponding |permission_type|.
+class MediaStreamDevicePermissionContext : public PermissionContextBase {
+ public:
+ MediaStreamDevicePermissionContext(
+ Profile* profile,
+ const ContentSettingsType content_settings_type);
+ ~MediaStreamDevicePermissionContext() override;
+
+ // PermissionContextBase:
+ void RequestPermission(content::WebContents* web_contents,
+ const PermissionRequestID& id,
+ const GURL& requesting_frame,
+ bool user_gesture,
+ const BrowserPermissionCallback& callback) override;
+
+ // TODO(xhwang): GURL.GetOrigin() shouldn't be used as the origin. Need to
+ // refactor to use url::Origin. crbug.com/527149 is filed for this.
+ ContentSetting GetPermissionStatus(
+ const GURL& requesting_origin,
+ const GURL& embedding_origin) const override;
+
+ void ResetPermission(const GURL& requesting_origin,
+ const GURL& embedding_origin) override;
+
+ void CancelPermissionRequest(content::WebContents* web_contents,
+ const PermissionRequestID& id) override;
+
+ private:
+ // PermissionContextBase:
+ bool IsRestrictedToSecureOrigins() const override;
+
+ ContentSettingsType content_settings_type_;
+
+ DISALLOW_COPY_AND_ASSIGN(MediaStreamDevicePermissionContext);
+};
+
+#endif // CHROME_BROWSER_MEDIA_MEDIA_STREAM_DEVICE_PERMISSION_CONTEXT_H_

Powered by Google App Engine
This is Rietveld 408576698