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

Unified Diff: android_webview/browser/aw_browser_policy_connector.cc

Issue 1852513003: Convert //android_webview to use std::unique_ptr (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: git is hard Created 4 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 side-by-side diff with in-line comments
Download patch
Index: android_webview/browser/aw_browser_policy_connector.cc
diff --git a/android_webview/browser/aw_browser_policy_connector.cc b/android_webview/browser/aw_browser_policy_connector.cc
index f3123f5328151390df7c2e67c332f7e4de8b6e23..8d366c59271063467c4597f83c728a80584af4ae 100644
--- a/android_webview/browser/aw_browser_policy_connector.cc
+++ b/android_webview/browser/aw_browser_policy_connector.cc
@@ -6,6 +6,7 @@
#include "android_webview/browser/aw_browser_context.h"
#include "base/bind.h"
+#include "base/memory/ptr_util.h"
#include "components/policy/core/browser/android/android_combined_policy_provider.h"
#include "components/policy/core/browser/configuration_policy_handler_list.h"
#include "components/policy/core/browser/url_blacklist_policy_handler.h"
@@ -28,25 +29,25 @@ const policy::PolicyDetails* GetChromePolicyDetails(const std::string& policy) {
// Factory for the handlers that will be responsible for converting the policies
// to the associated preferences.
-scoped_ptr<policy::ConfigurationPolicyHandlerList> BuildHandlerList(
+std::unique_ptr<policy::ConfigurationPolicyHandlerList> BuildHandlerList(
const policy::Schema& chrome_schema) {
- scoped_ptr<policy::ConfigurationPolicyHandlerList> handlers(
+ std::unique_ptr<policy::ConfigurationPolicyHandlerList> handlers(
new policy::ConfigurationPolicyHandlerList(
base::Bind(&PopulatePolicyHandlerParameters),
base::Bind(&GetChromePolicyDetails)));
// URL Filtering
- handlers->AddHandler(make_scoped_ptr(new policy::SimplePolicyHandler(
+ handlers->AddHandler(base::WrapUnique(new policy::SimplePolicyHandler(
policy::key::kURLWhitelist, policy::policy_prefs::kUrlWhitelist,
base::Value::TYPE_LIST)));
handlers->AddHandler(
- make_scoped_ptr(new policy::URLBlacklistPolicyHandler()));
+ base::WrapUnique(new policy::URLBlacklistPolicyHandler()));
// HTTP Negotiate authentication
- handlers->AddHandler(make_scoped_ptr(new policy::SimplePolicyHandler(
+ handlers->AddHandler(base::WrapUnique(new policy::SimplePolicyHandler(
policy::key::kAuthServerWhitelist, prefs::kAuthServerWhitelist,
base::Value::TYPE_STRING)));
- handlers->AddHandler(make_scoped_ptr(new policy::SimplePolicyHandler(
+ handlers->AddHandler(base::WrapUnique(new policy::SimplePolicyHandler(
policy::key::kAuthAndroidNegotiateAccountType,
prefs::kAuthAndroidNegotiateAccountType, base::Value::TYPE_STRING)));
@@ -57,7 +58,7 @@ scoped_ptr<policy::ConfigurationPolicyHandlerList> BuildHandlerList(
AwBrowserPolicyConnector::AwBrowserPolicyConnector()
: BrowserPolicyConnectorBase(base::Bind(&BuildHandlerList)) {
- SetPlatformPolicyProvider(make_scoped_ptr(
+ SetPlatformPolicyProvider(base::WrapUnique(
new policy::android::AndroidCombinedPolicyProvider(GetSchemaRegistry())));
InitPolicyProviders();
}
« no previous file with comments | « android_webview/browser/aw_browser_main_parts.h ('k') | android_webview/browser/aw_content_browser_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698