Standardize usage of virtual/override/final specifiers.

ash/first_run/desktop_cleaner.cc

 // Copyright 2013 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 "ash/first_run/desktop_cleaner.h"
 
 #include "ash/shell.h"
 #include "ash/shell_window_ids.h"
 #include "ui/aura/window_event_dispatcher.h"
 #include "ui/aura/window_observer.h"
@@ skipping 23 matching lines @@
       : container_was_hidden_(!container->IsVisible()),
         container_(container) {
     if (container_was_hidden_)
       return;
     ui::Layer* layer = container_->layer();
     ui::ScopedLayerAnimationSettings animation_settings(layer->GetAnimator());
     animation_settings.AddObserver(this);
     layer->SetOpacity(0.0);
   }
 
-  virtual ~ContainerHider() {
+  ~ContainerHider() override {
     if (container_was_hidden_ || !container_)
       return;
     if (!WasAnimationCompletedForProperty(ui::LayerAnimationElement::OPACITY)) {
       // We are in the middle of animation.
       StopObservingImplicitAnimations();
     } else {
       container_->Show();
     }
     ui::Layer* layer = container_->layer();
     ui::ScopedLayerAnimationSettings animation_settings(layer->GetAnimator());
     layer->SetOpacity(1.0);
   }
 
  private:
   // Overriden from ui::ImplicitAnimationObserver.
-  virtual void OnImplicitAnimationsCompleted() override {
-    container_->Hide();
-  }
+  void OnImplicitAnimationsCompleted() override { container_->Hide(); }
 
   // Overriden from aura::WindowObserver.
-  virtual void OnWindowDestroying(aura::Window* window) override {
+  void OnWindowDestroying(aura::Window* window) override {
     DCHECK(window == container_);
     container_ = NULL;
   }
 
   const bool container_was_hidden_;
   aura::Window* container_;
 
   DISALLOW_COPY_AND_ASSIGN(ContainerHider);
 };
 
 class NotificationBlocker : public message_center::NotificationBlocker {
  public:
   NotificationBlocker()
       : message_center::NotificationBlocker(
             message_center::MessageCenter::Get()) {
     NotifyBlockingStateChanged();
   }
 
-  virtual ~NotificationBlocker() {};
+  ~NotificationBlocker() override{};

[Daniel Erat, 2014/10/27 21:55:49]

i feel like the following, with a space between 'override' and '{' and no
trailing semicolon, is more common both in chrome and even just within this
change:

  ~NotificationBlocker() override {}

[dcheng, 2014/10/27 22:03:41]

I think this is a misformatting by clang-format. Thanks for catching this. I bet
the semicolon confuses it...

 
  private:
   // Overriden from message_center::NotificationBlocker.
-  virtual bool ShouldShowNotificationAsPopup(
+  bool ShouldShowNotificationAsPopup(
       const message_center::NotifierId& notifier_id) const override {
     return false;
   }
 
   DISALLOW_COPY_AND_ASSIGN(NotificationBlocker);
 };
 
 DesktopCleaner::DesktopCleaner() {
   // TODO(dzhioev): Add support for secondary displays.
   aura::Window* root_window = Shell::GetInstance()->GetPrimaryRootWindow();
   for (size_t i = 0; i < arraysize(kContainerIdsToHide); ++i) {
     aura::Window* container =
         Shell::GetContainer(root_window, kContainerIdsToHide[i]);
     container_hiders_.push_back(make_linked_ptr(new ContainerHider(container)));
   }
   notification_blocker_.reset(new NotificationBlocker());
 }
 
 DesktopCleaner::~DesktopCleaner() {}
 
 // static
 std::vector<int> DesktopCleaner::GetContainersToHideForTest() {
   return std::vector<int>(kContainerIdsToHide,
                           kContainerIdsToHide + arraysize(kContainerIdsToHide));
 }
 
 }  // namespace ash