Cleanup:...

Cleanup:
* Remove AlertInfoBarDelegate.  There are basically no users of this (the only one is in autofill which is really trying to make something like a confirm info bar).
* Change the default button set for a ConfirmInfoBarDelegate from 0 buttons to 2 to match how practically every caller uses it.

Note that this doesn't remove views' AlertInfoBar yet.  AlertInfoBar is now only used as a base for ConfirmInfoBar, but actually moving that functionality into ConfirmInfoBar is to come in a subsequent patch.  

BUG=none
TEST=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=72726

[Elliot Glaysher, 2011-01-26 20:52:22]

Patch LGTM.

Surprised about everyone reimplementing GetButtons(). Otherwise +1 to
simplification.

http://codereview.chromium.org/6262018/diff/1/chrome/browser/tab_contents/inf...
File chrome/browser/tab_contents/infobar_delegate.cc (right):

http://codereview.chromium.org/6262018/diff/1/chrome/browser/tab_contents/inf...
chrome/browser/tab_contents/infobar_delegate.cc:191: }
Linter wants newline

[Peter Kasting, 2011-01-26 20:55:59]

http://codereview.chromium.org/6262018/diff/1/chrome/browser/tab_contents/inf...
File chrome/browser/tab_contents/infobar_delegate.cc (right):

http://codereview.chromium.org/6262018/diff/1/chrome/browser/tab_contents/inf...
chrome/browser/tab_contents/infobar_delegate.cc:191: }
On 2011/01/26 20:52:22, Elliot Glaysher wrote:
> Linter wants newline

It has a newline.  The linter is wrong about this 98% of the time.  I don't know
why.