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

Unified Diff: Source/core/inspector/DevToolsHost.cpp

Issue 1232333002: Fix virtual/override/final usage in the rest of Source/core/. (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/core/inspector/DOMEditor.cpp ('k') | Source/core/inspector/InspectorAnimationAgent.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/inspector/DevToolsHost.cpp
diff --git a/Source/core/inspector/DevToolsHost.cpp b/Source/core/inspector/DevToolsHost.cpp
index cbac64de93792e2e4344f36e8afdfb5b907af107..6753becbd2670233165c68921a3870b813f6f334 100644
--- a/Source/core/inspector/DevToolsHost.cpp
+++ b/Source/core/inspector/DevToolsHost.cpp
@@ -63,7 +63,7 @@ public:
return adoptRefWillBeNoop(new FrontendMenuProvider(devtoolsHost, devtoolsApiObject, items));
}
- virtual ~FrontendMenuProvider()
+ ~FrontendMenuProvider() override
{
// Verify that this menu provider has been detached.
ASSERT(!m_devtoolsHost);
@@ -81,7 +81,7 @@ public:
m_devtoolsHost = nullptr;
}
- virtual void contextMenuCleared() override
+ void contextMenuCleared() override
{
if (m_devtoolsHost) {
ScriptFunctionCall function(m_devtoolsApiObject, "contextMenuCleared");
@@ -93,13 +93,13 @@ public:
m_items.clear();
}
- virtual void populateContextMenu(ContextMenu* menu) override
+ void populateContextMenu(ContextMenu* menu) override
{
for (size_t i = 0; i < m_items.size(); ++i)
menu->appendItem(m_items[i]);
}
- virtual void contextMenuItemSelected(const ContextMenuItem* item) override
+ void contextMenuItemSelected(const ContextMenuItem* item) override
{
if (!m_devtoolsHost)
return;
« no previous file with comments | « Source/core/inspector/DOMEditor.cpp ('k') | Source/core/inspector/InspectorAnimationAgent.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698