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

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

Issue 671653005: SetNeedsRedraw directly when updating a visible tile. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: pinchblurmerge-test: tweak Created 6 years, 1 month 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 #include "cc/layers/picture_layer_impl.h" 5 #include "cc/layers/picture_layer_impl.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <limits> 8 #include <limits>
9 #include <set> 9 #include <set>
10 10
(...skipping 1421 matching lines...) Expand 10 before | Expand all | Expand 10 after
1432 // iterators. 1432 // iterators.
1433 if (!tile) 1433 if (!tile)
1434 continue; 1434 continue;
1435 1435
1436 // We can't check tile->required_for_activation, because that value might 1436 // We can't check tile->required_for_activation, because that value might
1437 // be out of date. It is updated in the raster/eviction iterators. 1437 // be out of date. It is updated in the raster/eviction iterators.
1438 // TODO(vmpstr): Remove the comment once you can't access this information 1438 // TODO(vmpstr): Remove the comment once you can't access this information
1439 // from the tile. 1439 // from the tile.
1440 if (is_tile_required_callback.Run(tiling, tile) && 1440 if (is_tile_required_callback.Run(tiling, tile) &&
1441 !tile->IsReadyToDraw()) { 1441 !tile->IsReadyToDraw()) {
1442 TRACE_EVENT_INSTANT0("cc", 1442 TRACE_EVENT_INSTANT0("cc", "Tile required, but not ready to draw.",
1443 "Tile required, but not ready to draw.",
1444 TRACE_EVENT_SCOPE_THREAD); 1443 TRACE_EVENT_SCOPE_THREAD);
1445 return false; 1444 return false;
1446 } 1445 }
1447 } 1446 }
1448 } 1447 }
1449 1448
1450 return true; 1449 return true;
1451 } 1450 }
1452 1451
1453 namespace { 1452 namespace {
(...skipping 292 matching lines...) Expand 10 before | Expand all | Expand 10 after
1746 PictureLayerTilingSet::TilingRange tiling_range = CurrentTilingRange(); 1745 PictureLayerTilingSet::TilingRange tiling_range = CurrentTilingRange();
1747 size_t current_tiling_range_offset = current_tiling_ - tiling_range.start; 1746 size_t current_tiling_range_offset = current_tiling_ - tiling_range.start;
1748 return tiling_range.end - 1 - current_tiling_range_offset; 1747 return tiling_range.end - 1 - current_tiling_range_offset;
1749 } 1748 }
1750 } 1749 }
1751 NOTREACHED(); 1750 NOTREACHED();
1752 return 0; 1751 return 0;
1753 } 1752 }
1754 1753
1755 } // namespace cc 1754 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698