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

Unified Diff: third_party/WebKit/LayoutTests/accessibility/css-styles.html

Issue 2006043002: Improved the reporting of background color information. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added blank line to test expectations. Created 4 years, 7 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: third_party/WebKit/LayoutTests/accessibility/css-styles.html
diff --git a/third_party/WebKit/LayoutTests/accessibility/css-styles.html b/third_party/WebKit/LayoutTests/accessibility/css-styles.html
index 2c73453c233d7e23d0a7175e54ab7efabf52acdb..2c66b05b9ccd4c54fef47b346d2de2461e215c49 100644
--- a/third_party/WebKit/LayoutTests/accessibility/css-styles.html
+++ b/third_party/WebKit/LayoutTests/accessibility/css-styles.html
@@ -16,7 +16,7 @@
font-size: 14px;
}
input {
- background-color: white;
+ background-color: red;
color: blue;
font-size: 16px;
}
@@ -34,25 +34,25 @@
if (window.accessibilityController) {
var axHeading = accessibilityController.accessibleElementById("h1");
- // Black has the value 0x000000.
- shouldBeZero("axHeading.backgroundColor");
- shouldBeEqualToNumber("axHeading.color", 0xff0000);
+ shouldBeEqualToNumber("axHeading.backgroundColor", 0xff000000); // Black
+ shouldBeEqualToNumber("axHeading.color", 0xff0000); // Red
shouldBeEqualToNumber("axHeading.fontSize", 12);
var axParagraph = accessibilityController.accessibleElementById("p");
- shouldBeZero("axParagraph.backgroundColor");
+ shouldBeEqualToNumber("axParagraph.backgroundColor", 0xff000000); // Black
+ // 0xff00 is called "lime" in CSS.
shouldBeEqualToNumber("axParagraph.color", 0xff00);
shouldBeEqualToNumber("axParagraph.fontSize", 14);
var axButton = accessibilityController.accessibleElementById("button");
// Button's background is not transparent, so alpha value is included.
- shouldBeEqualToNumber("axButton.backgroundColor", 0xffffffff);
- shouldBeEqualToNumber("axButton.color", 0xff);
+ shouldBeEqualToNumber("axButton.backgroundColor", 0xffff0000); // Red
+ shouldBeEqualToNumber("axButton.color", 0xff); // Blue
shouldBeEqualToNumber("axButton.fontSize", 16);
var axText = accessibilityController.accessibleElementById("text");
- shouldBeEqualToNumber("axText.backgroundColor", 0xffffffff);
- shouldBeEqualToNumber("axText.color", 0xff);
+ shouldBeEqualToNumber("axText.backgroundColor", 0xffff0000); // Red
+ shouldBeEqualToNumber("axText.color", 0xff); // Blue
shouldBeEqualToNumber("axText.fontSize", 16);
}

Powered by Google App Engine
This is Rietveld 408576698