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

Side by Side Diff: chrome/browser/chrome_content_browser_client.cc

Issue 2334613003: Re-write many calls to WrapUnique() with MakeUnique() (Closed)
Patch Set: Changes from review by sky 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/chrome_content_browser_client.h" 5 #include "chrome/browser/chrome_content_browser_client.h"
6 6
7 #include <map> 7 #include <map>
8 #include <set> 8 #include <set>
9 #include <utility> 9 #include <utility>
10 #include <vector> 10 #include <vector>
(...skipping 2022 matching lines...) Expand 10 before | Expand all | Expand 10 after
2033 2033
2034 QuotaPermissionContext* 2034 QuotaPermissionContext*
2035 ChromeContentBrowserClient::CreateQuotaPermissionContext() { 2035 ChromeContentBrowserClient::CreateQuotaPermissionContext() {
2036 return new ChromeQuotaPermissionContext(); 2036 return new ChromeQuotaPermissionContext();
2037 } 2037 }
2038 2038
2039 std::unique_ptr<storage::QuotaEvictionPolicy> 2039 std::unique_ptr<storage::QuotaEvictionPolicy>
2040 ChromeContentBrowserClient::GetTemporaryStorageEvictionPolicy( 2040 ChromeContentBrowserClient::GetTemporaryStorageEvictionPolicy(
2041 content::BrowserContext* context) { 2041 content::BrowserContext* context) {
2042 return SiteEngagementEvictionPolicy::IsEnabled() 2042 return SiteEngagementEvictionPolicy::IsEnabled()
2043 ? base::WrapUnique(new SiteEngagementEvictionPolicy(context)) 2043 ? base::MakeUnique<SiteEngagementEvictionPolicy>(context)
2044 : nullptr; 2044 : nullptr;
2045 } 2045 }
2046 2046
2047 void ChromeContentBrowserClient::AllowCertificateError( 2047 void ChromeContentBrowserClient::AllowCertificateError(
2048 content::WebContents* web_contents, 2048 content::WebContents* web_contents,
2049 int cert_error, 2049 int cert_error,
2050 const net::SSLInfo& ssl_info, 2050 const net::SSLInfo& ssl_info,
2051 const GURL& request_url, 2051 const GURL& request_url,
2052 ResourceType resource_type, 2052 ResourceType resource_type,
2053 bool overridable, 2053 bool overridable,
(...skipping 1114 matching lines...) Expand 10 before | Expand all | Expand 10 after
3168 if (channel <= kMaxDisableEncryptionChannel) { 3168 if (channel <= kMaxDisableEncryptionChannel) {
3169 static const char* const kWebRtcDevSwitchNames[] = { 3169 static const char* const kWebRtcDevSwitchNames[] = {
3170 switches::kDisableWebRtcEncryption, 3170 switches::kDisableWebRtcEncryption,
3171 }; 3171 };
3172 to_command_line->CopySwitchesFrom(from_command_line, 3172 to_command_line->CopySwitchesFrom(from_command_line,
3173 kWebRtcDevSwitchNames, 3173 kWebRtcDevSwitchNames,
3174 arraysize(kWebRtcDevSwitchNames)); 3174 arraysize(kWebRtcDevSwitchNames));
3175 } 3175 }
3176 } 3176 }
3177 #endif // defined(ENABLE_WEBRTC) 3177 #endif // defined(ENABLE_WEBRTC)
OLDNEW
« no previous file with comments | « chrome/browser/chrome_browser_main.cc ('k') | chrome/browser/content_settings/local_shared_objects_container.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698