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

Unified Diff: net/proxy/proxy_resolver_v8_tracing.h

Issue 992733002: Remove //net (except for Android test stuff) and sdch (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/proxy/proxy_resolver_v8.cc ('k') | net/proxy/proxy_resolver_v8_tracing.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/proxy/proxy_resolver_v8_tracing.h
diff --git a/net/proxy/proxy_resolver_v8_tracing.h b/net/proxy/proxy_resolver_v8_tracing.h
deleted file mode 100644
index 85aca519e7e2aa68df30ccc06f9613a4a61cc677..0000000000000000000000000000000000000000
--- a/net/proxy/proxy_resolver_v8_tracing.h
+++ /dev/null
@@ -1,83 +0,0 @@
-// Copyright (c) 2013 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#ifndef NET_PROXY_PROXY_RESOLVER_V8_TRACING_H_
-#define NET_PROXY_PROXY_RESOLVER_V8_TRACING_H_
-
-#include "base/basictypes.h"
-#include "base/memory/ref_counted.h"
-#include "base/memory/scoped_ptr.h"
-#include "base/threading/non_thread_safe.h"
-#include "net/base/net_export.h"
-#include "net/proxy/proxy_resolver.h"
-
-namespace base {
-class Thread;
-class MessageLoopProxy;
-} // namespace base
-
-namespace net {
-
-class HostResolver;
-class NetLog;
-class ProxyResolverErrorObserver;
-class ProxyResolverV8;
-
-// ProxyResolverV8Tracing is a non-blocking ProxyResolver. It executes
-// ProxyResolverV8 on a single helper thread, and does some magic to avoid
-// blocking in DNS. For more details see the design document:
-// https://docs.google.com/a/google.com/document/d/16Ij5OcVnR3s0MH4Z5XkhI9VTPoMJdaBn9rKreAmGOdE/edit?pli=1
-class NET_EXPORT_PRIVATE ProxyResolverV8Tracing
- : public ProxyResolver,
- NON_EXPORTED_BASE(public base::NonThreadSafe) {
- public:
- // Constructs a ProxyResolver that will issue DNS requests through
- // |host_resolver|, forward Javascript errors through |error_observer|, and
- // log Javascript errors and alerts to |net_log|.
- //
- // Note that the constructor takes ownership of |error_observer|, whereas
- // |host_resolver| and |net_log| are expected to outlive |this|.
- ProxyResolverV8Tracing(HostResolver* host_resolver,
- ProxyResolverErrorObserver* error_observer,
- NetLog* net_log);
-
- ~ProxyResolverV8Tracing() override;
-
- // ProxyResolver implementation:
- int GetProxyForURL(const GURL& url,
- ProxyInfo* results,
- const CompletionCallback& callback,
- RequestHandle* request,
- const BoundNetLog& net_log) override;
- void CancelRequest(RequestHandle request) override;
- LoadState GetLoadState(RequestHandle request) const override;
- void CancelSetPacScript() override;
- int SetPacScript(const scoped_refptr<ProxyResolverScriptData>& script_data,
- const CompletionCallback& callback) override;
-
- private:
- class Job;
-
- // The worker thread on which the ProxyResolverV8 will be run.
- scoped_ptr<base::Thread> thread_;
- scoped_ptr<ProxyResolverV8> v8_resolver_;
-
- // Non-owned host resolver, which is to be operated on the origin thread.
- HostResolver* host_resolver_;
-
- scoped_ptr<ProxyResolverErrorObserver> error_observer_;
- NetLog* net_log_;
-
- // The outstanding SetPacScript operation, or NULL.
- scoped_refptr<Job> set_pac_script_job_;
-
- // The number of outstanding (non-cancelled) jobs.
- int num_outstanding_callbacks_;
-
- DISALLOW_COPY_AND_ASSIGN(ProxyResolverV8Tracing);
-};
-
-} // namespace net
-
-#endif // NET_PROXY_PROXY_RESOLVER_V8_TRACING_H_
« no previous file with comments | « net/proxy/proxy_resolver_v8.cc ('k') | net/proxy/proxy_resolver_v8_tracing.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698