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

Unified Diff: third_party/WebKit/LayoutTests/paint/tables/table-cell-changes-to-colgroup.html

Issue 2532283002: Minor test cleanup of table-cell-changes-to-colgroup.html (Closed)
Patch Set: Created 4 years, 1 month 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
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/paint/tables/table-cell-changes-to-colgroup-expected.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/paint/tables/table-cell-changes-to-colgroup.html
diff --git a/third_party/WebKit/LayoutTests/paint/tables/table-cell-changes-to-colgroup.html b/third_party/WebKit/LayoutTests/paint/tables/table-cell-changes-to-colgroup.html
index d1444acc6737650e8ac1d36d02436d4b88bbde15..aeff4715bf140609db6de68b3bcade1888ac2877 100644
--- a/third_party/WebKit/LayoutTests/paint/tables/table-cell-changes-to-colgroup.html
+++ b/third_party/WebKit/LayoutTests/paint/tables/table-cell-changes-to-colgroup.html
@@ -2,30 +2,30 @@
<script src="../../resources/run-after-layout-and-paint.js"></script>
<!-- Test passes if it does not crash on an ASAN build. -->
<style>
-.c7 {
+.tableColumnGroup {
background: white;
display: table-column-group;
}
-.c13 {
+.tableCell {
display: table-cell;
border-style: ridge;
will-change: transform;
}
</style>
<script>
-var label = document.createElement('label');
-label.setAttribute('class', 'c13');
-document.documentElement.appendChild(label);
-var colgroup = document.createElement('colgroup');
-colgroup.setAttribute('class', 'c13');
-document.documentElement.appendChild(colgroup);
-var aside = document.createElement('aside');
-aside.setAttribute('class', 'c13');
-document.documentElement.appendChild(aside);
+function createAndAppendWithClass(targetType, targetClass) {
+ var targetElement = document.createElement(targetType);
+ targetElement.setAttribute('class', targetClass);
+ document.documentElement.appendChild(targetElement);
+ return targetElement;
+}
+var label = createAndAppendWithClass('label', 'tableCell');
+var colgroup = createAndAppendWithClass('colgroup', 'tableCell');
+createAndAppendWithClass('aside', 'tableCell');
if (window.testRunner)
testRunner.waitUntilDone();
runAfterLayoutAndPaint(function() {
- colgroup.setAttribute('class', 'c11');
- label.setAttribute('class', 'c7');
+ colgroup.setAttribute('class', '');
+ label.setAttribute('class', 'tableColumnGroup');
}, true);
</script>
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/paint/tables/table-cell-changes-to-colgroup-expected.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698