Minor infobar cleanup.

chrome/browser/content_settings/permission_infobar_delegate.h

Index: chrome/browser/content_settings/permission_infobar_delegate.h
diff --git a/chrome/browser/content_settings/permission_infobar_delegate.h b/chrome/browser/content_settings/permission_infobar_delegate.h
index 1789058b3a2b56f9030a6623d82116b574ecfc76..dee1d38504616c2a25814730cb4c9cbaf177d6f0 100644
--- a/chrome/browser/content_settings/permission_infobar_delegate.h
+++ b/chrome/browser/content_settings/permission_infobar_delegate.h
@@ -27,19 +27,14 @@ class PermissionInfobarDelegate : public ConfirmInfoBarDelegate {
                             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;
-  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;
+  Type GetInfoBarType() const override;
+  base::string16 GetButtonLabel(InfoBarButton button) const override;
   bool Accept() override;
   bool Cancel() override;
 
- private:
   void SetPermission(bool update_content_setting, bool allowed);
 
   PermissionQueueController* controller_; // not owned by us