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

Unified Diff: Source/platform/text/LineEnding.cpp

Issue 1213003004: Fix virtual/override/final usage in Source/platform/. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 5 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/platform/text/DateTimeFormatTest.cpp ('k') | Source/platform/text/LocaleICU.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/platform/text/LineEnding.cpp
diff --git a/Source/platform/text/LineEnding.cpp b/Source/platform/text/LineEnding.cpp
index a37e63cf1e8a72be6c30f35d1a71c4b1f35d87fa..143b6e29962dc4facb0aa296cc8163c3a56475bd 100644
--- a/Source/platform/text/LineEnding.cpp
+++ b/Source/platform/text/LineEnding.cpp
@@ -50,16 +50,16 @@ public:
: m_buffer(buffer)
{
}
- virtual ~CStringBuffer() { }
+ ~CStringBuffer() override {}
- virtual char* allocate(size_t size) override
+ char* allocate(size_t size) override
{
char* ptr;
m_buffer = CString::newUninitialized(size, ptr);
return ptr;
}
- virtual void copy(const CString& source) override
+ void copy(const CString& source) override
{
m_buffer = source;
}
@@ -76,16 +76,16 @@ public:
: m_buffer(buffer)
{
}
- virtual ~VectorCharAppendBuffer() { }
+ ~VectorCharAppendBuffer() override {}
- virtual char* allocate(size_t size) override
+ char* allocate(size_t size) override
{
size_t oldSize = m_buffer.size();
m_buffer.grow(oldSize + size);
return m_buffer.data() + oldSize;
}
- virtual void copy(const CString& source) override
+ void copy(const CString& source) override
{
m_buffer.append(source.data(), source.length());
}
« no previous file with comments | « Source/platform/text/DateTimeFormatTest.cpp ('k') | Source/platform/text/LocaleICU.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698