Enable SitePerProcessAccessibilityBrowserTest on more platforms.

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 28 matching lines @@
 };
 
 // Utility function to determine if an accessibility tree has finished loading
 // or if the tree represents a page that hasn't finished loading yet.
 bool AccessibilityTreeIsLoaded(BrowserAccessibilityManager* manager) {
   BrowserAccessibility* root = manager->GetRoot();
   return (root->GetFloatAttribute(ui::AX_ATTR_DOC_LOADING_PROGRESS) == 1.0 &&
           root->GetStringAttribute(ui::AX_ATTR_DOC_URL) != url::kAboutBlankURL);
 }
 
-// TODO(nasko): try enabling this test on more platforms once
-// SitePerProcessBrowserTest.CrossSiteIframe is enabled everywhere.
-// http://crbug.com/399775
-#if defined(OS_LINUX) && !defined(OS_CHROMEOS)
+// Times out on Android, not clear if it's an actual bug or just slow.
+#if defined(OS_ANDROID)

[dmazzoni, 2015/01/28 19:29:00]

Currently Android only uses one process for all renderers anyway, so there
doesn't seem to be much point in testing this on Android.

[nasko, 2015/02/03 17:08:18]

Are you sure about that? We do run our site-per-process tests on Android and
those I think verify we use separate processes.

[dmazzoni, 2015/02/03 17:15:29]

I'll dig into it more. Maybe I'll land this change now and then try Android in a
follow-up.

+#define MAYBE_CrossSiteIframeAccessibility DISABLED_CrossSiteIframeAccessibility
+#else
 #define MAYBE_CrossSiteIframeAccessibility CrossSiteIframeAccessibility
-#else
-#define MAYBE_CrossSiteIframeAccessibility DISABLED_CrossSiteIframeAccessibility
 #endif
 IN_PROC_BROWSER_TEST_F(SitePerProcessAccessibilityBrowserTest,
                        MAYBE_CrossSiteIframeAccessibility) {
   // Enable full accessibility for all current and future WebContents.
   BrowserAccessibilityState::GetInstance()->EnableAccessibility();
 
   AccessibilityNotificationWaiter main_frame_accessibility_waiter(
       shell(), AccessibilityModeComplete,
       ui::AX_EVENT_LOAD_COMPLETE);
 
@@ skipping 90 matching lines @@
   BrowserAccessibility* ax_child_frame_static_text =
       ax_child_frame_group->PlatformGetChild(0);
   EXPECT_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.
   EXPECT_EQ(ax_child_frame_root->GetParent(), ax_scroll_area);
 }
 
 }  // namespace content