Removed DCHECK in ConnectionLoginFailed which was being hit on iOS

google_apis/gcm/engine/connection_event_tracker.cc

Index: google_apis/gcm/engine/connection_event_tracker.cc
diff --git a/google_apis/gcm/engine/connection_event_tracker.cc b/google_apis/gcm/engine/connection_event_tracker.cc
index d8c3313135f23abc256bb807c1b535449a61d483..cad77ebd37e1387c8c24e1fadc0d7ca85401beaa 100644
--- a/google_apis/gcm/engine/connection_event_tracker.cc
+++ b/google_apis/gcm/engine/connection_event_tracker.cc
@@ -62,8 +62,8 @@ void ConnectionEventTracker::ConnectionAttemptSucceeded() {
 void ConnectionEventTracker::ConnectionLoginFailed() {
   // A login failure would have originally been marked as a successful
   // connection, so now that it failed, that needs to be updated.
-  DCHECK_EQ(current_event_.type(),
-            mcs_proto::ClientEvent::SUCCESSFUL_CONNECTION);
+  // TODO(harkness): Add back DCHECK which was removed. See
+  // https://crbug.com/673706.
 
   current_event_.set_type(mcs_proto::ClientEvent::FAILED_CONNECTION);
   current_event_.clear_time_connection_established_ms();