Do not create proxy hosts in the subtree of navigating frame.

content/browser/accessibility/site_per_process_accessibility_browsertest.cc

 // Copyright (c) 2014 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/command_line.h"
 #include "base/strings/stringprintf.h"
 #include "content/browser/accessibility/browser_accessibility.h"
 #include "content/browser/accessibility/browser_accessibility_manager.h"
 #include "content/browser/accessibility/browser_accessibility_state_impl.h"
 #include "content/browser/frame_host/cross_process_frame_connector.h"
@@ skipping 62 matching lines @@
   GURL::Replacements replace_host;
   std::string foo_com("foo.com");
 
   // Load cross-site page into iframe.
   GURL cross_site_url(test_server()->GetURL("files/title2.html"));
   replace_host.SetHostStr(foo_com);
   cross_site_url = cross_site_url.ReplaceComponents(replace_host);
   NavigateFrameToURL(root->child_at(0), cross_site_url);
 
   // Ensure that we have created a new process for the subframe.
-  ASSERT_EQ(1U, root->child_count());
+  ASSERT_EQ(2U, root->child_count());
   SiteInstance* site_instance = child->current_frame_host()->GetSiteInstance();
   EXPECT_NE(shell()->web_contents()->GetSiteInstance(), site_instance);
 
   // Wait until the accessibility tree from both the main frame and
   // cross-process iframe load.
   RenderFrameHostImpl* child_frame = static_cast<RenderFrameHostImpl*>(
       child->current_frame_host());
   AccessibilityNotificationWaiter child_frame_accessibility_waiter(
       child_frame, ui::AX_EVENT_NONE);
   main_frame_accessibility_waiter.WaitForNotification();
@@ skipping 12 matching lines @@
           << child_frame_manager->SnapshotAXTreeForTesting().ToString();
 
   // Assert that we can walk from the main frame down into the child frame
   // directly, getting correct roles and data along the way.
   BrowserAccessibility* ax_root = main_frame_manager->GetRoot();
   ASSERT_EQ(ui::AX_ROLE_ROOT_WEB_AREA, ax_root->GetRole());
   ASSERT_EQ(1U, ax_root->PlatformChildCount());
 
   BrowserAccessibility* ax_group = ax_root->PlatformGetChild(0);
   ASSERT_EQ(ui::AX_ROLE_GROUP, ax_group->GetRole());
-  ASSERT_EQ(1U, ax_group->PlatformChildCount());
+  ASSERT_EQ(2U, ax_group->PlatformChildCount());

[dmazzoni, 2014/09/11 15:09:31]

Could you either access ax_group->PlatformGetChild(1) below and assert what it
is, or else comment what it is and why we should ignore it?

Use the VLOGs above for a nice text dump of the accessibility tree so you
shouldn't have to guess the role.

[nasko, 2014/09/11 17:10:36]

As discussed, I'll update this test in a separate CL to check for the entire
frame tree.

 
   BrowserAccessibility* ax_iframe = ax_group->PlatformGetChild(0);
   ASSERT_EQ(ui::AX_ROLE_IFRAME, ax_iframe->GetRole());
   ASSERT_EQ(1U, ax_iframe->PlatformChildCount());
 
   BrowserAccessibility* ax_scroll_area = ax_iframe->PlatformGetChild(0);
   ASSERT_EQ(ui::AX_ROLE_SCROLL_AREA, ax_scroll_area->GetRole());
   ASSERT_EQ(1U, ax_scroll_area->PlatformChildCount());
 
   BrowserAccessibility* ax_child_frame_root =
@@ skipping 10 matching lines @@
   BrowserAccessibility* ax_child_frame_static_text =
       ax_child_frame_group->PlatformGetChild(0);
   ASSERT_EQ(ui::AX_ROLE_STATIC_TEXT, ax_child_frame_static_text->GetRole());
   ASSERT_EQ(0U, ax_child_frame_static_text->PlatformChildCount());
 
   // Last, check that the parent of the child frame root is correct.
   ASSERT_EQ(ax_child_frame_root->GetParent(), ax_scroll_area);
 }
 
 }  // namespace content