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

Unified Diff: LayoutTests/platform/mac/fast/table/giantRowspan2-expected.txt

Issue 181573003: Auto-rebaseline for r168326 (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 10 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/platform/mac/fast/table/giantRowspan2-expected.txt
diff --git a/LayoutTests/platform/mac/fast/table/giantRowspan2-expected.txt b/LayoutTests/platform/mac/fast/table/giantRowspan2-expected.txt
index 309b696c3f472350637c8a7be00a9b8f75427649..b747b7208dfb57c20818bf9aef2995412cdefcf4 100644
--- a/LayoutTests/platform/mac/fast/table/giantRowspan2-expected.txt
+++ b/LayoutTests/platform/mac/fast/table/giantRowspan2-expected.txt
@@ -3,15 +3,15 @@ layer at (0,0) size 800x600
layer at (0,0) size 800x600
RenderBlock {HTML} at (0,0) size 800x600
RenderBody {BODY} at (8,8) size 784x584
- RenderTable {TABLE} at (0,0) size 784x4
- RenderTableSection {TBODY} at (0,0) size 784x4
+ RenderTable {TABLE} at (0,0) size 784x76
+ RenderTableSection {TBODY} at (0,0) size 784x76
RenderTableRow {TR} at (0,2) size 784x0
- RenderTableCell {TD} at (2,-52) size 780x109 [r=0 c=0 rs=8190 cs=1]
- RenderText {#text} at (1,1) size 778x107
+ RenderTableCell {TD} at (2,1) size 780x74 [r=0 c=0 rs=8190 cs=1]
+ RenderText {#text} at (1,1) size 778x72
text run at (1,1) width 673: "This test succeeds if it does not crash. We implemented a heuristic a while back to prevent giant rowspans. "
text run at (674,1) width 82: "The heuristic"
- text run at (1,54) width 696: "did not, however, fix this case. This rowpsan was small enough so that we thought it was okay to allocate, but "
- text run at (697,54) width 82: "large enough"
- text run at (1,72) width 684: "so that it actually wasn't okay, and we crashed. Firefox seems to hold 8k as the maximum size for rowspans. "
- text run at (685,72) width 92: "So this change"
- text run at (1,90) width 413: "matches Firefox and clamps all rowspans to 8k right in the DOM."
+ text run at (1,19) width 696: "did not, however, fix this case. This rowpsan was small enough so that we thought it was okay to allocate, but "
+ text run at (697,19) width 82: "large enough"
+ text run at (1,37) width 684: "so that it actually wasn't okay, and we crashed. Firefox seems to hold 8k as the maximum size for rowspans. "
+ text run at (685,37) width 92: "So this change"
+ text run at (1,55) width 413: "matches Firefox and clamps all rowspans to 8k right in the DOM."

Powered by Google App Engine
This is Rietveld 408576698