Make to call WebContentsImpl::RenderViewCreated() when we create child window.

content/browser/web_contents/web_contents_impl_browsertest.cc

 // Copyright 2013 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 "base/values.h"
 #include "content/browser/frame_host/navigation_entry_impl.h"
 #include "content/browser/web_contents/web_contents_impl.h"
 #include "content/public/browser/load_notification_details.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/notification_details.h"
@@ skipping 367 matching lines @@
   // Install the observer and navigate cross-site.
   RenderFrameCreatedObserver observer(shell());
   NavigateToURL(shell(), cross_site_url);
 
   // The observer should've seen a RenderFrameCreated call for the new frame
   // and not the old one.
   EXPECT_NE(observer.last_rfh(), orig_rfh);
   EXPECT_EQ(observer.last_rfh(), shell()->web_contents()->GetMainFrame());
 }
 
+class RenderViewCreatedObserver : public WebContentsObserver {
+ public:
+  static void Create(WebContents* web_contents) {
+    current_ = new RenderViewCreatedObserver(web_contents);
+  }
+
+  static void Clean() {
+    if (!current_) return;
+    delete current_;
+    current_ = NULL;
+  }
+
+ public:
+  RenderViewCreatedObserver(WebContents* web_contents)
+      : WebContentsObserver(web_contents),
+        call_render_view_created_(false) {
+  }
+
+  // WebContentsObserver:
+  virtual void RenderViewCreated(RenderViewHost* rvh) OVERRIDE {
+    call_render_view_created_ = true;
+  }
+
+  bool call_render_view_created_;
+  static RenderViewCreatedObserver* current_;
+};
+
+RenderViewCreatedObserver* RenderViewCreatedObserver::current_ = NULL;
+
+class NewChildWindowNotificationObserver : public WindowedNotificationObserver {
+ public:
+  NewChildWindowNotificationObserver(WebContents* web_contens)
+      : WindowedNotificationObserver(
+                                NOTIFICATION_WEB_CONTENTS_CREATE_NEW_WINDOW,
+                                Source<WebContents>(web_contens)) {
+  }
+  virtual void Observe(int type,
+                       const NotificationSource& source,
+                       const NotificationDetails& details) OVERRIDE {
+    if (type ==
+        NOTIFICATION_WEB_CONTENTS_CREATE_NEW_WINDOW) {
+      WebContents* child_web_contents =
+                    content::Details<WebContents>(details).ptr();
+      RenderViewCreatedObserver::Create(child_web_contents);
+    }
+    WindowedNotificationObserver::Observe(type, source, details);
+  }
+};
+
+IN_PROC_BROWSER_TEST_F(WebContentsImplBrowserTest,
+                       RenderViewCreatedForChildWindow) {
+  ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady());
+
+  NewChildWindowNotificationObserver observer(shell()->web_contents());

[Charlie Reis, 2014/04/29 16:55:25]

You can use a ShellAddedObserver here instead, without adding the new
notification.  See content_browser_test_utils, and
render_frame_host_manager_browsertest.cc for examples.

+  NavigateToURL(shell(),
+                embedded_test_server()->GetURL("/openChildWindow.html"));
+
+  observer.Wait();
+  EXPECT_TRUE(
+      RenderViewCreatedObserver::current_->call_render_view_created_ == true);

[Charlie Reis, 2014/04/29 16:55:25]

nit: No need for "== true"

+  RenderViewCreatedObserver::Clean();
+}
+
 }  // namespace content