RESOLVED FIXED 189302
REGRESSION: -webkit-appearance test case crashes 
https://bugs.webkit.org/show_bug.cgi?id=189302
Summary REGRESSION: -webkit-appearance test case crashes
Simon Pieters (:zcorpan)
Reported 2018-09-05 05:23:50 PDT
https://bug1368555.bmoattachments.org/attachment.cgi?id=8923730 Scroll somewhere to the right. Safari TP release 64 (Safari 12.1, WebKit 13607.1.3.3) crashes. Safari stable 11.1.2 does not crash.
Attachments
Test reduction (84 bytes, text/html)
2019-01-02 14:16 PST, zalan 		no flags
Details
Patch (4.72 KB, patch)
2019-01-03 14:33 PST, zalan 		no flags
DetailsFormatted DiffDiff
Patch (4.70 KB, patch)
2019-01-03 14:52 PST, zalan 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1 2018-09-05 10:06:31 PDT
Thread 0 Crashed:: Dispatch queue: com.apple.main-thread 0 com.apple.WebCore 0x00000001413b1913 WebCore::RenderThemeMac::paintSearchFieldCancelButton(WebCore::RenderBox const&, WebCore::PaintInfo const&, WebCore::IntRect const&) + 1363 1 com.apple.WebCore 0x000000014221c65c WebCore::RenderTheme::paint(WebCore::RenderBox const&, WebCore::ControlStates&, WebCore::PaintInfo const&, WebCore::LayoutRect const&) + 1564 2 com.apple.WebCore 0x0000000140df3fef WebCore::RenderBox::paintBoxDecorations(WebCore::PaintInfo&, WebCore::LayoutPoint const&) + 943 3 com.apple.WebCore 0x0000000140df143b WebCore::RenderBlock::paintObject(WebCore::PaintInfo&, WebCore::LayoutPoint const&) + 75 ...
Radar WebKit Bug Importer
Comment 2 2018-09-05 10:06:48 PDT
<rdar://problem/44143049>
zalan
Comment 3 2019-01-02 14:16:40 PST
Created attachment 358204 [details] Test reduction
zalan
Comment 4 2019-01-03 14:33:29 PST
Created attachment 358278 [details] Patch
Wenson Hsieh
Comment 5 2019-01-03 14:38:19 PST
Comment on attachment 358278 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=358278&action=review > Source/WebCore/rendering/RenderThemeMac.mm:2080 > + // Set horizontal position back to based on the content direction (cancelButtonRectForBounds() only knows about system direction). Nit - I think you missed a word in "…back to based on the…"
zalan
Comment 6 2019-01-03 14:52:21 PST
Created attachment 358280 [details] Patch
WebKit Commit Bot
Comment 7 2019-01-03 18:34:30 PST
Comment on attachment 358280 [details] Patch Clearing flags on attachment: 358280 Committed r239604: <https://trac.webkit.org/changeset/239604>
WebKit Commit Bot
Comment 8 2019-01-03 18:34:31 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.