Fix unit test style in Source/wtf/.

Source/wtf/DequeTest.cpp

Index: Source/wtf/DequeTest.cpp
diff --git a/Source/wtf/DequeTest.cpp b/Source/wtf/DequeTest.cpp
index 3a1b8c82c7eb4e657cb6fb0c2d6c8d03b393643c..6452173bc789601e19b07e043e9dafe61a4582eb 100644
--- a/Source/wtf/DequeTest.cpp
+++ b/Source/wtf/DequeTest.cpp
@@ -24,7 +24,6 @@
  */
 
 #include "config.h"
-
 #include "wtf/Deque.h"
 
 #include "wtf/HashSet.h"
@@ -32,6 +31,8 @@
 #include "wtf/PassOwnPtr.h"
 #include <gtest/gtest.h>
 
+namespace WTF {
+
 namespace {

[kochi, 2015/06/15 03:34:22]

Is this anonymous namespace necessary?

It seems anonymous namespace within namespace WTF is inconsistently
used in these tests in this CL.

Unless the name conflicts with symbols defined in other files,
technically it's fine to go without anonymous namespace,
but for consistency, do we enforce use of anonymous namespace
for file-local class/function/typedef etc. (i.e. anything without
test body)?

[tkent, 2015/06/15 03:49:23]

Yes, this CL keeps anonymous namespaces only if they are necessary.  I saw some
ODR violations without anonymous namespaces.
Any local helpers should be wrapped with anonymous namespace ideally.  However,
I don't intend to do it by myself at this moment.

[kochi, 2015/06/15 04:02:39]

Okay, then.

 
 TEST(DequeTest, Basic)
@@ -313,4 +314,6 @@ TEST(DequeTest, SwapWithoutInlineCapacity)
     dequeB.swap(dequeA);
 }
 
-} // namespace
+} // anonymous namespace
+
+} // namespace WTF