Fix unit test style in Source/wtf/.

Source/wtf/HashSetTest.cpp

 /*
  * Copyright (C) 2012 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  *
  * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
  * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
  * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
  * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
  * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
  * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
  * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
  * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
  * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
  * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
  * THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
+#include "wtf/HashSet.h"
 
-#include "wtf/HashSet.h"
 #include "wtf/OwnPtr.h"
 #include "wtf/PassOwnPtr.h"
 #include "wtf/RefCounted.h"
 #include <gtest/gtest.h>
 
-namespace {
+namespace WTF {
 
 template<int initialCapacity>
-    struct InitialCapacityTestHashTraits : public WTF::UnsignedWithZeroKeyHashTraits<int> {
+    struct InitialCapacityTestHashTraits : public UnsignedWithZeroKeyHashTraits<int> {
     static const int minimumTableSize = initialCapacity;
 };
 
 template<unsigned size>
 void testInitialCapacity()
 {
-    const unsigned initialCapacity = WTF::HashTableCapacityForSize<size>::value;
+    const unsigned initialCapacity = HashTableCapacityForSize<size>::value;
     HashSet<int, DefaultHash<int>::Hash, InitialCapacityTestHashTraits<initialCapacity>> testSet;
 
     // Initial capacity is null.
     EXPECT_EQ(0UL, testSet.capacity());
 
     // Adding items up to size should never change the capacity.
     for (size_t i = 0; i < size; ++i) {
         testSet.add(i);
         EXPECT_EQ(initialCapacity, testSet.capacity());
     }
@@ skipping 101 matching lines @@
         ownPtr2 = set.takeAny();
         EXPECT_TRUE(set.isEmpty());
     }
     EXPECT_FALSE(deleted1);
     EXPECT_FALSE(deleted2);
 
     EXPECT_EQ(ptr1, ownPtr1);
     EXPECT_EQ(ptr2, ownPtr2);
 }
 
 class DummyRefCounted: public WTF::RefCounted<DummyRefCounted> {

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

one space after DummyRefCounted.
WTF:: not necessary.

There are 3 DummyRefCounted defined (in HashMapTest, HashSetTest,
ListHashSetTest).
This one is defined in WTF:: namespace, and others are in WTF:{} namespace.
Could they be consistently defined?

[tkent, 2015/06/15 04:45:57]

Done.
Added namespace {} to this file.

 public:
     DummyRefCounted(bool& isDeleted) : m_isDeleted(isDeleted) { m_isDeleted = false; }
     ~DummyRefCounted() { m_isDeleted = true; }
 
     void ref()
     {
         WTF::RefCounted<DummyRefCounted>::ref();
         ++s_refInvokesCount;
     }
 
@@ skipping 24 matching lines @@
 
     ptr.clear();
     EXPECT_FALSE(isDeleted);
 
     set.remove(rawPtr);
     EXPECT_TRUE(isDeleted);
     EXPECT_TRUE(set.isEmpty());
     EXPECT_EQ(1, DummyRefCounted::s_refInvokesCount);
 }
 
-
-} // namespace
+} // namespace WTF