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

Unified Diff: cc/test/fake_picture_layer_impl.h

Issue 640063010: cc: Don't swap PictureLayerTilingSet on activate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: noswap: perftest Created 6 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
Index: cc/test/fake_picture_layer_impl.h
diff --git a/cc/test/fake_picture_layer_impl.h b/cc/test/fake_picture_layer_impl.h
index b10b357c7debacb6d23e01f761176d53cd629b18..95742954dba9746c4cdc5277c008a316fa019fdc 100644
--- a/cc/test/fake_picture_layer_impl.h
+++ b/cc/test/fake_picture_layer_impl.h
@@ -21,9 +21,10 @@ class FakePictureLayerImpl : public PictureLayerImpl {
static scoped_ptr<FakePictureLayerImpl> CreateWithRasterSource(
enne (OOO) 2014/12/01 22:08:17 I'd slightly prefer CreateMaskWithRasterSource and
danakj 2014/12/05 22:30:44 Done.
LayerTreeImpl* tree_impl,
int id,
- scoped_refptr<RasterSource> raster_source) {
+ scoped_refptr<RasterSource> raster_source,
+ bool is_mask) {
return make_scoped_ptr(
- new FakePictureLayerImpl(tree_impl, id, raster_source));
+ new FakePictureLayerImpl(tree_impl, id, raster_source, is_mask));
}
// Create layer from a raster source that only covers part of the layer.
@@ -31,12 +32,14 @@ class FakePictureLayerImpl : public PictureLayerImpl {
LayerTreeImpl* tree_impl,
int id,
scoped_refptr<RasterSource> raster_source,
+ bool is_mask,
const gfx::Size& layer_bounds) {
- return make_scoped_ptr(
- new FakePictureLayerImpl(tree_impl, id, raster_source, layer_bounds));
+ return make_scoped_ptr(new FakePictureLayerImpl(
+ tree_impl, id, raster_source, is_mask, layer_bounds));
}
scoped_ptr<LayerImpl> CreateLayerImpl(LayerTreeImpl* tree_impl) override;
+ void PushPropertiesTo(LayerImpl* layer_impl) override;
void AppendQuads(RenderPass* render_pass,
const Occlusion& occlusion_in_content_space,
AppendQuadsData* append_quads_data) override;
@@ -91,7 +94,9 @@ class FakePictureLayerImpl : public PictureLayerImpl {
PictureLayerTilingSet* tilings() { return tilings_.get(); }
RasterSource* raster_source() { return raster_source_.get(); }
- void SetRasterSource(scoped_refptr<RasterSource> raster_source);
+ void SetRasterSourceOnPending(scoped_refptr<RasterSource> raster_source,
+ const Region& invalidation,
+ bool is_mask);
size_t append_quads_count() { return append_quads_count_; }
const Region& invalidation() const { return invalidation_; }
@@ -103,7 +108,9 @@ class FakePictureLayerImpl : public PictureLayerImpl {
void set_fixed_tile_size(const gfx::Size& size) { fixed_tile_size_ = size; }
+ // TODO(danakj): Remove this darn thing.
void CreateDefaultTilingsAndTiles();
+
void SetAllTilesVisible();
void SetAllTilesReady();
void SetAllTilesReadyInTiling(PictureLayerTiling* tiling);
@@ -123,10 +130,12 @@ class FakePictureLayerImpl : public PictureLayerImpl {
protected:
FakePictureLayerImpl(LayerTreeImpl* tree_impl,
int id,
- scoped_refptr<RasterSource> raster_source);
+ scoped_refptr<RasterSource> raster_source,
+ bool is_mask);
FakePictureLayerImpl(LayerTreeImpl* tree_impl,
int id,
scoped_refptr<RasterSource> raster_source,
+ bool is_mask,
const gfx::Size& layer_bounds);
FakePictureLayerImpl(LayerTreeImpl* tree_impl, int id);

Powered by Google App Engine
This is Rietveld 408576698