Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1881)

Unified Diff: Source/web/tests/ScrollAnimatorNoneTest.cpp

Issue 1174283002: Fix unit test style in Source/web/. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: remove static Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/web/tests/ProgrammaticScrollTest.cpp ('k') | Source/web/tests/ScrollingCoordinatorTest.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/web/tests/ScrollAnimatorNoneTest.cpp
diff --git a/Source/web/tests/ScrollAnimatorNoneTest.cpp b/Source/web/tests/ScrollAnimatorNoneTest.cpp
index 75708a97e5aba8f97694f81d525905ed1bd573fe..30a3dab82004517db5e5d44b91117ba0deb8d39d 100644
--- a/Source/web/tests/ScrollAnimatorNoneTest.cpp
+++ b/Source/web/tests/ScrollAnimatorNoneTest.cpp
@@ -26,7 +26,6 @@
// Tests for the ScrollAnimatorNone class.
#include "config.h"
-
#include "platform/scroll/ScrollAnimatorNone.h"
#include "platform/Logging.h"
@@ -37,7 +36,7 @@
#include <gmock/gmock.h>
#include <gtest/gtest.h>
-using namespace blink;
+namespace blink {
using testing::AtLeast;
using testing::Return;
@@ -216,12 +215,12 @@ public:
{
}
- virtual void SetUp()
+ void SetUp() override
{
m_currentPosition = 100;
m_data = new ScrollAnimatorNone::PerAxisData(&m_currentPosition, 768);
}
- virtual void TearDown()
+ void TearDown() override
{
delete m_data;
}
@@ -1054,3 +1053,5 @@ TEST_F(ScrollAnimatorNoneTest, ReverseInMiddle)
result = result && animateScroll(t);
EXPECT_GE(before, m_currentPosition);
}
+
+} // namespace blink
« no previous file with comments | « Source/web/tests/ProgrammaticScrollTest.cpp ('k') | Source/web/tests/ScrollingCoordinatorTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698