Make TabContents own its infobar delegates....

chrome/browser/tab_contents/tab_contents.h

 // 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.
 
 #ifndef CHROME_BROWSER_TAB_CONTENTS_TAB_CONTENTS_H_
 #define CHROME_BROWSER_TAB_CONTENTS_TAB_CONTENTS_H_
 #pragma once
 
 #include "build/build_config.h"
 
@@ skipping 440 matching lines @@
   // the page contents. The view calls this function when the tab is focused
   // to see what it should do.
   virtual bool FocusLocationBarByDefault();
 
   // Focuses the location bar.
   virtual void SetFocusToLocationBar(bool select_all);
 
   // Infobars ------------------------------------------------------------------
 
   // Adds an InfoBar for the specified |delegate|.
   virtual void AddInfoBar(InfoBarDelegate* delegate);

[Ben Goodger (Google), 2010/11/10 21:30:59]

Can you add a note here about ownership/lifetime now that you've made it
consistent?

 
   // Removes the InfoBar for the specified |delegate|.
   void RemoveInfoBar(InfoBarDelegate* delegate);
 
   // Replaces one infobar with another, without any animation in between.
   void ReplaceInfoBar(InfoBarDelegate* old_delegate,
                       InfoBarDelegate* new_delegate);
 
   // Enumeration and access functions.
   int infobar_delegate_count() const { return infobar_delegates_.size(); }
   InfoBarDelegate* GetInfoBarDelegateAt(int index) {
-    return infobar_delegates_.at(index);
+    return infobar_delegates_[index];

[tfarina, 2010/11/10 13:11:57]

hum? Why? What is the benefit?

   }
 
   // Toolbars and such ---------------------------------------------------------
 
   // Returns true if a Bookmark Bar should be shown for this tab.
   virtual bool ShouldShowBookmarkBar();
 
   // Notifies the delegate that a download is about to be started.
   // This notification is fired before a local temporary file has been created.
   bool CanDownload(int request_id);
@@ skipping 700 matching lines @@
 
   // TODO(pkasting): Hack to try and fix Linux browser tests.
   bool dont_notify_render_view_;
 
   // True if this is a secure page which displayed insecure content.
   bool displayed_insecure_content_;
 
   // Data for shelves and stuff ------------------------------------------------
 
   // Delegates for InfoBars associated with this TabContents.
-  std::vector<InfoBarDelegate*> infobar_delegates_;
+  typedef std::vector<InfoBarDelegate*> InfoBarDelegates;
+  InfoBarDelegates infobar_delegates_;
 
   // Data for find in page -----------------------------------------------------
 
   // TODO(brettw) this should be separated into a helper class.
 
   // Each time a search request comes in we assign it an id before passing it
   // over the IPC so that when the results come in we can evaluate whether we
   // still care about the results of the search (in some cases we don't because
   // the user has issued a new search).
   static int find_request_id_counter_;
@@ skipping 110 matching lines @@
   // Content restrictions, used to disable print/copy etc based on content's
   // (full-page plugins for now only) permissions.
   int content_restrictions_;
 
   // ---------------------------------------------------------------------------
 
   DISALLOW_COPY_AND_ASSIGN(TabContents);
 };
 
 #endif  // CHROME_BROWSER_TAB_CONTENTS_TAB_CONTENTS_H_