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

Unified Diff: cc/layer_tiling_data.cc

Issue 11144023: cc: Store tiles, painters, tiling data in scoped_ptr (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 8 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/layer_tiling_data.h ('k') | cc/layer_tree_host_impl_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/layer_tiling_data.cc
diff --git a/cc/layer_tiling_data.cc b/cc/layer_tiling_data.cc
index 331589c67e4b91c1defc577ab290937d17c4a8d3..2e6eaab8b7758f3580b7c3c3e8cd4a72bdbf2343 100644
--- a/cc/layer_tiling_data.cc
+++ b/cc/layer_tiling_data.cc
@@ -11,9 +11,9 @@ using namespace std;
namespace cc {
-PassOwnPtr<CCLayerTilingData> CCLayerTilingData::create(const IntSize& tileSize, BorderTexelOption border)
+scoped_ptr<CCLayerTilingData> CCLayerTilingData::create(const IntSize& tileSize, BorderTexelOption border)
{
- return adoptPtr(new CCLayerTilingData(tileSize, border));
+ return make_scoped_ptr(new CCLayerTilingData(tileSize, border));
}
CCLayerTilingData::CCLayerTilingData(const IntSize& tileSize, BorderTexelOption border)
@@ -58,16 +58,16 @@ const CCLayerTilingData& CCLayerTilingData::operator=(const CCLayerTilingData& t
return *this;
}
-void CCLayerTilingData::addTile(PassOwnPtr<Tile> tile, int i, int j)
+void CCLayerTilingData::addTile(scoped_ptr<Tile> tile, int i, int j)
{
ASSERT(!tileAt(i, j));
tile->moveTo(i, j);
- m_tiles.add(make_pair(i, j), tile);
+ m_tiles.add(make_pair(i, j), tile.Pass());
}
-PassOwnPtr<CCLayerTilingData::Tile> CCLayerTilingData::takeTile(int i, int j)
+scoped_ptr<CCLayerTilingData::Tile> CCLayerTilingData::takeTile(int i, int j)
{
- return m_tiles.take(make_pair(i, j));
+ return m_tiles.take_and_erase(make_pair(i, j));
}
CCLayerTilingData::Tile* CCLayerTilingData::tileAt(int i, int j) const
@@ -134,16 +134,11 @@ void CCLayerTilingData::setBounds(const IntSize& size)
contentRectToTileIndices(IntRect(IntPoint(), size), left, top, right, bottom);
Vector<TileMapKey> invalidTileKeys;
for (TileMap::const_iterator it = m_tiles.begin(); it != m_tiles.end(); ++it) {
-#if WTF_NEW_HASHMAP_ITERATORS_INTERFACE
- if (it->key.first > right || it->key.second > bottom)
- invalidTileKeys.append(it->key);
-#else
if (it->first.first > right || it->first.second > bottom)
invalidTileKeys.append(it->first);
-#endif
}
for (size_t i = 0; i < invalidTileKeys.size(); ++i)
- m_tiles.remove(invalidTileKeys[i]);
+ m_tiles.erase(invalidTileKeys[i]);
}
IntSize CCLayerTilingData::bounds() const
« no previous file with comments | « cc/layer_tiling_data.h ('k') | cc/layer_tree_host_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698