Allow the computation of inflated bounds for non matrix transform operations.

Allow the computation of inflated bounds for non matrix transform operations.

This CL expands the list of transform operations for which we may compute inflated
bounds for animation.

R=hartmanng@chromium.org,ajuma@chromium.org
BUG=

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=232889

[ajuma, 2013-11-01 21:02:07]

https://codereview.chromium.org/55763004/diff/1/cc/animation/transform_operat...
File cc/animation/transform_operation.cc (right):

https://codereview.chromium.org/55763004/diff/1/cc/animation/transform_operat...
cc/animation/transform_operation.cc:290: }
One thing I find confusing is that we don't need to use |to|'s axis at all after
this point. What am I missing? :)

[Ian Vollick, 2013-11-03 19:05:19]

https://codereview.chromium.org/55763004/diff/1/cc/animation/transform_operat...
File cc/animation/transform_operation.cc (right):

https://codereview.chromium.org/55763004/diff/1/cc/animation/transform_operat...
cc/animation/transform_operation.cc:290: }
On 2013/11/01 21:02:08, ajuma wrote:
> One thing I find confusing is that we don't need to use |to|'s axis at all
after
> this point. What am I missing? :)

Whoops, nothing! I thought that we'd enforced that the rotation operations
shared an axis before we got to this point, but we hadn't. I've added that check
and a unit test to ensure that it works. So in future, it'll be ok to stop
referring to |to|'s axis because it'll be the same as |from|'s.

[ajuma, 2013-11-04 14:22:25]

On 2013/11/03 19:05:19, Ian Vollick wrote:
>
https://codereview.chromium.org/55763004/diff/1/cc/animation/transform_operat...
> File cc/animation/transform_operation.cc (right):
> 
>
https://codereview.chromium.org/55763004/diff/1/cc/animation/transform_operat...
> cc/animation/transform_operation.cc:290: }
> On 2013/11/01 21:02:08, ajuma wrote:
> > One thing I find confusing is that we don't need to use |to|'s axis at all
> after
> > this point. What am I missing? :)
> 
> Whoops, nothing! I thought that we'd enforced that the rotation operations
> shared an axis before we got to this point, but we hadn't. I've added that
check
> and a unit test to ensure that it works. So in future, it'll be ok to stop
> referring to |to|'s axis because it'll be the same as |from|'s.

[ajuma, 2013-11-04 14:22:59]

lgtm

[commit-bot: I haz the power, 2013-11-04 14:25:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/vollick@chromium.org/55763004/140001

[commit-bot: I haz the power, 2013-11-04 14:39:48]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[Ian Vollick, 2013-11-04 15:16:38]

+danakj for OWNERS.

[danakj, 2013-11-04 15:41:18]

lgtm

[commit-bot: I haz the power, 2013-11-04 15:51:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/vollick@chromium.org/55763004/140001

[commit-bot: I haz the power, 2013-11-04 17:45:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/vollick@chromium.org/55763004/140001

[commit-bot: I haz the power, 2013-11-05 02:35:34]

Change committed as 232889