Don't activate POPUP window

views/desktop/desktop_window_manager.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/desktop/desktop_window_manager.h"
 
 #include "views/events/event.h"
 #include "views/widget/widget.h"
 #include "ui/gfx/point.h"
 #include "ui/gfx/rect.h"
 #include "views/widget/widget.h"
 #include "views/widget/native_widget_private.h"
+#include "views/widget/native_widget_view.h"
 #include "views/widget/native_widget_views.h"
 #include "views/widget/widget_delegate.h"
 #if defined(OS_LINUX)
 #include "views/window/hit_test.h"
 #endif
 #include "views/window/non_client_view.h"
 
 namespace {
 
 class MoveWindowController : public views::desktop::WindowController {
@@ skipping 62 matching lines @@
 }
 
 WindowController::~WindowController() {
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // DesktopWindowManager, public:
 
 DesktopWindowManager::DesktopWindowManager(Widget* desktop)
     : desktop_(desktop),
-      mouse_capture_(NULL) {
+      mouse_capture_(NULL),
+      active_widget_(NULL) {
 }
 
 DesktopWindowManager::~DesktopWindowManager() {
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // DesktopWindowManager, WindowManager implementation:
 
 void DesktopWindowManager::StartMoveDrag(
     views::Widget* widget,
@@ skipping 41 matching lines @@
   DCHECK(HasMouseCapture());
   ReleaseMouseCapture();
   mouse_capture_ = NULL;
   return true;
 }
 
 bool DesktopWindowManager::HasMouseCapture(const views::Widget* widget) const {
   return widget && mouse_capture_ == widget;
 }
 
+bool DesktopWindowManager::HandleKeyEvent(
+    views::Widget* widget, const views::KeyEvent& event) {
+  return active_widget_ ?
+      static_cast<NativeWidgetViews*>(active_widget_->native_widget_private())
+          ->OnKeyEvent(event) : false;

[sadrul, 2011/08/26 16:21:36]

Could this just be 'active_widget_->OnKeyEvent(event)'?

[oshima, 2011/08/26 17:20:28]

We need to send to NativeViewViews as it needs to send key to InputMethod object
of the widget. It'd require some refactoring if we want to send it directly to
widget.

[sadrul, 2011/08/26 18:41:05]

Ah. That seems a little unfortunate. Thanks for the explanation.

+}
+
 bool DesktopWindowManager::HandleMouseEvent(
     views::Widget* widget, const views::MouseEvent& event) {
+  if (event.type() == ui::ET_MOUSE_PRESSED) {
+    View* target =
+        widget->GetRootView()->GetEventHandlerForPoint(event.location());
+
+    if (target->GetClassName() == internal::NativeWidgetView::kViewClassName) {
+      internal::NativeWidgetView* native_widget_view =
+          static_cast<internal::NativeWidgetView*>(target);
+      views::Widget* target_widget = native_widget_view->GetAssociatedWidget();
+      if (target_widget->CanActivate())
+        Activate(target_widget);
+    }
+  } else if (event.type() == ui::ET_MOUSEWHEEL && active_widget_) {
+    return active_widget_->OnMouseEvent(event);
+  }
 
   if (window_controller_.get()) {
     if (!window_controller_->OnMouseEvent(event)) {
       ReleaseMouseCapture();
       window_controller_.reset();
     }
     return true;
   }
 
   if (mouse_capture_) {
     views::MouseEvent translated(event, widget->GetRootView(),
                                  mouse_capture_->GetRootView());
     mouse_capture_->OnMouseEvent(translated);
     return true;
   }
   return false;
 }
 
+void DesktopWindowManager::Register(Widget* widget) {

[sadrul, 2011/08/26 16:21:36]

Perhaps DCHECK(!widget->HasObserver(this))?

[oshima, 2011/08/26 17:20:28]

Done.

+  widget->AddObserver(this);
+}
+
 ////////////////////////////////////////////////////////////////////////////////
 // DesktopWindowManager, private:
 
+void DesktopWindowManager::OnWidgetClosing(Widget* widget) {
+  if (active_widget_ && active_widget_ == widget)
+    active_widget_ = NULL;
+}
+
+void DesktopWindowManager::OnWidgetVisibilityChanged(Widget* widget,
+                                                     bool visible) {
+}
+
+void DesktopWindowManager::OnWidgetActivationChanged(Widget* widget,
+                                                     bool active) {
+  if (active) {
+    if (active_widget_)
+      active_widget_->Deactivate();
+    active_widget_ = widget;
+  } else if (widget == active_widget_) {
+    active_widget_ = NULL;
+  }
+}
+
 void DesktopWindowManager::SetMouseCapture() {
   return desktop_->native_widget_private()->SetMouseCapture();
 }
 
 void DesktopWindowManager::ReleaseMouseCapture() {
   return desktop_->native_widget_private()->ReleaseMouseCapture();
 }
 
 bool DesktopWindowManager::HasMouseCapture() const {
   return desktop_->native_widget_private()->HasMouseCapture();
 }
 
+void DesktopWindowManager::Activate(Widget* widget) {
+  if (widget && widget->IsActive())
+    return;
+
+  if (widget) {
+    if (!widget->HasObserver(this))
+      widget->AddObserver(this);

[sadrul, 2011/08/26 16:21:36]

Perhaps just 'DCHECK(widget->HasObserver(this));' here? this will make sure all
windows are registered to the WM.

+    widget->Activate();
+  }
+}
+
 }  // namespace desktop
 }  // namespace views