131775 2014-04-16 16:47:54 -0700 Images dragged from WebKit1 on 2x displays are too big 2014-04-16 17:00:37 -0700 1 1 1 Unclassified WebKit WebKit Misc. 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 thorton thorton burg simon.fraser oldest_to_newest 1001396 0 thorton 2014-04-16 16:47:54 -0700 WebCore hands us an NSImage with large logical size; WebKit2 scales down by the page's device scale factor; WebKit1 does not. WebCore handing WebKit2 an image with the right logical size won't work, because WebKit2 has to put it in a ShareableBitmap and throw it across the process boundary. So, instead, scale it down in WebKit1 just like we do in WebKit2. <rdar://problem/15933525> 1001401 1 229499 thorton 2014-04-16 16:52:04 -0700 Created attachment 229499 patch 1001406 2 thorton 2014-04-16 17:00:37 -0700 http://trac.webkit.org/changeset/167403 229499 2014-04-16 16:52:04 -0700 2014-04-16 16:56:00 -0700 patch drag-image-scale.diff text/plain 1536 thorton ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvbWFjL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQv bWFjL0NoYW5nZUxvZwppbmRleCAzNWI5YjI3Li4xZjQxZDA5IDEwMDY0NAotLS0gYS9Tb3VyY2Uv V2ViS2l0L21hYy9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdC9tYWMvQ2hhbmdlTG9nCkBA IC0xLDMgKzEsMTYgQEAKKzIwMTQtMDQtMTYgIFRpbSBIb3J0b24gIDx0aW1vdGh5X2hvcnRvbkBh cHBsZS5jb20+CisKKyAgICAgICAgSW1hZ2VzIGRyYWdnZWQgZnJvbSBXZWJLaXQxIG9uIDJ4IGRp c3BsYXlzIGFyZSB0b28gYmlnCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3df YnVnLmNnaT9pZD0xMzE3NzUKKyAgICAgICAgPHJkYXI6Ly9wcm9ibGVtLzE1OTMzNTI1PgorCisg ICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogV2ViQ29yZVN1 cHBvcnQvV2ViRHJhZ0NsaWVudC5tbToKKyAgICAgICAgKFdlYkRyYWdDbGllbnQ6OnN0YXJ0RHJh Zyk6CisgICAgICAgIFNjYWxlIHRoZSBkcmFnIGltYWdlJ3MgbG9naWNhbCBzaXplIGRvd24gYnkg dGhlIGRldmljZSBzY2FsZSBmYWN0b3IKKyAgICAgICAgaW4gV2ViS2l0MSBqdXN0IGxpa2Ugd2Ug ZG8gaW4gV2ViS2l0Mi4KKwogMjAxNC0wNC0xMyAgQW5keSBFc3RlcyAgPGFlc3Rlc0BhcHBsZS5j b20+CiAKICAgICAgICAgW1F1aWNrTG9va10gTW92ZSBmaWxlIHN5c3RlbS1yZWxhdGVkIGNvZGUg aW50byBXZWJLaXQKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvbWFjL1dlYkNvcmVTdXBwb3J0 L1dlYkRyYWdDbGllbnQubW0gYi9Tb3VyY2UvV2ViS2l0L21hYy9XZWJDb3JlU3VwcG9ydC9XZWJE cmFnQ2xpZW50Lm1tCmluZGV4IDgwN2Q2MDguLjk2ODIzZmYgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9X ZWJLaXQvbWFjL1dlYkNvcmVTdXBwb3J0L1dlYkRyYWdDbGllbnQubW0KKysrIGIvU291cmNlL1dl YktpdC9tYWMvV2ViQ29yZVN1cHBvcnQvV2ViRHJhZ0NsaWVudC5tbQpAQCAtMTA1LDYgKzEwNSwx MCBAQCB2b2lkIFdlYkRyYWdDbGllbnQ6OnN0YXJ0RHJhZyhEcmFnSW1hZ2VSZWYgZHJhZ0ltYWdl LCBjb25zdCBJbnRQb2ludCYgYXQsIGNvbnN0CiAgICAgTlNJbWFnZSAqZHJhZ05TSW1hZ2UgPSBk cmFnSW1hZ2UuZ2V0KCk7CiAgICAgV2ViSFRNTFZpZXcgKnNvdXJjZUhUTUxWaWV3ID0gaHRtbFZp ZXcuZ2V0KCk7CiAKKyAgICBJbnRTaXplIHNpemUoW2RyYWdOU0ltYWdlIHNpemVdKTsKKyAgICBz aXplLnNjYWxlKDEgLyBmcmFtZS5wYWdlKCktPmRldmljZVNjYWxlRmFjdG9yKCkpOworICAgIFtk cmFnTlNJbWFnZSBzZXRTaXplOnNpemVdOworCiAgICAgaWQgZGVsZWdhdGUgPSBbbV93ZWJWaWV3 IFVJRGVsZWdhdGVdOwogICAgIFNFTCBzZWxlY3RvciA9IEBzZWxlY3Rvcih3ZWJWaWV3OmRyYWdJ bWFnZTphdDpvZmZzZXQ6ZXZlbnQ6cGFzdGVib2FyZDpzb3VyY2U6c2xpZGVCYWNrOmZvclZpZXc6 KTsKICAgICBpZiAoW2RlbGVnYXRlIHJlc3BvbmRzVG9TZWxlY3RvcjpzZWxlY3Rvcl0pIHsK