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

Side by Side Diff: chromecast/browser/cast_permission_manager.cc

Issue 2110343002: Reintroduce plumbing for user gesture into permission system. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Nits Created 4 years, 5 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
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 #include "chromecast/browser/cast_permission_manager.h" 5 #include "chromecast/browser/cast_permission_manager.h"
6 6
7 #include "base/callback.h" 7 #include "base/callback.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "content/public/browser/permission_type.h" 9 #include "content/public/browser/permission_type.h"
10 10
11 namespace chromecast { 11 namespace chromecast {
12 namespace shell { 12 namespace shell {
13 13
14 CastPermissionManager::CastPermissionManager() 14 CastPermissionManager::CastPermissionManager()
15 : content::PermissionManager() { 15 : content::PermissionManager() {
16 } 16 }
17 17
18 CastPermissionManager::~CastPermissionManager() { 18 CastPermissionManager::~CastPermissionManager() {
19 } 19 }
20 20
21 int CastPermissionManager::RequestPermission( 21 int CastPermissionManager::RequestPermission(
22 content::PermissionType permission, 22 content::PermissionType permission,
23 content::RenderFrameHost* render_frame_host, 23 content::RenderFrameHost* render_frame_host,
24 const GURL& origin, 24 const GURL& origin,
25 bool user_gesture,
25 const base::Callback<void(blink::mojom::PermissionStatus)>& callback) { 26 const base::Callback<void(blink::mojom::PermissionStatus)>& callback) {
26 LOG(INFO) << __FUNCTION__ << ": " << static_cast<int>(permission); 27 LOG(INFO) << __FUNCTION__ << ": " << static_cast<int>(permission);
27 callback.Run(blink::mojom::PermissionStatus::GRANTED); 28 callback.Run(blink::mojom::PermissionStatus::GRANTED);
28 return kNoPendingOperation; 29 return kNoPendingOperation;
29 } 30 }
30 31
31 int CastPermissionManager::RequestPermissions( 32 int CastPermissionManager::RequestPermissions(
32 const std::vector<content::PermissionType>& permissions, 33 const std::vector<content::PermissionType>& permissions,
33 content::RenderFrameHost* render_frame_host, 34 content::RenderFrameHost* render_frame_host,
34 const GURL& requesting_origin, 35 const GURL& requesting_origin,
36 bool user_gesture,
35 const base::Callback< 37 const base::Callback<
36 void(const std::vector<blink::mojom::PermissionStatus>&)>& callback) { 38 void(const std::vector<blink::mojom::PermissionStatus>&)>& callback) {
37 callback.Run(std::vector<blink::mojom::PermissionStatus>( 39 callback.Run(std::vector<blink::mojom::PermissionStatus>(
38 permissions.size(), blink::mojom::PermissionStatus::GRANTED)); 40 permissions.size(), blink::mojom::PermissionStatus::GRANTED));
39 return kNoPendingOperation; 41 return kNoPendingOperation;
40 } 42 }
41 43
42 void CastPermissionManager::CancelPermissionRequest(int request_id) { 44 void CastPermissionManager::CancelPermissionRequest(int request_id) {
43 } 45 }
44 46
(...skipping 24 matching lines...) Expand all
69 const base::Callback<void(blink::mojom::PermissionStatus)>& callback) { 71 const base::Callback<void(blink::mojom::PermissionStatus)>& callback) {
70 return kNoPendingOperation; 72 return kNoPendingOperation;
71 } 73 }
72 74
73 void CastPermissionManager::UnsubscribePermissionStatusChange( 75 void CastPermissionManager::UnsubscribePermissionStatusChange(
74 int subscription_id) { 76 int subscription_id) {
75 } 77 }
76 78
77 } // namespace shell 79 } // namespace shell
78 } // namespace chromecast 80 } // namespace chromecast
OLDNEW
« no previous file with comments | « chromecast/browser/cast_permission_manager.h ('k') | content/browser/permissions/permission_service_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698