Roll mojo SDK to 734c6e1652ff2f3b696e441722838f453f4f9b42

mojo/services/network/cookie_store_impl.h

Index: mojo/services/network/cookie_store_impl.h
diff --git a/mojo/services/network/cookie_store_impl.h b/mojo/services/network/cookie_store_impl.h
index e3e9711caf72f74a48abc724732b8433bc85bd20..429b054c9255c9c37704ff1980d3e07b1e9a8b26 100644
--- a/mojo/services/network/cookie_store_impl.h
+++ b/mojo/services/network/cookie_store_impl.h
@@ -6,14 +6,17 @@
 #define MOJO_SERVICES_NETWORK_COOKIE_STORE_IMPL_H_
 
 #include "mojo/services/network/public/interfaces/cookie_store.mojom.h"
+#include "third_party/mojo/src/mojo/public/cpp/bindings/strong_binding.h"
 #include "url/gurl.h"
 
 namespace mojo {
 class NetworkContext;
 
-class CookieStoreImpl : public InterfaceImpl<CookieStore> {
+class CookieStoreImpl : public CookieStore {
  public:
-  CookieStoreImpl(NetworkContext* context, const GURL& origin);
+  CookieStoreImpl(InterfaceRequest<CookieStore> request,
+                  NetworkContext* context,
+                  const GURL& origin);
   ~CookieStoreImpl() override;
 
  private:
@@ -25,6 +28,7 @@ class CookieStoreImpl : public InterfaceImpl<CookieStore> {
 
   NetworkContext* context_;
   GURL origin_;
+  mojo::StrongBinding<CookieStore> binding_;

[viettrungluu, 2015/07/09 16:41:52]

Here and elsewhere: I have a slight preference for having binding_ be declared
first, basically matching the order of the arguments to the constructor
(logically, the request "gets turned into" a binding).

[qsr, 2015/07/10 07:53:17]

Done. But I personally have a preference for the other way around. My reasoning
is that a bound Binding has a reference to the object, and if something makes it
do a call, it should not happen with an partially destructed object. Now, I
agree that this should not happen, but having it destroyed first ensure it will
not.

 
   DISALLOW_COPY_AND_ASSIGN(CookieStoreImpl);
 };