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

Side by Side Diff: chrome/browser/extensions/api/proxy/proxy_api_helpers.h

Issue 2785883003: Use unique_ptr<DictionaryValue> in ProxyConfigDictionary (Closed)
Patch Set: Created 3 years, 8 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 // Definition of helper functions for the Chrome Extensions Proxy Settings API. 5 // Definition of helper functions for the Chrome Extensions Proxy Settings API.
6 6
7 #ifndef CHROME_BROWSER_EXTENSIONS_API_PROXY_PROXY_API_HELPERS_H_ 7 #ifndef CHROME_BROWSER_EXTENSIONS_API_PROXY_PROXY_API_HELPERS_H_
8 #define CHROME_BROWSER_EXTENSIONS_API_PROXY_PROXY_API_HELPERS_H_ 8 #define CHROME_BROWSER_EXTENSIONS_API_PROXY_PROXY_API_HELPERS_H_
9 9
10 #include <string> 10 #include <string>
jdoerrie 2017/03/30 11:00:39 nit: add #include <memory>
vabr (Chromium) 2017/03/30 12:07:41 Done.
11 11
12 #include "components/proxy_config/proxy_prefs.h" 12 #include "components/proxy_config/proxy_prefs.h"
13 #include "net/proxy/proxy_config.h" 13 #include "net/proxy/proxy_config.h"
14 14
15 class ProxyConfigDictionary; 15 class ProxyConfigDictionary;
16 16
17 namespace base { 17 namespace base {
18 class DictionaryValue; 18 class DictionaryValue;
19 class ListValue; 19 class ListValue;
20 } 20 }
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
71 std::string* error, 71 std::string* error,
72 bool* bad_message); 72 bool* bad_message);
73 bool GetBypassListFromExtensionPref(const base::DictionaryValue* proxy_config, 73 bool GetBypassListFromExtensionPref(const base::DictionaryValue* proxy_config,
74 std::string* out, 74 std::string* out,
75 std::string* error, 75 std::string* error,
76 bool* bad_message); 76 bool* bad_message);
77 77
78 // Creates and returns a ProxyConfig dictionary (as defined in the extension 78 // Creates and returns a ProxyConfig dictionary (as defined in the extension
79 // API) from the given parameters. Ownership is passed to the caller. 79 // API) from the given parameters. Ownership is passed to the caller.
80 // Depending on the value of |mode_enum|, several of the strings may be empty. 80 // Depending on the value of |mode_enum|, several of the strings may be empty.
81 base::DictionaryValue* CreateProxyConfigDict( 81 std::unique_ptr<base::DictionaryValue> CreateProxyConfigDict(
82 ProxyPrefs::ProxyMode mode_enum, 82 ProxyPrefs::ProxyMode mode_enum,
83 bool pac_mandatory, 83 bool pac_mandatory,
84 const std::string& pac_url, 84 const std::string& pac_url,
85 const std::string& pac_data, 85 const std::string& pac_data,
86 const std::string& proxy_rules_string, 86 const std::string& proxy_rules_string,
87 const std::string& bypass_list, 87 const std::string& bypass_list,
88 std::string* error); 88 std::string* error);
89 89
90 // Converts a ProxyServer dictionary instance (as defined in the extension API) 90 // Converts a ProxyServer dictionary instance (as defined in the extension API)
91 // |proxy_server| to a net::ProxyServer. 91 // |proxy_server| to a net::ProxyServer.
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
128 128
129 // Tokenizes the |in| at delimiters |delims| and returns a new ListValue with 129 // Tokenizes the |in| at delimiters |delims| and returns a new ListValue with
130 // StringValues created from the tokens. Ownership is passed to the caller. 130 // StringValues created from the tokens. Ownership is passed to the caller.
131 base::ListValue* TokenizeToStringList(const std::string& in, 131 base::ListValue* TokenizeToStringList(const std::string& in,
132 const std::string& delims); 132 const std::string& delims);
133 133
134 } // namespace proxy_api_helpers 134 } // namespace proxy_api_helpers
135 } // namespace extensions 135 } // namespace extensions
136 136
137 #endif // CHROME_BROWSER_EXTENSIONS_API_PROXY_PROXY_API_HELPERS_H_ 137 #endif // CHROME_BROWSER_EXTENSIONS_API_PROXY_PROXY_API_HELPERS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698