Bug 196743 – [AutoSizing] Avoid making text paragraphs scroll horizontally when there is a wide table

• Component: Layout and Rendering
• Status: RESOLVED FIXED

[zalan]

<rdar://problem/43897551>

[zalan]

Created attachment 367072 [details]
Patch

[zalan]

Committed r244098: <https://trac.webkit.org/changeset/244098>

[Radar WebKit Bug Importer]

<rdar://problem/49756050>

[Truitt Savell]

It looks like the changes in https://trac.webkit.org/changeset/244098/webkit

are most likely responsible for the test fast/dynamic/paused-event-dispatch.html becoming a timeout. 

History:
https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=fast%2Fdynamic%2Fpaused-event-dispatch.html

build:
https://build.webkit.org/builders/Apple%20Mojave%20Debug%20WK1%20(Tests)/builds/2992

[zalan]

(In reply to Truitt Savell from comment #4)
> It looks like the changes in https://trac.webkit.org/changeset/244098/webkit
> 
> are most likely responsible for the test
> fast/dynamic/paused-event-dispatch.html becoming a timeout. 
> 
> History:
> https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.
> html#showAllRuns=true&tests=fast%2Fdynamic%2Fpaused-event-dispatch.html
> 
> build:
> https://build.webkit.org/builders/Apple%20Mojave%20Debug%20WK1%20(Tests)/
> builds/2992
That's quite surprising because this testcase does not exercise auto-sizing at all.

[Truitt Savell]

We were able to reproduce this failure using command:

run-webkit-tests fast/dynamic/crash-subtree-layout-when-auto-size-enabled.html fast/dynamic/paused-event-dispatch.html  -1 --no-retry

running the test fast/dynamic/crash-subtree-layout-when-auto-size-enabled.html, added in 244098, in front of the timing out test causes the timeout.

[zalan]

(In reply to Truitt Savell from comment #6)
> We were able to reproduce this failure using command:
> 
> run-webkit-tests
> fast/dynamic/crash-subtree-layout-when-auto-size-enabled.html
> fast/dynamic/paused-event-dispatch.html  -1 --no-retry
> 
> running the test
> fast/dynamic/crash-subtree-layout-when-auto-size-enabled.html, added in
> 244098, in front of the timing out test causes the timeout.
Ok. Thanks! I'll look into it.

[zalan]

This patch actually revealed a bug in the test framework. We don't reset the auto-sizing mode for subsequent runs.

[Truitt Savell]

*** Bug 196789 has been marked as a duplicate of this bug. ***

[Truitt Savell]

Tracking the test failure and underlying framework bug in https://bugs.webkit.org/show_bug.cgi?id=196789