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

Unified Diff: mojo/public/cpp/bindings/lib/connector.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/connector.cc
diff --git a/mojo/public/cpp/bindings/lib/connector.cc b/mojo/public/cpp/bindings/lib/connector.cc
index f01bba0ef45c2c66f4e8ea5a93d04fa5f5ccc097..f4bd4b5e46d0898b96f6a7235897aa67b7129651 100644
--- a/mojo/public/cpp/bindings/lib/connector.cc
+++ b/mojo/public/cpp/bindings/lib/connector.cc
@@ -4,10 +4,10 @@
#include "mojo/public/cpp/bindings/lib/connector.h"
-#include <assert.h>
-#include <stdlib.h>
+#include <stddef.h>
#include "mojo/public/cpp/bindings/error_handler.h"
+#include "mojo/public/cpp/environment/logging.h"
namespace mojo {
namespace internal {
@@ -66,7 +66,7 @@ bool Connector::WaitForIncomingMessage() {
}
bool Connector::Accept(Message* message) {
- assert(message_pipe_.is_valid());
+ MOJO_DCHECK(message_pipe_.is_valid());
if (error_)
return false;
@@ -112,7 +112,7 @@ void Connector::CallOnHandleReady(void* closure, MojoResult result) {
}
void Connector::OnHandleReady(MojoResult result) {
- assert(async_wait_id_ != 0);
+ MOJO_DCHECK(async_wait_id_ != 0);
async_wait_id_ = 0;
if (result != MOJO_RESULT_OK) {
NotifyError();

Powered by Google App Engine
This is Rietveld 408576698