Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5)

Side by Side Diff: content/browser/site_per_process_browsertest.cc

Issue 1847303002: Disable two flakily timing out SitePerProcessBrowserTests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix link in comment Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/site_per_process_browsertest.h" 5 #include "content/browser/site_per_process_browsertest.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <algorithm> 10 #include <algorithm>
(...skipping 4922 matching lines...) Expand 10 before | Expand all | Expand 10 after
4933 // The test sets up six inputs fields in a page with two cross-process 4933 // The test sets up six inputs fields in a page with two cross-process
4934 // subframes: 4934 // subframes:
4935 // child1 child2 4935 // child1 child2
4936 // /------------\ /------------\. 4936 // /------------\ /------------\.
4937 // | 2. <input> | | 4. <input> | 4937 // | 2. <input> | | 4. <input> |
4938 // 1. <input> | 3. <input> | | 5. <input> | 6. <input> 4938 // 1. <input> | 3. <input> | | 5. <input> | 6. <input>
4939 // \------------/ \------------/. 4939 // \------------/ \------------/.
4940 // 4940 //
4941 // The test then presses <tab> six times to cycle through focused elements 1-6. 4941 // The test then presses <tab> six times to cycle through focused elements 1-6.
4942 // The test then repeats this with <shift-tab> to cycle in reverse order. 4942 // The test then repeats this with <shift-tab> to cycle in reverse order.
4943 IN_PROC_BROWSER_TEST_F(SitePerProcessBrowserTest, SequentialFocusNavigation) { 4943
4944 // Freqently times out. https://crbug.com/599730.
4945 IN_PROC_BROWSER_TEST_F(SitePerProcessBrowserTest,
4946 DISABLED_SequentialFocusNavigation) {
4944 GURL main_url(embedded_test_server()->GetURL( 4947 GURL main_url(embedded_test_server()->GetURL(
4945 "a.com", "/cross_site_iframe_factory.html?a(b,c)")); 4948 "a.com", "/cross_site_iframe_factory.html?a(b,c)"));
4946 EXPECT_TRUE(NavigateToURL(shell(), main_url)); 4949 EXPECT_TRUE(NavigateToURL(shell(), main_url));
4947 4950
4948 WebContents* contents = shell()->web_contents(); 4951 WebContents* contents = shell()->web_contents();
4949 FrameTreeNode* root = 4952 FrameTreeNode* root =
4950 static_cast<WebContentsImpl*>(contents)->GetFrameTree()->root(); 4953 static_cast<WebContentsImpl*>(contents)->GetFrameTree()->root();
4951 4954
4952 // Assign a name to each frame. This will be sent along in test messages 4955 // Assign a name to each frame. This will be sent along in test messages
4953 // from focus events. 4956 // from focus events.
(...skipping 322 matching lines...) Expand 10 before | Expand all | Expand 10 after
5276 5279
5277 // Test that clicking a select element in a nested out-of-process iframe creates 5280 // Test that clicking a select element in a nested out-of-process iframe creates
5278 // a popup menu in the correct position, even if the top-level page repositions 5281 // a popup menu in the correct position, even if the top-level page repositions
5279 // its out-of-process iframe. This verifies that screen positioning information 5282 // its out-of-process iframe. This verifies that screen positioning information
5280 // is propagating down the frame tree correctly. 5283 // is propagating down the frame tree correctly.
5281 #if defined(OS_ANDROID) 5284 #if defined(OS_ANDROID)
5282 // Surface-based hit testing and coordinate translation is not yet avaiable on 5285 // Surface-based hit testing and coordinate translation is not yet avaiable on
5283 // Android. 5286 // Android.
5284 #define MAYBE_NestedPopupMenuTest DISABLED_NestedPopupMenuTest 5287 #define MAYBE_NestedPopupMenuTest DISABLED_NestedPopupMenuTest
5285 #else 5288 #else
5286 #define MAYBE_NestedPopupMenuTest NestedPopupMenuTest 5289 // Times out frequently. https://crbug.com/599730.
5290 #define MAYBE_NestedPopupMenuTest DISABLED_NestedPopupMenuTest
5287 #endif 5291 #endif
5288 IN_PROC_BROWSER_TEST_F(SitePerProcessBrowserTest, MAYBE_NestedPopupMenuTest) { 5292 IN_PROC_BROWSER_TEST_F(SitePerProcessBrowserTest, MAYBE_NestedPopupMenuTest) {
5289 GURL main_url(embedded_test_server()->GetURL( 5293 GURL main_url(embedded_test_server()->GetURL(
5290 "/cross_site_iframe_factory.html?a(b(c))")); 5294 "/cross_site_iframe_factory.html?a(b(c))"));
5291 NavigateToURL(shell(), main_url); 5295 NavigateToURL(shell(), main_url);
5292 5296
5293 FrameTreeNode* root = static_cast<WebContentsImpl*>(shell()->web_contents()) 5297 FrameTreeNode* root = static_cast<WebContentsImpl*>(shell()->web_contents())
5294 ->GetFrameTree() 5298 ->GetFrameTree()
5295 ->root(); 5299 ->root();
5296 5300
(...skipping 1009 matching lines...) Expand 10 before | Expand all | Expand 10 after
6306 EXPECT_EQ(b_url, root->current_url()); 6310 EXPECT_EQ(b_url, root->current_url());
6307 6311
6308 // Verify that the same RenderViewHost is preserved and that it is no longer 6312 // Verify that the same RenderViewHost is preserved and that it is no longer
6309 // in swapped out state. 6313 // in swapped out state.
6310 EXPECT_EQ(rvh, contents->GetFrameTree()->GetRenderViewHost( 6314 EXPECT_EQ(rvh, contents->GetFrameTree()->GetRenderViewHost(
6311 root->current_frame_host()->GetSiteInstance())); 6315 root->current_frame_host()->GetSiteInstance()));
6312 EXPECT_FALSE(rvh->is_swapped_out_); 6316 EXPECT_FALSE(rvh->is_swapped_out_);
6313 } 6317 }
6314 6318
6315 } // namespace content 6319 } // namespace content
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698