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

Unified Diff: third_party/WebKit/Source/core/css/ActiveStyleSheets.cpp

Issue 2755493004: Replace ASSERT, ASSERT_NOT_REACHED, and RELEASE_ASSERT in core/css/ (Closed)
Patch Set: All windows error are Resolved now. Created 3 years, 9 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
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/css/AffectedByFocusTest.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/css/ActiveStyleSheets.cpp
diff --git a/third_party/WebKit/Source/core/css/ActiveStyleSheets.cpp b/third_party/WebKit/Source/core/css/ActiveStyleSheets.cpp
index 893c7905fa93b8dfdfade2ed3cd1569b89c3cd1c..3c19d3a7086896b09ef64c29921ec597bdb0f71b 100644
--- a/third_party/WebKit/Source/core/css/ActiveStyleSheets.cpp
+++ b/third_party/WebKit/Source/core/css/ActiveStyleSheets.cpp
@@ -63,7 +63,8 @@ ActiveSheetsChange compareActiveStyleSheets(
: ActiveSheetsChanged;
}
- DCHECK(index < oldStyleSheetCount && index < newStyleSheetCount);
+ DCHECK_LT(index, oldStyleSheetCount);
+ DCHECK_LT(index, newStyleSheetCount);
// Both the new and old active stylesheet vectors have stylesheets following
// the common prefix. Figure out which were added or removed by sorting the
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/css/AffectedByFocusTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698