aura: Ask the WindowDelegate before a Window is Focus()ed.

. In aura, before a window is focused, ask the delegate if it can be focused.
. Make a popup RWHVA non-focusable.

This is to prevent the focus from moving from the main web-page to the combobox (which causes the combobox to get destroyed). This fixes an issue where comboboxes in the login screen could not be used properly.

BUG=105786
TEST=manually

Review URL: http://codereview.chromium.org/8770011

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112909 0039d316-1c4b-4281-b951-d872f2087c98

content/browser/renderer_host/render_widget_host_view_aura.cc

@@ -531,6 +531,10 @@ ui::TouchStatus RenderWidgetHostViewAura::OnTouchEvent(
   return DecideTouchStatus(touch_event_, point);
 }
 
+bool RenderWidgetHostViewAura::CanFocus() {
+  return popup_type_ == WebKit::WebPopupTypeNone;
+}
+
 bool RenderWidgetHostViewAura::ShouldActivate(aura::Event* event) {
   return false;
 }

content/browser/renderer_host/render_widget_host_view_aura.h

@@ -114,6 +114,7 @@ class CONTENT_EXPORT RenderWidgetHostViewAura
   virtual int GetNonClientComponent(const gfx::Point& point) const OVERRIDE;
   virtual bool OnMouseEvent(aura::MouseEvent* event) OVERRIDE;
   virtual ui::TouchStatus OnTouchEvent(aura::TouchEvent* event) OVERRIDE;
+  virtual bool CanFocus() OVERRIDE;
   virtual bool ShouldActivate(aura::Event* event) OVERRIDE;
   virtual void OnActivated() OVERRIDE;
   virtual void OnLostActive() OVERRIDE;

ui/aura/demo/demo_main.cc

@@ -53,6 +53,7 @@ class DemoWindowDelegate : public aura::WindowDelegate {
   virtual ui::TouchStatus OnTouchEvent(aura::TouchEvent* event) OVERRIDE {
     return ui::TOUCH_STATUS_END;
   }
+  virtual bool CanFocus() OVERRIDE { return true; }
   virtual bool ShouldActivate(aura::Event* event) OVERRIDE { return true; }
   virtual void OnActivated() OVERRIDE {}
   virtual void OnLostActive() OVERRIDE {}

ui/aura/test/test_window_delegate.cc

@@ -56,6 +56,10 @@ ui::TouchStatus TestWindowDelegate::OnTouchEvent(TouchEvent* event) {
   return ui::TOUCH_STATUS_UNKNOWN;
 }
 
+bool TestWindowDelegate::CanFocus() {
+  return true;
+}
+
 bool TestWindowDelegate::ShouldActivate(Event* event) {
   return true;
 }

ui/aura/test/test_window_delegate.h

@@ -30,6 +30,7 @@ class TestWindowDelegate : public WindowDelegate {
   virtual int GetNonClientComponent(const gfx::Point& point) const OVERRIDE;
   virtual bool OnMouseEvent(MouseEvent* event) OVERRIDE;
   virtual ui::TouchStatus OnTouchEvent(TouchEvent* event) OVERRIDE;
+  virtual bool CanFocus() OVERRIDE;
   virtual bool ShouldActivate(Event* event) OVERRIDE;
   virtual void OnActivated() OVERRIDE;
   virtual void OnLostActive() OVERRIDE;

ui/aura/window.cc

@@ -368,8 +368,7 @@ bool Window::HasFocus() const {
 // propagation of events that would otherwise be targeted at windows behind it.
 // We then perform this same check on every window up to the root.
 bool Window::CanFocus() const {
-  // TODO(beng): Figure out how to consult the delegate wrt. focusability also.
-  if (!IsVisible() || !parent_)
+  if (!IsVisible() || !parent_ || (delegate_ && !delegate_->CanFocus()))
     return false;
 
   Windows::const_iterator i = std::find(parent_->children().begin(),

ui/aura/window_delegate.h

@@ -52,6 +52,9 @@ class AURA_EXPORT WindowDelegate {
 
   virtual ui::TouchStatus OnTouchEvent(TouchEvent* event) = 0;
 
+  // Returns true of the window can be focused.
+  virtual bool CanFocus() = 0;
+
   // Returns true if the window should be activated. |event| is either the mouse
   // event supplied if the activation is the result of a mouse, or the touch
   // event if the activation is the result of a touch, or NULL if activation is

ui/views/widget/native_widget_aura.cc

@@ -611,6 +611,10 @@ ui::TouchStatus NativeWidgetAura::OnTouchEvent(aura::TouchEvent* event) {
   return delegate_->OnTouchEvent(touch_event);
 }
 
+bool NativeWidgetAura::CanFocus() {
+  return true;
+}
+
 bool NativeWidgetAura::ShouldActivate(aura::Event* event) {
   return can_activate_;
 }

ui/views/widget/native_widget_aura.h

@@ -131,6 +131,7 @@ class VIEWS_EXPORT NativeWidgetAura : public internal::NativeWidgetPrivate,
   virtual int GetNonClientComponent(const gfx::Point& point) const OVERRIDE;
   virtual bool OnMouseEvent(aura::MouseEvent* event) OVERRIDE;
   virtual ui::TouchStatus OnTouchEvent(aura::TouchEvent* event) OVERRIDE;
+  virtual bool CanFocus() OVERRIDE;
   virtual bool ShouldActivate(aura::Event* event) OVERRIDE;
   virtual void OnActivated() OVERRIDE;
   virtual void OnLostActive() OVERRIDE;