Standardize usage of virtual/override/final in chrome/browser/ui/

chrome/browser/ui/cocoa/extensions/extension_popup_controller.mm

 // Copyright (c) 2012 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.
 
 #import "chrome/browser/ui/cocoa/extensions/extension_popup_controller.h"
 
 #include <algorithm>
 
 #include "base/callback.h"
 #include "chrome/browser/chrome_notification_types.h"
@@ skipping 55 matching lines @@
 - (void)onWindowChanged;
 
 @end
 
 class ExtensionPopupContainer : public ExtensionViewMac::Container {
  public:
   explicit ExtensionPopupContainer(ExtensionPopupController* controller)
       : controller_(controller) {
   }
 
-  virtual void OnExtensionSizeChanged(
-      ExtensionViewMac* view,
-      const gfx::Size& new_size) override {
+  void OnExtensionSizeChanged(ExtensionViewMac* view,
+                              const gfx::Size& new_size) override {
     [controller_ onSizeChanged:
         NSMakeSize(new_size.width(), new_size.height())];
   }
 
-  virtual void OnExtensionViewDidShow(ExtensionViewMac* view) override {
+  void OnExtensionViewDidShow(ExtensionViewMac* view) override {
     [controller_ onViewDidShow];
   }
 
  private:
   ExtensionPopupController* controller_; // Weak; owns this.
 };
 
 class DevtoolsNotificationBridge : public content::NotificationObserver {
  public:
   explicit DevtoolsNotificationBridge(ExtensionPopupController* controller)
     : controller_(controller),
       web_contents_([controller_ extensionViewHost]->host_contents()),
       devtools_callback_(base::Bind(
           &DevtoolsNotificationBridge::OnDevToolsStateChanged,
           base::Unretained(this))) {
     content::DevToolsAgentHost::AddAgentStateCallback(devtools_callback_);
   }
 
-  virtual ~DevtoolsNotificationBridge() {
+  ~DevtoolsNotificationBridge() override {
     content::DevToolsAgentHost::RemoveAgentStateCallback(devtools_callback_);
   }
 
   void OnDevToolsStateChanged(content::DevToolsAgentHost* agent_host,
                               bool attached) {
     if (agent_host->GetWebContents() != web_contents_)
       return;
 
     if (attached) {
       // Set the flag on the controller so the popup is not hidden when
       // the dev tools get focus.
       [controller_ setBeingInspected:YES];
     } else {
       // Allow the devtools to finish detaching before we close the popup.
       [controller_ performSelector:@selector(close)
                         withObject:nil
                         afterDelay:0.0];
     }
   }
 
-  virtual void Observe(
-      int type,
-      const content::NotificationSource& source,
-      const content::NotificationDetails& details) override {
+  void Observe(int type,
+               const content::NotificationSource& source,
+               const content::NotificationDetails& details) override {
     switch (type) {
       case extensions::NOTIFICATION_EXTENSION_HOST_DID_STOP_LOADING: {
         if (content::Details<extensions::ExtensionViewHost>(
                 [controller_ extensionViewHost]) == details) {
           [controller_ showDevTools];
         }
         break;
       }
       default: {
         NOTREACHED() << "Received unexpected notification";
@@ skipping 297 matching lines @@
   return minSize;
 }
 
 // Private (TestingAPI)
 + (NSSize)maxPopupSize {
   NSSize maxSize = {ExtensionViewMac::kMaxWidth, ExtensionViewMac::kMaxHeight};
   return maxSize;
 }
 
 @end