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

Unified Diff: third_party/WebKit/LayoutTests/fast/table/colspan-with-empty-cells-needing-extra-width.html

Issue 1368243002: Don't deduct min-width from the available width for empty cells (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 3 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/fast/table/colspan-with-empty-cells-needing-extra-width.html
diff --git a/third_party/WebKit/LayoutTests/fast/table/colspan-with-empty-cells-needing-extra-width.html b/third_party/WebKit/LayoutTests/fast/table/colspan-with-empty-cells-needing-extra-width.html
new file mode 100644
index 0000000000000000000000000000000000000000..9c058eabcabbe479f2cb8e52fb6be608bda4c851
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/fast/table/colspan-with-empty-cells-needing-extra-width.html
@@ -0,0 +1,19 @@
+<!DOCTYPE html>
+<style>
+table td {
+ padding: 0;
+ margin: 0;
mstensho (USE GERRIT) 2015/09/30 11:09:39 No UA-defined margins on those elements.
+}
+</style>
+<p>crbug.com/534830: Don't deduct the min-width of empty cells from the available table width. </p>
mstensho (USE GERRIT) 2015/09/30 11:09:39 Can you please write a test with a human readable
+<table style="width: 800px; background-color: black;">
mstensho (USE GERRIT) 2015/09/30 11:09:39 This is wider than the initial viewport when start
+ <tr>
+ <td style="background-color:red; width: 10px; height:20px;"></td>
mstensho (USE GERRIT) 2015/09/30 11:09:39 Red means danger. Maybe set the background of the
+ <td style="background-color:blue;"></td>
+ </tr>
+ <tr>
+ <td colspan="2" style="background-color:yellow;">
+ TestTestTest
+ </td>
+ </tr>
+</table>

Powered by Google App Engine
This is Rietveld 408576698