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

Unified Diff: LayoutTests/fast/css/first-letter-table.html

Issue 370593002: Remove updateFirstLetter and firstLineBlock from RenderTable. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 5 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
« no previous file with comments | « LayoutTests/TestExpectations ('k') | Source/core/rendering/RenderTable.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/fast/css/first-letter-table.html
diff --git a/LayoutTests/fast/css/first-letter-table.html b/LayoutTests/fast/css/first-letter-table.html
new file mode 100644
index 0000000000000000000000000000000000000000..d7f916dbb8335275671384dfa9e355e836c0ac55
--- /dev/null
+++ b/LayoutTests/fast/css/first-letter-table.html
@@ -0,0 +1,32 @@
+<!doctype html>
+
Julien - ping for review 2014/07/14 15:44:13 Nit: Unneeded empty space.
dsinclair 2014/07/14 15:50:18 Done.
+<script>
Julien - ping for review 2014/07/14 15:44:14 if (window.testRunner) testRunner.dumpAsText()
dsinclair 2014/07/14 15:50:18 Done.
+ function makeChild()
+ {
+ var child = document.createElement('div');
+ child.textContent = 'my text content';
+ return child;
+ }
+
+ function runTest()
+ {
+ document.querySelector('table').appendChild(makeChild());
+ document.querySelector('tbody').appendChild(makeChild());
+ document.querySelector('tr').appendChild(makeChild());
+ document.querySelector('td').appendChild(makeChild());
+ }
+ onload = runTest;
+</script>
+<style>
+ #target {
+ background-color: green;
+ }
+ #target:first-letter, #target div:first-letter { color: red; }
+</style>
+
+<div>This test is successfully if it does not crash.</div>
Julien - ping for review 2014/07/14 15:44:14 Let's add a description about what you are testing
dsinclair 2014/07/14 15:50:18 Better?
+
+<table id="target">
+ a
+ <tr><td>Test</td></tr>
+</table>
« no previous file with comments | « LayoutTests/TestExpectations ('k') | Source/core/rendering/RenderTable.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698