Implement the logic to kick off image decoding jobs for TileManager

cc/tile_manager.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_TILE_MANAGER_H_
 #define CC_TILE_MANAGER_H_
 
+#include <list>
 #include <queue>
 #include <vector>
 
 #include "base/memory/scoped_ptr.h"
 #include "base/values.h"
 #include "cc/resource_pool.h"
 #include "cc/tile_priority.h"
 
 namespace cc {
 
@@ skipping 28 matching lines @@
  public:
   ManagedTileState();
   ~ManagedTileState();
 
   // Persisted state: valid all the time.
   bool can_use_gpu_memory;
   bool can_be_freed;
   scoped_ptr<ResourcePool::Resource> resource;
   bool resource_is_being_initialized;
   bool contents_swizzled;
+  bool has_image_decoding_info;
+  std::list<skia::LazyPixelRef*> pending_pixel_refs;

[reveman, 2012/12/11 01:35:42]

nit: can you make it obvious that these two members are related. maybe change
has_image_decoding_info to need_to_gather_pixel_refs or just add a comment.

[qinmin, 2012/12/11 04:30:15]

Done.

 
   // Ephemeral state, valid only during Manage.
   TileManagerBin bin;
   TileResolution resolution;
   float time_to_needed_in_seconds;
 };
 
 // This class manages tiles, deciding which should get rasterized and which
 // should no longer have any memory assigned to them. Tile objects are "owned"
 // by layers; they automatically register with the manager when they are
@@ skipping 18 matching lines @@
   friend class Tile;
   void RegisterTile(Tile*);
   void UnregisterTile(Tile*);
   void WillModifyTilePriority(Tile*, WhichTree, const TilePriority& new_priority);
 
  private:
   void AssignGpuMemoryToTiles();
   void FreeResourcesForTile(Tile*);
   void ScheduleManageTiles();
   void ScheduleCheckForCompletedSetPixels();
-  void DispatchMoreRasterTasks();
+  void DispatchMoreTasks();
   void DispatchOneRasterTask(RasterThread*, scoped_refptr<Tile>);
   void OnRasterTaskCompleted(
       scoped_refptr<Tile>,
       scoped_ptr<ResourcePool::Resource>,
       scoped_refptr<PicturePileImpl>,
       RenderingStats*);
   void DidFinishTileInitialization(Tile*);
+  bool HasImageDecodingTasks(Tile*);
+  void OnImageDecodingTaskCompleted(scoped_refptr<Tile>, uint32_t);
+  void DispatchOneImageDecodingTask(
+      RasterThread*, scoped_refptr<Tile>, skia::LazyPixelRef*);
+  void GetImageInformationForTile(Tile*);
+  RasterThread* GetFreeRasterThread();
 
   TileManagerClient* client_;
   scoped_ptr<ResourcePool> resource_pool_;
   bool manage_tiles_pending_;
   bool check_for_completed_set_pixels_pending_;
 
   GlobalStateThatImpactsTilePriority global_state_;
 
   typedef std::vector<Tile*> TileVector;
   TileVector tiles_;
   TileVector tiles_that_need_to_be_rasterized_;
 
+  typedef std::list<Tile*> TileList;
+  // Tiles that are has image decoding tasks.

[reveman, 2012/12/11 01:35:42]

I guess the proper name for the list is
tiles_that_need_to_be_rasterized_but_have_image_decoding_tasks_ but that's
awkwardly long. Can you just make it clear in this comment that these are tiles
that need to be rasterized?

[qinmin, 2012/12/11 04:30:15]

Done.

+  TileList tiles_have_image_decoding_tasks_;

[reveman, 2012/12/11 01:35:42]

nit: tiles_that_have_image_decoding_tasks_ or tiles_with_image_decoding_tasks_. 

[qinmin, 2012/12/11 04:30:15]

Done.

+
+  typedef base::hash_map<uint32_t, skia::LazyPixelRef*> PixelRefMap;
+  PixelRefMap pending_decode_tasks_;
+
   typedef std::queue<scoped_refptr<Tile> > TileQueue;
   TileQueue tiles_with_pending_set_pixels_;
 
   typedef ScopedPtrVector<RasterThread> RasterThreadVector;
   RasterThreadVector raster_threads_;
 
   RenderingStats rendering_stats_;
 
   DISALLOW_COPY_AND_ASSIGN(TileManager);
 };
 
 }  // namespace cc
 
 #endif  // CC_TILE_MANAGER_H_