Revert of Don't allow a widget to send close notifications more than once.

ui/views/widget/widget_unittest.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 <algorithm>
 #include <memory>
 #include <set>
 
 #include "base/bind.h"
 #include "base/macros.h"
@@ skipping 720 matching lines @@
 
   // Set a widget to Close() the next time the Widget being observed is hidden.
   void CloseOnNextHide(Widget* widget) {
     widget_to_close_on_hide_ = widget;
   }
 
   // Overridden from WidgetObserver:
   void OnWidgetDestroying(Widget* widget) override {
     if (active_ == widget)
       active_ = nullptr;
-    if (widget_activated_ == widget)
-      widget_activated_ = nullptr;
     widget_closed_ = widget;
   }
 
   void OnWidgetActivationChanged(Widget* widget, bool active) override {
     if (active) {
       if (widget_activated_)
         widget_activated_->Deactivate();
       widget_activated_ = widget;
       active_ = widget;
     } else {
@@ skipping 50 matching lines @@
   Widget* widget_closed_;
   Widget* widget_activated_;
   Widget* widget_deactivated_;
   Widget* widget_shown_;
   Widget* widget_hidden_;
   Widget* widget_bounds_changed_;
 
   Widget* widget_to_close_on_hide_;
 };
 
-TEST_F(WidgetObserverTest, ActivationChange) {
+TEST_F(WidgetObserverTest, DISABLED_ActivationChange) {
   WidgetAutoclosePtr toplevel(CreateTopLevelPlatformWidget());
-  WidgetAutoclosePtr toplevel1(NewWidget());
-  WidgetAutoclosePtr toplevel2(NewWidget());
+
+  Widget* toplevel1 = NewWidget();
+  Widget* toplevel2 = NewWidget();
 
   toplevel1->Show();
   toplevel2->Show();
 
   reset();
 
   toplevel1->Activate();
 
   RunPendingMessages();
-  EXPECT_EQ(toplevel1.get(), widget_activated());
+  EXPECT_EQ(toplevel1, widget_activated());
 
   toplevel2->Activate();
   RunPendingMessages();
-  EXPECT_EQ(toplevel1.get(), widget_deactivated());
-  EXPECT_EQ(toplevel2.get(), widget_activated());
-  EXPECT_EQ(toplevel2.get(), active());
+  EXPECT_EQ(toplevel1, widget_deactivated());
+  EXPECT_EQ(toplevel2, widget_activated());
+  EXPECT_EQ(toplevel2, active());
 }
 
-namespace {
+TEST_F(WidgetObserverTest, DISABLED_VisibilityChange) {
+  WidgetAutoclosePtr toplevel(CreateTopLevelPlatformWidget());
 
-// This class simulates a focus manager that moves focus to a second widget when
-// the first one is closed. It simulates a situation where a sequence of widget
-// observers might try to call Widget::Close in response to a OnWidgetClosing().
-class WidgetActivationForwarder : public TestWidgetObserver {
- public:
-  WidgetActivationForwarder(Widget* current_active_widget,
-                            Widget* widget_to_activate)
-      : TestWidgetObserver(current_active_widget),
-        widget_to_activate_(widget_to_activate) {}
-
-  ~WidgetActivationForwarder() override {}
-
- private:
-  // WidgetObserver overrides:
-  void OnWidgetClosing(Widget* widget) override {
-    widget->OnNativeWidgetActivationChanged(false);
-    widget_to_activate_->Activate();
-  }
-  void OnWidgetActivationChanged(Widget* widget, bool active) override {
-    if (!active)
-      widget->Close();
-  }
-
-  Widget* widget_to_activate_;
-
-  DISALLOW_COPY_AND_ASSIGN(WidgetActivationForwarder);
-};
-
-// This class observes a widget and counts the number of times OnWidgetClosing
-// is called.
-class WidgetCloseCounter : public TestWidgetObserver {
- public:
-  explicit WidgetCloseCounter(Widget* widget) : TestWidgetObserver(widget) {}
-
-  ~WidgetCloseCounter() override {}
-
-  int close_count() const { return close_count_; }
-
- private:
-  // WidgetObserver overrides:
-  void OnWidgetClosing(Widget* widget) override { close_count_++; }
-
-  int close_count_ = 0;
-
-  DISALLOW_COPY_AND_ASSIGN(WidgetCloseCounter);
-};
-
-}  // namespace
-
-// Makes sure close notifications aren't sent more than once when a Widget is
-// shutting down. Test for crbug.com/714334
-TEST_F(WidgetObserverTest, CloseReentrancy) {
-  Widget* widget1 = CreateTopLevelPlatformWidget();
-  Widget* widget2 = CreateTopLevelPlatformWidget();
-  WidgetCloseCounter counter(widget1);
-  WidgetActivationForwarder focus_manager(widget1, widget2);
-  widget1->Close();
-  EXPECT_EQ(1, counter.close_count());
-  widget2->Close();
-}
-
-TEST_F(WidgetObserverTest, VisibilityChange) {
-  WidgetAutoclosePtr toplevel(CreateTopLevelPlatformWidget());
-  WidgetAutoclosePtr child1(NewWidget());
-  WidgetAutoclosePtr child2(NewWidget());
+  Widget* child1 = NewWidget();
+  Widget* child2 = NewWidget();
 
   toplevel->Show();
   child1->Show();
   child2->Show();
 
   reset();
 
   child1->Hide();
-  EXPECT_EQ(child1.get(), widget_hidden());
+  EXPECT_EQ(child1, widget_hidden());
 
   child2->Hide();
-  EXPECT_EQ(child2.get(), widget_hidden());
+  EXPECT_EQ(child2, widget_hidden());
 
   child1->Show();
-  EXPECT_EQ(child1.get(), widget_shown());
+  EXPECT_EQ(child1, widget_shown());
 
   child2->Show();
-  EXPECT_EQ(child2.get(), widget_shown());
+  EXPECT_EQ(child2, widget_shown());
 }
 
 TEST_F(WidgetObserverTest, DestroyBubble) {
   // This test expect NativeWidgetAura, force its creation.
   ViewsDelegate::GetInstance()->set_native_widget_factory(
       ViewsDelegate::NativeWidgetFactory());
 
   WidgetAutoclosePtr anchor(CreateTopLevelPlatformWidget());
   anchor->Show();
 
@@ skipping 3005 matching lines @@
 }
 
 TEST_F(CompositingWidgetTest, Transparency_DesktopWidgetTranslucent) {
   CheckAllWidgetsForOpacity(Widget::InitParams::TRANSLUCENT_WINDOW);
 }
 
 #endif  // !defined(OS_CHROMEOS)
 
 }  // namespace test
 }  // namespace views