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

Side by Side Diff: cc/layers/picture_layer_impl.h

Issue 555183002: cc: Fix layer raster tile iterator not to skip a tile erroneously. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: DCHECK, cleaned up unit test Created 6 years, 3 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
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 2012 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 #ifndef CC_LAYERS_PICTURE_LAYER_IMPL_H_ 5 #ifndef CC_LAYERS_PICTURE_LAYER_IMPL_H_
6 #define CC_LAYERS_PICTURE_LAYER_IMPL_H_ 6 #define CC_LAYERS_PICTURE_LAYER_IMPL_H_
7 7
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
44 ~LayerRasterTileIterator(); 44 ~LayerRasterTileIterator();
45 45
46 Tile* operator*(); 46 Tile* operator*();
47 const Tile* operator*() const; 47 const Tile* operator*() const;
48 LayerRasterTileIterator& operator++(); 48 LayerRasterTileIterator& operator++();
49 operator bool() const; 49 operator bool() const;
50 50
51 private: 51 private:
52 enum IteratorType { LOW_RES, HIGH_RES, NUM_ITERATORS }; 52 enum IteratorType { LOW_RES, HIGH_RES, NUM_ITERATORS };
53 53
54 void AdvanceToNextStage();
55
54 PictureLayerImpl* layer_; 56 PictureLayerImpl* layer_;
55 57
56 struct IterationStage { 58 struct IterationStage {
57 IteratorType iterator_type; 59 IteratorType iterator_type;
58 TilePriority::PriorityBin tile_type; 60 TilePriority::PriorityBin tile_type;
59 }; 61 };
60 62
61 size_t current_stage_; 63 size_t current_stage_;
62 64
63 // One low res stage, and three high res stages. 65 // One low res stage, and three high res stages.
(...skipping 164 matching lines...) Expand 10 before | Expand all | Expand 10 after
228 gfx::Rect viewport_rect_for_tile_priority_; 230 gfx::Rect viewport_rect_for_tile_priority_;
229 gfx::Transform screen_space_transform_for_tile_priority_; 231 gfx::Transform screen_space_transform_for_tile_priority_;
230 232
231 friend class PictureLayer; 233 friend class PictureLayer;
232 DISALLOW_COPY_AND_ASSIGN(PictureLayerImpl); 234 DISALLOW_COPY_AND_ASSIGN(PictureLayerImpl);
233 }; 235 };
234 236
235 } // namespace cc 237 } // namespace cc
236 238
237 #endif // CC_LAYERS_PICTURE_LAYER_IMPL_H_ 239 #endif // CC_LAYERS_PICTURE_LAYER_IMPL_H_
OLDNEW
« no previous file with comments | « no previous file | cc/layers/picture_layer_impl.cc » ('j') | cc/layers/picture_layer_impl_unittest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698