Cleanup duplicated namespace in AnimationTranslationUtil

Cleanup duplicated namespace in AnimationTranslationUtil

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=180044

[kangil_, 2014-08-11 10:26:24]

This is a blink name space clean-up patch. Not an owner lgtm.

[kangil_, 2014-08-11 10:29:44]

On 2014/08/11 10:26:24, kangil_ wrote:
> This is a blink name space clean-up patch. Not an owner lgtm.

oops.. not lgtm

[haraken, 2014-08-11 10:34:53]

https://codereview.chromium.org/453393002/diff/1/Source/core/animation/Animat...
File Source/core/animation/AnimationTranslationUtil.h (right):

https://codereview.chromium.org/453393002/diff/1/Source/core/animation/Animat...
Source/core/animation/AnimationTranslationUtil.h:39: class WebFilterOperations;

Alphabetical order please. I guess this will be caught by the style checker in
'git cl upload'...

[kangil_, 2014-08-11 10:39:22]

@heeyoun: Would you incorporate review comments in next patch set?

https://codereview.chromium.org/453393002/diff/1/Source/core/animation/Animat...
File Source/core/animation/AnimationTranslationUtilTest.cpp (right):

https://codereview.chromium.org/453393002/diff/1/Source/core/animation/Animat...
Source/core/animation/AnimationTranslationUtilTest.cpp:90:
ops.operations().append(TranslateTransformOperation::create(Length(2,
blink::Fixed), Length(0, blink::Fixed), TransformOperation::TranslateX));
s/blink::Fixed/Fixed

[kangil_, 2014-08-11 10:53:38]

Exclude owners till peer review is finished. :)

[kangil_, 2014-08-12 02:46:50]

Not an owner lgtm.
@haraken: Would you please take a look? Thanks!

[haraken, 2014-08-12 03:21:05]

LGTM

[heeyoun.lee, 2014-08-12 04:02:51]

The CQ bit was checked by heeyoun.lee@samsung.com

[commit-bot: I haz the power, 2014-08-12 04:03:42]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/heeyoun.lee@samsung.com/453393002/20001

[commit-bot: I haz the power, 2014-08-12 04:49:52]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/2...)
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/2...)
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/19381)
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/22174)

[commit-bot: I haz the power, 2014-08-12 04:56:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-08-12 04:56:09]

Try jobs failed on following builders:
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/19386)

[heeyoun.lee, 2014-08-12 06:39:33]

The CQ bit was checked by heeyoun.lee@samsung.com

[commit-bot: I haz the power, 2014-08-12 06:40:31]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/heeyoun.lee@samsung.com/453393002/20001

[commit-bot: I haz the power, 2014-08-12 07:45:43]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/22212)

[commit-bot: I haz the power, 2014-08-12 08:31:22]

Change committed as 180044