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

Side by Side Diff: chrome/browser/permissions/permission_prompt_android.cc

Issue 2440403002: Make GroupedPermissionInfoBarDelegate's methods call through PermissionPromptAndroid (Closed)
Patch Set: address Created 4 years, 1 month 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 | « chrome/browser/permissions/permission_prompt_android.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "chrome/browser/permissions/permission_prompt_android.h" 5 #include "chrome/browser/permissions/permission_prompt_android.h"
6 6
7 #include "base/memory/ptr_util.h" 7 #include "base/memory/ptr_util.h"
8 #include "chrome/browser/infobars/infobar_service.h" 8 #include "chrome/browser/infobars/infobar_service.h"
9 #include "chrome/browser/media/webrtc/media_stream_devices_controller.h" 9 #include "chrome/browser/media/webrtc/media_stream_devices_controller.h"
10 #include "chrome/browser/permissions/grouped_permission_infobar_delegate_android .h" 10 #include "chrome/browser/permissions/grouped_permission_infobar_delegate_android .h"
11 #include "chrome/browser/permissions/permission_request.h" 11 #include "chrome/browser/permissions/permission_request.h"
12 #include "chrome/browser/permissions/permission_request_id.h" 12 #include "chrome/browser/permissions/permission_request_id.h"
13 #include "components/infobars/core/infobar.h" 13 #include "components/infobars/core/infobar.h"
14 #include "content/public/browser/web_contents.h" 14 #include "content/public/browser/web_contents.h"
15 15
16 PermissionPromptAndroid::PermissionPromptAndroid( 16 PermissionPromptAndroid::PermissionPromptAndroid(
17 content::WebContents* web_contents) 17 content::WebContents* web_contents)
18 : web_contents_(web_contents), delegate_(nullptr), infobar_(nullptr) { 18 : web_contents_(web_contents), delegate_(nullptr), infobar_(nullptr) {
19 DCHECK(web_contents); 19 DCHECK(web_contents);
20 } 20 }
21 21
22 PermissionPromptAndroid::~PermissionPromptAndroid() {} 22 PermissionPromptAndroid::~PermissionPromptAndroid() {
23 GroupedPermissionInfoBarDelegate* infobar_delegate =
24 static_cast<GroupedPermissionInfoBarDelegate*>(infobar_->delegate());
raymes 2016/11/01 05:29:15 Could infobar_ be an GroupedPermissionInfoBar? The
lshang 2016/11/01 22:58:55 GetDelegate() is a private method of infobar, so I
25 infobar_delegate->PermissionPromptDestroyed();
26 }
23 27
24 void PermissionPromptAndroid::SetDelegate(Delegate* delegate) { 28 void PermissionPromptAndroid::SetDelegate(Delegate* delegate) {
25 delegate_ = delegate; 29 delegate_ = delegate;
26 } 30 }
27 31
28 void PermissionPromptAndroid::Show( 32 void PermissionPromptAndroid::Show(
29 const std::vector<PermissionRequest*>& requests, 33 const std::vector<PermissionRequest*>& requests,
30 const std::vector<bool>& values) { 34 const std::vector<bool>& values) {
31 InfoBarService* infobar_service = 35 InfoBarService* infobar_service =
32 InfoBarService::FromWebContents(web_contents_); 36 InfoBarService::FromWebContents(web_contents_);
33 if (!infobar_service) 37 if (!infobar_service)
34 return; 38 return;
35 39
36 infobar_ = GroupedPermissionInfoBarDelegate::Create( 40 infobar_ = GroupedPermissionInfoBarDelegate::Create(
37 infobar_service, requests[0]->GetOrigin(), requests); 41 this, infobar_service, requests[0]->GetOrigin(), requests);
38 } 42 }
39 43
40 bool PermissionPromptAndroid::CanAcceptRequestUpdate() { 44 bool PermissionPromptAndroid::CanAcceptRequestUpdate() {
41 return false; 45 return false;
42 } 46 }
43 47
44 void PermissionPromptAndroid::Hide() { 48 void PermissionPromptAndroid::Hide() {
45 InfoBarService* infobar_service = 49 InfoBarService* infobar_service =
46 InfoBarService::FromWebContents(web_contents_); 50 InfoBarService::FromWebContents(web_contents_);
47 if (infobar_ && infobar_service) { 51 if (infobar_ && infobar_service) {
(...skipping 14 matching lines...) Expand all
62 NOTREACHED() << "GetNativeWindow is not implemented"; 66 NOTREACHED() << "GetNativeWindow is not implemented";
63 return nullptr; 67 return nullptr;
64 } 68 }
65 69
66 void PermissionPromptAndroid::Closing() { 70 void PermissionPromptAndroid::Closing() {
67 infobar_ = nullptr; 71 infobar_ = nullptr;
68 if (delegate_) 72 if (delegate_)
69 delegate_->Closing(); 73 delegate_->Closing();
70 } 74 }
71 75
76 void PermissionPromptAndroid::Accept() {
77 if (delegate_)
78 delegate_->Accept();
79 }
80
81 void PermissionPromptAndroid::Deny() {
82 if (delegate_)
83 delegate_->Deny();
84 }
85
72 // static 86 // static
73 std::unique_ptr<PermissionPrompt> PermissionPrompt::Create( 87 std::unique_ptr<PermissionPrompt> PermissionPrompt::Create(
74 content::WebContents* web_contents) { 88 content::WebContents* web_contents) {
75 return base::MakeUnique<PermissionPromptAndroid>(web_contents); 89 return base::MakeUnique<PermissionPromptAndroid>(web_contents);
76 } 90 }
OLDNEW
« no previous file with comments | « chrome/browser/permissions/permission_prompt_android.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698