Remove unnecessary 'virtual' annotation from mojo

mojo/services/html_viewer/webthread_impl.h

 // 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.
 
 #ifndef MOJO_SERVICES_HTML_VIEWER_WEBTHREAD_IMPL_H_
 #define MOJO_SERVICES_HTML_VIEWER_WEBTHREAD_IMPL_H_
 
 #include <map>
 
 #include "base/memory/scoped_ptr.h"
 #include "base/threading/thread.h"
 #include "third_party/WebKit/public/platform/WebThread.h"
 
 namespace html_viewer {
 
 class WebThreadBase : public blink::WebThread {
  public:
-  virtual ~WebThreadBase();
+  ~WebThreadBase() override;
 
-  virtual void addTaskObserver(TaskObserver* observer);
-  virtual void removeTaskObserver(TaskObserver* observer);
+  void addTaskObserver(TaskObserver* observer) override;
+  void removeTaskObserver(TaskObserver* observer) override;
 
-  virtual bool isCurrentThread() const = 0;
-  virtual blink::PlatformThreadId threadId() const = 0;
+  bool isCurrentThread() const override = 0;
+  blink::PlatformThreadId threadId() const override = 0;

[sky, 2015/03/31 17:15:57]

I don't think this is necessary, can you remove it?

 
  protected:
   WebThreadBase();
 
  private:
   class TaskObserverAdapter;
 
   typedef std::map<TaskObserver*, TaskObserverAdapter*> TaskObserverMap;
   TaskObserverMap task_observer_map_;
 };
 
 class WebThreadImpl : public WebThreadBase {
  public:
   explicit WebThreadImpl(const char* name);
-  virtual ~WebThreadImpl();
+  ~WebThreadImpl() override;
 
-  virtual void postTask(const blink::WebTraceLocation& location, Task* task);
-  virtual void postDelayedTask(const blink::WebTraceLocation& location,
-                               Task* task,
-                               long long delay_ms);
+  void postTask(const blink::WebTraceLocation& location, Task* task) override;
+  void postDelayedTask(const blink::WebTraceLocation& location,
+                       Task* task,
+                       long long delay_ms) override;
 
-  virtual void enterRunLoop();
-  virtual void exitRunLoop();
+  void enterRunLoop() override;
+  void exitRunLoop() override;
 
   base::MessageLoop* message_loop() const { return thread_->message_loop(); }
 
-  virtual bool isCurrentThread() const;
-  virtual blink::PlatformThreadId threadId() const;
+  bool isCurrentThread() const override;
+  blink::PlatformThreadId threadId() const override;
 
  private:
   scoped_ptr<base::Thread> thread_;
 };
 
 class WebThreadImplForMessageLoop : public WebThreadBase {
  public:
   explicit WebThreadImplForMessageLoop(
       base::MessageLoopProxy* message_loop);
-  virtual ~WebThreadImplForMessageLoop();
+  ~WebThreadImplForMessageLoop() override;
 
-  virtual void postTask(const blink::WebTraceLocation& location, Task* task);
-  virtual void postDelayedTask(const blink::WebTraceLocation& location,
-                               Task* task,
-                               long long delay_ms);
+  void postTask(const blink::WebTraceLocation& location, Task* task) override;
+  void postDelayedTask(const blink::WebTraceLocation& location,
+                       Task* task,
+                       long long delay_ms) override;
 
-  virtual void enterRunLoop();
-  virtual void exitRunLoop();
+  void enterRunLoop() override;
+  void exitRunLoop() override;
 
  private:
-  virtual bool isCurrentThread() const;
-  virtual blink::PlatformThreadId threadId() const;
+  bool isCurrentThread() const override;
+  blink::PlatformThreadId threadId() const override;
 
   scoped_refptr<base::MessageLoopProxy> message_loop_;
   blink::PlatformThreadId thread_id_;
 };
 
 }  // namespace html_viewer
 
 #endif  // MOJO_SERVICES_HTML_VIEWER_WEBTHREAD_IMPL_H_