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

Unified Diff: mojo/public/cpp/bindings/lib/message_queue.cc

Issue 399313007: Mojo: Convert assert()s under mojo/public/cpp/bindings/... to MOJO_DCHECK()s. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 months 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: mojo/public/cpp/bindings/lib/message_queue.cc
diff --git a/mojo/public/cpp/bindings/lib/message_queue.cc b/mojo/public/cpp/bindings/lib/message_queue.cc
index 1982ccb14c20523e128a923e22efda00d715a817..fd701e9791440eb76c510c57d8f9e2a189ab2da7 100644
--- a/mojo/public/cpp/bindings/lib/message_queue.cc
+++ b/mojo/public/cpp/bindings/lib/message_queue.cc
@@ -4,10 +4,8 @@
#include "mojo/public/cpp/bindings/lib/message_queue.h"
-#include <assert.h>
-#include <stddef.h>
-
#include "mojo/public/cpp/bindings/message.h"
+#include "mojo/public/cpp/environment/logging.h"
namespace mojo {
namespace internal {
@@ -25,7 +23,7 @@ bool MessageQueue::IsEmpty() const {
}
Message* MessageQueue::Peek() {
- assert(!queue_.empty());
+ MOJO_DCHECK(!queue_.empty());
return queue_.front();
}
@@ -35,13 +33,13 @@ void MessageQueue::Push(Message* message) {
}
void MessageQueue::Pop(Message* message) {
- assert(!queue_.empty());
+ MOJO_DCHECK(!queue_.empty());
queue_.front()->Swap(message);
Pop();
}
void MessageQueue::Pop() {
- assert(!queue_.empty());
+ MOJO_DCHECK(!queue_.empty());
delete queue_.front();
queue_.pop();
}

Powered by Google App Engine
This is Rietveld 408576698