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

Unified Diff: chromeos/printing/ppd_provider.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: chromeos/printing/ppd_provider.cc
diff --git a/chromeos/printing/ppd_provider.cc b/chromeos/printing/ppd_provider.cc
index 1d85f0a339ec6a2c633331cb3bf0af3b637bf379..06ccd780c492a6eab4b6ae12fdd55d4579e3566f 100644
--- a/chromeos/printing/ppd_provider.cc
+++ b/chromeos/printing/ppd_provider.cc
@@ -108,8 +108,7 @@ class PpdProviderImpl : public PpdProvider {
void Resolve(const Printer::PpdReference& ppd_reference,
const PpdProvider::ResolveCallback& cb) override {
CHECK(!cb.is_null());
- CHECK(base::SequencedTaskRunnerHandle::IsSet())
- << "Resolve must be called from a SequencedTaskRunner context";
+ CHECK(base::SequencedTaskRunnerHandle::IsSet());
auto cache_result = base::MakeUnique<base::Optional<base::FilePath>>();
auto* raw_cache_result_ptr = cache_result.get();
bool post_result = io_task_runner_->PostTaskAndReply(
@@ -124,8 +123,7 @@ class PpdProviderImpl : public PpdProvider {
void QueryAvailable(const QueryAvailableCallback& cb) override {
CHECK(!cb.is_null());
- CHECK(base::SequencedTaskRunnerHandle::IsSet())
- << "QueryAvailable() must be called from a SequencedTaskRunner context";
+ CHECK(base::SequencedTaskRunnerHandle::IsSet());
auto cache_result =
base::MakeUnique<base::Optional<PpdProvider::AvailablePrintersMap>>();
auto* raw_cache_result_ptr = cache_result.get();

Powered by Google App Engine
This is Rietveld 408576698