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

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

Issue 2387533003: Accessibility: Ignore all anonymous blocks (Closed)
Patch Set: Update some Android test results 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 fd13b952a4ec066687cada119173469cdfce6f72..66b3cf63f249147c23ba8313d7be801fb06bce92 100644
--- a/content/browser/accessibility/dump_accessibility_browsertest_base.cc
+++ b/content/browser/accessibility/dump_accessibility_browsertest_base.cc
@@ -197,12 +197,6 @@ 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;
@@ -229,6 +223,13 @@ 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