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

Side by Side Diff: content/renderer/media/media_permission_dispatcher.h

Issue 1470233002: media: Generalize MediaPermissionDispatcher. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase only Created 4 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 | content/renderer/media/media_permission_dispatcher.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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 CONTENT_RENDERER_MEDIA_MEDIA_PERMISSION_DISPATCHER_H_ 5 #ifndef CONTENT_RENDERER_MEDIA_MEDIA_PERMISSION_DISPATCHER_H_
6 #define CONTENT_RENDERER_MEDIA_MEDIA_PERMISSION_DISPATCHER_H_ 6 #define CONTENT_RENDERER_MEDIA_MEDIA_PERMISSION_DISPATCHER_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <map> 10 #include <map>
11 11
12 #include "base/callback.h"
12 #include "base/macros.h" 13 #include "base/macros.h"
13 #include "base/memory/ref_counted.h" 14 #include "base/memory/ref_counted.h"
14 #include "base/memory/weak_ptr.h" 15 #include "base/memory/weak_ptr.h"
15 #include "content/common/content_export.h" 16 #include "content/common/content_export.h"
16 #include "content/common/permission_service.mojom.h" 17 #include "content/common/permission_service.mojom.h"
17 #include "content/public/renderer/render_frame_observer.h"
18 #include "media/base/media_permission.h" 18 #include "media/base/media_permission.h"
19 19
20 namespace base { 20 namespace base {
21 class SingleThreadTaskRunner; 21 class SingleThreadTaskRunner;
22 } 22 }
23 23
24 namespace content { 24 namespace content {
25 25
26 // MediaPermission implementation. 26 // MediaPermission implementation using content PermissionService.
27 class CONTENT_EXPORT MediaPermissionDispatcher : public media::MediaPermission, 27 class CONTENT_EXPORT MediaPermissionDispatcher : public media::MediaPermission {
28 public RenderFrameObserver {
29 public: 28 public:
30 explicit MediaPermissionDispatcher(RenderFrame* render_frame); 29 using ConnectToServiceCB =
30 base::Callback<void(mojo::InterfaceRequest<PermissionService>)>;
31
32 explicit MediaPermissionDispatcher(
33 const ConnectToServiceCB& connect_to_service_cb);
31 ~MediaPermissionDispatcher() override; 34 ~MediaPermissionDispatcher() override;
32 35
33 // media::MediaPermission implementation. 36 // media::MediaPermission implementation.
34 // Note: Can be called on any thread. The |permission_status_cb| will always 37 // Note: Can be called on any thread. The |permission_status_cb| will always
35 // be fired on the thread where these methods are called. 38 // be fired on the thread where these methods are called.
36 void HasPermission(Type type, 39 void HasPermission(Type type,
37 const GURL& security_origin, 40 const GURL& security_origin,
38 const PermissionStatusCB& permission_status_cb) override; 41 const PermissionStatusCB& permission_status_cb) override;
39 void RequestPermission( 42 void RequestPermission(
40 Type type, 43 Type type,
41 const GURL& security_origin, 44 const GURL& security_origin,
42 const PermissionStatusCB& permission_status_cb) override; 45 const PermissionStatusCB& permission_status_cb) override;
43 46
44 private: 47 private:
45 // Map of request IDs and pending PermissionStatusCBs. 48 // Map of request IDs and pending PermissionStatusCBs.
46 typedef std::map<uint32_t, PermissionStatusCB> RequestMap; 49 typedef std::map<uint32_t, PermissionStatusCB> RequestMap;
47 50
48 // Register PermissionStatusCBs. Returns |request_id| that can be used to make 51 // Register PermissionStatusCBs. Returns |request_id| that can be used to make
49 // PermissionService calls. 52 // PermissionService calls.
50 uint32_t RegisterCallback(const PermissionStatusCB& permission_status_cb); 53 uint32_t RegisterCallback(const PermissionStatusCB& permission_status_cb);
51 54
52 // Callback for |permission_service_| calls. 55 // Callback for |permission_service_| calls.
53 void OnPermissionStatus(uint32_t request_id, PermissionStatus status); 56 void OnPermissionStatus(uint32_t request_id, PermissionStatus status);
54 57
58 ConnectToServiceCB connect_to_service_cb_;
55 scoped_refptr<base::SingleThreadTaskRunner> task_runner_; 59 scoped_refptr<base::SingleThreadTaskRunner> task_runner_;
56 uint32_t next_request_id_; 60 uint32_t next_request_id_;
57 RequestMap requests_; 61 RequestMap requests_;
58 PermissionServicePtr permission_service_; 62 PermissionServicePtr permission_service_;
59 63
60 // Used to safely post MediaPermission calls for execution on |task_runner_|. 64 // Used to safely post MediaPermission calls for execution on |task_runner_|.
61 base::WeakPtr<MediaPermissionDispatcher> weak_ptr_; 65 base::WeakPtr<MediaPermissionDispatcher> weak_ptr_;
62 66
63 base::WeakPtrFactory<MediaPermissionDispatcher> weak_factory_; 67 base::WeakPtrFactory<MediaPermissionDispatcher> weak_factory_;
64 68
65 DISALLOW_COPY_AND_ASSIGN(MediaPermissionDispatcher); 69 DISALLOW_COPY_AND_ASSIGN(MediaPermissionDispatcher);
66 }; 70 };
67 71
68 } // namespace content 72 } // namespace content
69 73
70 #endif // CONTENT_RENDERER_MEDIA_MEDIA_PERMISSION_DISPATCHER_H_ 74 #endif // CONTENT_RENDERER_MEDIA_MEDIA_PERMISSION_DISPATCHER_H_
OLDNEW
« no previous file with comments | « no previous file | content/renderer/media/media_permission_dispatcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698