PaymentRequest should be a ContextLifecycleObserver

mojo/public/cpp/bindings/binding.h

 // 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 MOJO_PUBLIC_CPP_BINDINGS_BINDING_H_
 #define MOJO_PUBLIC_CPP_BINDINGS_BINDING_H_
 
 #include <utility>
 
 #include "base/macros.h"
@@ skipping 138 matching lines @@
   // This method may only be called if the object has been bound to a message
   // pipe and there are no associated interfaces running.
   bool WaitForIncomingMethodCall(
       MojoDeadline deadline = MOJO_DEADLINE_INDEFINITE) {
     CHECK(!HasAssociatedInterfaces());
     return internal_state_.WaitForIncomingMethodCall(deadline);
   }
 
   // Closes the message pipe that was previously bound. Put this object into a
   // state where it can be rebound to a new pipe.
-  void Close() { internal_state_.Close(); }
+  void Close() { if (internal_state_.is_bound()) internal_state_.Close(); }

[please use gerrit instead, 2016/04/13 22:56:46]

Instead of checking is_bound() here, let's check it in
PaymentRequest::cleanUp().

[rwlbuis, 2016/04/14 00:28:33]

Done.

 
   // Unbinds the underlying pipe from this binding and returns it so it can be
   // used in another context, such as on another thread or with a different
   // implementation. Put this object into a state where it can be rebound to a
   // new pipe.
   //
   // This method may only be called if the object has been bound to a message
   // pipe and there are no associated interfaces running.
   //
   // TODO(yzshen): For now, users need to make sure there is no one holding
@@ skipping 44 matching lines @@
  private:
   internal::BindingState<Interface, Interface::PassesAssociatedKinds_>
       internal_state_;
 
   DISALLOW_COPY_AND_ASSIGN(Binding);
 };
 
 }  // namespace mojo
 
 #endif  // MOJO_PUBLIC_CPP_BINDINGS_BINDING_H_