[Mac] Make BookmarkBubbleController is-a BaseBubbleController.

[Mac] Make BookmarkBubbleController is-a BaseBubbleController.

XIB changes:
* Update the base class for BookmarkBubbleController to reflect new outlet location.

BUG=116179
TEST=Click the star icon in the omnibox. You can edit bookmarks.
TEST=Covered by unit tests.


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

[Robert Sesek, 2012-03-15 21:08:05]

Last one!

[Nico, 2012-03-15 21:21:34]

lgtm

https://chromiumcodereview.appspot.com/9703084/diff/1/chrome/browser/ui/cocoa...
File chrome/browser/ui/cocoa/bookmarks/bookmark_bubble_controller.mm (right):

https://chromiumcodereview.appspot.com/9703084/diff/1/chrome/browser/ui/cocoa...
chrome/browser/ui/cocoa/bookmarks/bookmark_bubble_controller.mm:177: //
unfortunately, [NSWindowController showWindow:] will also bring it
Is this comment still needed? Maybe just "Override BaseBubbleController's
-showWindow to tweak the arrow location"? (Also in the other files, sorry)

[Robert Sesek, 2012-03-16 15:31:07]

https://chromiumcodereview.appspot.com/9703084/diff/1/chrome/browser/ui/cocoa...
File chrome/browser/ui/cocoa/bookmarks/bookmark_bubble_controller.mm (right):

https://chromiumcodereview.appspot.com/9703084/diff/1/chrome/browser/ui/cocoa...
chrome/browser/ui/cocoa/bookmarks/bookmark_bubble_controller.mm:177: //
unfortunately, [NSWindowController showWindow:] will also bring it
On 2012/03/15 21:21:34, Nico wrote:
> Is this comment still needed? Maybe just "Override BaseBubbleController's
> -showWindow to tweak the arrow location"? (Also in the other files, sorry)

Done.

[commit-bot: I haz the power, 2012-03-16 15:31:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rsesek@chromium.org/9703084/5001

[commit-bot: I haz the power, 2012-03-16 16:19:29]

Try job failure for 9703084-5001 (retry) on mac_rel for step "compile" (clobber
build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2012-03-16 16:22:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rsesek@chromium.org/9703084/5001

[commit-bot: I haz the power, 2012-03-16 18:01:10]

Change committed as 127200