FBTF: Move virtual methods to implementation files.

chrome/browser/extensions/extension_infobar_delegate.cc

 // Copyright (c) 2010 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 "chrome/browser/extensions/extension_infobar_delegate.h"
 
 #include "chrome/browser/browser.h"
 #include "chrome/browser/profile.h"
 #include "chrome/browser/extensions/extension_host.h"
 #include "chrome/browser/extensions/extension_process_manager.h"
@@ skipping 40 matching lines @@
 
   // Only allow one InfoBar at a time per extension.
   return extension_delegate->extension_host()->extension() ==
          extension_host_->extension();
 }
 
 void ExtensionInfoBarDelegate::InfoBarClosed() {
   delete this;
 }
 
+ExtensionInfoBarDelegate* ExtensionInfoBarDelegate::
+AsExtensionInfoBarDelegate() {

[Lei Zhang, 2010/09/23 22:05:41]

style-wise, is this the better way to wrap really long method names or is it
better to write:

ExtensionInfoBarDelegate*
ExtensionInfoBarDelegate::AsExtensionInfoBarDelegate()

?

+  return this;
+}
+
+InfoBarDelegate::Type ExtensionInfoBarDelegate::GetInfoBarType() {
+  return PAGE_ACTION_TYPE;
+}
+
 void ExtensionInfoBarDelegate::Observe(NotificationType type,
                                        const NotificationSource& source,
                                        const NotificationDetails& details) {
   switch (type.value) {
     case NotificationType::EXTENSION_HOST_VIEW_SHOULD_CLOSE: {
       const ExtensionHost* result = Details<ExtensionHost>(details).ptr();
       if (extension_host_.get() == result)
         tab_contents_->RemoveInfoBar(this);
       break;
     }
     case NotificationType::EXTENSION_UNLOADED: {
       Extension* extension = Details<Extension>(details).ptr();
       if (extension_ == extension)
         tab_contents_->RemoveInfoBar(this);
       break;
     }
     default: {
       NOTREACHED() << "Unknown message";
       break;
     }
   }
 }