cc: Separate the priority from the tile and put in new PrioritizedTile

cc/resources/picture_layer_tiling.cc

Index: cc/resources/picture_layer_tiling.cc
diff --git a/cc/resources/picture_layer_tiling.cc b/cc/resources/picture_layer_tiling.cc
index 015c492dcc8600d711791d76a64d4213efdf90a4..fd25b763148a9384f26599f809c8a9ea32a9888b 100644
--- a/cc/resources/picture_layer_tiling.cc
+++ b/cc/resources/picture_layer_tiling.cc
@@ -13,6 +13,7 @@
 #include "base/trace_event/trace_event.h"
 #include "base/trace_event/trace_event_argument.h"
 #include "cc/base/math_util.h"
+#include "cc/resources/prioritized_tile.h"
 #include "cc/resources/tile.h"
 #include "cc/resources/tile_priority.h"
 #include "ui/gfx/geometry/point_conversions.h"
@@ -809,13 +810,26 @@ bool PictureLayerTiling::IsTileRequiredForDraw(const Tile* tile) const {
   return true;
 }
 
-void PictureLayerTiling::UpdateTilePriority(Tile* tile) const {
-  tile->set_priority(ComputePriorityForTile(tile));
-  tile->set_is_occluded(IsTileOccluded(tile));
+void PictureLayerTiling::UpdateTilePriority(

[vmpstr, 2015/05/08 18:11:06]

If you do go the route of GetPrioritizedTile(tile), then I think you would also
want to have a separate call for UpdateRequiredStateForTile(tile) that changes
the required_for_* flags.

Yes, it's going to be two calls, but it's more explicit.

[hendrikw, 2015/05/08 19:21:02]

messier, dangerous, but I like to live dangerously.  hopefully we can move the
required stuff within the prioritized tile.

+    PrioritizedTile* prioritized_tile) const {
+  Tile* tile = prioritized_tile->tile();
+  prioritized_tile->set_priority(ComputePriorityForTile(tile));
+  prioritized_tile->set_is_occluded(IsTileOccluded(tile));
   tile->set_required_for_activation(IsTileRequiredForActivation(tile));
   tile->set_required_for_draw(IsTileRequiredForDraw(tile));
 }
 
+std::map<const Tile*, PrioritizedTile>
+PictureLayerTiling::UpdateAndGetAllPrioritizedTilesForTesting() {
+  std::map<const Tile*, PrioritizedTile> result;
+  for (auto& key_tile_pair : tiles_) {

[vmpstr, 2015/05/08 18:11:06]

const auto&?

[hendrikw, 2015/05/08 19:21:02]

Done.

+    PrioritizedTile prioritized_tile;
+    prioritized_tile.UpdateTile(key_tile_pair.second.get(), this);
+    result.insert(std::make_pair(prioritized_tile.tile(), prioritized_tile));
+  }
+  return result;
+}
+
 void PictureLayerTiling::VerifyAllTilesHaveCurrentRasterSource() const {
 #if DCHECK_IS_ON()
   for (const auto& tile_pair : tiles_)