Chromium Code Reviews| Index: content/browser/accessibility/site_per_process_accessibility_browsertest.cc |
| diff --git a/content/browser/accessibility/site_per_process_accessibility_browsertest.cc b/content/browser/accessibility/site_per_process_accessibility_browsertest.cc |
| index 6daf5abbeb5007ed4610f08043f3367698e9ce80..13eb5897757e6c4f23218ecc87ccb65e8ef9e223 100644 |
| --- a/content/browser/accessibility/site_per_process_accessibility_browsertest.cc |
| +++ b/content/browser/accessibility/site_per_process_accessibility_browsertest.cc |
| @@ -46,13 +46,11 @@ bool AccessibilityTreeIsLoaded(BrowserAccessibilityManager* manager) { |
| 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) |
| -#define MAYBE_CrossSiteIframeAccessibility CrossSiteIframeAccessibility |
| -#else |
| +// 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 re
nasko
2015/02/03 17:08:18
Are you sure about that? We do run our site-per-pr
dmazzoni
2015/02/03 17:15:29
I'll dig into it more. Maybe I'll land this change
|
| #define MAYBE_CrossSiteIframeAccessibility DISABLED_CrossSiteIframeAccessibility |
| +#else |
| +#define MAYBE_CrossSiteIframeAccessibility CrossSiteIframeAccessibility |
| #endif |
| IN_PROC_BROWSER_TEST_F(SitePerProcessAccessibilityBrowserTest, |
| MAYBE_CrossSiteIframeAccessibility) { |