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

Unified Diff: components/copresence/rpc/http_post.h

Issue 433283002: Adding the Copresence RpcHandler and HttpPost helper. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@directive-handler
Patch Set: More WeakPtr fixes Created 6 years, 4 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: components/copresence/rpc/http_post.h
diff --git a/components/copresence/rpc/http_post.h b/components/copresence/rpc/http_post.h
new file mode 100644
index 0000000000000000000000000000000000000000..f511ae7efccdd214f1d2e2578792cda795fb7240
--- /dev/null
+++ b/components/copresence/rpc/http_post.h
@@ -0,0 +1,74 @@
+// Copyright 2014 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 COMPONENTS_COPRESENCE_RPC_HTTP_POST_H_
+#define COMPONENTS_COPRESENCE_RPC_HTTP_POST_H_
+
+#include <string>
+
+#include "base/callback.h"
+#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
+#include "net/url_request/url_fetcher_delegate.h"
+
+namespace google {
+namespace protobuf {
+class MessageLite;
+}
+}
+
+namespace net {
+class URLRequestContextGetter;
+}
+
+namespace copresence {
+
+// This class handles all Apiary calls to the Copresence server.
+// It configures the HTTP request appropriately and reports any errors.
+// Clients should not attempt to delete this class, as it needs to
+// persist until the HTTP request completes (or times out).
+//
+// TODO(ckehoe): Add retry logic.
+class HttpPost : public net::URLFetcherDelegate {
+ public:
+ // Callback to receive the HTTP status code and body of the response
+ // (if any). A pointer to this HttpPost object is also passed along,
+ // which will be deleted right after the callback completes.
+ typedef base::Callback<void(int, const std::string&, HttpPost*)>
+ ResponseCallback;
+
+ // An id for testing url fetching.
+ static const int kUrlFetcherId = 1;
+
+ // Send a request to the Copresence server. After calling the callback, this
+ // object will delete itself. |url_context_getter| is owned by the caller,
+ // and the context it provides must be available until the request completes.
+ // If this context is being destructed, call Cancel().
+ HttpPost(net::URLRequestContextGetter* url_context_getter,
+ const std::string& server_host,
+ const std::string& rpc_name,
+ const google::protobuf::MessageLite& request_proto,
+ const ResponseCallback& response_callback);
+
+ // Cancels the HTTP request and deletes this object. Call this
+ // if the URLRequestContext passed into the constructor is going away.
+ void Cancel();
+
+ private:
+ // This class deletes itself when the request completes (or times out).
+ // To delete it earlier, call Cancel().
+ virtual ~HttpPost();
+
+ // Overridden from net::URLFetcherDelegate.
+ virtual void OnURLFetchComplete(const net::URLFetcher* source) OVERRIDE;
+
+ ResponseCallback response_callback_;
+ scoped_ptr<net::URLFetcher> url_fetcher_;
+
+ DISALLOW_COPY_AND_ASSIGN(HttpPost);
+};
+
+} // namespace copresence
+
+#endif // COMPONENTS_COPRESENCE_RPC_HTTP_POST_H_

Powered by Google App Engine
This is Rietveld 408576698