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

Unified Diff: chrome/browser/net/dns_probe_runner.h

Issue 13270005: Display DNS probe results. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Tweak things, clean up unit tests Created 7 years, 6 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
Index: chrome/browser/net/dns_probe_runner.h
diff --git a/chrome/browser/net/dns_probe_runner.h b/chrome/browser/net/dns_probe_runner.h
new file mode 100644
index 0000000000000000000000000000000000000000..3d6d90e959031aabe1fc20ec6274507c73c43ecd
--- /dev/null
+++ b/chrome/browser/net/dns_probe_runner.h
@@ -0,0 +1,70 @@
+// 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 CHROME_BROWSER_NET_DNS_PROBE_RUNNER_H
+#define CHROME_BROWSER_NET_DNS_PROBE_RUNNER_H
+
+#include "base/basictypes.h"
+#include "base/bind.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/memory/weak_ptr.h"
+
+namespace net {
+class DnsClient;
+class DnsResponse;
+class DnsTransaction;
+}
+
+namespace chrome_browser_net {
+
+// Runs DNS probes using a single DnsClient and evaluates the responses.
+// (Currently requests A records for google.com and expects at least one IP
+// address in the response.)
+// Used by DnsProbeService to probe the system and public DNS configurations.
+class DnsProbeRunner {
+ public:
+ enum Result {
+ UNKNOWN,
+ CORRECT, // Response contains at least one A record
+ INCORRECT, // Response claimed success but included no A records
+ FAILING, // Response included an error or was malformed
+ UNREACHABLE // No response received (timeout, network unreachable, etc.)
+ };
+
+ typedef base::Callback<void(Result)> ProbeCallback;
+
+ DnsProbeRunner();
+ ~DnsProbeRunner();
+
+ // Starts a probe using the client specified with SetClient, which must have
+ // been called before RunProbe. |callback| will be called asynchronously
+ // with the result (even if it is known instantly).
szym 2013/06/13 20:50:48 Maybe add comment that must not be called again un
Deprecated (see juliatuttle) 2013/06/18 19:36:37 Done.
+ void RunProbe(const ProbeCallback& callback);
+
+ // Sets the DnsClient that will be used for DNS probes sent by this runner.
+ // Must be called before RunProbe; can be called repeatedly, including during
+ // a probe. (It will not, of course, affect the in-flight probe.)
+ void SetClient(scoped_ptr<net::DnsClient> client);
+
+ // Returns true if a probe is running. Guaranteed to return true after
+ // RunProbe returns, and false during and after the callback.
+ bool IsRunning() const;
+
+ private:
+ void OnTransactionComplete(net::DnsTransaction* transaction,
+ int net_error,
+ const net::DnsResponse* response);
+ void CallCallback(Result result);
+
+ base::WeakPtrFactory<DnsProbeRunner> weak_factory_;
+ scoped_ptr<net::DnsClient> client_;
+ ProbeCallback callback_;
+ scoped_ptr<net::DnsTransaction> transaction_;
+
+ DISALLOW_COPY_AND_ASSIGN(DnsProbeRunner);
+};
+
+} // namespace chrome_browser_net
+
+#endif // CHROME_BROWSER_NET_DNS_PROBE_RUNNER_H

Powered by Google App Engine
This is Rietveld 408576698