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

Unified Diff: cc/resources/prioritized_tile_set.cc

Issue 367833003: cc: Start using raster/eviction iterators. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 2 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « cc/resources/prioritized_tile_set.h ('k') | cc/resources/prioritized_tile_set_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/resources/prioritized_tile_set.cc
diff --git a/cc/resources/prioritized_tile_set.cc b/cc/resources/prioritized_tile_set.cc
deleted file mode 100644
index d0104cbe331f8351dea111d8bdc6deaef5321eba..0000000000000000000000000000000000000000
--- a/cc/resources/prioritized_tile_set.cc
+++ /dev/null
@@ -1,155 +0,0 @@
-// Copyright 2013 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "cc/resources/prioritized_tile_set.h"
-
-#include <algorithm>
-
-#include "cc/resources/managed_tile_state.h"
-#include "cc/resources/tile.h"
-
-namespace cc {
-
-class BinComparator {
- public:
- bool operator()(const Tile* a,
- const Tile* b) const {
- const ManagedTileState& ams = a->managed_state();
- const ManagedTileState& bms = b->managed_state();
-
- if (ams.priority_bin != bms.priority_bin)
- return ams.priority_bin < bms.priority_bin;
-
- if (ams.required_for_activation != bms.required_for_activation)
- return ams.required_for_activation;
-
- if (ams.resolution != bms.resolution)
- return ams.resolution < bms.resolution;
-
- if (ams.distance_to_visible != bms.distance_to_visible)
- return ams.distance_to_visible < bms.distance_to_visible;
-
- gfx::Rect a_rect = a->content_rect();
- gfx::Rect b_rect = b->content_rect();
- if (a_rect.y() != b_rect.y())
- return a_rect.y() < b_rect.y();
- return a_rect.x() < b_rect.x();
- }
-};
-
-namespace {
-
-bool TilePriorityTieBreaker(const Tile* tile_i, const Tile* tile_j) {
- // When two tiles has same priority use Id as tie breaker.
- return tile_i->id() < tile_j->id();
-}
-
-typedef std::vector<Tile*> TileVector;
-
-void SortBinTiles(ManagedTileBin bin, TileVector* tiles) {
- switch (bin) {
- case NEVER_BIN:
- break;
- case NOW_AND_READY_TO_DRAW_BIN:
- std::sort(tiles->begin(), tiles->end(), TilePriorityTieBreaker);
- break;
- case NOW_BIN:
- case SOON_BIN:
- case EVENTUALLY_AND_ACTIVE_BIN:
- case EVENTUALLY_BIN:
- case AT_LAST_AND_ACTIVE_BIN:
- case AT_LAST_BIN:
- std::sort(tiles->begin(), tiles->end(), BinComparator());
- break;
- default:
- NOTREACHED();
- }
-}
-
-} // namespace
-
-PrioritizedTileSet::PrioritizedTileSet() {
- for (int bin = 0; bin < NUM_BINS; ++bin)
- bin_sorted_[bin] = true;
-}
-
-PrioritizedTileSet::~PrioritizedTileSet() {}
-
-void PrioritizedTileSet::InsertTile(Tile* tile, ManagedTileBin bin) {
- tiles_[bin].push_back(tile);
- bin_sorted_[bin] = false;
-}
-
-void PrioritizedTileSet::Clear() {
- for (int bin = 0; bin < NUM_BINS; ++bin) {
- tiles_[bin].clear();
- bin_sorted_[bin] = true;
- }
-}
-
-bool PrioritizedTileSet::IsEmpty() {
- for (int bin = 0; bin < NUM_BINS; ++bin)
- if (!tiles_[bin].empty())
- return false;
-
- return true;
-}
-
-void PrioritizedTileSet::SortBinIfNeeded(ManagedTileBin bin) {
- if (!bin_sorted_[bin]) {
- SortBinTiles(bin, &tiles_[bin]);
- bin_sorted_[bin] = true;
- }
-}
-
-PrioritizedTileSet::Iterator::Iterator(
- PrioritizedTileSet* tile_set, bool use_priority_ordering)
- : tile_set_(tile_set),
- current_bin_(NOW_AND_READY_TO_DRAW_BIN),
- use_priority_ordering_(use_priority_ordering) {
- if (use_priority_ordering_)
- tile_set_->SortBinIfNeeded(current_bin_);
- iterator_ = tile_set->tiles_[current_bin_].begin();
- if (iterator_ == tile_set_->tiles_[current_bin_].end())
- AdvanceList();
-}
-
-PrioritizedTileSet::Iterator::~Iterator() {}
-
-void PrioritizedTileSet::Iterator::DisablePriorityOrdering() {
- use_priority_ordering_ = false;
-}
-
-PrioritizedTileSet::Iterator&
-PrioritizedTileSet::Iterator::operator++() {
- // We can't increment past the end of the tiles.
- DCHECK(iterator_ != tile_set_->tiles_[current_bin_].end());
-
- ++iterator_;
- if (iterator_ == tile_set_->tiles_[current_bin_].end())
- AdvanceList();
- return *this;
-}
-
-Tile* PrioritizedTileSet::Iterator::operator*() {
- DCHECK(iterator_ != tile_set_->tiles_[current_bin_].end());
- return *iterator_;
-}
-
-void PrioritizedTileSet::Iterator::AdvanceList() {
- DCHECK(iterator_ == tile_set_->tiles_[current_bin_].end());
-
- while (current_bin_ != NEVER_BIN) {
- current_bin_ = static_cast<ManagedTileBin>(current_bin_ + 1);
-
- if (use_priority_ordering_)
- tile_set_->SortBinIfNeeded(current_bin_);
-
- iterator_ = tile_set_->tiles_[current_bin_].begin();
- if (iterator_ != tile_set_->tiles_[current_bin_].end())
- break;
- }
-}
-
-} // namespace cc
« no previous file with comments | « cc/resources/prioritized_tile_set.h ('k') | cc/resources/prioritized_tile_set_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698