Enable KeyboardOverlayDialog on aura.

ui/views/widget/widget.cc

 // Copyright (c) 2012 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 "ui/views/widget/widget.h"
 
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/utf_string_conversions.h"
 #include "ui/base/hit_test.h"
@@ skipping 203 matching lines @@
   params.delegate = delegate;
 #if defined(OS_WIN) || defined(USE_AURA)
   params.parent = parent;
 #endif
   params.bounds = bounds;
   widget->Init(params);
   return widget;
 }
 
 // static
+Widget* Widget::CreateFramelessWindowWithParentAndBounds(
+    WidgetDelegate* delegate,
+    gfx::NativeWindow parent,
+    const gfx::Rect& bounds) {

[Ben Goodger (Google), 2012/01/30 21:16:16]

Rather than do this in Widget, can you just move this code into the window.cc
file in src/chrome/browser... since you only use this in one place.

[jennyz, 2012/01/30 23:13:28]

Done.

+  Widget* widget = new Widget;
+  Widget::InitParams params(views::Widget::InitParams::TYPE_WINDOW_FRAMELESS);
+  params.delegate = delegate;
+  // Will this function be called if !defined(USE_AURA)?
+#if defined(OS_WIN) || defined(USE_AURA)
+  params.parent = parent;
+#endif
+  params.bounds = bounds;
+  widget->Init(params);
+  widget->SetContentsView(delegate->GetContentsView());
+  widget->SetInitialBoundsForFramelessWindow(bounds);

[jennyz, 2012/01/27 23:17:48]

I was thinking for moving SetContentsView(delegate->GetContentsView()) and
SetInitialBoundsForFramelessWindow into Widget::Init() for else part of if
(RequiresNonClientView(params.type)), but I found some existing codes are
calling widget->SetContentsView(xx) after widget->Init(), although most time
delegate and the parameter passing to SetContentsView are the same, but I am a
little concerned if we can assume that all the time, therefore, I left this
outside Widget::Init(), the code maybe cleaner the other way. Any suggestion?

+  return widget;
+}
+
+// static
 void Widget::SetPureViews(bool pure) {
   use_pure_views = pure;
 }
 
 // static
 bool Widget::IsPureViews() {
 #if defined(USE_AURA)
   return true;
 #else
   return use_pure_views;
@@ skipping 963 matching lines @@
       // No initial bounds supplied, so size the window to its content and
       // center over its parent.
       native_widget_->CenterWindow(non_client_view_->GetPreferredSize());
     } else {
       // Use the supplied initial bounds.
       SetBoundsConstrained(bounds);
     }
   }
 }
 
+void Widget::SetInitialBoundsForFramelessWindow(const gfx::Rect& bounds) {
+  View* contents_view = GetContentsView();
+  DCHECK(contents_view != NULL);
+  if (bounds.IsEmpty()) {
+    // No initial bounds supplied, so size the window to its content and
+    // center over its parent.
+    native_widget_->CenterWindow(contents_view->GetPreferredSize());
+  } else {
+    // Use the supplied initial bounds.
+    SetBoundsConstrained(bounds);
+  }
+}
+
 bool Widget::GetSavedWindowPlacement(gfx::Rect* bounds,
                                      ui::WindowShowState* show_state) {
   // First we obtain the window's saved show-style and store it. We need to do
   // this here, rather than in Show() because by the time Show() is called,
   // the window's size will have been reset (below) and the saved maximized
   // state will have been lost. Sadly there's no way to tell on Windows when
   // a window is restored from maximized state, so we can't more accurately
   // track maximized state independently of sizing information.
 
   // Restore the window's placement from the controller.
@@ skipping 28 matching lines @@
 
 ////////////////////////////////////////////////////////////////////////////////
 // internal::NativeWidgetPrivate, NativeWidget implementation:
 
 internal::NativeWidgetPrivate* NativeWidgetPrivate::AsNativeWidgetPrivate() {
   return this;
 }
 
 }  // namespace internal
 }  // namespace views