Merge pull request #491 from syjer/482-infinite-loop-table

add failing test that highlight the infinite loop issue in the inline box layout algorithm #482
danfickle · May 26, 2020 · ba8dc5c · ba8dc5c
2 parents 75ca103 + 3decd12
commit ba8dc5c
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 0 deletions.
diff --git a/openhtmltopdf-examples/src/main/resources/visualtest/html/issue-482-infinite-loop-table.html b/openhtmltopdf-examples/src/main/resources/visualtest/html/issue-482-infinite-loop-table.html
@@ -0,0 +1,19 @@
+<html>
+<head>
+<style>
+    * {
+        font-family: 'Liberation Sans', sans-serif;
+        margin: 0;
+        padding: 0;
+    }
+
+    .content {
+        word-wrap: break-word;
+        white-space:pre-wrap;
+    }
+</style>
+</head>
+<body>
+<table class="content"><tbody><tr><td width="10"></td></tr></tbody></table>
+</body>
+</html>
diff --git a/...-examples/src/test/java/com/openhtmltopdf/visualregressiontests/VisualRegressionTest.java b/...-examples/src/test/java/com/openhtmltopdf/visualregressiontests/VisualRegressionTest.java
@@ -1154,6 +1154,15 @@ public Reader getReader() {
         })));
     }
 
+    @Test
+    @Ignore
+    public void testIssue482InfiniteLoopTable() throws IOException {
+        assertTrue(vt.runTest("issue-482-infinite-loop-table", builder -> {
+            builder.useFont(() -> VisualRegressionTest.class.getClassLoader().getResourceAsStream("org/apache/pdfbox/resources/ttf/LiberationSans-Regular.ttf"),
+                    "Liberation Sans");
+        }));
+    }
+
     // TODO:
     // + Elements that appear just on generated overflow pages.
     // + content property (page counters, etc)