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

Side by Side Diff: cc/tiles/picture_layer_tiling.h

Issue 2816943004: cc: Fix bugs found by fuzzer due to floating point imprecision. (Closed)
Patch Set: layouttests Created 3 years, 7 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
« no previous file with comments | « no previous file | cc/tiles/picture_layer_tiling.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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_TILES_PICTURE_LAYER_TILING_H_ 5 #ifndef CC_TILES_PICTURE_LAYER_TILING_H_
6 #define CC_TILES_PICTURE_LAYER_TILING_H_ 6 #define CC_TILES_PICTURE_LAYER_TILING_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 219 matching lines...) Expand 10 before | Expand all | Expand 10 after
230 Tile* operator->() const { return current_tile_; } 230 Tile* operator->() const { return current_tile_; }
231 Tile* operator*() const { return current_tile_; } 231 Tile* operator*() const { return current_tile_; }
232 232
233 CoverageIterator& operator++(); 233 CoverageIterator& operator++();
234 operator bool() const { return tile_j_ <= bottom_; } 234 operator bool() const { return tile_j_ <= bottom_; }
235 235
236 int i() const { return tile_i_; } 236 int i() const { return tile_i_; }
237 int j() const { return tile_j_; } 237 int j() const { return tile_j_; }
238 238
239 private: 239 private:
240 gfx::Rect ComputeGeometryRect() const;
241
240 const PictureLayerTiling* tiling_ = nullptr; 242 const PictureLayerTiling* tiling_ = nullptr;
241 gfx::Size coverage_rect_max_bounds_; 243 gfx::Size coverage_rect_max_bounds_;
242 gfx::Rect coverage_rect_; 244 gfx::Rect coverage_rect_;
243 gfx::AxisTransform2d coverage_to_content_; 245 gfx::AxisTransform2d coverage_to_content_;
244 246
245 Tile* current_tile_ = nullptr; 247 Tile* current_tile_ = nullptr;
246 gfx::Rect current_geometry_rect_; 248 gfx::Rect current_geometry_rect_;
247 int tile_i_ = 0; 249 int tile_i_ = 0;
248 int tile_j_ = 0; 250 int tile_j_ = 0;
249 int left_ = 0; 251 int left_ = 0;
(...skipping 138 matching lines...) Expand 10 before | Expand all | Expand 10 after
388 bool has_eventually_rect_tiles_ = false; 390 bool has_eventually_rect_tiles_ = false;
389 bool all_tiles_done_ = true; 391 bool all_tiles_done_ = true;
390 392
391 private: 393 private:
392 DISALLOW_COPY_AND_ASSIGN(PictureLayerTiling); 394 DISALLOW_COPY_AND_ASSIGN(PictureLayerTiling);
393 }; 395 };
394 396
395 } // namespace cc 397 } // namespace cc
396 398
397 #endif // CC_TILES_PICTURE_LAYER_TILING_H_ 399 #endif // CC_TILES_PICTURE_LAYER_TILING_H_
OLDNEW
« no previous file with comments | « no previous file | cc/tiles/picture_layer_tiling.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698