cc: Remove forbidden using namespace directives.

cc/active_animation_unittest.cc

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 
 #include "cc/active_animation.h"
 
 #include "cc/test/animation_test_common.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using namespace WebKitTests;
-using namespace cc;
 
+namespace cc {
 namespace {

[danakj, 2012/11/10 01:15:43]

That looks like a script mistake? That's not how ui/gfx tests are..

[tfarina, 2012/11/10 01:18:45]

What is the problem? :)

Is perfectly fine to have them in unnamed namespace. And if you look elsewhere
not just ui/gfx/ you will notice that there are unittests only in unnamed
namespace, unittests in global namespace, unittests in specific namespaces, in
both as it's here, etc.

Leave it in the unnamed namespace + the cc namespace made possible to do less
changes.

And, no, I haven't used a script :/

[jamesr, 2012/11/10 01:20:12]

ui/gfx/point_unittest.cc seems similar, except for different newlines.
quad_unittest.cc isn't in an anonymous namespace, but you run the risk of ODR
violation rules in the unit test binary without the namespace

 
 scoped_ptr<ActiveAnimation> createActiveAnimation(int iterations, double duration)
 {
     scoped_ptr<ActiveAnimation> toReturn(ActiveAnimation::create(make_scoped_ptr(new FakeFloatAnimationCurve(duration)).PassAs<AnimationCurve>(), 0, 1, ActiveAnimation::Opacity));
     toReturn->setIterations(iterations);
     return toReturn.Pass();
 }
 
 scoped_ptr<ActiveAnimation> createActiveAnimation(int iterations)
 {
@@ skipping 184 matching lines @@
     scoped_ptr<ActiveAnimation> anim(createActiveAnimation(1));
     anim->suspend(0);
     EXPECT_EQ(ActiveAnimation::Paused, anim->runState());
     anim->setRunState(ActiveAnimation::Running, 0);
     EXPECT_EQ(ActiveAnimation::Paused, anim->runState());
     anim->resume(0);
     anim->setRunState(ActiveAnimation::Running, 0);
     EXPECT_EQ(ActiveAnimation::Running, anim->runState());
 }
 
-}  // anonymous namespace
+}  // namespace
+}  // namespace cc