Change WebLayer::addAnimation to transfer ownership of WebAnimation.

Source/core/platform/graphics/GraphicsLayer.cpp

 /*
  * Copyright (C) 2009 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 1005 matching lines @@
 void GraphicsLayer::setContentsToCanvas(WebLayer* layer)
 {
     setContentsTo(ContentsLayerForCanvas, layer);
 }
 
 void GraphicsLayer::setContentsToMedia(WebLayer* layer)
 {
     setContentsTo(ContentsLayerForVideo, layer);
 }
 
-bool GraphicsLayer::addAnimation(WebAnimation* animation)
+bool GraphicsLayer::addAnimation(PassOwnPtr<WebAnimation> popAnimation)
 {
+    OwnPtr<WebAnimation> animation(popAnimation);
     ASSERT(animation);
     platformLayer()->setAnimationDelegate(this);
 
     // Remove any existing animations with the same animation id and target property.
     platformLayer()->removeAnimation(animation->id(), animation->targetProperty());
-    return platformLayer()->addAnimation(animation);
+#if defined(ANIMATION_OWNERSHIP_NOT_TRANSFER)

[jamesr, 2013/10/15 21:14:05]

there's no need to test for this macro in the Blink side of the patch since you
are setting it here.  Just add the new path (which won't need the intermediate 
OwnPtr<> at all)

[dshwang, 2013/10/16 11:40:05]

I don't think so.
If I changes it to "animation.leakPtr()", Chromium leaks memory until
https://codereview.chromium.org/24165003/ is landed.
Is it accepted?

I'll handle it by following step.
1. land this CL
2. land https://codereview.chromium.org/24165003/
3. remove ANIMATION_OWNERSHIP_NOT_TRANSFER flag in Blink
4. remove ANIMATION_OWNERSHIP_NOT_TRANSFER flag in Chromium

WDYT?

+    return platformLayer()->addAnimation(animation.get());
+#else
+    return platformLayer()->addAnimation(animation.leakPtr());
+#endif
 }
 
 void GraphicsLayer::pauseAnimation(int animationId, double timeOffset)
 {
     platformLayer()->pauseAnimation(animationId, timeOffset);
 }
 
 void GraphicsLayer::removeAnimation(int animationId)
 {
     platformLayer()->removeAnimation(animationId);
@@ skipping 176 matching lines @@
 #ifndef NDEBUG
 void showGraphicsLayerTree(const WebCore::GraphicsLayer* layer)
 {
     if (!layer)
         return;
 
     String output = layer->layerTreeAsText(WebCore::LayerTreeIncludesDebugInfo);
     fprintf(stderr, "%s\n", output.utf8().data());
 }
 #endif