cc: Without valid frame, delegated renderer layer is not drawable

cc/delegated_renderer_layer_impl.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/delegated_renderer_layer_impl.h"
 
 #include "cc/append_quads_data.h"
 #include "cc/delegated_frame_data.h"
 #include "cc/layer_tree_impl.h"
 #include "cc/math_util.h"
@@ skipping 34 matching lines @@
   // Display size is already set so we can compute what the damage rect
   // will be in layer space.
   RenderPass* new_root_pass = frame_data->render_pass_list.empty() ?
                               NULL : frame_data->render_pass_list.back();
   bool new_frame_is_empty = !new_root_pass;
 
   RenderPass* old_root_pass = render_passes_in_draw_order_.empty() ?
                               NULL : render_passes_in_draw_order_.back();
   bool old_frame_is_empty = !old_root_pass;
 
   gfx::RectF damage_in_layer;

[piman, 2013/02/27 20:51:03]

nit: you can now move this and the setUpdateRect inside the if.

[danakj, 2013/02/27 20:53:45]

Thanks! I've already uploaded another version with that :D (also deleted the
old_stuff above now unusued.)

-  if (new_frame_is_empty) {
-    if (!old_frame_is_empty)
-      damage_in_layer = gfx::Rect(bounds());
-  } else {
+  if (!new_frame_is_empty) {
     DCHECK(!new_root_pass->output_rect.IsEmpty());
     damage_in_layer = MathUtil::mapClippedRect(
         DelegatedFrameToLayerSpaceTransform(new_root_pass->output_rect.size()),
         damage_in_frame);
   }
   setUpdateRect(gfx::UnionRects(updateRect(), damage_in_layer));
 
   // TODO(danakj): Convert the resource ids the render passes and return data
   // for a frame ack.
   SetRenderPasses(frame_data->render_pass_list);
@@ skipping 217 matching lines @@
 void DelegatedRendererLayerImpl::ClearChildId() {
   if (!child_id_)
     return;
 
   ResourceProvider* resource_provider = layerTreeImpl()->resource_provider();
   resource_provider->destroyChild(child_id_);
   child_id_ = 0;
 }
 
 }  // namespace cc