Support Windows native textfield, combobox, etc. in new bubbles.

views/examples/bubble_example.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 "views/examples/bubble_example.h"
 
 #include "base/utf_string_conversions.h"
 #include "views/bubble/bubble_delegate.h"
 #include "views/controls/button/text_button.h"
 #include "views/controls/label.h"
@@ skipping 72 matching lines @@
     config = kFadeOutConfig;
 
   config.anchor_point.set_x(sender->width() / 2);
   config.anchor_point.set_y(sender->height() / 2);
   views::View::ConvertPointToScreen(sender, &config.anchor_point);
 
   ExampleBubbleDelegateView* bubble_delegate =
       new ExampleBubbleDelegateView(config);
   views::Widget* bubble = views::BubbleDelegateView::CreateBubble(
       bubble_delegate, example_view()->GetWidget());
+  DCHECK(bubble);

[Ben Goodger (Google), 2011/10/31 17:43:02]

Looking at your code, CreateBubble can never return NULL, so why bother?

[msw, 2011/10/31 17:47:37]

To silence a clang error for the otherwise unused variable :)

[Ben Goodger (Google), 2011/10/31 17:48:29]

Why not nix the variable then?

[msw, 2011/10/31 17:56:33]

Done. I thought it might help fill out the example, but yeah, it's useless here.

 
   if (config.fade_in)
     bubble_delegate->StartFade(true);
   else
-    bubble->Show();
+    bubble_delegate->Show();
 
   if (config.fade_out) {
     bubble_delegate->set_close_on_esc(false);
     bubble_delegate->StartFade(false);
   }
 }
 
 }  // namespace examples