Change DCHECK_IS_ON() to DCHECK_IS_ON

sync/internal_api/http_bridge.cc

 // Copyright 2012 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 "sync/internal_api/public/http_bridge.h"
 
 #include "base/message_loop/message_loop.h"
 #include "base/message_loop/message_loop_proxy.h"
 #include "base/strings/string_number_conversions.h"
 #include "net/base/load_flags.h"
@@ skipping 180 matching lines @@
 HttpBridge::~HttpBridge() {
 }
 
 void HttpBridge::SetExtraRequestHeaders(const char * headers) {
   DCHECK(extra_headers_.empty())
       << "HttpBridge::SetExtraRequestHeaders called twice.";
   extra_headers_.assign(headers);
 }
 
 void HttpBridge::SetURL(const char* url, int port) {
+#if DCHECK_IS_ON
   DCHECK_EQ(base::MessageLoop::current(), created_on_loop_);
-  if (DCHECK_IS_ON()) {
+  {
     base::AutoLock lock(fetch_state_lock_);
     DCHECK(!fetch_state_.request_completed);
   }
   DCHECK(url_for_request_.is_empty())
       << "HttpBridge::SetURL called more than once?!";
+#endif
   GURL temp(url);
   GURL::Replacements replacements;
   std::string port_str = base::IntToString(port);
   replacements.SetPort(port_str.c_str(),
                        url_parse::Component(0, port_str.length()));
   url_for_request_ = temp.ReplaceComponents(replacements);
 }
 
 void HttpBridge::SetPostPayload(const char* content_type,
                                 int content_length,
                                 const char* content) {
+#if DCHECK_IS_ON
   DCHECK_EQ(base::MessageLoop::current(), created_on_loop_);
-  if (DCHECK_IS_ON()) {
+  {
     base::AutoLock lock(fetch_state_lock_);
     DCHECK(!fetch_state_.request_completed);
   }
   DCHECK(content_type_.empty()) << "Bridge payload already set.";
   DCHECK_GE(content_length, 0) << "Content length < 0";
+#endif
   content_type_ = content_type;
   if (!content || (content_length == 0)) {
     DCHECK_EQ(content_length, 0);
     request_content_ = " ";  // TODO(timsteele): URLFetcher requires non-empty
                              // content for POSTs whereas CURL does not, for now
                              // we hack this to support the sync backend.
   } else {
     request_content_.assign(content, content_length);
   }
 }
 
 bool HttpBridge::MakeSynchronousPost(int* error_code, int* response_code) {
+#if DCHECK_IS_ON
   DCHECK_EQ(base::MessageLoop::current(), created_on_loop_);
-  if (DCHECK_IS_ON()) {
+  {
     base::AutoLock lock(fetch_state_lock_);
     DCHECK(!fetch_state_.request_completed);
   }
   DCHECK(url_for_request_.is_valid()) << "Invalid URL for request";
   DCHECK(!content_type_.empty()) << "Payload not set";
+#endif
 
   if (!network_task_runner_->PostTask(
           FROM_HERE,
           base::Bind(&HttpBridge::CallMakeAsynchronousPost, this))) {
     // This usually happens when we're in a unit test.
     LOG(WARNING) << "Could not post CallMakeAsynchronousPost task";
     return false;
   }
 
   // Block until network request completes or is aborted. See
@@ skipping 133 matching lines @@
   int64 sane_time_ms = 0;
   if (base::StringToInt64(sane_time_str, &sane_time_ms)) {
     network_time_update_callback_.Run(
         base::Time::FromJsTime(sane_time_ms),
         base::TimeDelta::FromMilliseconds(1),
         fetch_state_.end_time - fetch_state_.start_time);
   }
 }
 
 }  // namespace syncer