GCM Checkin implementation with unit tests and protobufs

google_apis/gcm/engine/checkin_request.h

Index: google_apis/gcm/engine/checkin_request.h
diff --git a/google_apis/gcm/engine/checkin_request.h b/google_apis/gcm/engine/checkin_request.h
new file mode 100644
index 0000000000000000000000000000000000000000..58b580bf70c409e7825c0681716135925626fe22
--- /dev/null
+++ b/google_apis/gcm/engine/checkin_request.h
@@ -0,0 +1,58 @@
+// Copyright (c) 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 GOOGLE_APIS_GCM_ENGINE_CHECKIN_REQUEST_H_
+#define GOOGLE_APIS_GCM_ENGINE_CHECKIN_REQUEST_H_
+
+#include "base/basictypes.h"
+#include "base/callback.h"
+#include "google_apis/gcm/base/gcm_export.h"
+#include "google_apis/gcm/protocol/android_checkin.pb.h"
+#include "net/url_request/url_fetcher_delegate.h"
+
+namespace net {
+class URLRequestContextGetter;
+}
+
+namespace gcm {
+
+// Enables making check-in requests with the GCM infrastructure. When call with

[Nicolas Zea, 2014/01/08 19:53:36]

nit: when call -> when called

[fgorski, 2014/01/08 21:26:39]

Done.

+// android_id and security_token both set to 0 it is an initial check-in used to
+// obtain credentials. These should be persisted and used for subsequent
+// check-ins.
+class GCM_EXPORT CheckinRequest : public net::URLFetcherDelegate {
+ public:
+  // A callback function for the checkin request, accepting |android_id| and
+  // |security_token|.
+  typedef base::Callback<void(uint64, uint64)> CheckinRequestCallback;

[Nicolas Zea, 2014/01/08 19:53:36]

nit: I find it useful to include parameter names in callback definition like
this, so it's clear the first uint64 is the android id, and the second is the
security token.

[fgorski, 2014/01/08 21:26:39]

Done.

+
+  CheckinRequest(const CheckinRequestCallback& callback,
+                 const checkin_proto::ChromeBuildProto& chrome_build_proto,
+                 int64 user_serial_number,
+                 uint64 android_id,
+                 uint64 secret_token,
+                 net::URLRequestContextGetter* request_context_getter);
+  virtual ~CheckinRequest();
+
+  void Start();
+
+  // URLFetcherDelegate implementation.
+  virtual void OnURLFetchComplete(const net::URLFetcher* source) OVERRIDE;
+
+ private:
+  net::URLRequestContextGetter* request_context_getter_;
+  CheckinRequestCallback callback_;
+
+  scoped_ptr<net::URLFetcher> url_fetcher_;
+  checkin_proto::ChromeBuildProto chrome_build_proto_;
+  uint64 android_id_;
+  uint64 security_token_;
+  int64 user_serial_number_;
+
+  DISALLOW_COPY_AND_ASSIGN(CheckinRequest);
+};
+
+}  // namespace gcm
+
+#endif  // GOOGLE_APIS_GCM_ENGINE_CHECKIN_REQUEST_H_