Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(44)

Unified Diff: components/sync/test/fake_server/fake_server.cc

Issue 2561963002: base: Remove the string logging from CHECK(). (Closed)
Patch Set: checkstring: rebase Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/sync/test/fake_server/fake_server.cc
diff --git a/components/sync/test/fake_server/fake_server.cc b/components/sync/test/fake_server/fake_server.cc
index 66b720a6b06c40e359d76a7ac4c1d0f7553e8f3f..2cdb52b464dcab1f32a9a4d589856edf5e62b704 100644
--- a/components/sync/test/fake_server/fake_server.cc
+++ b/components/sync/test/fake_server/fake_server.cc
@@ -118,8 +118,8 @@ class UpdateSieve {
std::unique_ptr<UpdateSieve> UpdateSieve::Create(
const sync_pb::GetUpdatesMessage& get_updates_message) {
- CHECK_GT(get_updates_message.from_progress_marker_size(), 0)
- << "A GetUpdates request must have at least one progress marker.";
+ // A GetUpdates request must have at least one progress marker.
+ CHECK_GT(get_updates_message.from_progress_marker_size(), 0);
UpdateSieve::ModelTypeToVersionMap request_from_version;
int64_t min_version = std::numeric_limits<int64_t>::max();
@@ -132,7 +132,8 @@ std::unique_ptr<UpdateSieve> UpdateSieve::Create(
// first request for this type).
if (marker.has_token() && !marker.token().empty()) {
bool parsed = base::StringToInt64(marker.token(), &version);
- CHECK(parsed) << "Unable to parse progress marker token.";
+ // Unable to parse progress marker token.
+ CHECK(parsed);
}
ModelType model_type =
@@ -252,7 +253,8 @@ void FakeServer::HandleCommand(const string& request,
sync_pb::ClientToServerMessage message;
bool parsed = message.ParseFromString(request);
- CHECK(parsed) << "Unable to parse the ClientToServerMessage.";
+ // Unable to parse the ClientToServerMessage.
+ CHECK(parsed);
sync_pb::ClientToServerResponse response_proto;

Powered by Google App Engine
This is Rietveld 408576698