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

Side by Side Diff: chrome/browser/chromeos/extensions/public_session_permission_helper_unittest.cc

Issue 2721553004: Remove auto raw pointer deduction from non-linux specific code. (Closed)
Patch Set: rebase Created 3 years, 9 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 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 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/chromeos/extensions/public_session_permission_helper.h" 5 #include "chrome/browser/chromeos/extensions/public_session_permission_helper.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <string> 8 #include <string>
9 #include <utility> 9 #include <utility>
10 #include <vector> 10 #include <vector>
(...skipping 124 matching lines...) Expand 10 before | Expand all | Expand 10 after
135 PublicSessionPermissionHelperTest::ReturnPrompt( 135 PublicSessionPermissionHelperTest::ReturnPrompt(
136 std::unique_ptr<ExtensionInstallPrompt> prompt, 136 std::unique_ptr<ExtensionInstallPrompt> prompt,
137 WebContents* web_contents) { 137 WebContents* web_contents) {
138 return prompt; 138 return prompt;
139 } 139 }
140 140
141 base::WeakPtr<ProgrammableInstallPrompt> 141 base::WeakPtr<ProgrammableInstallPrompt>
142 PublicSessionPermissionHelperTest::CallHandlePermissionRequest( 142 PublicSessionPermissionHelperTest::CallHandlePermissionRequest(
143 const scoped_refptr<Extension>& extension, 143 const scoped_refptr<Extension>& extension,
144 const PermissionIDSet& permissions) { 144 const PermissionIDSet& permissions) {
145 auto prompt = new ProgrammableInstallPrompt(web_contents()); 145 auto* prompt = new ProgrammableInstallPrompt(web_contents());
146 auto prompt_weak_ptr = prompt->AsWeakPtr(); 146 auto prompt_weak_ptr = prompt->AsWeakPtr();
147 auto factory_callback = base::Bind( 147 auto factory_callback = base::Bind(
148 &PublicSessionPermissionHelperTest::ReturnPrompt, base::Unretained(this), 148 &PublicSessionPermissionHelperTest::ReturnPrompt, base::Unretained(this),
149 base::Passed(base::WrapUnique<ExtensionInstallPrompt>(prompt))); 149 base::Passed(base::WrapUnique<ExtensionInstallPrompt>(prompt)));
150 HandlePermissionRequest( 150 HandlePermissionRequest(
151 *extension.get(), permissions, web_contents(), 151 *extension.get(), permissions, web_contents(),
152 base::Bind(&PublicSessionPermissionHelperTest::RequestResolved, 152 base::Bind(&PublicSessionPermissionHelperTest::RequestResolved,
153 base::Unretained(this)), 153 base::Unretained(this)),
154 factory_callback); 154 factory_callback);
155 // In case all permissions were already prompted, ReturnPrompt isn't called 155 // In case all permissions were already prompted, ReturnPrompt isn't called
(...skipping 90 matching lines...) Expand 10 before | Expand all | Expand 10 after
246 EXPECT_TRUE(prompt2); 246 EXPECT_TRUE(prompt2);
247 EXPECT_TRUE(get_did_show_dialog_and_reset()); 247 EXPECT_TRUE(get_did_show_dialog_and_reset());
248 prompt1->Resolve(Result::USER_CANCELED); 248 prompt1->Resolve(Result::USER_CANCELED);
249 EXPECT_TRUE(allowed_permissions_.at(0).Equals({})); 249 EXPECT_TRUE(allowed_permissions_.at(0).Equals({}));
250 prompt2->Resolve(Result::ACCEPTED); 250 prompt2->Resolve(Result::ACCEPTED);
251 EXPECT_TRUE(allowed_permissions_.at(1).Equals({permission_b})); 251 EXPECT_TRUE(allowed_permissions_.at(1).Equals({permission_b}));
252 } 252 }
253 253
254 } // namespace permission_helper 254 } // namespace permission_helper
255 } // namespace extensions 255 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698