Tests for bug 5988.

content/browser/renderer_host/resource_dispatcher_host_uitest.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 <sstream>
 #include <string>
 
 #include "base/command_line.h"
 #include "base/file_path.h"
 #include "base/path_service.h"
 #include "base/string_util.h"
+#include "base/utf_string_conversions.h"
 #include "base/test/test_timeouts.h"
 #include "chrome/browser/net/url_request_failed_dns_job.h"
 #include "chrome/browser/net/url_request_mock_http_job.h"
 #include "chrome/test/automation/browser_proxy.h"
 #include "chrome/test/automation/tab_proxy.h"
+#include "chrome/test/automation/window_proxy.h"
 #include "chrome/test/ui/ui_test.h"
 #include "content/common/test_url_constants.h"
 #include "content/common/url_constants.h"
+#include "chrome/test/ui_test_utils.h"

[Paweł Hajdan Jr., 2011/06/01 07:23:44]

nit: Is this used?

 #include "net/base/net_util.h"
 #include "net/test/test_server.h"
 
 namespace {
 
 class ResourceDispatcherTest : public UITest {
  public:
   void CheckTitleTest(const std::string& file,
                       const std::string& expected_title,
                       int expected_navigations) {
@@ skipping 361 matching lines @@
   // Visit a URL that fails without calling ResourceDispatcherHost::Read.
   GURL broken_url("chrome://theme");
   ASSERT_EQ(AUTOMATION_MSG_NAVIGATION_SUCCESS, tab->NavigateToURL(broken_url));
 
   // Make sure the navigation finishes.
   std::wstring tab_title;
   EXPECT_TRUE(tab->GetTabTitle(&tab_title));
   EXPECT_EQ(L"chrome://theme/ is not available", tab_title);
 }
 
+// Test title for content dynamically created by javascript.
+// See bug 5988
+TEST_F(ResourceDispatcherTest, DynamicTitle1) {
+  dom_automation_enabled_ = true;
+
+  net::TestServer test_server(net::TestServer::TYPE_HTTP,
+                              FilePath(FILE_PATH_LITERAL("chrome/test/data")));
+  ASSERT_TRUE(test_server.Start());
+
+  scoped_refptr<BrowserProxy> browser_proxy(automation()->GetBrowserWindow(0));
+  ASSERT_TRUE(browser_proxy.get());
+  scoped_refptr<TabProxy> tab(browser_proxy->GetActiveTab());
+  ASSERT_TRUE(tab.get());
+
+  GURL url(test_server.GetURL("files/dynamic1.html"));
+  ASSERT_EQ(AUTOMATION_MSG_NAVIGATION_SUCCESS, tab->NavigateToURL(url));
+
+  // Make sure the navigation finishes.
+  std::wstring tab_title;
+  EXPECT_TRUE(tab->GetTabTitle(&tab_title));
+
+  // Create dynamic popup.
+  std::wstring jscript =
+      L"OpenPopup(); window.domAutomationController.send('string');";
+  std::wstring value;
+  ASSERT_TRUE(tab->ExecuteAndExtractString(L"", jscript, &value));
+
+  scoped_refptr<BrowserProxy> browser_proxy2(automation()->GetBrowserWindow(1));
+  ASSERT_TRUE(browser_proxy2.get());
+  scoped_refptr<WindowProxy> window2(browser_proxy2->GetWindow());

[Paweł Hajdan Jr., 2011/06/01 07:23:44]

WindowProxy are discouraged, do you really need it?

The problem with WindowProxy is that it uses OS calls that might behave in
unexpected ways. I'm not sure if GetWindowTitle is one of those, but to be safe
I just recommend avoiding WindowProxy unless you absolutely have to use it.

+  ASSERT_TRUE(window2.get());
+
+  base::PlatformThread::Sleep(TestTimeouts::action_timeout_ms());

[Paweł Hajdan Jr., 2011/06/01 07:23:44]

Are you trying to wait for the load to finish or for the window to appear?

Please consider writing a _browser_ test instead, there are many functions in
ui_test_utils that could help you (like WaitForNavigationInCurrentTab or
similar).

There's also WaitForNavigation for UI tests, but it's a bit more ugly. I don't
really recommend it.

If you just need to wait for a window to appear, you can use
WaitForWindowCountToBecome.

If my guesses don't really cover your scenario, please explain in more detail
what you need to wait for (I'm just not familiar with every part of the browser,
especially the WebKit side), and I'll see if we have facilities for waiting for
those kinds of events or suggest how to add one.

+
+  string16 tab_title2;
+  EXPECT_TRUE(window2->GetWindowTitle(&tab_title2));
+
+  // tab_title3 should start with 'My Dynamic Title'
+  ASSERT_EQ(string16::size_type(0),
+            tab_title2.find(ASCIIToUTF16("My Popup Title")));
+}
+
+// Test title for content dynamically created by javascript.
+// See Bug 5988
+TEST_F(ResourceDispatcherTest, DynamicTitle2) {
+  dom_automation_enabled_ = true;
+
+  net::TestServer test_server(net::TestServer::TYPE_HTTP,
+                              FilePath(FILE_PATH_LITERAL("chrome/test/data")));
+  ASSERT_TRUE(test_server.Start());
+
+  scoped_refptr<BrowserProxy> browser_proxy(automation()->GetBrowserWindow(0));
+  ASSERT_TRUE(browser_proxy.get());
+  scoped_refptr<TabProxy> tab(browser_proxy->GetActiveTab());
+  ASSERT_TRUE(tab.get());
+
+  GURL url(test_server.GetURL("files/dynamic2.html"));
+  ASSERT_EQ(AUTOMATION_MSG_NAVIGATION_SUCCESS, tab->NavigateToURL(url));
+
+  // Make sure the navigation finishes.
+  std::wstring tab_title;
+  EXPECT_TRUE(tab->GetTabTitle(&tab_title));
+
+  // Create dynamic popup.
+  std::wstring jscript =
+      L"OpenPopup(); window.domAutomationController.send('string');";
+  std::wstring value;
+  ASSERT_TRUE(tab->ExecuteAndExtractString(L"", jscript, &value));
+
+  scoped_refptr<BrowserProxy> browser_proxy2(automation()->GetBrowserWindow(1));
+  ASSERT_TRUE(browser_proxy2.get());
+  scoped_refptr<WindowProxy> window2(browser_proxy2->GetWindow());
+  ASSERT_TRUE(window2.get());
+
+  base::PlatformThread::Sleep(TestTimeouts::action_timeout_ms());
+
+  string16 tab_title2;
+  EXPECT_TRUE(window2->GetWindowTitle(&tab_title2));
+
+  // tab_title3 should start with 'My Dynamic Title'
+  ASSERT_EQ(string16::size_type(0),
+            tab_title2.find(ASCIIToUTF16("My Dynamic Title")));
+}
+
 }  // namespace