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

Unified Diff: cc/layers/picture_layer_impl.cc

Issue 195973002: Change DCHECK_IS_ON() to DCHECK_IS_ON (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Keep a comment 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
« no previous file with comments | « cc/layers/delegated_frame_provider.cc ('k') | cc/trees/tree_synchronizer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/layers/picture_layer_impl.cc
diff --git a/cc/layers/picture_layer_impl.cc b/cc/layers/picture_layer_impl.cc
index a2b5ca3952a6e79b641109a46fb20edacc5fd4c7..628f419837db141cc3e9087582aa1a1e10b0e659 100644
--- a/cc/layers/picture_layer_impl.cc
+++ b/cc/layers/picture_layer_impl.cc
@@ -1192,9 +1192,7 @@ bool PictureLayerImpl::CanHaveTilingWithScale(float contents_scale) const {
}
void PictureLayerImpl::SanityCheckTilingState() const {
- if (!DCHECK_IS_ON())
- return;
-
+#if DCHECK_IS_ON
if (!CanHaveTilings()) {
DCHECK_EQ(0u, tilings_->num_tilings());
return;
@@ -1205,6 +1203,7 @@ void PictureLayerImpl::SanityCheckTilingState() const {
// MarkVisibleResourcesAsRequired depends on having exactly 1 high res
// tiling to mark its tiles as being required for activation.
DCHECK_EQ(1, tilings_->NumHighResTilings());
+#endif
}
void PictureLayerImpl::GetDebugBorderProperties(
« no previous file with comments | « cc/layers/delegated_frame_provider.cc ('k') | cc/trees/tree_synchronizer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698