WebKit memory management: Safari jetsams on some websites when zooming and scrolling

[Antti Koivisto]

Summary:
Using the zoom function on a few websites is causing a crash of Safari

Steps to Reproduce:
Go on www.uk-koeln.de or www.huettenhospital.de and use the zoom function a few times in a short time.

[Antti Koivisto]

<rdar://problem/36315010>

[Antti Koivisto]

Created attachment 341572 [details]
patch

[Geoffrey Garen]

Comment on attachment 341572 [details]
patch

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

r=me

> Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm:2896
> -    
> +

Boo whitespace change.

[Simon Fraser (smfr)]

Comment on attachment 341572 [details]
patch

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

> Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm:308
> +    return FloatRect(exposedRect.x() - horizontalChange / 2, exposedRect.y() - verticalChange / 2, exposedRect.width() + horizontalChange, exposedRect.height() + verticalChange);

Use FloatRect::expand()!

[Antti Koivisto]

Created attachment 341657 [details]
patch

[Antti Koivisto]

Created attachment 341658 [details]
patch

[WebKit Commit Bot]

Comment on attachment 341658 [details]
patch

Clearing flags on attachment: 341658

Committed r232356: <https://trac.webkit.org/changeset/232356>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.