Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1581)

Unified Diff: cc/tile.h

Issue 11417002: First draft of TileManager's tile prioritzation system (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Compiles Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « cc/layer_tree_host_impl.h ('k') | cc/tile.cc » ('j') | cc/tile_priority.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/tile.h
diff --git a/cc/tile.h b/cc/tile.h
index 2f555c339514fe398993594356f398886ae2f660..133d4fe3072736af693e1ba942e4ae30be8d1796 100644
--- a/cc/tile.h
+++ b/cc/tile.h
@@ -8,8 +8,10 @@
#include "base/memory/ref_counted.h"
#include "base/memory/scoped_ptr.h"
#include "base/memory/scoped_vector.h"
+#include "cc/layer_tree_host_impl.h"
#include "cc/picture_pile.h"
#include "cc/resource_provider.h"
+#include "cc/tile_manager.h"
#include "cc/tile_priority.h"
#include "ui/gfx/rect.h"
#include "ui/gfx/size.h"
@@ -17,80 +19,55 @@
namespace cc {
class Tile;
-class TileManager;
-enum TileQuality {
- LOW_TILE_QUALITY,
- NORMAL_TILE_QUALITY
-};
-
-class TileVersion {
-public:
- TileVersion(Tile* tile, int frame_number,
- PicturePile* picture_pile)
- : tile_(tile),
- frame_number_(frame_number),
- picture_pile_(picture_pile),
- resource_id_(0) {}
-
- int frame_number() const { return frame_number_; }
+class Tile : public base::RefCounted<Tile> {
+ public:
+ Tile(TileManager* tile_manager,
+ PicturePile* picture_pile,
+ gfx::Size tile_size,
+ GLenum format,
+ gfx::Rect rect_inside_picture);
const PicturePile* picture_pile() const {
return picture_pile_;
}
- const TilePriority& priority() const {
- return priority_;
+ const TilePriority& priority(WhichTree tree) const {
+ return priority_[tree];
}
- void ModifyPriority(const TilePriority& priority) {
- priority_ = priority;
+ TilePriority combined_priority() const {
+ return TilePriority(priority_[ACTIVE_TREE],
+ priority_[PENDING_TREE]);
}
- ResourceProvider::ResourceId resource_id() const {
- return resource_id_;
- }
-
-private:
- Tile* tile_;
- int frame_number_;
- PicturePile* picture_pile_;
- TilePriority priority_;
- ResourceProvider::ResourceId resource_id_;
-};
-
-class Tile : public base::RefCounted<Tile> {
- public:
- Tile(TileManager* tile_manager,
- gfx::Size tile_size,
- GLenum format,
- gfx::Rect rect_inside_picture,
- TileQuality quality);
-
- void SetPicturePile(int frame_number, PicturePile* picture_pile);
- void ModifyPriority(int frame_number, const TilePriority& priority);
+ void set_priority(WhichTree tree, const TilePriority& priority);
// Returns 0 if not drawable.
- ResourceProvider::ResourceId GetDrawableResourceId(int frame_number);
+ ResourceProvider::ResourceId resource_id() const { return managed_state_.resource_id; }
- protected:
- // Methods called by TileManager.
- void DeleteVersionOnRequestOfTileManager(int frame_number);
+ size_t bytes_consumed_if_allocated() const;
private:
- friend class base::RefCounted<Tile>;
+ // Methods called by by tile manager.
friend class TileManager;
+ friend class BinComparator;
+ ManagedTileState& managed_state() { return managed_state_; }
+ const ManagedTileState& managed_state() const { return managed_state_; }
- TileVersion* GetVersion(int frame_number);
+ // Normal private methods.
+ friend class base::RefCounted<Tile>;
~Tile();
TileManager* tile_manager_;
+ PicturePile* picture_pile_;
gfx::Rect tile_size_;
GLenum format_;
gfx::Rect rect_inside_picture_;
- TileQuality quality_;
- ScopedVector<TileVersion> versions_;
-};
+
+ TilePriority priority_[2];
+ ManagedTileState managed_state_;
+ };
} // namespace cc
« no previous file with comments | « cc/layer_tree_host_impl.h ('k') | cc/tile.cc » ('j') | cc/tile_priority.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698