Enable accessibility testing for the bookmark browser test.

chrome/test/base/in_process_browser_test_browsertest.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <string.h>
 
+#include "base/files/file_util.h"
+#include "base/path_service.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_observer.h"
+#include "net/base/filename_util.h"
 #include "net/base/net_errors.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
 
 class InProcessBrowserTestP
     : public InProcessBrowserTest,
       public ::testing::WithParamInterface<const char*> {
 };
 
@@ skipping 49 matching lines @@
   for (size_t i = 0; i < arraysize(kURLs); ++i) {
     GURL url(kURLs[i]);
     LoadFailObserver observer(contents);
     ui_test_utils::NavigateToURL(browser(), url);
     EXPECT_TRUE(observer.failed_load());
     EXPECT_EQ(net::ERR_NOT_IMPLEMENTED, observer.error_code());
     EXPECT_EQ(url, observer.validated_url());
   }
 }
 
+// Paths are to very simple HTML files. One is accessible, the other is not.
+const base::FilePath kPassHTML(
+    FILE_PATH_LITERAL("chrome/test/data/accessibility_pass.html"));
+const base::FilePath kFailHTML(
+    FILE_PATH_LITERAL("chrome/test/data/accessibility_fail.html"));
+
+/*
+ * This class is meant as a test for the accessibility audit in the
+ * InProcessBrowserTest. These test do NOT validate the accessibility audit,

[Jay Civelli, 2015/02/05 23:29:09]

Typo: These tests

[hcarmona, 2015/02/06 22:50:53]

Done.

+ * just the ability to run it.
+ */
+class InProcessAccessibilityBrowsertest : public InProcessBrowserTest {
+ protected:
+  // Construct a URL from a file path that can be used to get to a web page.
+  base::FilePath BuildURLToFile(const base::FilePath file_path) {

[Jay Civelli, 2015/02/05 23:29:10]

const base::FilePath&

[hcarmona, 2015/02/06 22:50:53]

Done.

+    base::FilePath source_root;
+    if (!PathService::Get(base::DIR_SOURCE_ROOT, &source_root))
+      return base::FilePath();
+    return source_root.Append(file_path);
+  }
+
+  bool NavigateToURL(base::FilePath address) {

[Jay Civelli, 2015/02/05 23:29:09]

const base::FilePath&

[hcarmona, 2015/02/06 22:50:53]

Done.

+    GURL url = net::FilePathToFileURL(BuildURLToFile(address));
+
+    if (!url.is_valid() || url.is_empty() || !browser())
+      return false;
+
+    ui_test_utils::NavigateToURLBlockUntilNavigationsComplete(
+        browser(), url, 1);
+    return true;
+  }
+};
+
+// Test that an accessible page doesn't fail the accessibility audit.
+IN_PROC_BROWSER_TEST_F(
+    InProcessAccessibilityBrowsertest, VerifyAccessibilityPass) {
+
+  ASSERT_TRUE(NavigateToURL(kPassHTML));
+
+  std::string test_result;
+  RunAccessibilityChecks(&test_result);

[Jay Civelli, 2015/02/05 23:29:10]

Would it make sense for RunAccessibilityChecks to also return false when it
fails? (and it would accept nullptr as a param, so if you are not interested in
the test result, the check becomes a one-liner. And then you don't need to
expose kExpectedAccessibilityResults).

[hcarmona, 2015/02/06 22:50:53]

Awesome idea. Done.

+
+  EXPECT_EQ(InProcessBrowserTest::kExpectedAccessibilityResults, test_result);
+}
+
+// Test that a page that is not accessible will fail the accessibility audit.
+IN_PROC_BROWSER_TEST_F(
+    InProcessAccessibilityBrowsertest, VerifyAccessibilityFail) {
+
+  ASSERT_TRUE(NavigateToURL(kFailHTML));
+
+  std::string test_result;
+  RunAccessibilityChecks(&test_result);
+
+  EXPECT_NE(InProcessBrowserTest::kExpectedAccessibilityResults, test_result);
+}
+
 }  // namespace