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

Unified Diff: cc/resources/picture_layer_tiling_set_unittest.cc

Issue 62283012: cc: Added tile bundles (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years 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/picture_layer_tiling_set.cc ('k') | cc/resources/picture_layer_tiling_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/resources/picture_layer_tiling_set_unittest.cc
diff --git a/cc/resources/picture_layer_tiling_set_unittest.cc b/cc/resources/picture_layer_tiling_set_unittest.cc
index 96129c64fe5452aa0fc260c12cd4b7e1df728868..0ab1c664f8abedbb22de815dc9ec9ec8ef518fe5 100644
--- a/cc/resources/picture_layer_tiling_set_unittest.cc
+++ b/cc/resources/picture_layer_tiling_set_unittest.cc
@@ -12,6 +12,7 @@
#include "cc/test/fake_output_surface.h"
#include "cc/test/fake_output_surface_client.h"
#include "cc/test/fake_picture_layer_tiling_client.h"
+#include "cc/test/fake_tile_manager.h"
#include "cc/test/fake_tile_manager_client.h"
#include "testing/gtest/include/gtest/gtest.h"
#include "ui/gfx/size_conversions.h"
@@ -20,7 +21,9 @@ namespace cc {
namespace {
TEST(PictureLayerTilingSetTest, NoResources) {
- FakePictureLayerTilingClient client;
+ FakeTileManagerClient tile_manager_client;
+ FakeTileManager tile_manager(&tile_manager_client);
+ FakePictureLayerTilingClient client(&tile_manager);
gfx::Size layer_bounds(1000, 800);
PictureLayerTilingSet set(&client, layer_bounds);
client.SetTileSize(gfx::Size(256, 256));
@@ -68,7 +71,10 @@ class PictureLayerTilingSetTestWithResources : public testing::Test {
scoped_ptr<ResourceProvider> resource_provider =
ResourceProvider::Create(output_surface.get(), NULL, 0, false, 1);
- FakePictureLayerTilingClient client(resource_provider.get());
+ FakeTileManagerClient tile_manager_client;
+ FakeTileManager tile_manager(&tile_manager_client, resource_provider.get());
+
+ FakePictureLayerTilingClient client(&tile_manager);
client.SetTileSize(gfx::Size(256, 256));
gfx::Size layer_bounds(1000, 800);
PictureLayerTilingSet set(&client, layer_bounds);
@@ -150,10 +156,24 @@ class PictureLayerTilingSetSyncTest : public testing::Test {
: tile_size_(gfx::Size(10, 10)),
source_bounds_(gfx::Size(30, 20)),
target_bounds_(gfx::Size(30, 30)) {
- source_client_.SetTileSize(tile_size_);
- target_client_.SetTileSize(tile_size_);
- source_.reset(new PictureLayerTilingSet(&source_client_, source_bounds_));
- target_.reset(new PictureLayerTilingSet(&target_client_, target_bounds_));
+ output_surface_ = FakeOutputSurface::Create3d();
+ CHECK(output_surface_->BindToClient(&output_surface_client_));
+ resource_provider_ =
+ ResourceProvider::Create(output_surface_.get(), NULL, 0, false, 1);
+ tile_manager_ = make_scoped_ptr(
+ new FakeTileManager(&tile_manager_client_, resource_provider_.get()));
+ source_client_ =
+ make_scoped_ptr(new FakePictureLayerTilingClient(tile_manager_.get()));
+ target_client_ =
+ make_scoped_ptr(new FakePictureLayerTilingClient(tile_manager_.get()));
+
+ source_client_->SetTileSize(tile_size_);
+ target_client_->SetTileSize(tile_size_);
+
+ source_.reset(
+ new PictureLayerTilingSet(source_client_.get(), source_bounds_));
+ target_.reset(
+ new PictureLayerTilingSet(target_client_.get(), target_bounds_));
}
// Sync from source to target.
@@ -161,9 +181,9 @@ class PictureLayerTilingSetSyncTest : public testing::Test {
const Region& invalidation,
float minimum_scale) {
for (size_t i = 0; i < source_->num_tilings(); ++i)
- source_->tiling_at(i)->CreateAllTilesForTesting();
+ source_->tiling_at(i)->CreateTilesForTesting(ACTIVE_TREE);
for (size_t i = 0; i < target_->num_tilings(); ++i)
- target_->tiling_at(i)->CreateAllTilesForTesting();
+ target_->tiling_at(i)->CreateTilesForTesting(PENDING_TREE);
target_->SyncTilings(
*source_.get(), new_bounds, invalidation, minimum_scale);
@@ -195,8 +215,8 @@ class PictureLayerTilingSetSyncTest : public testing::Test {
target_tiling->contents_scale());
}
- EXPECT_EQ(source_->client(), &source_client_);
- EXPECT_EQ(target_->client(), &target_client_);
+ EXPECT_EQ(source_->client(), source_client_.get());
+ EXPECT_EQ(target_->client(), target_client_.get());
ValidateTargetTilingSet();
}
@@ -212,7 +232,7 @@ class PictureLayerTilingSetSyncTest : public testing::Test {
}
for (size_t i = 0; i < target_->num_tilings(); ++i)
- ValidateTiling(target_->tiling_at(i), target_client_.pile());
+ ValidateTiling(target_->tiling_at(i), target_client_->pile());
}
void ValidateTiling(const PictureLayerTiling* tiling,
@@ -222,7 +242,7 @@ class PictureLayerTilingSetSyncTest : public testing::Test {
else if (!tiling->live_tiles_rect().IsEmpty())
EXPECT_TRUE(tiling->ContentRect().Contains(tiling->live_tiles_rect()));
- std::vector<Tile*> tiles = tiling->AllTilesForTesting();
+ std::vector<Tile*> tiles = tiling->TilesForTesting(PENDING_TREE);
for (size_t i = 0; i < tiles.size(); ++i) {
const Tile* tile = tiles[i];
ASSERT_TRUE(!!tile);
@@ -239,13 +259,19 @@ class PictureLayerTilingSetSyncTest : public testing::Test {
}
}
+ scoped_ptr<FakeOutputSurface> output_surface_;
+ FakeOutputSurfaceClient output_surface_client_;
+
gfx::Size tile_size_;
+ scoped_ptr<ResourceProvider> resource_provider_;
+ scoped_ptr<FakeTileManager> tile_manager_;
+ FakeTileManagerClient tile_manager_client_;
- FakePictureLayerTilingClient source_client_;
+ scoped_ptr<FakePictureLayerTilingClient> source_client_;
gfx::Size source_bounds_;
scoped_ptr<PictureLayerTilingSet> source_;
- FakePictureLayerTilingClient target_client_;
+ scoped_ptr<FakePictureLayerTilingClient> target_client_;
gfx::Size target_bounds_;
scoped_ptr<PictureLayerTilingSet> target_;
};
@@ -348,7 +374,7 @@ TEST_F(PictureLayerTilingSetSyncTest, MinimumScale) {
TEST_F(PictureLayerTilingSetSyncTest, Invalidation) {
source_->AddTiling(2.f);
target_->AddTiling(2.f);
- target_->tiling_at(0)->CreateAllTilesForTesting();
+ target_->tiling_at(0)->CreateTilesForTesting(PENDING_TREE);
Region layer_invalidation;
layer_invalidation.Union(gfx::Rect(0, 0, 1, 1));
@@ -392,7 +418,7 @@ TEST_F(PictureLayerTilingSetSyncTest, TileSizeChange) {
target_->tiling_at(0)->AllTilesForTesting();
EXPECT_GT(original_tiles.size(), 0u);
gfx::Size new_tile_size(100, 100);
- target_client_.SetTileSize(new_tile_size);
+ target_client_->SetTileSize(new_tile_size);
EXPECT_NE(target_->tiling_at(0)->tile_size().ToString(),
new_tile_size.ToString());
« no previous file with comments | « cc/resources/picture_layer_tiling_set.cc ('k') | cc/resources/picture_layer_tiling_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698