Minor infobar cleanup.

chrome/browser/content_settings/permission_infobar_delegate.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 CHROME_BROWSER_CONTENT_SETTINGS_PERMISSION_INFOBAR_DELEGATE_H_
 #define CHROME_BROWSER_CONTENT_SETTINGS_PERMISSION_INFOBAR_DELEGATE_H_
 
 #include "chrome/browser/infobars/infobar_service.h"
 #include "components/content_settings/core/common/content_settings_types.h"
 #include "components/content_settings/core/common/permission_request_id.h"
 #include "components/infobars/core/confirm_infobar_delegate.h"
 #include "content/public/browser/web_contents.h"
 
 class NavigationDetails;
 class PermissionQueueController;
 
 // Base class for permission infobars, it implements the default behavior
 // so that the accept/deny buttons grant/deny the relevant permission.
 // A basic implementor only needs to implement the methods that
 // provide an icon and a message text to the infobar.
 class PermissionInfobarDelegate : public ConfirmInfoBarDelegate {
 
  protected:
   PermissionInfobarDelegate(PermissionQueueController* controller,
                             const PermissionRequestID& id,
                             const GURL& requesting_origin,
                             ContentSettingsType type);
   ~PermissionInfobarDelegate() override;
 
+ private:
   // ConfirmInfoBarDelegate:
-  base::string16 GetMessageText() const override = 0;

[Peter Kasting, 2015/02/13 03:22:26]

This is already pure in the base class, so there's no need to redeclare it here,
the compiler will still enforce that subclasses override this.

-
-  infobars::InfoBarDelegate::Type GetInfoBarType() const override;
+  void InfoBarDismissed() override;
+  Type GetInfoBarType() const override;
   base::string16 GetButtonLabel(InfoBarButton button) const override;
-
-  // Remember to call RegisterActionTaken for these methods if you are
-  // overriding them.

[Peter Kasting, 2015/02/13 03:22:26]

No one overrides these, so no need for a comment.

-  void InfoBarDismissed() override;
   bool Accept() override;
   bool Cancel() override;
 
- private:
   void SetPermission(bool update_content_setting, bool allowed);
 
   PermissionQueueController* controller_; // not owned by us
   const PermissionRequestID id_;
   GURL requesting_origin_;
   bool action_taken_;
   ContentSettingsType type_;
 
   DISALLOW_COPY_AND_ASSIGN(PermissionInfobarDelegate);
 };
 
 #endif  // CHROME_BROWSER_CONTENT_SETTINGS_PERMISSION_INFOBAR_DELEGATE_H_