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

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

Issue 862813002: WIP: Prototype OOP V8 PAC. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Slight cleanup and report JS memory usage. Created 5 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
« no previous file with comments | « net/proxy/proxy_resolver_mojo_host.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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_V8_H_ 5 #ifndef NET_PROXY_PROXY_RESOLVER_V8_H_
6 #define NET_PROXY_PROXY_RESOLVER_V8_H_ 6 #define NET_PROXY_PROXY_RESOLVER_V8_H_
7 7
8 #include "base/compiler_specific.h" 8 #include "base/compiler_specific.h"
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "net/base/net_export.h" 10 #include "net/base/net_export.h"
(...skipping 21 matching lines...) Expand all
32 // since only one will be running inside V8 at a time. 32 // since only one will be running inside V8 at a time.
33 // 33 //
34 // It is important that *ALL* instances of V8 in the process be using 34 // It is important that *ALL* instances of V8 in the process be using
35 // v8::Locker. If not there can be race conditions between the non-locked V8 35 // v8::Locker. If not there can be race conditions between the non-locked V8
36 // instances and the locked V8 instances used by ProxyResolverV8 (assuming they 36 // instances and the locked V8 instances used by ProxyResolverV8 (assuming they
37 // run on different threads). 37 // run on different threads).
38 // 38 //
39 // This is the case with the V8 instance used by chromium's renderer -- it runs 39 // This is the case with the V8 instance used by chromium's renderer -- it runs
40 // on a different thread from ProxyResolver (renderer thread vs PAC thread), 40 // on a different thread from ProxyResolver (renderer thread vs PAC thread),
41 // and does not use locking since it expects to be alone. 41 // and does not use locking since it expects to be alone.
42 class NET_EXPORT_PRIVATE ProxyResolverV8 : public ProxyResolver { 42 class NET_EXPORT ProxyResolverV8 : public ProxyResolver {
43 public: 43 public:
44 // Interface for the javascript bindings. 44 // Interface for the javascript bindings.
45 class NET_EXPORT_PRIVATE JSBindings { 45 class NET_EXPORT_PRIVATE JSBindings {
46 public: 46 public:
47 enum ResolveDnsOperation { 47 enum ResolveDnsOperation {
48 DNS_RESOLVE, 48 DNS_RESOLVE,
49 DNS_RESOLVE_EX, 49 DNS_RESOLVE_EX,
50 MY_IP_ADDRESS, 50 MY_IP_ADDRESS,
51 MY_IP_ADDRESS_EX, 51 MY_IP_ADDRESS_EX,
52 }; 52 };
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
116 scoped_ptr<Context> context_; 116 scoped_ptr<Context> context_;
117 117
118 JSBindings* js_bindings_; 118 JSBindings* js_bindings_;
119 119
120 DISALLOW_COPY_AND_ASSIGN(ProxyResolverV8); 120 DISALLOW_COPY_AND_ASSIGN(ProxyResolverV8);
121 }; 121 };
122 122
123 } // namespace net 123 } // namespace net
124 124
125 #endif // NET_PROXY_PROXY_RESOLVER_V8_H_ 125 #endif // NET_PROXY_PROXY_RESOLVER_V8_H_
OLDNEW
« no previous file with comments | « net/proxy/proxy_resolver_mojo_host.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698