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

Side by Side Diff: android_webview/browser/aw_browser_context.cc

Issue 1356933002: make ProxyService::CreateSystemProxyConfigService return scoped_ptrs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: small ios fix 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 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 "android_webview/browser/aw_browser_context.h" 5 #include "android_webview/browser/aw_browser_context.h"
6 6
7 #include "android_webview/browser/aw_browser_policy_connector.h" 7 #include "android_webview/browser/aw_browser_policy_connector.h"
8 #include "android_webview/browser/aw_form_database_service.h" 8 #include "android_webview/browser/aw_form_database_service.h"
9 #include "android_webview/browser/aw_permission_manager.h" 9 #include "android_webview/browser/aw_permission_manager.h"
10 #include "android_webview/browser/aw_pref_store.h" 10 #include "android_webview/browser/aw_pref_store.h"
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
56 void DeleteDirRecursively(const base::FilePath& path) { 56 void DeleteDirRecursively(const base::FilePath& path) {
57 if (!base::DeleteFile(path, true)) { 57 if (!base::DeleteFile(path, true)) {
58 // Deleting a non-existent file is considered successful, so this will 58 // Deleting a non-existent file is considered successful, so this will
59 // trigger only in case of real errors. 59 // trigger only in case of real errors.
60 LOG(WARNING) << "Failed to delete " << path.AsUTF8Unsafe(); 60 LOG(WARNING) << "Failed to delete " << path.AsUTF8Unsafe();
61 } 61 }
62 } 62 }
63 63
64 AwBrowserContext* g_browser_context = NULL; 64 AwBrowserContext* g_browser_context = NULL;
65 65
66 net::ProxyConfigService* CreateProxyConfigService() { 66 net::ProxyConfigService* CreateProxyConfigService() {
mmenke 2015/09/18 18:58:53 Should probably modify this to return a scoped_ptr
Charlie Harrison 2015/09/18 19:56:06 Done.
67 net::ProxyConfigServiceAndroid* config_service = 67 net::ProxyConfigServiceAndroid* config_service =
68 static_cast<net::ProxyConfigServiceAndroid*>( 68 static_cast<net::ProxyConfigServiceAndroid*>(
69 net::ProxyService::CreateSystemProxyConfigService( 69 net::ProxyService::CreateSystemProxyConfigService(
70 BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO), 70 BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO),
71 nullptr /* Ignored on Android */ )); 71 nullptr /* Ignored on Android */ ).release());
72 config_service->set_exclude_pac_url(true); 72 config_service->set_exclude_pac_url(true);
73 return config_service; 73 return config_service;
74 } 74 }
75 75
76 bool OverrideBlacklistForURL(const GURL& url, bool* block, int* reason) { 76 bool OverrideBlacklistForURL(const GURL& url, bool* block, int* reason) {
77 // We don't have URLs that should never be blacklisted here. 77 // We don't have URLs that should never be blacklisted here.
78 return false; 78 return false;
79 } 79 }
80 80
81 policy::URLBlacklistManager* CreateURLBlackListManager( 81 policy::URLBlacklistManager* CreateURLBlackListManager(
(...skipping 352 matching lines...) Expand 10 before | Expand all | Expand 10 after
434 return; 434 return;
435 // We don't care about commit_delay for now. It is just a dummy value. 435 // We don't care about commit_delay for now. It is just a dummy value.
436 base::TimeDelta commit_delay = base::TimeDelta::FromMinutes(60); 436 base::TimeDelta commit_delay = base::TimeDelta::FromMinutes(60);
437 data_reduction_proxy_service->EnableCompressionStatisticsLogging( 437 data_reduction_proxy_service->EnableCompressionStatisticsLogging(
438 user_pref_service_.get(), 438 user_pref_service_.get(),
439 BrowserThread::GetMessageLoopProxyForThread(BrowserThread::UI), 439 BrowserThread::GetMessageLoopProxyForThread(BrowserThread::UI),
440 commit_delay); 440 commit_delay);
441 } 441 }
442 442
443 } // namespace android_webview 443 } // namespace android_webview
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698