cc: Remove forbidden using namespace directives.

cc/active_animation_unittest.cc

Index: cc/active_animation_unittest.cc
diff --git a/cc/active_animation_unittest.cc b/cc/active_animation_unittest.cc
index f9b8fcf4df2660cef4f21a5fd668c9bac2f10084..b4a340d48cfc4f83140f8c062c9f370d4f9659a6 100644
--- a/cc/active_animation_unittest.cc
+++ b/cc/active_animation_unittest.cc
@@ -11,8 +11,8 @@
 #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)
@@ -218,4 +218,5 @@ TEST(ActiveAnimationTest, RunStateChangesIgnoredWhileSuspended)
     EXPECT_EQ(ActiveAnimation::Running, anim->runState());
 }
 
-}  // anonymous namespace
+}  // namespace
+}  // namespace cc