NativeWidgetViews: Implement widget activation/deactivation.

BUG=none
TEST=WidgetObserverTest.ActivationChange

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

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

views/desktop/desktop_window_view.cc

@@ -138,15 +138,10 @@ void DesktopWindowView::ActivateWidget(Widget* widget) {
   if (widget && widget->IsActive())
     return;
 
-  if (active_native_widget_)
-    active_native_widget_->OnActivate(false);
   if (widget) {
-    widget->MoveToTop();
-    active_native_widget_ =
-        static_cast<NativeWidgetViews*>(widget->native_widget());
-    active_native_widget_->OnActivate(true);
     if (!widget->HasObserver(this))
       widget->AddObserver(this);
+    widget->Activate();
   }
 }
 
@@ -252,6 +247,14 @@ void DesktopWindowView::OnWidgetVisibilityChanged(Widget* widget,
 
 void DesktopWindowView::OnWidgetActivationChanged(Widget* widget,
                                                   bool active) {
+  if (active) {
+    if (active_native_widget_)
+      active_native_widget_->GetWidget()->Deactivate();
+    active_native_widget_ =
+        static_cast<NativeWidgetViews*>(widget->native_widget());
+  } else if (widget == active_native_widget_->GetWidget()) {
+    active_native_widget_ = NULL;
+  }
 }
 
 }  // namespace desktop

views/widget/native_widget_views.cc

@@ -378,11 +378,12 @@ bool NativeWidgetViews::IsVisible() const {
 
 void NativeWidgetViews::Activate() {
   // Enable WidgetObserverTest.ActivationChange when this is implemented.
-  NOTIMPLEMENTED();
+  MoveToTop();
+  OnActivate(true);
 }
 
 void NativeWidgetViews::Deactivate() {
-  NOTIMPLEMENTED();
+  OnActivate(false);
 }
 
 bool NativeWidgetViews::IsActive() const {

views/widget/native_widget_views.h

@@ -14,6 +14,10 @@
 #include "views/widget/native_widget_private.h"
 
 namespace views {
+namespace desktop {
+class DesktopWindowView;
+}
+
 namespace internal {
 class NativeWidgetView;
 }
@@ -128,6 +132,8 @@ class VIEWS_EXPORT NativeWidgetViews : public internal::NativeWidgetPrivate,
   virtual void DispatchKeyEventPostIME(const KeyEvent& key) OVERRIDE;
 
  private:
+  friend class desktop::DesktopWindowView;
+
   // These functions may return NULL during Widget destruction.
   internal::NativeWidgetPrivate* GetParentNativeWidget();
   const internal::NativeWidgetPrivate* GetParentNativeWidget() const;

views/widget/widget_unittest.cc

@@ -586,10 +586,15 @@ class WidgetObserverTest : public WidgetTest,
   virtual void OnWidgetActivationChanged(Widget* widget,
                                          bool active) OVERRIDE {
     if (active) {
+      if (widget_activated_)
+        widget_activated_->Deactivate();
       widget_activated_ = widget;
       active_ = widget;
-    } else
+    } else {
+      if (widget_activated_ == widget)
+        widget_activated_ = NULL;
       widget_deactivated_ = widget;
+    }
   }
 
   virtual void OnWidgetVisibilityChanged(Widget* widget,
@@ -633,9 +638,7 @@ class WidgetObserverTest : public WidgetTest,
   Widget* widget_hidden_;
 };
 
-// TODO: This test should be enabled when NativeWidgetViews::Activate is
-// implemented.
-TEST_F(WidgetObserverTest, DISABLED_ActivationChange) {
+TEST_F(WidgetObserverTest, ActivationChange) {
   Widget* toplevel = CreateTopLevelPlatformWidget();
   views_delegate.set_default_parent_view(toplevel->GetRootView());