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

Side by Side Diff: chrome/browser/apps/guest_view/web_view_browsertest.cc

Issue 2674353003: Enable WebViewAccessiblityTest for OOPIF webview. (Closed)
Patch Set: Fixed comment and spacing. Created 3 years, 10 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 <queue> 5 #include <queue>
6 #include <set> 6 #include <set>
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/callback_helpers.h" 9 #include "base/callback_helpers.h"
10 #include "base/files/file_util.h" 10 #include "base/files/file_util.h"
(...skipping 862 matching lines...) Expand 10 before | Expand all | Expand 10 after
873 INSTANTIATE_TEST_CASE_P(WebViewTests, WebViewSizeTest, testing::Bool()); 873 INSTANTIATE_TEST_CASE_P(WebViewTests, WebViewSizeTest, testing::Bool());
874 874
875 class WebViewVisibilityTest : public WebViewTest {}; 875 class WebViewVisibilityTest : public WebViewTest {};
876 INSTANTIATE_TEST_CASE_P(WebViewTests, WebViewVisibilityTest, testing::Bool()); 876 INSTANTIATE_TEST_CASE_P(WebViewTests, WebViewVisibilityTest, testing::Bool());
877 877
878 class WebViewSpeechAPITest : public WebViewTest {}; 878 class WebViewSpeechAPITest : public WebViewTest {};
879 INSTANTIATE_TEST_CASE_P(WebViewTests, WebViewSpeechAPITest, testing::Bool()); 879 INSTANTIATE_TEST_CASE_P(WebViewTests, WebViewSpeechAPITest, testing::Bool());
880 880
881 // The following test suites are created to group tests based on specific 881 // The following test suites are created to group tests based on specific
882 // features of <webview>. 882 // features of <webview>.
883 // These features currently would not work with
884 // --enable-features=GuestViewCrossProcessFrames and is disabled on
885 // GuestViewCrossProcessFrames.
886 // TODO(avallee): https://crbug.com/610795: Enable this for testing::Bool().
887 class WebViewAccessibilityTest : public WebViewTest {}; 883 class WebViewAccessibilityTest : public WebViewTest {};
888 INSTANTIATE_TEST_CASE_P(WebViewTests, 884 INSTANTIATE_TEST_CASE_P(WebViewTests,
889 WebViewAccessibilityTest, 885 WebViewAccessibilityTest,
890 testing::Values(false)); 886 testing::Bool());
891
892 887
893 class WebViewDPITest : public WebViewTest { 888 class WebViewDPITest : public WebViewTest {
894 protected: 889 protected:
895 void SetUpCommandLine(base::CommandLine* command_line) override { 890 void SetUpCommandLine(base::CommandLine* command_line) override {
896 WebViewTest::SetUpCommandLine(command_line); 891 WebViewTest::SetUpCommandLine(command_line);
897 command_line->AppendSwitchASCII(switches::kForceDeviceScaleFactor, 892 command_line->AppendSwitchASCII(switches::kForceDeviceScaleFactor,
898 base::StringPrintf("%f", scale())); 893 base::StringPrintf("%f", scale()));
899 } 894 }
900 895
901 static float scale() { return 2.0f; } 896 static float scale() { return 2.0f; }
(...skipping 3002 matching lines...) Expand 10 before | Expand all | Expand 10 after
3904 ClosingChromeSignInShouldNotCrash) { 3899 ClosingChromeSignInShouldNotCrash) {
3905 GURL signin_url{"chrome://chrome-signin"}; 3900 GURL signin_url{"chrome://chrome-signin"};
3906 3901
3907 AddTabAtIndex(0, signin_url, ui::PAGE_TRANSITION_TYPED); 3902 AddTabAtIndex(0, signin_url, ui::PAGE_TRANSITION_TYPED);
3908 AddTabAtIndex(1, signin_url, ui::PAGE_TRANSITION_TYPED); 3903 AddTabAtIndex(1, signin_url, ui::PAGE_TRANSITION_TYPED);
3909 WaitForWebViewInDom(); 3904 WaitForWebViewInDom();
3910 3905
3911 chrome::CloseTab(browser()); 3906 chrome::CloseTab(browser());
3912 } 3907 }
3913 #endif 3908 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698