[webkit-reviews] review granted: [Bug 221635] Crash in WebCore::RenderThemeMac::paintSearchFieldResultsDecorationPart() from large scale : [Attachment 419774] Patch

[bugzilla-daemon at webkit.org]

Tim Horton <thorton at apple.com> has granted Julian Gonzalez
<julian_a_gonzalez at apple.com>'s request for review:
Bug 221635: Crash in
WebCore::RenderThemeMac::paintSearchFieldResultsDecorationPart() from large
scale
https://bugs.webkit.org/show_bug.cgi?id=221635

Attachment 419774: Patch

https://bugs.webkit.org/attachment.cgi?id=419774&action=review




--- Comment #2 from Tim Horton <thorton at apple.com> ---
Comment on attachment 419774
  --> https://bugs.webkit.org/attachment.cgi?id=419774
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=419774&action=review

> LayoutTests/platform/mac/editing/style/large-scale-crash-expected.txt:1
> +* { -webkit-appearance: searchfield-results-decoration; transform:
scale(100); writing-mode: vertical-rl; } if (window.testRunner)
testRunner.dumpAsText(); onload = () => { document.styleSheets[0].insertRule(`*
{ all: initial; }`); }; This tests that we do not hit an assertion while
rendering the page. (which prints out its style). PASS

Is this all scrunched on one line for a reason? Usually we format -expected
HTML nicely like we do the real test.