Fix unit test style in Source/web/.

Source/web/tests/WebDocumentTest.cpp

 // Copyright 2014 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 "public/web/WebDocument.h"
 
 #include "core/CSSPropertyNames.h"
 #include "core/HTMLNames.h"
 #include "core/dom/NodeComputedStyle.h"
 #include "core/dom/StyleEngine.h"
 #include "core/frame/LocalFrame.h"
 #include "core/html/HTMLElement.h"
 #include "core/html/HTMLLinkElement.h"
 #include "core/style/ComputedStyle.h"
 #include "core/page/Page.h"
 #include "platform/RuntimeEnabledFeatures.h"
 #include "platform/graphics/Color.h"
 #include "platform/testing/URLTestHelpers.h"
 #include "platform/weborigin/SecurityOrigin.h"
 #include "web/tests/FrameTestHelpers.h"
 #include <gtest/gtest.h>
 
-namespace {
+namespace blink {
 
 using blink::FrameTestHelpers::WebViewHelper;
 using blink::URLTestHelpers::toKURL;
-using namespace blink;
 
 const char* kDefaultOrigin = "https://example.test/";
 const char* kManifestDummyFilePath = "manifest-dummy.html";
 
 class WebDocumentTest : public ::testing::Test {
 protected:
     static void SetUpTestCase();
 
     void loadURL(const std::string& url);
     Document* topDocument() const;
@@ skipping 91 matching lines @@
     // Crossorigin set to 'anonymous' shouldn't trigger using credentials.
     linkManifest->setAttribute(HTMLNames::crossoriginAttr, "anonymous");
     ASSERT_FALSE(webDoc.manifestUseCredentials());
 
     // Crossorigin set to 'use-credentials' should trigger using credentials.
     linkManifest->setAttribute(HTMLNames::crossoriginAttr, "use-credentials");
     ASSERT_TRUE(webDoc.manifestUseCredentials());
 }
 
 namespace {
-    const char* baseURLOriginA = "http://example.test:0/";
-    const char* baseURLOriginSubA = "http://subdomain.example.test:0/";
-    const char* baseURLOriginB = "http://not-example.test:0/";
-    const char* emptyFile = "first_party/empty.html";
-    const char* nestedData = "first_party/nested-data.html";
-    const char* nestedOriginA = "first_party/nested-originA.html";
-    const char* nestedOriginSubA = "first_party/nested-originSubA.html";
-    const char* nestedOriginAInOriginA = "first_party/nested-originA-in-originA.html";
-    const char* nestedOriginAInOriginB = "first_party/nested-originA-in-originB.html";
-    const char* nestedOriginB = "first_party/nested-originB.html";
-    const char* nestedOriginBInOriginA = "first_party/nested-originB-in-originA.html";
-    const char* nestedOriginBInOriginB = "first_party/nested-originB-in-originB.html";
-    const char* nestedSrcDoc = "first_party/nested-srcdoc.html";
 
-    static KURL toOriginA(const char* file)
-    {
-        return toKURL(std::string(baseURLOriginA) + file);
-    }
+const char* baseURLOriginA = "http://example.test:0/";
+const char* baseURLOriginSubA = "http://subdomain.example.test:0/";
+const char* baseURLOriginB = "http://not-example.test:0/";
+const char* emptyFile = "first_party/empty.html";
+const char* nestedData = "first_party/nested-data.html";
+const char* nestedOriginA = "first_party/nested-originA.html";
+const char* nestedOriginSubA = "first_party/nested-originSubA.html";
+const char* nestedOriginAInOriginA = "first_party/nested-originA-in-originA.html";
+const char* nestedOriginAInOriginB = "first_party/nested-originA-in-originB.html";
+const char* nestedOriginB = "first_party/nested-originB.html";
+const char* nestedOriginBInOriginA = "first_party/nested-originB-in-originA.html";
+const char* nestedOriginBInOriginB = "first_party/nested-originB-in-originB.html";
+const char* nestedSrcDoc = "first_party/nested-srcdoc.html";
 
-    static KURL toOriginSubA(const char* file)
-    {
-        return toKURL(std::string(baseURLOriginSubA) + file);
-    }
+static KURL toOriginA(const char* file)

[kochi, 2015/06/11 07:34:50]

nit: don't need 'static' in anonymous namespace.

[tkent, 2015/06/11 07:38:35]

Done.

+{
+    return toKURL(std::string(baseURLOriginA) + file);
+}
 
-    static KURL toOriginB(const char* file)
-    {
-        return toKURL(std::string(baseURLOriginB) + file);
-    }
+static KURL toOriginSubA(const char* file)

[kochi, 2015/06/11 07:34:50]

nit: ditto

[tkent, 2015/06/11 07:38:35]

Done.

+{
+    return toKURL(std::string(baseURLOriginSubA) + file);
 }
 
+static KURL toOriginB(const char* file)

[kochi, 2015/06/11 07:34:50]

nit: ditto

[tkent, 2015/06/11 07:38:35]

Done.

+{
+    return toKURL(std::string(baseURLOriginB) + file);
+}
+
+} // anonymous namespace
+
 class WebDocumentFirstPartyTest : public WebDocumentTest {
 public:
     static void SetUpTestCase();
 
 protected:
     void load(const char*);
     Document* nestedDocument() const;
     Document* nestedNestedDocument() const;
 };
 
@@ skipping 108 matching lines @@
 }
 
 TEST_F(WebDocumentFirstPartyTest, NestedData)
 {
     load(nestedData);
 
     ASSERT_EQ(toOriginA(nestedData), topDocument()->firstPartyForCookies());
     ASSERT_EQ(SecurityOrigin::urlWithUniqueSecurityOrigin(), nestedDocument()->firstPartyForCookies());
 }
 
-}
+} // namespace blink