Add InfoBarIdentifier to GroupedPermissionInfoBarDelegate

chrome/browser/permissions/grouped_permission_infobar_delegate.h

Index: chrome/browser/permissions/grouped_permission_infobar_delegate.h
diff --git a/chrome/browser/permissions/grouped_permission_infobar_delegate.h b/chrome/browser/permissions/grouped_permission_infobar_delegate.h
index f0a22ced3e5f36a28108eeb6af278b9dcfd82d44..773ef79842a61d214f8a08aac39690b8ab8b50d4 100644
--- a/chrome/browser/permissions/grouped_permission_infobar_delegate.h
+++ b/chrome/browser/permissions/grouped_permission_infobar_delegate.h
@@ -50,6 +50,7 @@ class GroupedPermissionInfoBarDelegate : public ConfirmInfoBarDelegate {
   // ConfirmInfoBarDelegate:
   base::string16 GetButtonLabel(InfoBarButton button) const override;
   int GetButtons() const override;
+  infobars::InfoBarDelegate::InfoBarIdentifier GetIdentifier() const override;

[Peter Kasting, 2016/09/08 04:06:24]

Nit: Move this function to the top of this group of functions (match vtable
order).  See also next comment.

Is infobars::InfoBarDelegate:: really necessary here?

[lshang, 2016/09/21 07:09:33]

Done.
Done.

 
   // InfoBarDelegate:

[Peter Kasting, 2016/09/08 04:06:24]

Nit: While here: Remove this comment, move the function here up to join the
three above, place above GetIdentifier() (match order in base class)

(Make sure to fix .cc order to match the .h order too)

[lshang, 2016/09/21 07:09:33]

Done.

   Type GetInfoBarType() const override;