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

Side by Side Diff: chrome/browser/extensions/crx_installer_browsertest.cc

Issue 2257113002: Re-write many calls to WrapUnique() with MakeUnique() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 3 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/extensions/crx_installer.h" 5 #include "chrome/browser/extensions/crx_installer.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <utility> 9 #include <utility>
10 10
(...skipping 167 matching lines...) Expand 10 before | Expand all | Expand 10 after
178 178
179 MockPromptProxy::~MockPromptProxy() {} 179 MockPromptProxy::~MockPromptProxy() {}
180 180
181 std::unique_ptr<ExtensionInstallPrompt> MockPromptProxy::CreatePrompt() { 181 std::unique_ptr<ExtensionInstallPrompt> MockPromptProxy::CreatePrompt() {
182 return std::unique_ptr<MockInstallPrompt>( 182 return std::unique_ptr<MockInstallPrompt>(
183 new MockInstallPrompt(web_contents_, this)); 183 new MockInstallPrompt(web_contents_, this));
184 } 184 }
185 185
186 std::unique_ptr<MockPromptProxy> CreateMockPromptProxyForBrowser( 186 std::unique_ptr<MockPromptProxy> CreateMockPromptProxyForBrowser(
187 Browser* browser) { 187 Browser* browser) {
188 return base::WrapUnique( 188 return base::MakeUnique<MockPromptProxy>(
189 new MockPromptProxy(browser->tab_strip_model()->GetActiveWebContents())); 189 browser->tab_strip_model()->GetActiveWebContents());
190 } 190 }
191 191
192 class ManagementPolicyMock : public extensions::ManagementPolicy::Provider { 192 class ManagementPolicyMock : public extensions::ManagementPolicy::Provider {
193 public: 193 public:
194 ManagementPolicyMock() {} 194 ManagementPolicyMock() {}
195 195
196 std::string GetDebugPolicyProviderName() const override { 196 std::string GetDebugPolicyProviderName() const override {
197 return "ManagementPolicyMock"; 197 return "ManagementPolicyMock";
198 } 198 }
199 199
(...skipping 490 matching lines...) Expand 10 before | Expand all | Expand 10 after
690 EXPECT_TRUE(WaitForCrxInstallerDone()); 690 EXPECT_TRUE(WaitForCrxInstallerDone());
691 const Extension* extension = installer->extension(); 691 const Extension* extension = installer->extension();
692 ASSERT_TRUE(extension); 692 ASSERT_TRUE(extension);
693 ASSERT_EQ(extension_id, extension->id()); 693 ASSERT_EQ(extension_id, extension->id());
694 EXPECT_TRUE(ExtensionPrefs::Get(profile())->AllowFileAccess(extension_id)); 694 EXPECT_TRUE(ExtensionPrefs::Get(profile())->AllowFileAccess(extension_id));
695 EXPECT_TRUE(extension->creation_flags() & Extension::ALLOW_FILE_ACCESS); 695 EXPECT_TRUE(extension->creation_flags() & Extension::ALLOW_FILE_ACCESS);
696 } 696 }
697 } 697 }
698 698
699 } // namespace extensions 699 } // namespace extensions
OLDNEW
« no previous file with comments | « chrome/browser/extensions/crx_installer.cc ('k') | chrome/browser/extensions/extension_action_runner.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698