aura: Initialize Widget to avoid a crash when showing bubbles.

aura: Initialize Widget to avoid a crash when showing bubbles.

BUG=97248
TEST=manually

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

[sadrul, 2011-10-27 16:14:51]

http://codereview.chromium.org/8402031/diff/1/chrome/browser/ui/views/bubble/...
File chrome/browser/ui/views/bubble/bubble.cc (right):

http://codereview.chromium.org/8402031/diff/1/chrome/browser/ui/views/bubble/...
chrome/browser/ui/views/bubble/bubble.cc:212: SetOpacity(0);
Are there more things to do here? Should I leave the TODO?

[Ben Goodger (Google), 2011-10-27 16:16:38]

LGTM for wallpapering over the crash.

http://codereview.chromium.org/8402031/diff/1/chrome/browser/ui/views/bubble/...
File chrome/browser/ui/views/bubble/bubble.cc (right):

http://codereview.chromium.org/8402031/diff/1/chrome/browser/ui/views/bubble/...
chrome/browser/ui/views/bubble/bubble.cc:212: SetOpacity(0);
On 2011/10/27 16:14:51, sadrul wrote:
> Are there more things to do here? Should I leave the TODO?

Probably leave the TODO & the notimplemented. alicet/msw are rewriting all the
bubbles to use the new infrastructure.

[sadrul, 2011-10-27 16:29:50]

http://codereview.chromium.org/8402031/diff/1/chrome/browser/ui/views/bubble/...
File chrome/browser/ui/views/bubble/bubble.cc (right):

http://codereview.chromium.org/8402031/diff/1/chrome/browser/ui/views/bubble/...
chrome/browser/ui/views/bubble/bubble.cc:212: SetOpacity(0);
On 2011/10/27 16:16:38, Ben Goodger (Google) wrote:
> On 2011/10/27 16:14:51, sadrul wrote:
> > Are there more things to do here? Should I leave the TODO?
> 
> Probably leave the TODO & the notimplemented. alicet/msw are rewriting all the
> bubbles to use the new infrastructure.

Added back the TODO and NOTIMPLEMENTED, and an explanation NOTE.