base: Remove the string logging from CHECK().

components/sync/engine_impl/loopback_server/persistent_permanent_entity.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/sync/engine_impl/loopback_server/persistent_permanent_entity.h"
 
 #include <memory>
 #include <string>
 
 #include "base/logging.h"
@@ skipping 15 matching lines @@
 namespace syncer {
 
 PersistentPermanentEntity::~PersistentPermanentEntity() {}
 
 // static
 std::unique_ptr<LoopbackServerEntity> PersistentPermanentEntity::Create(
     const ModelType& model_type,
     const string& server_tag,
     const string& name,
     const string& parent_server_tag) {
-  CHECK(model_type != syncer::UNSPECIFIED) << "The entity's ModelType is "
-                                           << "invalid.";
-  CHECK(!server_tag.empty())
-      << "A PersistentPermanentEntity must have a server tag.";
-  CHECK(!name.empty()) << "The entity must have a non-empty name.";
-  CHECK(!parent_server_tag.empty())
-      << "A PersistentPermanentEntity must have a parent "
-      << "server tag.";
-  CHECK(parent_server_tag != kRootParentTag) << "Top-level entities should not "
-                                             << "be created with this factory.";
+  // The entity's ModelType is invalid.
+  CHECK(model_type != syncer::UNSPECIFIED);
+  // A PersistentPermanentEntity must have a server tag.
+  CHECK(!server_tag.empty());
+  // The entity must have a non-empty name.
+  CHECK(!name.empty());
+  // A PersistentPermanentEntity must have a parent server tag.
+  CHECK(!parent_server_tag.empty());
+  // Top-level entities should not be created with this factory.
+  CHECK(parent_server_tag != kRootParentTag);
 
   string id = LoopbackServerEntity::CreateId(model_type, server_tag);
   string parent_id =
       LoopbackServerEntity::CreateId(model_type, parent_server_tag);
   sync_pb::EntitySpecifics entity_specifics;
   AddDefaultFieldValue(model_type, &entity_specifics);
   return std::unique_ptr<LoopbackServerEntity>(new PersistentPermanentEntity(
       id, 0, model_type, name, parent_id, server_tag, entity_specifics));
 }
 
 // static
 std::unique_ptr<LoopbackServerEntity> PersistentPermanentEntity::CreateTopLevel(
     const ModelType& model_type) {
-  CHECK(model_type != syncer::UNSPECIFIED) << "The entity's ModelType is "
-                                           << "invalid.";
+  // The entity's ModelType is invalid.
+  CHECK(model_type != syncer::UNSPECIFIED);
   string server_tag = syncer::ModelTypeToRootTag(model_type);
   string name = syncer::ModelTypeToString(model_type);
   string id = LoopbackServerEntity::GetTopLevelId(model_type);
   sync_pb::EntitySpecifics entity_specifics;
   AddDefaultFieldValue(model_type, &entity_specifics);
   return std::unique_ptr<LoopbackServerEntity>(new PersistentPermanentEntity(
       id, 0, model_type, name, kRootParentTag, server_tag, entity_specifics));
 }
 
 // static
 std::unique_ptr<LoopbackServerEntity>
 PersistentPermanentEntity::CreateUpdatedNigoriEntity(
     const sync_pb::SyncEntity& client_entity,
     const LoopbackServerEntity& current_server_entity) {
   ModelType model_type = current_server_entity.GetModelType();
-  CHECK(model_type == syncer::NIGORI) << "This factory only supports NIGORI "
-                                      << "entities.";
+  // This factory only supports NIGORI entities.
+  CHECK(model_type == syncer::NIGORI);
 
   return base::WrapUnique<LoopbackServerEntity>(new PersistentPermanentEntity(
       current_server_entity.GetId(), current_server_entity.GetVersion(),
       model_type, current_server_entity.GetName(),
       current_server_entity.GetParentId(),
       syncer::ModelTypeToRootTag(model_type), client_entity.specifics()));
 }
 
 PersistentPermanentEntity::PersistentPermanentEntity(
     const string& id,
@@ skipping 30 matching lines @@
 bool PersistentPermanentEntity::IsPermanent() const {
   return true;
 }
 
 sync_pb::LoopbackServerEntity_Type
 PersistentPermanentEntity::GetLoopbackServerEntityType() const {
   return sync_pb::LoopbackServerEntity_Type_PERMANENT;
 }
 
 }  // namespace syncer