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

Unified Diff: third_party/WebKit/Source/core/html/canvas/CanvasFontCache.cpp

Issue 2896443003: Replace remaining ASSERT with DCHECK/_EQ as appropriate (Closed)
Patch Set: 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/html/canvas/CanvasFontCache.cpp
diff --git a/third_party/WebKit/Source/core/html/canvas/CanvasFontCache.cpp b/third_party/WebKit/Source/core/html/canvas/CanvasFontCache.cpp
index f855d4768eef17e1eab4f1ee01cbf7b6e332b504..b5c526018e836751b1ff0dbe79afe6ce776bf5d5 100644
--- a/third_party/WebKit/Source/core/html/canvas/CanvasFontCache.cpp
+++ b/third_party/WebKit/Source/core/html/canvas/CanvasFontCache.cpp
@@ -105,8 +105,8 @@ MutableStylePropertySet* CanvasFontCache::ParseFont(const String& font_string) {
// Hard limit is applied here, on the fly, while the soft limit is
// applied at the end of the task.
if (fetched_fonts_.size() > HardMaxFonts()) {
- ASSERT(fetched_fonts_.size() == HardMaxFonts() + 1);
- ASSERT(font_lru_list_.size() == HardMaxFonts() + 1);
+ DCHECK_EQ(fetched_fonts_.size(), HardMaxFonts() + 1);
+ DCHECK_EQ(font_lru_list_.size(), HardMaxFonts() + 1);
fetched_fonts_.erase(font_lru_list_.front());
fonts_resolved_using_default_style_.erase(font_lru_list_.front());
font_lru_list_.RemoveFirst();

Powered by Google App Engine
This is Rietveld 408576698