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

Side by Side Diff: chrome/browser/ui/webui/bookmarks_ui_browsertest.cc

Issue 582493002: Enable accessibility testing for the bookmark browser test. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Applied Feedback Created 5 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 (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 "base/test/test_timeouts.h" 5 #include "base/test/test_timeouts.h"
6 #include "chrome/browser/ui/browser.h" 6 #include "chrome/browser/ui/browser.h"
7 #include "chrome/browser/ui/chrome_pages.h" 7 #include "chrome/browser/ui/chrome_pages.h"
8 #include "chrome/browser/ui/tabs/tab_strip_model.h" 8 #include "chrome/browser/ui/tabs/tab_strip_model.h"
9 #include "chrome/common/url_constants.h" 9 #include "chrome/common/url_constants.h"
10 #include "chrome/test/base/in_process_browser_test.h" 10 #include "chrome/test/base/in_process_browser_test.h"
11 #include "chrome/test/base/ui_test_utils.h" 11 #include "chrome/test/base/ui_test_utils.h"
12 #include "content/public/browser/notification_service.h" 12 #include "content/public/browser/notification_service.h"
13 #include "content/public/browser/web_contents.h" 13 #include "content/public/browser/web_contents.h"
14 #include "content/public/test/browser_test_utils.h" 14 #include "content/public/test/browser_test_utils.h"
15 #include "content/public/test/test_navigation_observer.h" 15 #include "content/public/test/test_navigation_observer.h"
16 16
17 class BookmarksTest : public InProcessBrowserTest { 17 class BookmarksTest : public InProcessBrowserTest {
18 public: 18 public:
19 BookmarksTest() {} 19 BookmarksTest() {}
20 20
21 void SetUpOnMainThread() override {
22 InProcessBrowserTest::SetUpOnMainThread();
23 EnableAccessibilityChecksForTestCase(true);
24 };
Jay Civelli 2015/02/07 01:11:12 Nit: no comma needed after the }
hcarmona 2015/02/07 02:22:12 Done.
25
21 void OpenBookmarksManager() { 26 void OpenBookmarksManager() {
22 content::TestNavigationObserver navigation_observer( 27 content::TestNavigationObserver navigation_observer(
23 browser()->tab_strip_model()->GetActiveWebContents(), 2); 28 browser()->tab_strip_model()->GetActiveWebContents(), 2);
24 navigation_observer.StartWatchingNewWebContents(); 29 navigation_observer.StartWatchingNewWebContents();
25 30
26 // Bring up the bookmarks manager tab. 31 // Bring up the bookmarks manager tab.
27 chrome::ShowBookmarkManager(browser()); 32 chrome::ShowBookmarkManager(browser());
28 navigation_observer.Wait(); 33 navigation_observer.Wait();
29 } 34 }
30 35
(...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
98 navigation_observer.Wait(); 103 navigation_observer.Wait();
99 104
100 ASSERT_EQ(1, browser()->tab_strip_model()->count()); 105 ASSERT_EQ(1, browser()->tab_strip_model()->count());
101 } 106 }
102 107
103 IN_PROC_BROWSER_TEST_F(BookmarksTest, BookmarksLoaded) { 108 IN_PROC_BROWSER_TEST_F(BookmarksTest, BookmarksLoaded) {
104 ui_test_utils::NavigateToURL(browser(), GURL(chrome::kChromeUIBookmarksURL)); 109 ui_test_utils::NavigateToURL(browser(), GURL(chrome::kChromeUIBookmarksURL));
105 ASSERT_EQ(1, browser()->tab_strip_model()->count()); 110 ASSERT_EQ(1, browser()->tab_strip_model()->count());
106 AssertIsBookmarksPage(browser()->tab_strip_model()->GetActiveWebContents()); 111 AssertIsBookmarksPage(browser()->tab_strip_model()->GetActiveWebContents());
107 } 112 }
OLDNEW
« no previous file with comments | « no previous file | chrome/test/base/in_process_browser_test.h » ('j') | chrome/test/base/in_process_browser_test.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698