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

Unified Diff: LayoutTests/fast/pagination/first-letter-inherit-all-crash.html

Issue 1304123006: Applying the "all" property needs to check all properties against the whitelist. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 4 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: LayoutTests/fast/pagination/first-letter-inherit-all-crash.html
diff --git a/LayoutTests/fast/pagination/first-letter-inherit-all-crash.html b/LayoutTests/fast/pagination/first-letter-inherit-all-crash.html
new file mode 100644
index 0000000000000000000000000000000000000000..b23b65ad6241a511ad11b8e303d0d13848215796
--- /dev/null
+++ b/LayoutTests/fast/pagination/first-letter-inherit-all-crash.html
@@ -0,0 +1,20 @@
+<!DOCTYPE html>
+<style>
+ #elm { overflow-y:-webkit-paged-y; }
+
+ /* Need to float the first letter, to turn it into a block. Otherwise, overflow-y wouldn't be
+ an applicable property in the first place. */
+ #elm::first-letter { all:inherit; float:left; }
+</style>
+<p>overflow-y isn't a valid property for ::first-letter. We'd crash when extracting innerText,
+ because the implementation expects that the first letter text is a direct child of the
+ ::first-letter pseudo object. The paged overflow / multicol implementation would violate this
+ assumption by inserting a flow thread object between the ::first-letter pseudo object and the
+ actual text.</p>
+<p>PASS if no crash or assertion failure.</p>
+<div id="elm">x</div>
+<script>
+ if (window.testRunner)
+ testRunner.dumpAsText();
+ document.getElementById("elm").innerText;
+</script>

Powered by Google App Engine
This is Rietveld 408576698