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

Unified Diff: chrome/browser/extensions/extension_install_prompt_unittest.cc

Issue 1349613003: [Extensions] Un-refcount PermissionSet (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 5 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/extension_install_prompt_unittest.cc
diff --git a/chrome/browser/extensions/extension_install_prompt_unittest.cc b/chrome/browser/extensions/extension_install_prompt_unittest.cc
index 31f7c86341ca528cd7e9f662b9401efedbb20728..8102a1f7d4dd04b579b17422871300a86353f913 100644
--- a/chrome/browser/extensions/extension_install_prompt_unittest.cc
+++ b/chrome/browser/extensions/extension_install_prompt_unittest.cc
@@ -42,9 +42,9 @@ TEST(ExtensionInstallPromptUnittest, PromptShowsPermissionWarnings) {
content::TestBrowserThreadBundle thread_bundle;
APIPermissionSet api_permissions;
api_permissions.insert(APIPermission::kTab);
- scoped_refptr<const PermissionSet> permission_set =
+ scoped_ptr<const PermissionSet> permission_set(
new PermissionSet(api_permissions, ManifestPermissionSet(),
- URLPatternSet(), URLPatternSet());
+ URLPatternSet(), URLPatternSet()));
scoped_refptr<const Extension> extension =
ExtensionBuilder().SetManifest(
DictionaryBuilder().Set("name", "foo")
@@ -59,8 +59,7 @@ TEST(ExtensionInstallPromptUnittest, PromptShowsPermissionWarnings) {
1u, // |regular_permissions_count|.
0u)); // |withheld_permissions_count|.
prompt.ConfirmPermissions(nullptr, // no delegate
- extension.get(),
- permission_set.get());
+ extension.get(), permission_set.Pass());
run_loop.Run();
}
« no previous file with comments | « chrome/browser/extensions/extension_install_prompt.cc ('k') | chrome/browser/extensions/extension_management.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698