[GCM] DCHECK should not be used since it will skip the code inside

components/gcm_driver/gcm_channel_status_request.cc

 // 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.
 
 #include "components/gcm_driver/gcm_channel_status_request.h"
 
 #include "base/bind.h"
 #include "base/message_loop/message_loop.h"
 #include "components/gcm_driver/gcm_backoff_policy.h"
 #include "net/base/escape.h"
@@ skipping 42 matching lines @@
 }
 
 void GCMChannelStatusRequest::Start() {
   DCHECK(!url_fetcher_.get());
 
   GURL request_url(channel_status_request_url_);
 
   sync_pb::ExperimentStatusRequest proto_data;
   proto_data.add_experiment_name(kGCMChannelTag);
   std::string upload_data;
-  DCHECK(proto_data.SerializeToString(&upload_data));
+  if (!proto_data.SerializeToString(&upload_data)) {
+     NOTREACHED();
+  }
 
   url_fetcher_.reset(
       net::URLFetcher::Create(request_url, net::URLFetcher::POST, this));
   url_fetcher_->SetRequestContext(request_context_getter_.get());
   url_fetcher_->AddExtraRequestHeader("User-Agent: " + user_agent_);
   url_fetcher_->SetUploadData(kRequestContentType, upload_data);
   url_fetcher_->SetLoadFlags(net::LOAD_DO_NOT_SEND_COOKIES |
                              net::LOAD_DO_NOT_SAVE_COOKIES);
   url_fetcher_->Start();
 }
@@ skipping 72 matching lines @@
                    weak_ptr_factory_.GetWeakPtr(),
                    false),
         backoff_entry_.GetTimeUntilRelease());
     return;
   }
 
   Start();
 }
 
 }  // namespace gcm