OLD | NEW |
---|---|
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2013 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_V8_TRACING_H_ | 5 #ifndef NET_PROXY_PROXY_RESOLVER_V8_TRACING_H_ |
6 #define NET_PROXY_PROXY_RESOLVER_V8_TRACING_H_ | 6 #define NET_PROXY_PROXY_RESOLVER_V8_TRACING_H_ |
7 | 7 |
8 #include "base/macros.h" | 8 #include "base/macros.h" |
9 #include "base/memory/ref_counted.h" | 9 #include "base/memory/ref_counted.h" |
10 #include "base/memory/scoped_ptr.h" | 10 #include "base/memory/scoped_ptr.h" |
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
82 scoped_ptr<ProxyResolverV8Tracing>* resolver, | 82 scoped_ptr<ProxyResolverV8Tracing>* resolver, |
83 const CompletionCallback& callback, | 83 const CompletionCallback& callback, |
84 scoped_ptr<ProxyResolverFactory::Request>* request) = 0; | 84 scoped_ptr<ProxyResolverFactory::Request>* request) = 0; |
85 | 85 |
86 static scoped_ptr<ProxyResolverV8TracingFactory> Create(); | 86 static scoped_ptr<ProxyResolverV8TracingFactory> Create(); |
87 | 87 |
88 private: | 88 private: |
89 DISALLOW_COPY_AND_ASSIGN(ProxyResolverV8TracingFactory); | 89 DISALLOW_COPY_AND_ASSIGN(ProxyResolverV8TracingFactory); |
90 }; | 90 }; |
91 | 91 |
92 // TODO(eroman): Remove when done gathering data for crbug.com/593759 | |
93 enum class PacResultForStrippedUrl { | |
94 // Did NOT measure the impact of running FindProxyForURL() with a modified | |
95 // URL path, because the original URL could not complete using the | |
96 // non-blocking DNS mode. | |
97 SKIPPED_FALLBACK_BLOCKING_DNS = 0, | |
98 | |
99 // The result of running FindProxyForURL() with a modified URL path appears | |
100 // to be indistinguishable. (Although there may have been sideffects to the | |
101 // script state that won't manifest until later invocations). | |
102 SUCCESS = 1, | |
103 | |
104 // Calling FindProxyForURL() with a modified URL path returned the same proxy | |
105 // list, but had measurable sideffects in calls to alert(). | |
cbentzel
2016/03/18 18:54:53
I had no idea you could alert in PAC eval.
eroman
2016/03/18 20:20:16
tl;dr: Yes, alert() basically just exists for prin
| |
106 SUCCESS_DIFFERENT_ALERTS = 2, | |
107 | |
108 // Calling FindProxyForURL() with a modified URL path returned the same proxy | |
109 // list, but invoked a different sequence of DNS resolutions. This would | |
110 // require a rather unusual script to trigger. | |
111 SUCCESS_DIFFERENT_NUM_DNS = 3, | |
112 | |
113 // Calling FindProxyForURL() with a modified URL path resulted in a different | |
114 // set of DNS dependencies. | |
115 FAIL_ABANDONED = 4, | |
116 | |
117 // Calling FindProxyForURL() with a modified URL path caused a different | |
118 // execution flow. Whereas with the original URL it succeeded with | |
119 // non-blocking DNS, this attempt requires fallback to blocking DNS (and was | |
120 // not attempted). | |
121 FAIL_FALLBACK_BLOCKING_DNS = 5, | |
122 | |
123 // Calling FindProxyForURL() with a modified URL path caused a script error. | |
124 FAIL_ERROR = 6, | |
125 | |
126 // Calling FindProxyForURL() with a modified URL path returned a different | |
127 // proxy list. | |
128 FAIL_DIFFERENT_PROXY_LIST = 7, | |
129 | |
130 MAX_VALUE, | |
131 }; | |
132 | |
133 // TODO(eroman): Remove when done gathering data for crbug.com/593759 | |
134 extern NET_EXPORT_PRIVATE const char kHistogramPacResultForStrippedUrl[]; | |
135 | |
92 } // namespace net | 136 } // namespace net |
93 | 137 |
94 #endif // NET_PROXY_PROXY_RESOLVER_V8_TRACING_H_ | 138 #endif // NET_PROXY_PROXY_RESOLVER_V8_TRACING_H_ |
OLD | NEW |