cc: Heapify pointers instead of objects in eviction tile priority queue.

cc/resources/eviction_tile_priority_queue.h

 // Copyright 2014 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_EVICTION_TILE_PRIORITY_QUEUE_H_
 #define CC_RESOURCES_EVICTION_TILE_PRIORITY_QUEUE_H_
 
 #include <utility>
 #include <vector>
 
@@ skipping 29 matching lines @@
 
   void Build(const std::vector<PictureLayerImpl::Pair>& paired_layers,
              TreePriority tree_priority);
   void Reset();
 
   bool IsEmpty() const;
   Tile* Top();
   void Pop();
 
  private:
-  std::vector<PairedPictureLayerQueue> paired_queues_;
+  // TODO(vmpstr): This is potentially unnecessary if it becomes the case that
+  // PairedPictureLayerQueue is fast enough to copy. In that case, we can use
+  // objects directly (ie std::vector<PairedPictureLayerQueue>).
+  ScopedPtrVector<PairedPictureLayerQueue> paired_queues_heap_;

[reveman, 2014/08/06 01:37:42]

nit: not sure it's necessary to add the "heap_" suffix. we usually avoid having
the type be part of the container member name. you're call but whatever you do,
keep RasterTilePriorityQueue consistent.

[vmpstr, 2014/08/06 15:54:19]

Done.

   TreePriority tree_priority_;
 
   DISALLOW_COPY_AND_ASSIGN(EvictionTilePriorityQueue);
 };
 
 }  // namespace cc
 
 #endif  // CC_RESOURCES_EVICTION_TILE_PRIORITY_QUEUE_H_