LayoutTests/virtual/fasttextautosizing/fast/text-autosizing/tables/nested-tables.html - Issue 236033005: [FastTextAutosizer] Do not inflate table parts that do not need layout

Side by Side Diff: LayoutTests/virtual/fasttextautosizing/fast/text-autosizing/tables/nested-tables.html

Issue 236033005: [FastTextAutosizer] Do not inflate table parts that do not need layout (Closed) Base URL: svn://svn.chromium.org/blink/trunk

Patch Set: Fix TestExpectations Created 6 years, 8 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 unified diff | Download patch | Annotate | Revision Log

« no previous file with comments | « LayoutTests/platform/mac/virtual/fasttextautosizing/fast/text-autosizing/tables/nested-tables-expected.txt ('k') | Source/core/rendering/FastTextAutosizer.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 <!DOCTYPE html>	1 <!DOCTYPE html>

2 <html>	2 <html>

3 <head>	3 <head>

4 <meta name="viewport" content="width=800">	4 <meta name="viewport" content="width=800">

5 <style>	5 <style>

6 body {	6 body {

7 width: 800px;	7 width: 800px;

8 margin: 0;	8 margin: 0;

9 overflow-y: hidden;	9 overflow-y: hidden;

10 font-size: 12px;	10 font-size: 12px;

(...skipping 14 matching lines...) Expand all Loading...
25 }	25 }

26 </style>	26 </style>

27	27

28 <script>	28 <script>

29 if (window.internals) {	29 if (window.internals) {

30 window.internals.settings.setTextAutosizingEnabled(true);	30 window.internals.settings.setTextAutosizingEnabled(true);

31 window.internals.settings.setTextAutosizingWindowSizeOverride(320, 480);	31 window.internals.settings.setTextAutosizingWindowSizeOverride(320, 480);

32 } else if (window.console && console.warn) {	32 } else if (window.console && console.warn) {

33 console.warn("This test depends on the Text Autosizing setting being true, s o run it in DumpRenderTree, or manually enable Text Autosizing, and either use a mobile device with 320px device-width (like Nexus S or iPhone), or define HACK_ FORCE_TEXT_AUTOSIZING_ON_DESKTOP.");	33 console.warn("This test depends on the Text Autosizing setting being true, s o run it in DumpRenderTree, or manually enable Text Autosizing, and either use a mobile device with 320px device-width (like Nexus S or iPhone), or define HACK_ FORCE_TEXT_AUTOSIZING_ON_DESKTOP.");

34 }	34 }

	35

	36 if (window.testRunner) {

	37 testRunner.waitUntilDone();

	38

	39 window.requestAnimationFrame(function() {

	40 testRunner.notifyDone();

	41 });

	42 }

35 </script>	43 </script>

36	44

37 </head>	45 </head>

38 <body>	46 <body>

39 <div id="description">	47 <div id="description">

40 Table autosizing tests - narrow-specified-width.html<br/>	48 Table autosizing tests - nested-tables.html<br/>

41 This test passes if none of the 6 cells are autosized.	49 This test passes if "This text should not overlap itself" doesn't overlap itself .

42 </div>	50 </div>

43 <table width="200px">	51 <table>

44 <tr>	52 <tr>

45 <td>hello</td>	53 <td>

46 <td>hello</td>	54 Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosiz e Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosize Autosi ze Autosize

47 <td>hello</td>	55 <table border="1">

48 </tr>	56 <tr>

49 <tr>	57 <td>This text<br/>should not overlap<br/>itself.</td>

50 <td>	58 </tr>

51 hello	59 <tr>

52 </td>	60 <td><img src="broken_image" width="5" height="5" /></td>

53 <td>	61 </tr>

54 hello hello hello hello hello hello hello hello hello hello hello hello he llo hello hello hello hello hello hello hello hello hello hello hello hello hell o	62 </table>

55 </td>	63 </td>

56 <td>	64 </tr>

57 hello	65 </table>

58 </td>

59 </tr>

60 </table>

61 </body>	66 </body>

62 </html>	67 </html>

OLD	NEW