Converts some of the omnibox related classes to support multiple

chrome/browser/ui/views/frame/browser_view.cc

Index: chrome/browser/ui/views/frame/browser_view.cc
diff --git a/chrome/browser/ui/views/frame/browser_view.cc b/chrome/browser/ui/views/frame/browser_view.cc
index d2e936451a3a18e5e226bdeae2a94e39ddef1e3e..2bfb0b297def1be8ce9f0064f69a6f21e27b87fa 100644
--- a/chrome/browser/ui/views/frame/browser_view.cc
+++ b/chrome/browser/ui/views/frame/browser_view.cc
@@ -1863,7 +1863,7 @@ void BrowserView::PaintChildren(gfx::Canvas* canvas) {
 void BrowserView::ViewHierarchyChanged(bool is_add,
                                        views::View* parent,
                                        views::View* child) {
-  if (is_add && child == this && GetWidget() && !initialized_) {
+  if (!initialized_ && is_add && child == this && GetWidget()) {

[sky, 2012/11/08 02:04:24]

This in no way relates to this patch, but checking initializes_ first allows the
if to quickly terminate for the common case.

[Peter Kasting, 2012/11/08 18:55:47]

GetWidget() is expensive?

I don't mind the change, I'm just surprised it would make any meaningful
difference.

     Init();
     initialized_ = true;
   }