Change WebLayer::addAnimation to transfer ownership of WebAnimation.

public/platform/WebLayer.h

 /*
  * Copyright (C) 2011 Google 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
@@ skipping 125 matching lines @@
     // Provides a bitfield that describe why this composited layer was created.
     virtual void setCompositingReasons(WebCompositingReasons) = 0;
 
     // An animation delegate is notified when animations are started and
     // stopped. The WebLayer does not take ownership of the delegate, and it is
     // the responsibility of the client to reset the layer's delegate before
     // deleting the delegate.
     virtual void setAnimationDelegate(WebAnimationDelegate*) = 0;
 
     // Returns false if the animation cannot be added.
+    // After it, the given WebAnimation object is invalid because it passes
+    // the ownership of internal structure of WebAnimation to Chromium.

[jamesr, 2013/09/20 18:52:14]

I would simplify this to "The WebLayer takes ownership of the WebAnimation"

[dshwang, 2013/09/20 19:03:58]

It is because now we does pass ownership of the WebAnimation. we pass just
ownership of cc::Animation that is member of WebAnimation.

     virtual bool addAnimation(WebAnimation*) = 0;
 
     // Removes all animations with the given id.
     virtual void removeAnimation(int animationId) = 0;
 
     // Removes all animations with the given id targeting the given property.
     virtual void removeAnimation(int animationId, WebAnimation::TargetProperty) = 0;
 
     // Pauses all animations with the given id.
     virtual void pauseAnimation(int animationId, double timeOffset) = 0;
@@ skipping 59 matching lines @@
 
     // True if the layer is not part of a tree attached to a WebLayerTreeView.
     virtual bool isOrphan() const = 0;
 
     virtual void setWebLayerClient(WebLayerClient*) = 0;
 };
 
 } // namespace WebKit
 
 #endif // WebLayer_h