cc: Remove DCHECK code from production.

cc/layers/picture_layer.cc

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/layers/picture_layer.h"
 
 #include "base/auto_reset.h"
 #include "cc/layers/content_layer_client.h"
 #include "cc/layers/picture_layer_impl.h"
 #include "cc/trees/layer_tree_impl.h"
@@ skipping 23 matching lines @@
 
 void PictureLayer::PushPropertiesTo(LayerImpl* base_layer) {
   Layer::PushPropertiesTo(base_layer);
   PictureLayerImpl* layer_impl = static_cast<PictureLayerImpl*>(base_layer);
 
   if (layer_impl->bounds().IsEmpty()) {
     // Update may not get called for an empty layer, so resize here instead.
     // Using layer_impl because either bounds() or paint_properties().bounds
     // may disagree and either one could have been pushed to layer_impl.
     pile_->SetEmptyBounds();
-  } else if (update_source_frame_number_ ==
-             layer_tree_host()->source_frame_number()) {
+  } else {
     // TODO(ernstm): This DCHECK is only valid as long as the pile's tiling_rect

[vmpstr, 2014/10/24 20:41:12]

I'm not sure what the todo here is, it seems to be just a comment?

[danakj, 2014/10/24 20:43:07]

Ya.. not sure either.

[danakj, 2014/10/24 20:44:30]

I think this is from when we were going to use non-0 origin for tilingdata
maybe. I'm going to remove it.

[enne (OOO), 2014/10/24 20:50:24]

Yes, that's exactly it.

     // is identical to the layer_rect.
     // If update called, then pile size must match bounds pushed to impl layer.
-    DCHECK_EQ(layer_impl->bounds().ToString(), pile_->tiling_size().ToString());
+    DCHECK_IMPLIES(
+        update_source_frame_number_ == layer_tree_host()->source_frame_number(),
+        layer_impl->bounds().ToString() == pile_->tiling_size().ToString());
   }
 
   // Unlike other properties, invalidation must always be set on layer_impl.
   // See PictureLayerImpl::PushPropertiesTo for more details.
   layer_impl->invalidation_.Clear();
   layer_impl->invalidation_.Swap(&pile_invalidation_);
   layer_impl->pile_ = PicturePileImpl::CreateFromOther(pile_.get());
 }
 
 void PictureLayer::SetLayerTreeHost(LayerTreeHost* host) {
@@ skipping 131 matching lines @@
 
 bool PictureLayer::HasDrawableContent() const {
   return client_ && Layer::HasDrawableContent();
 }
 
 void PictureLayer::RunMicroBenchmark(MicroBenchmark* benchmark) {
   benchmark->RunOnLayer(this);
 }
 
 }  // namespace cc