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

Unified Diff: third_party/WebKit/Source/core/frame/PlatformEventDispatcher.cpp

Issue 2858963002: Replace ASSERT with DCHECK in core/ (Closed)
Patch Set: WorkerBackingThread Created 3 years, 7 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: third_party/WebKit/Source/core/frame/PlatformEventDispatcher.cpp
diff --git a/third_party/WebKit/Source/core/frame/PlatformEventDispatcher.cpp b/third_party/WebKit/Source/core/frame/PlatformEventDispatcher.cpp
index 464c28a16598136ddc51a8ffc1b6caa0400dfaec..90ccfd1203f220ce8284969b1be3bb5bcbc627a5 100644
--- a/third_party/WebKit/Source/core/frame/PlatformEventDispatcher.cpp
+++ b/third_party/WebKit/Source/core/frame/PlatformEventDispatcher.cpp
@@ -14,7 +14,7 @@ PlatformEventDispatcher::PlatformEventDispatcher()
void PlatformEventDispatcher::AddController(
PlatformEventController* controller) {
- ASSERT(controller);
+ DCHECK(controller);
// TODO: If we can avoid to register a same controller twice, we can change
// this 'if' to ASSERT.
if (controllers_.Contains(controller))
@@ -30,7 +30,7 @@ void PlatformEventDispatcher::AddController(
void PlatformEventDispatcher::RemoveController(
PlatformEventController* controller) {
- ASSERT(controllers_.Contains(controller));
+ DCHECK(controllers_.Contains(controller));
controllers_.erase(controller);
if (!is_dispatching_ && controllers_.IsEmpty()) {
« no previous file with comments | « third_party/WebKit/Source/core/frame/PlatformEventController.cpp ('k') | third_party/WebKit/Source/core/frame/RemoteFrame.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698