MacViews: Gets a webview working in views_examples_with_content_exe

ui/views/controls/native/native_view_host_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 "ui/views/controls/native/native_view_host.h"
 
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
-#include "ui/aura/window.h"
 #include "ui/views/test/views_test_base.h"
 #include "ui/views/widget/widget.h"
 
+#if defined(USE_AURA)
+#include "ui/aura/window.h"
+#endif
+
 namespace views {
 
 class NativeViewHostTest : public ViewsTestBase {
  public:
   NativeViewHostTest() {
   }
 
   virtual void SetUp() OVERRIDE {
     ViewsTestBase::SetUp();
 
@@ skipping 23 matching lines @@
     parent_view->AddChildView(host);
     host->Attach(child->GetNativeView());
 
     return child;
   }
 
   Widget* toplevel() {
     return toplevel_.get();
   }
 
- private:
+ protected:
   scoped_ptr<Widget> toplevel_;
 
+ private:
   DISALLOW_COPY_AND_ASSIGN(NativeViewHostTest);
 };
 
 namespace {
 
 // View implementation used by NativeViewHierarchyChanged to count number of
 // times NativeViewHierarchyChanged() is invoked.
 class NativeViewHierarchyChangedTestView : public View {
  public:
   NativeViewHierarchyChangedTestView() : notification_count_(0) {
   }
 
+  virtual ~NativeViewHierarchyChangedTestView() {
+    if (g_expect_destroy)
+      EXPECT_EQ(g_expect_destroy_object, this);
+    g_expect_destroy_object = NULL;
+  }
+
   void ResetCount() {
     notification_count_ = 0;
   }
 
+  void ExpectDestroy() {
+    DCHECK(!g_expect_destroy);
+    g_expect_destroy = true;
+    g_expect_destroy_object = this;
+  }
+
+  static bool PopExpectDestroy() {
+    DCHECK(g_expect_destroy);
+    g_expect_destroy = false;
+    return g_expect_destroy_object == NULL;
+  }
+
   int notification_count() const { return notification_count_; }
 
   // Overriden from View:
   virtual void NativeViewHierarchyChanged() OVERRIDE {
     ++notification_count_;
     View::NativeViewHierarchyChanged();
   }
 
  private:
+  static bool g_expect_destroy;
+  static NativeViewHierarchyChangedTestView* g_expect_destroy_object;
+
   int notification_count_;
 
   DISALLOW_COPY_AND_ASSIGN(NativeViewHierarchyChangedTestView);
 };
 
+bool NativeViewHierarchyChangedTestView::g_expect_destroy = false;
+NativeViewHierarchyChangedTestView*
+    NativeViewHierarchyChangedTestView::g_expect_destroy_object = NULL;
+
+#if defined(USE_AURA)
 aura::Window* GetNativeParent(aura::Window* window) {
   return window->parent();
 }
+#else
+gfx::NativeView GetNativeParent(gfx::NativeView native_view) {
+  Widget* widget = Widget::GetWidgetForNativeView(native_view);
+  return widget ? widget->GetNativeView() : NULL;
+}
+#endif
 
 class ViewHierarchyChangedTestHost : public NativeViewHost {
  public:
   ViewHierarchyChangedTestHost()
       : num_parent_changes_(0) {
   }
 
   void ResetParentChanges() {
     num_parent_changes_ = 0;
   }
@@ skipping 17 matching lines @@
  private:
   int num_parent_changes_;
 
   DISALLOW_COPY_AND_ASSIGN(ViewHierarchyChangedTestHost);
 };
 
 }  // namespace
 
 // Verifies NativeViewHierarchyChanged is sent.
 TEST_F(NativeViewHostTest, NativeViewHierarchyChanged) {
+  // Put a test view in the parent widget to track changes.
+  views::View* parent_view = toplevel()->GetRootView();
+  NativeViewHierarchyChangedTestView* test_view_in_parent =
+      new NativeViewHierarchyChangedTestView;
+  parent_view->AddChildView(test_view_in_parent);
+
   // Create a child widget.
-  NativeViewHierarchyChangedTestView* test_view =
+  NativeViewHierarchyChangedTestView* test_view_in_child =
       new NativeViewHierarchyChangedTestView;
+
   NativeViewHost* host = new NativeViewHost;
-  scoped_ptr<Widget> child(CreateChildForHost(toplevel()->GetNativeView(),
-                                              toplevel()->GetRootView(),
-                                              test_view,
-                                              host));
+  scoped_ptr<Widget> child(CreateChildForHost(
+      toplevel()->GetNativeView(), parent_view, test_view_in_child, host));
 #if defined(USE_AURA)
   // Two notifications are generated from inserting the native view into the
   // clipping window and then inserting the clipping window into the root
   // window.
-  EXPECT_EQ(2, test_view->notification_count());
+  EXPECT_EQ(2, test_view_in_child->notification_count());
 #else
-  EXPECT_EQ(0, test_view->notification_count());
+  // One notification results from inserting the native contents view of the
+  // Widget created in CreateChildForHost() into the toplevel Widget's native
+  // hierarchy.
+  EXPECT_EQ(1, test_view_in_child->notification_count());
+  EXPECT_EQ(1, test_view_in_parent->notification_count());
 #endif
-  test_view->ResetCount();
+  test_view_in_child->ResetCount();
+  test_view_in_parent->ResetCount();
 
   // Detaching should send a NativeViewHierarchyChanged() notification and
   // change the parent.
   host->Detach();
 #if defined(USE_AURA)
   // Two notifications are generated from removing the native view from the
   // clipping window and then reparenting it to the root window.
-  EXPECT_EQ(2, test_view->notification_count());
+  EXPECT_EQ(2, test_view_in_child->notification_count());
 #else
-  EXPECT_EQ(1, test_view->notification_count());
+  // There are no notifications in the child view because it is not associated
+  // with a Widget after being detached. The parent gets one change.
+  EXPECT_EQ(0, test_view_in_child->notification_count());
+  EXPECT_EQ(1, test_view_in_parent->notification_count());
 #endif
   EXPECT_NE(toplevel()->GetNativeView(),
             GetNativeParent(child->GetNativeView()));
-  test_view->ResetCount();
+  test_view_in_child->ResetCount();
+  test_view_in_parent->ResetCount();
 
   // Attaching should send a NativeViewHierarchyChanged() notification and
   // reset the parent.
   host->Attach(child->GetNativeView());
 #if defined(USE_AURA)
   // There is a clipping window inserted above the native view that needs to be
   // accounted for when looking at the relationship between the native views.
-  EXPECT_EQ(2, test_view->notification_count());
+  EXPECT_EQ(2, test_view_in_child->notification_count());
   EXPECT_EQ(toplevel()->GetNativeView(),
             GetNativeParent(GetNativeParent(child->GetNativeView())));
 #else
-  EXPECT_EQ(1, test_view->notification_count());
+  EXPECT_EQ(0, test_view_in_child->notification_count());
+  EXPECT_EQ(1, test_view_in_parent->notification_count());
   EXPECT_EQ(toplevel()->GetNativeView(),
             GetNativeParent(child->GetNativeView()));
 #endif
+
+  // Ownership of the views should still rest with the original Widgets.
+  test_view_in_child->ExpectDestroy();
+  child.reset();
+  EXPECT_TRUE(NativeViewHierarchyChangedTestView::PopExpectDestroy());
+
+  test_view_in_parent->ExpectDestroy();
+  toplevel_.reset();
+  EXPECT_TRUE(NativeViewHierarchyChangedTestView::PopExpectDestroy());
 }
 
 // Verifies ViewHierarchyChanged handles NativeViewHost remove, add and move
 // (reparent) operations with correct parent changes.
 // This exercises the non-recursive code paths in
 // View::PropagateRemoveNotifications() and View::PropagateAddNotifications().
 TEST_F(NativeViewHostTest, ViewHierarchyChangedForHost) {
   // Original tree:
   // toplevel
   // +-- host0 (NativeViewHost)
@@ skipping 111 matching lines @@
   host0->ResetParentChanges();
   host1->ResetParentChanges();
   EXPECT_EQ(0, host0->num_parent_changes());
   EXPECT_EQ(0, host1->num_parent_changes());
   child0->GetContentsView()->AddChildView(view1);
   EXPECT_EQ(0, host0->num_parent_changes());
   EXPECT_EQ(2, host1->num_parent_changes());
 }
 
 }  // namespace views