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

Unified Diff: third_party/WebKit/Source/core/layout/ScrollAnchor.cpp

Issue 2770203002: Replace DCHECK with DCHECK_op and split some DCHECKs wherever necessary (Closed)
Patch Set: Add few more Created 3 years, 8 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/layout/ScrollAnchor.cpp
diff --git a/third_party/WebKit/Source/core/layout/ScrollAnchor.cpp b/third_party/WebKit/Source/core/layout/ScrollAnchor.cpp
index f0925ed7b28bf512b8f5cfa1849b22d0ecfb5b0c..e196f22724d9fb709ac9b273124bb291e13a02e1 100644
--- a/third_party/WebKit/Source/core/layout/ScrollAnchor.cpp
+++ b/third_party/WebKit/Source/core/layout/ScrollAnchor.cpp
@@ -30,7 +30,7 @@ ScrollAnchor::ScrollAnchor(ScrollableArea* scroller) : ScrollAnchor() {
ScrollAnchor::~ScrollAnchor() {}
void ScrollAnchor::setScroller(ScrollableArea* scroller) {
- DCHECK(m_scroller != scroller);
+ DCHECK_NE(m_scroller, scroller);
DCHECK(scroller);
DCHECK(scroller->isRootFrameViewport() || scroller->isFrameView() ||
scroller->isPaintLayerScrollableArea());
@@ -201,7 +201,7 @@ bool ScrollAnchor::findAnchorRecursive(LayoutObject* candidate) {
if (result.status == Constrain)
return true;
- DCHECK(result.status == Continue);
+ DCHECK_EQ(result.status, Continue);
return false;
}

Powered by Google App Engine
This is Rietveld 408576698