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

Side by Side Diff: cc/test/fake_picture_pile_impl.cc

Issue 196023015: Revert of cc: Replace recorded region with direct map lookup (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « cc/test/fake_picture_pile_impl.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "cc/test/fake_picture_pile_impl.h" 5 #include "cc/test/fake_picture_pile_impl.h"
6 6
7 #include <limits> 7 #include <limits>
8 #include <utility> 8 #include <utility>
9 9
10 #include "cc/test/impl_side_painting_settings.h" 10 #include "cc/test/impl_side_painting_settings.h"
11 #include "testing/gtest/include/gtest/gtest.h" 11 #include "testing/gtest/include/gtest/gtest.h"
12 12
13 namespace cc { 13 namespace cc {
14 14
15 FakePicturePileImpl::FakePicturePileImpl() {} 15 FakePicturePileImpl::FakePicturePileImpl() {}
16 16
17 FakePicturePileImpl::~FakePicturePileImpl() {} 17 FakePicturePileImpl::~FakePicturePileImpl() {}
18 18
19 scoped_refptr<FakePicturePileImpl> FakePicturePileImpl::CreateFilledPile( 19 scoped_refptr<FakePicturePileImpl> FakePicturePileImpl::CreateFilledPile(
20 const gfx::Size& tile_size, 20 const gfx::Size& tile_size,
21 const gfx::Size& layer_bounds) { 21 const gfx::Size& layer_bounds) {
22 scoped_refptr<FakePicturePileImpl> pile(new FakePicturePileImpl()); 22 scoped_refptr<FakePicturePileImpl> pile(new FakePicturePileImpl());
23 pile->tiling().SetTotalSize(layer_bounds); 23 pile->tiling().SetTotalSize(layer_bounds);
24 pile->tiling().SetMaxTextureSize(tile_size); 24 pile->tiling().SetMaxTextureSize(tile_size);
25 pile->SetTileGridSize(ImplSidePaintingSettings().default_tile_size); 25 pile->SetTileGridSize(ImplSidePaintingSettings().default_tile_size);
26 pile->recorded_viewport_ = gfx::Rect(layer_bounds);
27 pile->has_any_recordings_ = true;
28 for (int x = 0; x < pile->tiling().num_tiles_x(); ++x) { 26 for (int x = 0; x < pile->tiling().num_tiles_x(); ++x) {
29 for (int y = 0; y < pile->tiling().num_tiles_y(); ++y) 27 for (int y = 0; y < pile->tiling().num_tiles_y(); ++y)
30 pile->AddRecordingAt(x, y); 28 pile->AddRecordingAt(x, y);
31 } 29 }
30 pile->UpdateRecordedRegion();
32 return pile; 31 return pile;
33 } 32 }
34 33
35 scoped_refptr<FakePicturePileImpl> FakePicturePileImpl::CreateEmptyPile( 34 scoped_refptr<FakePicturePileImpl> FakePicturePileImpl::CreateEmptyPile(
36 const gfx::Size& tile_size, 35 const gfx::Size& tile_size,
37 const gfx::Size& layer_bounds) { 36 const gfx::Size& layer_bounds) {
38 scoped_refptr<FakePicturePileImpl> pile(new FakePicturePileImpl()); 37 scoped_refptr<FakePicturePileImpl> pile(new FakePicturePileImpl());
39 pile->tiling().SetTotalSize(layer_bounds); 38 pile->tiling().SetTotalSize(layer_bounds);
40 pile->tiling().SetMaxTextureSize(tile_size); 39 pile->tiling().SetMaxTextureSize(tile_size);
41 pile->SetTileGridSize(ImplSidePaintingSettings().default_tile_size); 40 pile->SetTileGridSize(ImplSidePaintingSettings().default_tile_size);
42 pile->recorded_viewport_ = gfx::Rect(); 41 pile->UpdateRecordedRegion();
43 pile->has_any_recordings_ = false;
44 return pile; 42 return pile;
45 } 43 }
46 44
47 scoped_refptr<FakePicturePileImpl> 45 scoped_refptr<FakePicturePileImpl>
48 FakePicturePileImpl::CreateEmptyPileThatThinksItHasRecordings( 46 FakePicturePileImpl::CreatePileWithRecordedRegion(
49 const gfx::Size& tile_size, 47 const gfx::Size& tile_size,
50 const gfx::Size& layer_bounds) { 48 const gfx::Size& layer_bounds,
49 const Region& recorded_region) {
51 scoped_refptr<FakePicturePileImpl> pile(new FakePicturePileImpl()); 50 scoped_refptr<FakePicturePileImpl> pile(new FakePicturePileImpl());
52 pile->tiling().SetTotalSize(layer_bounds); 51 pile->tiling().SetTotalSize(layer_bounds);
53 pile->tiling().SetMaxTextureSize(tile_size); 52 pile->tiling().SetMaxTextureSize(tile_size);
54 pile->SetTileGridSize(ImplSidePaintingSettings().default_tile_size); 53 pile->SetTileGridSize(ImplSidePaintingSettings().default_tile_size);
55 // This simulates a false positive for this flag. 54 pile->SetRecordedRegionForTesting(recorded_region);
56 pile->recorded_viewport_ = gfx::Rect();
57 pile->has_any_recordings_ = true;
58 return pile; 55 return pile;
59 } 56 }
60 57
61 scoped_refptr<FakePicturePileImpl> 58 scoped_refptr<FakePicturePileImpl> FakePicturePileImpl::CreatePile() {
62 FakePicturePileImpl::CreateInfiniteFilledPile() {
63 scoped_refptr<FakePicturePileImpl> pile(new FakePicturePileImpl()); 59 scoped_refptr<FakePicturePileImpl> pile(new FakePicturePileImpl());
64 gfx::Size size(std::numeric_limits<int>::max(), 60 gfx::Size size(std::numeric_limits<int>::max(),
65 std::numeric_limits<int>::max()); 61 std::numeric_limits<int>::max());
66 pile->Resize(size); 62 pile->Resize(size);
67 pile->tiling().SetTotalSize(size); 63 pile->recorded_region_ = Region(gfx::Rect(size));
68 pile->tiling().SetMaxTextureSize(size);
69 pile->SetTileGridSize(size);
70 pile->recorded_viewport_ = gfx::Rect(size);
71 pile->has_any_recordings_ = true;
72 pile->AddRecordingAt(0, 0);
73 return pile; 64 return pile;
74 } 65 }
75 66
76 void FakePicturePileImpl::AddRecordingAt(int x, int y) { 67 void FakePicturePileImpl::AddRecordingAt(int x, int y) {
77 EXPECT_GE(x, 0); 68 EXPECT_GE(x, 0);
78 EXPECT_GE(y, 0); 69 EXPECT_GE(y, 0);
79 EXPECT_LT(x, tiling_.num_tiles_x()); 70 EXPECT_LT(x, tiling_.num_tiles_x());
80 EXPECT_LT(y, tiling_.num_tiles_y()); 71 EXPECT_LT(y, tiling_.num_tiles_y());
81 72
82 if (HasRecordingAt(x, y)) 73 if (HasRecordingAt(x, y))
83 return; 74 return;
84 gfx::Rect bounds(tiling().TileBounds(x, y)); 75 gfx::Rect bounds(tiling().TileBounds(x, y));
85 bounds.Inset(-buffer_pixels(), -buffer_pixels()); 76 bounds.Inset(-buffer_pixels(), -buffer_pixels());
86 77
87 scoped_refptr<Picture> picture( 78 scoped_refptr<Picture> picture(
88 Picture::Create(bounds, &client_, tile_grid_info_, true, 0)); 79 Picture::Create(bounds, &client_, tile_grid_info_, true, 0));
89 picture_map_[std::pair<int, int>(x, y)].SetPicture(picture); 80 picture_map_[std::pair<int, int>(x, y)].SetPicture(picture);
90 EXPECT_TRUE(HasRecordingAt(x, y)); 81 EXPECT_TRUE(HasRecordingAt(x, y));
91 82
92 has_any_recordings_ = true; 83 UpdateRecordedRegion();
93 } 84 }
94 85
95 void FakePicturePileImpl::RemoveRecordingAt(int x, int y) { 86 void FakePicturePileImpl::RemoveRecordingAt(int x, int y) {
96 EXPECT_GE(x, 0); 87 EXPECT_GE(x, 0);
97 EXPECT_GE(y, 0); 88 EXPECT_GE(y, 0);
98 EXPECT_LT(x, tiling_.num_tiles_x()); 89 EXPECT_LT(x, tiling_.num_tiles_x());
99 EXPECT_LT(y, tiling_.num_tiles_y()); 90 EXPECT_LT(y, tiling_.num_tiles_y());
100 91
101 if (!HasRecordingAt(x, y)) 92 if (!HasRecordingAt(x, y))
102 return; 93 return;
103 picture_map_.erase(std::pair<int, int>(x, y)); 94 picture_map_.erase(std::pair<int, int>(x, y));
104 EXPECT_FALSE(HasRecordingAt(x, y)); 95 EXPECT_FALSE(HasRecordingAt(x, y));
96
97 UpdateRecordedRegion();
105 } 98 }
106 99
107 void FakePicturePileImpl::RerecordPile() { 100 void FakePicturePileImpl::RerecordPile() {
108 for (int y = 0; y < num_tiles_y(); ++y) { 101 for (int y = 0; y < num_tiles_y(); ++y) {
109 for (int x = 0; x < num_tiles_x(); ++x) { 102 for (int x = 0; x < num_tiles_x(); ++x) {
110 RemoveRecordingAt(x, y); 103 RemoveRecordingAt(x, y);
111 AddRecordingAt(x, y); 104 AddRecordingAt(x, y);
112 } 105 }
113 } 106 }
114 } 107 }
115 108
116 } // namespace cc 109 } // namespace cc
OLDNEW
« no previous file with comments | « cc/test/fake_picture_pile_impl.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698