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

Side by Side Diff: blimp/engine/app/blimp_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 "blimp/engine/app/blimp_permission_manager.h" 5 #include "blimp/engine/app/blimp_permission_manager.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "base/callback.h" 9 #include "base/callback.h"
10 #include "content/public/browser/permission_type.h" 10 #include "content/public/browser/permission_type.h"
11 11
12 namespace blimp { 12 namespace blimp {
13 namespace engine { 13 namespace engine {
14 14
15 BlimpPermissionManager::BlimpPermissionManager() 15 BlimpPermissionManager::BlimpPermissionManager()
16 : content::PermissionManager() {} 16 : content::PermissionManager() {}
17 17
18 BlimpPermissionManager::~BlimpPermissionManager() {} 18 BlimpPermissionManager::~BlimpPermissionManager() {}
19 19
20 int BlimpPermissionManager::RequestPermission( 20 int BlimpPermissionManager::RequestPermission(
21 content::PermissionType permission, 21 content::PermissionType permission,
22 content::RenderFrameHost* render_frame_host, 22 content::RenderFrameHost* render_frame_host,
23 const GURL& origin, 23 const GURL& origin,
24 bool user_gesture,
24 const base::Callback<void(blink::mojom::PermissionStatus)>& callback) { 25 const base::Callback<void(blink::mojom::PermissionStatus)>& callback) {
25 callback.Run(blink::mojom::PermissionStatus::DENIED); 26 callback.Run(blink::mojom::PermissionStatus::DENIED);
26 return kNoPendingOperation; 27 return kNoPendingOperation;
27 } 28 }
28 29
29 int BlimpPermissionManager::RequestPermissions( 30 int BlimpPermissionManager::RequestPermissions(
30 const std::vector<content::PermissionType>& permission, 31 const std::vector<content::PermissionType>& permission,
31 content::RenderFrameHost* render_frame_host, 32 content::RenderFrameHost* render_frame_host,
32 const GURL& requesting_origin, 33 const GURL& requesting_origin,
34 bool user_gesture,
33 const base::Callback< 35 const base::Callback<
34 void(const std::vector<blink::mojom::PermissionStatus>&)>& callback) { 36 void(const std::vector<blink::mojom::PermissionStatus>&)>& callback) {
35 callback.Run(std::vector<blink::mojom::PermissionStatus>( 37 callback.Run(std::vector<blink::mojom::PermissionStatus>(
36 permission.size(), blink::mojom::PermissionStatus::DENIED)); 38 permission.size(), blink::mojom::PermissionStatus::DENIED));
37 return kNoPendingOperation; 39 return kNoPendingOperation;
38 } 40 }
39 41
40 void BlimpPermissionManager::CancelPermissionRequest(int request_id) {} 42 void BlimpPermissionManager::CancelPermissionRequest(int request_id) {}
41 43
42 void BlimpPermissionManager::ResetPermission(content::PermissionType permission, 44 void BlimpPermissionManager::ResetPermission(content::PermissionType permission,
(...skipping 18 matching lines...) Expand all
61 const GURL& embedding_origin, 63 const GURL& embedding_origin,
62 const base::Callback<void(blink::mojom::PermissionStatus)>& callback) { 64 const base::Callback<void(blink::mojom::PermissionStatus)>& callback) {
63 return -1; 65 return -1;
64 } 66 }
65 67
66 void BlimpPermissionManager::UnsubscribePermissionStatusChange( 68 void BlimpPermissionManager::UnsubscribePermissionStatusChange(
67 int subscription_id) {} 69 int subscription_id) {}
68 70
69 } // namespace engine 71 } // namespace engine
70 } // namespace blimp 72 } // namespace blimp
OLDNEW
« no previous file with comments | « blimp/engine/app/blimp_permission_manager.h ('k') | chrome/browser/background_sync/background_sync_permission_context.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698