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

Unified Diff: cc/resources/tiling_set_raster_queue_all.h

Issue 816453008: cc: Split tiling set raster queues into all and required. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years, 11 months 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/resources/tiling_set_raster_queue.cc ('k') | cc/resources/tiling_set_raster_queue_all.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/resources/tiling_set_raster_queue_all.h
diff --git a/cc/resources/tiling_set_raster_queue.h b/cc/resources/tiling_set_raster_queue_all.h
similarity index 72%
copy from cc/resources/tiling_set_raster_queue.h
copy to cc/resources/tiling_set_raster_queue_all.h
index e3250369fd60fc2055f7ef423a42c107d9df3571..bea70d3f7c847a56ee8cebab371f126334457892 100644
--- a/cc/resources/tiling_set_raster_queue.h
+++ b/cc/resources/tiling_set_raster_queue_all.h
@@ -2,27 +2,30 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef CC_RESOURCES_TILING_SET_RASTER_QUEUE_H_
-#define CC_RESOURCES_TILING_SET_RASTER_QUEUE_H_
+#ifndef CC_RESOURCES_TILING_SET_RASTER_QUEUE_ALL_H_
+#define CC_RESOURCES_TILING_SET_RASTER_QUEUE_ALL_H_
#include "cc/base/cc_export.h"
#include "cc/resources/picture_layer_tiling_set.h"
#include "cc/resources/tile.h"
#include "cc/resources/tile_priority.h"
+#include "cc/resources/tiling_set_raster_queue.h"
namespace cc {
-class CC_EXPORT TilingSetRasterQueue {
+// This queue returns all tiles required to be rasterized from HIGH_RESOLUTION
+// and LOW_RESOLUTION tilings.
+class CC_EXPORT TilingSetRasterQueueAll
+ : public NON_EXPORTED_BASE(TilingSetRasterQueue) {
public:
- TilingSetRasterQueue();
- TilingSetRasterQueue(PictureLayerTilingSet* tiling_set,
- bool prioritize_low_res);
- ~TilingSetRasterQueue();
+ TilingSetRasterQueueAll(PictureLayerTilingSet* tiling_set,
+ bool prioritize_low_res);
+ ~TilingSetRasterQueueAll() override;
- Tile* Top();
- const Tile* Top() const;
- void Pop();
- bool IsEmpty() const;
+ Tile* Top() override;
+ const Tile* Top() const override;
+ void Pop() override;
+ bool IsEmpty() const override;
private:
class TilingIterator {
@@ -32,7 +35,7 @@ class CC_EXPORT TilingSetRasterQueue {
TilingData* tiling_data);
~TilingIterator();
- operator bool() const { return !!current_tile_; }
+ bool done() const { return current_tile_ == nullptr; }
const Tile* operator*() const { return current_tile_; }
Tile* operator*() { return current_tile_; }
TilePriority::PriorityBin type() const {
@@ -94,4 +97,4 @@ class CC_EXPORT TilingSetRasterQueue {
} // namespace cc
-#endif // CC_RESOURCES_TILING_SET_RASTER_QUEUE_H_
+#endif // CC_RESOURCES_TILING_SET_RASTER_QUEUE_ALL_H_
« no previous file with comments | « cc/resources/tiling_set_raster_queue.cc ('k') | cc/resources/tiling_set_raster_queue_all.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698