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

Side by Side Diff: net/proxy/proxy_resolver_error_observer.h

Issue 11885009: Improve performance of proxy resolver by tracing DNS dependencies. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: that's what she said Created 7 years, 10 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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 #ifndef NET_PROXY_PROXY_RESOLVER_ERROR_OBSERVER_H_ 5 #ifndef NET_PROXY_PROXY_RESOLVER_ERROR_OBSERVER_H_
6 #define NET_PROXY_PROXY_RESOLVER_ERROR_OBSERVER_H_ 6 #define NET_PROXY_PROXY_RESOLVER_ERROR_OBSERVER_H_
7 7
8 #include "base/basictypes.h" 8 #include "base/basictypes.h"
9 #include "base/string16.h" 9 #include "base/string16.h"
10 #include "net/base/net_export.h" 10 #include "net/base/net_export.h"
11 11
12 namespace net { 12 namespace net {
13 13
14 // Interface for observing JavaScript error messages from PAC scripts. The 14 // Interface for observing JavaScript error messages from PAC scripts.
mmenke 2013/01/29 20:19:08 May want to mention that this can be called on any
eroman 2013/01/29 22:51:23 Done. It is ambiguous what thread this will be ca
15 // default implementation of the ProxyResolverJSBindings takes a class
16 // implementing this interface and forwards all JavaScript errors related to
17 // PAC scripts.
18 class NET_EXPORT_PRIVATE ProxyResolverErrorObserver { 15 class NET_EXPORT_PRIVATE ProxyResolverErrorObserver {
19 public: 16 public:
20 ProxyResolverErrorObserver() {} 17 ProxyResolverErrorObserver() {}
21 virtual ~ProxyResolverErrorObserver() {} 18 virtual ~ProxyResolverErrorObserver() {}
22 19
23 // Handler for when an error is encountered. |line_number| may be -1 20 // Handler for when an error is encountered. |line_number| may be -1
24 // if a line number is not applicable to this error. |error| is a message 21 // if a line number is not applicable to this error. |error| is a message
25 // describing the error. 22 // describing the error.
26 virtual void OnPACScriptError(int line_number, const string16& error) = 0; 23 virtual void OnPACScriptError(int line_number, const string16& error) = 0;
27 24
28 private: 25 private:
29 DISALLOW_COPY_AND_ASSIGN(ProxyResolverErrorObserver); 26 DISALLOW_COPY_AND_ASSIGN(ProxyResolverErrorObserver);
30 }; 27 };
31 28
32 } // namespace net 29 } // namespace net
33 30
34 #endif // NET_PROXY_PROXY_RESOLVER_ERROR_OBSERVER_H_ 31 #endif // NET_PROXY_PROXY_RESOLVER_ERROR_OBSERVER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698