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

Side by Side Diff: chrome/browser/extensions/web_view_interactive_browsertest.cc

Issue 13865008: Fix WebViewInteractiveTest.NewWindow (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Updated Created 7 years, 6 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 | content/browser/browser_plugin/browser_plugin_guest.cc » ('j') | 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) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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 "base/utf_string_conversions.h" 5 #include "base/utf_string_conversions.h"
6 #include "chrome/browser/extensions/extension_test_message_listener.h" 6 #include "chrome/browser/extensions/extension_test_message_listener.h"
7 #include "chrome/browser/extensions/platform_app_browsertest_util.h" 7 #include "chrome/browser/extensions/platform_app_browsertest_util.h"
8 #include "chrome/browser/extensions/shell_window_registry.h" 8 #include "chrome/browser/extensions/shell_window_registry.h"
9 #include "chrome/browser/ui/extensions/shell_window.h" 9 #include "chrome/browser/ui/extensions/shell_window.h"
10 #include "chrome/test/base/interactive_test_utils.h" 10 #include "chrome/test/base/interactive_test_utils.h"
(...skipping 364 matching lines...) Expand 10 before | Expand all | Expand 10 after
375 375
376 // Flush any pending events to make sure we start with a clean slate. 376 // Flush any pending events to make sure we start with a clean slate.
377 content::RunAllPendingInMessageLoop(); 377 content::RunAllPendingInMessageLoop();
378 378
379 ExtensionTestMessageListener start_of_line_listener("StartOfLine", false); 379 ExtensionTestMessageListener start_of_line_listener("StartOfLine", false);
380 SendStartOfLineKeyPressToPlatformApp(); 380 SendStartOfLineKeyPressToPlatformApp();
381 // Wait for the guest to receive a 'copy' edit command. 381 // Wait for the guest to receive a 'copy' edit command.
382 ASSERT_TRUE(start_of_line_listener.WaitUntilSatisfied()); 382 ASSERT_TRUE(start_of_line_listener.WaitUntilSatisfied());
383 } 383 }
384 384
385 // Disabled due to timeouts/crashing on several platforms. crbug.com/241912 385 // Disabled due to timeouts/crashing on several platforms. crbug.com/241912
lazyboy 2013/06/04 15:51:44 Update the comment.
386 IN_PROC_BROWSER_TEST_F(WebViewInteractiveTest, DISABLED_NewWindow) { 386 IN_PROC_BROWSER_TEST_F(WebViewInteractiveTest, NewWindow) {
387 ASSERT_TRUE(StartTestServer()); // For serving guest pages. 387 ASSERT_TRUE(StartTestServer()); // For serving guest pages.
388 ASSERT_TRUE(RunPlatformAppTest("platform_apps/web_view/newwindow")) 388 ASSERT_TRUE(RunPlatformAppTest("platform_apps/web_view/newwindow"))
389 << message_; 389 << message_;
390 } 390 }
391 391
392 IN_PROC_BROWSER_TEST_F(WebViewInteractiveTest, ExecuteCode) { 392 IN_PROC_BROWSER_TEST_F(WebViewInteractiveTest, ExecuteCode) {
393 ASSERT_TRUE(StartTestServer()); // For serving guest pages. 393 ASSERT_TRUE(StartTestServer()); // For serving guest pages.
394 ASSERT_TRUE(RunPlatformAppTest("platform_apps/web_view/execute_code")) 394 ASSERT_TRUE(RunPlatformAppTest("platform_apps/web_view/execute_code"))
395 << message_; 395 << message_;
396 } 396 }
(...skipping 17 matching lines...) Expand all
414 // Started flakily failing after a Blink roll: http://crbug.com/245332 414 // Started flakily failing after a Blink roll: http://crbug.com/245332
415 IN_PROC_BROWSER_TEST_F(WebViewInteractiveTest, DISABLED_PopupPositioningMoved) { 415 IN_PROC_BROWSER_TEST_F(WebViewInteractiveTest, DISABLED_PopupPositioningMoved) {
416 SetupTest( 416 SetupTest(
417 "web_view/popup_positioning_moved", 417 "web_view/popup_positioning_moved",
418 "files/extensions/platform_apps/web_view/popup_positioning_moved" 418 "files/extensions/platform_apps/web_view/popup_positioning_moved"
419 "/guest.html"); 419 "/guest.html");
420 ASSERT_TRUE(guest_web_contents()); 420 ASSERT_TRUE(guest_web_contents());
421 421
422 PopupTestHelper(gfx::Point(20, 0)); 422 PopupTestHelper(gfx::Point(20, 0));
423 } 423 }
OLDNEW
« no previous file with comments | « no previous file | content/browser/browser_plugin/browser_plugin_guest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698