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

Unified Diff: cc/LayerChromium.cpp

Issue 11086053: Revert 161133 - [cc] Use base ptr types for cc's CSS animation classes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: 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/LayerChromium.h ('k') | cc/LayerChromiumTest.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/LayerChromium.cpp
===================================================================
--- cc/LayerChromium.cpp (revision 161134)
+++ cc/LayerChromium.cpp (working copy)
@@ -656,7 +656,7 @@
void LayerChromium::createRenderSurface()
{
ASSERT(!m_renderSurface);
- m_renderSurface = make_scoped_ptr(new RenderSurfaceChromium(this));
+ m_renderSurface = adoptPtr(new RenderSurfaceChromium(this));
setRenderTarget(this);
}
@@ -700,7 +700,7 @@
m_transform = transform;
}
-bool LayerChromium::addAnimation(scoped_ptr <CCActiveAnimation> animation)
+bool LayerChromium::addAnimation(PassOwnPtr<CCActiveAnimation> animation)
{
// WebCore currently assumes that accelerated animations will start soon
// after the animation is added. However we cannot guarantee that if we do
@@ -711,7 +711,7 @@
if (!CCSettings::acceleratedAnimationEnabled())
return false;
- m_layerAnimationController->addAnimation(animation.Pass());
+ m_layerAnimationController->addAnimation(animation);
if (m_layerTreeHost) {
m_layerTreeHost->didAddAnimation();
setNeedsCommit();
@@ -743,9 +743,9 @@
setNeedsCommit();
}
-void LayerChromium::setLayerAnimationController(scoped_ptr<CCLayerAnimationController> layerAnimationController)
+void LayerChromium::setLayerAnimationController(PassOwnPtr<CCLayerAnimationController> layerAnimationController)
{
- m_layerAnimationController = layerAnimationController.Pass();
+ m_layerAnimationController = layerAnimationController;
if (m_layerAnimationController) {
m_layerAnimationController->setClient(this);
m_layerAnimationController->setForceSync();
@@ -753,11 +753,11 @@
setNeedsCommit();
}
-scoped_ptr<CCLayerAnimationController> LayerChromium::releaseLayerAnimationController()
+PassOwnPtr<CCLayerAnimationController> LayerChromium::releaseLayerAnimationController()
{
- scoped_ptr<CCLayerAnimationController> toReturn = m_layerAnimationController.Pass();
+ OwnPtr<CCLayerAnimationController> toReturn = m_layerAnimationController.release();
m_layerAnimationController = CCLayerAnimationController::create(this);
- return toReturn.Pass();
+ return toReturn.release();
}
bool LayerChromium::hasActiveAnimation() const
« no previous file with comments | « cc/LayerChromium.h ('k') | cc/LayerChromiumTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698