Fix unit test style in Source/web/.

Source/web/ExternalPopupMenuTest.cpp

 // Copyright (c) 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 "web/ExternalPopupMenu.h"
 
 #include "core/HTMLNames.h"
 #include "core/frame/FrameHost.h"
 #include "core/frame/PinchViewport.h"
 #include "core/html/HTMLSelectElement.h"
 #include "core/html/forms/PopupMenuClient.h"
 #include "core/layout/LayoutMenuList.h"
 #include "core/page/Page.h"
 #include "core/testing/DummyPageHolder.h"
 #include "platform/PopupMenu.h"
 #include "platform/testing/URLTestHelpers.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebUnitTestSupport.h"
 #include "public/web/WebExternalPopupMenu.h"
 #include "public/web/WebPopupMenuInfo.h"
 #include "public/web/WebSettings.h"
 #include "web/WebLocalFrameImpl.h"
 #include "web/tests/FrameTestHelpers.h"
 #include <gtest/gtest.h>
 
-using namespace blink;
-
-namespace {
+namespace blink {
 
 const size_t kListSize = 7;
 
 class TestPopupMenuClient : public PopupMenuClient {
 public:
     TestPopupMenuClient() : m_listSize(0) { }
-    virtual ~TestPopupMenuClient() { }
+    ~TestPopupMenuClient() override { }
 
-    virtual void valueChanged(unsigned listIndex, bool fireEvents = true) override { }
-    virtual void selectionChanged(unsigned listIndex, bool fireEvents = true) override { }
-    virtual void selectionCleared() override { }
+    void valueChanged(unsigned listIndex, bool fireEvents = true) override { }
+    void selectionChanged(unsigned listIndex, bool fireEvents = true) override { }
+    void selectionCleared() override { }
 
-    virtual String itemText(unsigned listIndex) const override { return emptyString(); }
-    virtual String itemToolTip(unsigned listIndex) const override { return emptyString(); }
-    virtual String itemAccessibilityText(unsigned listIndex) const override { return emptyString(); }
-    virtual bool itemIsEnabled(unsigned listIndex) const override { return true; }
-    virtual PopupMenuStyle itemStyle(unsigned listIndex) const override
+    String itemText(unsigned listIndex) const override { return emptyString(); }
+    String itemToolTip(unsigned listIndex) const override { return emptyString(); }
+    String itemAccessibilityText(unsigned listIndex) const override { return emptyString(); }
+    bool itemIsEnabled(unsigned listIndex) const override { return true; }
+    PopupMenuStyle itemStyle(unsigned listIndex) const override
     {
         FontDescription fontDescription;
         fontDescription.setComputedSize(12.0);
         Font font(fontDescription);
         font.update(nullptr);
         bool displayNone = m_displayNoneIndexSet.find(listIndex) != m_displayNoneIndexSet.end();
         return PopupMenuStyle(Color::black, Color::white, font, true, displayNone, Length(), TextDirection(), false);
     }
-    virtual PopupMenuStyle menuStyle() const override { return itemStyle(0); }
-    virtual LayoutUnit clientPaddingLeft() const override { return 0; }
-    virtual LayoutUnit clientPaddingRight() const override { return 0; }
-    virtual int listSize() const override { return m_listSize; }
-    virtual int selectedIndex() const override { return 0; }
-    virtual void popupDidHide() override { }
-    virtual void popupDidCancel() override { }
-    virtual bool itemIsSeparator(unsigned listIndex) const override { return false;}
-    virtual bool itemIsLabel(unsigned listIndex) const override { return false; }
-    virtual bool itemIsSelected(unsigned listIndex) const override { return listIndex == 0;}
-    virtual void provisionalSelectionChanged(unsigned listIndex) override { }
-    virtual bool multiple() const override { return false; }
-    virtual IntRect elementRectRelativeToViewport() const override { return IntRect(); }
-    virtual Element& ownerElement() const override { return *m_ownerElement; }
-    virtual const ComputedStyle* computedStyleForItem(Element& element) const override { return nullptr; }
+    PopupMenuStyle menuStyle() const override { return itemStyle(0); }
+    LayoutUnit clientPaddingLeft() const override { return 0; }
+    LayoutUnit clientPaddingRight() const override { return 0; }
+    int listSize() const override { return m_listSize; }
+    int selectedIndex() const override { return 0; }
+    void popupDidHide() override { }
+    void popupDidCancel() override { }
+    bool itemIsSeparator(unsigned listIndex) const override { return false;}
+    bool itemIsLabel(unsigned listIndex) const override { return false; }
+    bool itemIsSelected(unsigned listIndex) const override { return listIndex == 0;}
+    void provisionalSelectionChanged(unsigned listIndex) override { }
+    bool multiple() const override { return false; }
+    IntRect elementRectRelativeToViewport() const override { return IntRect(); }
+    Element& ownerElement() const override { return *m_ownerElement; }
+    const ComputedStyle* computedStyleForItem(Element& element) const override { return nullptr; }
 
     void setListSize(size_t size) { m_listSize = size; }
     void setDisplayNoneIndex(unsigned index) { m_displayNoneIndexSet.insert(index); }
     void setOwnerElement(PassRefPtrWillBeRawPtr<Element> element) { m_ownerElement = element; }
+
 private:
     size_t m_listSize;
     std::set<unsigned> m_displayNoneIndexSet;
     RefPtrWillBePersistent<Element> m_ownerElement;
 };
 
 class ExternalPopupMenuDisplayNoneItemsTest : public testing::Test {
 public:
     ExternalPopupMenuDisplayNoneItemsTest() { }
 
@@ skipping 61 matching lines @@
     };
     WebRect m_shownBounds;
     MockWebExternalPopupMenu m_mockWebExternalPopupMenu;
 };
 
 class ExternalPopupMenuTest : public testing::Test {
 public:
     ExternalPopupMenuTest() : m_baseURL("http://www.test.com") { }
 
 protected:
-    virtual void SetUp() override
+    void SetUp() override
     {
         m_helper.initialize(false, &m_webFrameClient, &m_webViewClient);
         webView()->setUseExternalPopupMenus(true);
     }
-    virtual void TearDown() override
+    void TearDown() override
     {
         Platform::current()->unitTestSupport()->unregisterAllMockedURLs();
     }
 
     void registerMockedURLLoad(const std::string& fileName)
     {
         URLTestHelpers::registerMockedURLLoad(URLTestHelpers::toKURL(m_baseURL + fileName), WebString::fromUTF8(fileName.c_str()), WebString::fromUTF8("popup/"), WebString::fromUTF8("text/html"));
     }
 
     void loadFrame(const std::string& fileName)
@@ skipping 90 matching lines @@
     menuList->showPopup();
     ASSERT_TRUE(menuList->popupIsVisible());
 
     WebExternalPopupMenuClient* client = static_cast<ExternalPopupMenu*>(menuList->popup());
     WebVector<int> indices;
     client->didAcceptIndices(indices);
     EXPECT_FALSE(menuList->popupIsVisible());
     EXPECT_EQ(-1, select->selectedIndex());
 }
 
-} // namespace
+} // namespace blink