[Chromoting] Hook up host talkgadget policy checks.

remoting/host/connection_block_checker.cc

+// Copyright (c) 2012 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.
+
+#include "remoting/host/connection_block_checker.h"
+
+#include "googleurl/src/gurl.h"
+#include "net/url_request/url_fetcher.h"
+#include "remoting/host/chromoting_host_context.h"
+#include "remoting/host/constants.h"
+
+namespace remoting {
+
+const int kMaxConnectionBlockChecks = 5;
+
+ConnectionBlockChecker::ConnectionBlockChecker(
+    ChromotingHostContext* context,
+    std::string talkgadget_prefix,
+    const base::Callback<void(bool)>& callback)
+    : context_(context),
+      talkgadget_prefix_(talkgadget_prefix),
+      callback_(callback) {
+}
+
+ConnectionBlockChecker::~ConnectionBlockChecker() {
+}
+
+// This is called in response to the TalkGadget http request initiated from
+// CheckStatus().
+void ConnectionBlockChecker::OnURLFetchComplete(const net::URLFetcher* source) {
+  int response = source->GetResponseCode();
+  int allow = false;
+  if (source->GetResponseCode() == 200) {
+    LOG(INFO) << "Successfully connected to host talkgadget.";
+    allow = true;
+  } else {
+    LOG(INFO) << "Unable to connect to host talkgadget (" << response << ")";
+  }
+  url_fetcher_.reset(NULL);
+  callback_.Run(allow);
+}
+
+void ConnectionBlockChecker::CheckStatus() {
+  // Make sure we're not currently in the middle of a connection check.
+  if (!url_fetcher_.get()) {
+    std::string talkgadget_url("https://");
+    if (talkgadget_prefix_.empty()) {
+      talkgadget_url += kDefaultHostTalkGadgetPrefix;
+    } else {
+      talkgadget_url += talkgadget_prefix_;
+    }
+    talkgadget_url += kTalkGadgetUrl;
+    LOG(INFO) << "Verifying connection to " << talkgadget_url;
+    url_fetcher_.reset(net::URLFetcher::Create(GURL(talkgadget_url),
+                                               net::URLFetcher::GET, this));
+    url_fetcher_->SetRequestContext(context_->url_request_context_getter());
+    url_fetcher_->Start();
+  } else {
+    LOG(INFO) << "Pending connection check";

[Sergey Ulanov, 2012/08/29 20:10:29]

I think it's better to DCHECK(!url_fetcher_.get()) in the beginning of this
method.

[garykac, 2012/08/29 22:30:51]

A DCHECK for this is wrong since CheckStatus can possibly be called before the
previous check finishes.  We just don't want to fire off another check in that
case.

+  }
+}
+
+}  // namespace remoting