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

chrome/browser/ui/views/bubble/bubble.cc

 // Copyright (c) 2011 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.
 
 #include "chrome/browser/ui/views/bubble/bubble.h"
 
 #include <vector>
 
 #include "chrome/browser/ui/views/bubble/border_contents.h"
 #include "chrome/common/chrome_notification_types.h"
@@ skipping 185 matching lines @@
                         views::View* contents,
                         BubbleDelegate* delegate) {
   delegate_ = delegate;
   position_relative_to_ = position_relative_to;
   arrow_location_ = arrow_location;
   contents_ = contents;
   const bool fade_in = delegate_ && delegate_->FadeInOnShow();
 
   // Create the main window.
 #if defined(USE_AURA)
-  // TODO(beng):
-  NOTIMPLEMENTED();
+  views::Widget::InitParams params;
+  params.transparent = true;
+  params.parent_widget = parent;
+  params.native_widget = this;
+  GetWidget()->Init(params);
+  if (fade_in)
+    SetOpacity(0);

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

Are there more things to do here? Should I leave the TODO?

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

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]

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

 #elif defined(OS_WIN)
   views::Widget* parent_window = parent->GetTopLevelWidget();
   if (parent_window)
     parent_window->DisableInactiveRendering();
   set_window_style(WS_POPUP | WS_CLIPCHILDREN);
   int extended_style = WS_EX_TOOLWINDOW;
   // During FadeIn we need to turn on the layered window style to deal with
   // transparency. This flag needs to be reset after fading in is complete.
   if (fade_in)
     extended_style |= WS_EX_LAYERED;
@@ skipping 82 matching lines @@
   // |contents_view| has no layout manager, so we have to explicitly position
   // its children.
   border_contents_->SetBoundsRect(
       gfx::Rect(gfx::Point(), window_bounds.size()));
   contents->SetBoundsRect(contents_bounds);
 #endif
   GetWidget()->SetBounds(window_bounds);
 
   // Show the window.
 #if defined(USE_AURA)
-  // TODO(beng):
-  NOTIMPLEMENTED();
+  GetWidget()->Show();
 #elif defined(OS_WIN)
   border_->ShowWindow(SW_SHOW);
   ShowWindow(SW_SHOW);
 #elif defined(TOOLKIT_USES_GTK)
   GetWidget()->Show();
 #endif
 
   if (fade_in)
     FadeIn();
 }
@@ skipping 124 matching lines @@
     animation_->Hide();
 }
 
 bool Bubble::AcceleratorPressed(const views::Accelerator& accelerator) {
   if (!delegate_ || delegate_->CloseOnEscape()) {
     DoClose(true);
     return true;
   }
   return false;
 }