Make InstanceIDBridge fully async to fix strict mode violations

components/gcm_driver/instance_id/instance_id.h

Index: components/gcm_driver/instance_id/instance_id.h
diff --git a/components/gcm_driver/instance_id/instance_id.h b/components/gcm_driver/instance_id/instance_id.h
index 3aa7658e9a3738c20df0b43c148a780dcadb877b..46ceb5c3d95a9881ab0b9f5a3a5200404361c5d2 100644
--- a/components/gcm_driver/instance_id/instance_id.h
+++ b/components/gcm_driver/instance_id/instance_id.h
@@ -20,8 +20,8 @@ class InstanceIDHandler;
 namespace instance_id {
 
 // Encapsulates Instance ID functionalities that need to be implemented for
-// different platform. One instance is created per application. Life of
-// Instance ID is managed by the InstanceIdDriver.
+// different platforms. One instance is created per application. Life of
+// Instance ID is managed by the InstanceIDDriver.
 class InstanceID {
  public:
   enum Result {
@@ -41,7 +41,8 @@ class InstanceID {
     UNKNOWN_ERROR
   };
 
-  // Asynchronous callbacks.
+  // Asynchronous callbacks. Must not synchronously delete this (using

[Peter Beverloo, 2016/04/27 10:56:37]

nit: reads slightly easier if you'd use |this| (zeh pipes)

[johnme, 2016/04/27 18:03:57]

Done.

+  // InstanceIDDriver::RemoveInstanceID).
   typedef base::Callback<void(const std::string& app_id,
                               bool update_id)> TokenRefreshCallback;
   typedef base::Callback<void(const std::string& id)> GetIDCallback;