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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: ui/gfx/font_list_impl.cc

Issue 195973002: Change DCHECK_IS_ON() to DCHECK_IS_ON (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: if() -> #if Created 6 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
« gpu/command_buffer/tests/gl_shared_resources_unittest.cc ('K') | « sync/internal_api/http_bridge.cc ('k') | ui/wm/core/input_method_event_filter.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/gfx/font_list_impl.cc
diff --git a/ui/gfx/font_list_impl.cc b/ui/gfx/font_list_impl.cc
index b08d395ac315e0ef034fccbd5dc32724cefb3911..8c00fcc82785b2194b892f7736c0d4fcf76ca1fa 100644
--- a/ui/gfx/font_list_impl.cc
+++ b/ui/gfx/font_list_impl.cc
@@ -104,12 +104,12 @@ FontListImpl::FontListImpl(const std::vector<Font>& fonts)
DCHECK(!fonts.empty());
font_style_ = fonts[0].GetStyle();
font_size_ = fonts[0].GetFontSize();
- if (DCHECK_IS_ON()) {
- for (size_t i = 1; i < fonts.size(); ++i) {
- DCHECK_EQ(fonts[i].GetStyle(), font_style_);
- DCHECK_EQ(fonts[i].GetFontSize(), font_size_);
- }
+#if DCHECK_IS_ON
+ for (size_t i = 1; i < fonts.size(); ++i) {
+ DCHECK_EQ(fonts[i].GetStyle(), font_style_);
+ DCHECK_EQ(fonts[i].GetFontSize(), font_size_);
}
+#endif
}
FontListImpl::FontListImpl(const Font& font)
« gpu/command_buffer/tests/gl_shared_resources_unittest.cc ('K') | « sync/internal_api/http_bridge.cc ('k') | ui/wm/core/input_method_event_filter.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698