cc: ReadyToDraw notifications.

cc/resources/tile.h

 // 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.
 
 #ifndef CC_RESOURCES_TILE_H_
 #define CC_RESOURCES_TILE_H_
 
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/scoped_vector.h"
@@ skipping 73 matching lines @@
         NOTREACHED();
         return false;
     }
   }
 
   // TODO(vmpstr): Move this to the iterators.
   bool required_for_activation() const { return required_for_activation_; }
   void set_required_for_activation(bool is_required) {
     required_for_activation_ = is_required;
   }
+  bool required_for_draw() const { return required_for_draw_; }

[reveman, 2014/10/24 18:16:31]

"required_for_draw" makes it sound like drawing is blocked until this tile has
been initialized. shouldn't it be "visible" instead? which is a concept that
already exists. having both "required_for_draw" and "visible" is confusing..

[ernstm, 2014/10/27 21:13:56]

It is tiles that are visible, high-res and on the active tree. Those are the
ones that are required for drawing without an approximation such as low-res or
checkerboard.

+  void set_required_for_draw(bool is_required) {
+    required_for_draw_ = is_required;
+  }
 
   bool use_picture_analysis() const {
     return !!(flags_ & USE_PICTURE_ANALYSIS);
   }
 
   bool HasResources() const { return managed_state_.draw_info.has_resource(); }
   bool NeedsRaster() const {
     return managed_state_.draw_info.mode() ==
                ManagedTileState::DrawInfo::PICTURE_PILE_MODE ||
            !managed_state_.draw_info.IsReadyToDraw();
@@ skipping 69 matching lines @@
 
   TilePriority priority_[NUM_TREES];
   ManagedTileState managed_state_;
   int layer_id_;
   int source_frame_number_;
   int flags_;
   bool is_shared_;
   int tiling_i_index_;
   int tiling_j_index_;
   bool required_for_activation_;
+  bool required_for_draw_;

[vmpstr, 2014/10/24 18:02:24]

Initialize in the ctor please.

[ernstm, 2014/10/27 21:13:56]

Done.

 
   Id id_;
   static Id s_next_id_;
 
   DISALLOW_COPY_AND_ASSIGN(Tile);
 };
 
 }  // namespace cc
 
 #endif  // CC_RESOURCES_TILE_H_