Mac: Make bubble windows (not their controller) responsible for calling removeChildWindow

Mac: Make bubble windows (not their controller) responsible for calling removeChildWindow

InfoBubbleWindow overrides -[NSWindow close] to schedule a fade
animation instead of closing. Currently, BaseBubbleController calls
this, but first removes the bubble from its parent NSWindow. This means
that if the close is due to a click on a browser window, the
parent->child relationship is already gone, and the clicked window is
raised on top of the bubble, making it seem to disappear immediately
rather than fade out.

This changes the flow so that InfoBubbleWindow only removes itself as a
child before calling [super close]; once the animation is complete.

BUG=459881
Committed: https://crrev.com/c780b64584b33eb0f045fbda5c9b4c2ccd4c8815
Cr-Commit-Position: refs/heads/master@{#317962}

[tapted, 2015-02-24 05:45:02]

tapted@chromium.org changed reviewers:
+ rsesek@chromium.org

[tapted, 2015-02-24 22:23:49]

whoops - forgot to click publish. robert - please take a look. (no rush) Thanks!

[Robert Sesek, 2015-02-25 01:38:07]

LGTM

[tapted, 2015-02-25 01:57:39]

The CQ bit was checked by tapted@chromium.org

[commit-bot: I haz the power, 2015-02-25 01:58:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/944533004/1

[commit-bot: I haz the power, 2015-02-25 03:10:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-02-25 03:10:34]

Patchset 1 (id:??) landed as
https://crrev.com/c780b64584b33eb0f045fbda5c9b4c2ccd4c8815
Cr-Commit-Position: refs/heads/master@{#317962}