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

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

Issue 23514016: <webview>: Cleanup WebRequest event listeners when embedder destroyed (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: More fixes Created 7 years, 3 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
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 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 "apps/shell_window.h" 5 #include "apps/shell_window.h"
6 #include "apps/shell_window_registry.h" 6 #include "apps/shell_window_registry.h"
7 #include "base/strings/stringprintf.h" 7 #include "base/strings/stringprintf.h"
8 #include "base/strings/utf_string_conversions.h" 8 #include "base/strings/utf_string_conversions.h"
9 #include "chrome/browser/apps/app_browsertest_util.h" 9 #include "chrome/browser/apps/app_browsertest_util.h"
10 #include "chrome/browser/extensions/extension_test_message_listener.h" 10 #include "chrome/browser/extensions/extension_test_message_listener.h"
(...skipping 558 matching lines...) Expand 10 before | Expand all | Expand 10 after
569 } 569 }
570 570
571 IN_PROC_BROWSER_TEST_F(WebViewInteractiveTest, 571 IN_PROC_BROWSER_TEST_F(WebViewInteractiveTest,
572 NewWindow_WebRequestRemoveElement) { 572 NewWindow_WebRequestRemoveElement) {
573 TestHelper("testNewWindowWebRequestRemoveElement", 573 TestHelper("testNewWindowWebRequestRemoveElement",
574 "DoneNewWindowTest.PASSED", 574 "DoneNewWindowTest.PASSED",
575 "DoneNewWindowTest.FAILED", 575 "DoneNewWindowTest.FAILED",
576 "web_view/newwindow"); 576 "web_view/newwindow");
577 } 577 }
578 578
579 IN_PROC_BROWSER_TEST_F(WebViewInteractiveTest,
lazyboy 2013/08/30 17:10:11 A bug number here for reference to added DISABLED_
580 DISABLED_NewWindow_WebRequestCloseWindow) {
581 TestHelper("testNewWindowWebRequestCloseWindow",
582 "DoneNewWindowTest.PASSED",
583 "DoneNewWindowTest.FAILED",
584 "web_view/newwindow");
585 }
586
579 IN_PROC_BROWSER_TEST_F(WebViewInteractiveTest, ExecuteCode) { 587 IN_PROC_BROWSER_TEST_F(WebViewInteractiveTest, ExecuteCode) {
580 ASSERT_TRUE(StartEmbeddedTestServer()); // For serving guest pages. 588 ASSERT_TRUE(StartEmbeddedTestServer()); // For serving guest pages.
581 ASSERT_TRUE(RunPlatformAppTestWithArg( 589 ASSERT_TRUE(RunPlatformAppTestWithArg(
582 "platform_apps/web_view/common", "execute_code")) << message_; 590 "platform_apps/web_view/common", "execute_code")) << message_;
583 } 591 }
584 592
585 // This test used the old Autofill UI, which has been removed. 593 // This test used the old Autofill UI, which has been removed.
586 // See crbug.com/259438 594 // See crbug.com/259438
587 IN_PROC_BROWSER_TEST_F(WebViewInteractiveTest, DISABLED_PopupPositioning) { 595 IN_PROC_BROWSER_TEST_F(WebViewInteractiveTest, DISABLED_PopupPositioning) {
588 SetupTest( 596 SetupTest(
(...skipping 87 matching lines...) Expand 10 before | Expand all | Expand 10 after
676 ASSERT_TRUE(done_listener.WaitUntilSatisfied()); 684 ASSERT_TRUE(done_listener.WaitUntilSatisfied());
677 } 685 }
678 686
679 IN_PROC_BROWSER_TEST_F(WebViewInteractiveTest, 687 IN_PROC_BROWSER_TEST_F(WebViewInteractiveTest,
680 PointerLock_PointerLockLostWithFocus) { 688 PointerLock_PointerLockLostWithFocus) {
681 TestHelper("testPointerLockLostWithFocus", 689 TestHelper("testPointerLockLostWithFocus",
682 "DonePointerLockTest.PASSED", 690 "DonePointerLockTest.PASSED",
683 "DonePointerLockTest.FAILED", 691 "DonePointerLockTest.FAILED",
684 "web_view/pointerlock"); 692 "web_view/pointerlock");
685 } 693 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698