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

Unified Diff: content/browser/accessibility/dump_accessibility_browsertest_base.cc

Issue 2440833002: Revert of Accessibility: Ignore all anonymous blocks (Closed)
Patch Set: Rebase after revert of another change touching cursors_test.js Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/accessibility/dump_accessibility_browsertest_base.cc
diff --git a/content/browser/accessibility/dump_accessibility_browsertest_base.cc b/content/browser/accessibility/dump_accessibility_browsertest_base.cc
index 66b3cf63f249147c23ba8313d7be801fb06bce92..fd13b952a4ec066687cada119173469cdfce6f72 100644
--- a/content/browser/accessibility/dump_accessibility_browsertest_base.cc
+++ b/content/browser/accessibility/dump_accessibility_browsertest_base.cc
@@ -197,6 +197,12 @@ void DumpAccessibilityTestBase::RunTestForPlatform(
NavigateToURL(shell(), GURL(url::kAboutBlankURL));
+ // Output the test path to help anyone who encounters a failure and needs
+ // to know where to look.
+ LOG(INFO) << "Testing: " << file_path.LossyDisplayName()
+ << (is_blink_pass_ ? " (internal Blink accessibility tree)"
+ : " (native accessibility tree for this platform)");
+
std::string html_contents;
base::FilePath expected_file;
std::string expected_contents_raw;
@@ -223,13 +229,6 @@ void DumpAccessibilityTestBase::RunTestForPlatform(
base::ReadFileToString(expected_file, &expected_contents_raw);
}
- // Output the test path to help anyone who encounters a failure and needs
- // to know where to look.
- LOG(INFO) << "Testing: "
- << file_path.NormalizePathSeparatorsTo('/').LossyDisplayName();
- LOG(INFO) << "Expected output: "
- << expected_file.NormalizePathSeparatorsTo('/').LossyDisplayName();
-
// Tolerate Windows-style line endings (\r\n) in the expected file:
// normalize by deleting all \r from the file (if any) to leave only \n.
std::string expected_contents;

Powered by Google App Engine
This is Rietveld 408576698