[InfoBar] Add InfoBarDelegate::GetIconID()

chrome/browser/infobars/infobar_delegate.cc

 // 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.
 
 #include "chrome/browser/infobars/infobar_delegate.h"
 
 #include "base/logging.h"
 #include "build/build_config.h"
 #include "chrome/browser/infobars/infobar_service.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/navigation_details.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/web_contents.h"
+#include "ui/base/resource/resource_bundle.h"
 
 using content::NavigationEntry;
 
 // InfoBarDelegate ------------------------------------------------------------
 
 InfoBarDelegate::~InfoBarDelegate() {
 }
 
 InfoBarDelegate::InfoBarAutomationType
     InfoBarDelegate::GetInfoBarAutomationType() const {
   return UNKNOWN_INFOBAR;
 }
 
 bool InfoBarDelegate::EqualsDelegate(InfoBarDelegate* delegate) const {
   return false;
 }
 
 bool InfoBarDelegate::ShouldExpire(
     const content::LoadCommittedDetails& details) const {
   if (!details.is_navigation_to_different_page())
     return false;
 
   return ShouldExpireInternal(details);
 }
 
 void InfoBarDelegate::InfoBarDismissed() {
 }
 
+int InfoBarDelegate::GetIconID() const {
+  return -1;
+}
+
 gfx::Image* InfoBarDelegate::GetIcon() const {
-  return NULL;
+  if (GetIconID() == -1) {
+    return NULL;
+  } else {

[Peter Kasting, 2013/05/14 00:50:50]

Nit: No else after return.  Do this:

  int icon_id = GetIconID();
  return (icon_id == -1) ?
      NULL : &ResourceBundle::GetSharedInstance().GetNativeImageNamed(icon_id);

[gone, 2013/05/14 17:50:17]

Done.

+    return &ResourceBundle::GetSharedInstance().GetNativeImageNamed(
+        GetIconID());
+  }
 }
 
 InfoBarDelegate::Type InfoBarDelegate::GetInfoBarType() const {
   return WARNING_TYPE;
 }
 
 AutoLoginInfoBarDelegate* InfoBarDelegate::AsAutoLoginInfoBarDelegate() {
   return NULL;
 }
 
@@ skipping 58 matching lines @@
   return (contents_unique_id_ != details.entry->GetUniqueID()) ||
       (content::PageTransitionStripQualifier(
           details.entry->GetTransitionType()) ==
               content::PAGE_TRANSITION_RELOAD);
 }
 
 void InfoBarDelegate::RemoveSelf() {
   if (owner_)
     owner_->RemoveInfoBar(this);  // Clears |owner_|.
 }