132160 2014-04-24 18:12:34 -0700 [iOS WebKit2] RemoteLayerTreeDrawingArea should force CA to garbage collect surfaces. 2014-04-24 23:29:13 -0700 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 kling kling kling thorton oldest_to_newest 1003922 0 kling 2014-04-24 18:12:34 -0700 Add gratuitous flush for <rdar://problem/16110687> 1003924 1 230125 kling 2014-04-24 18:20:50 -0700 Created attachment 230125 Patch 1003929 2 230125 thorton 2014-04-24 18:27:18 -0700 Comment on attachment 230125 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=230125&action=review > Source/WebKit2/WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm:288 > + [CATransaction begin]; I wonder if it's better to do this in didUpdate() instead; we're more likely to actually do something more immediately useful then, because by that point we would (ideally but not always) have pushed the new front surfaces to CA. 1003980 3 kling 2014-04-24 23:26:18 -0700 (In reply to comment #2) > (From update of attachment 230125 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=230125&action=review > > > Source/WebKit2/WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm:288 > > + [CATransaction begin]; > > I wonder if it's better to do this in didUpdate() instead; we're more likely to actually do something more immediately useful then, because by that point we would (ideally but not always) have pushed the new front surfaces to CA. All right. Will move the "logic" there instead. 1003981 4 kling 2014-04-24 23:29:13 -0700 Committed r167792: <http://trac.webkit.org/changeset/167792> 230125 2014-04-24 18:20:50 -0700 2014-04-24 18:27:17 -0700 Patch bug-132160.diff text/plain 1899 kling ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0No YW5nZUxvZwppbmRleCA0MmZkMWI4Li44ZjYxNzAyIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0 Mi9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTgg QEAKKzIwMTQtMDQtMjQgIEFuZHJlYXMgS2xpbmcgIDxha2xpbmdAYXBwbGUuY29tPgorCisgICAg ICAgIFtpT1MgV2ViS2l0Ml0gUmVtb3RlTGF5ZXJUcmVlRHJhd2luZ0FyZWE6OmZsdXNoTGF5ZXJz KCkgc2hvdWxkIGZvcmNlIENBIHRvIGdhcmJhZ2UgY29sbGVjdCBzdXJmYWNlcy4KKyAgICAgICAg PGh0dHBzOi8vd2Via2l0Lm9yZy9iLzEzMjE2MD4KKyAgICAgICAgPHJkYXI6Ly9wcm9ibGVtLzE2 MTEwNjg3PgorCisgICAgICAgIERvIGFuIGVtcHR5IENBVHJhbnNhY3Rpb24gYXQgdGhlIGVuZCBv ZiBSZW1vdGVMYXllclRyZWVEcmF3aW5nQXJlYTo6Zmx1c2hMYXllcnMoKQorICAgICAgICB0byBn ZXQgQ0EgdG8gZ2FyYmFnZSBjb2xsZWN0IGl0cyBJT1N1cmZhY2VzLiBUaGlzIGhlbHBzIGNsZWFu IHVwIHVudXNlZCBzdXJmYWNlcworICAgICAgICB0aGF0IHdlJ2Qgb3RoZXJ3aXNlIGVuZCB1cCBo YW5naW5nIG9uIHRvIGZvciBhIGxvbmcgdGltZS4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JP RFkgKE9PUFMhKS4KKworICAgICAgICAqIFdlYlByb2Nlc3MvV2ViUGFnZS9tYWMvUmVtb3RlTGF5 ZXJUcmVlRHJhd2luZ0FyZWEubW06CisgICAgICAgIChXZWJLaXQ6OlJlbW90ZUxheWVyVHJlZURy YXdpbmdBcmVhOjpmbHVzaExheWVycyk6CisKIDIwMTQtMDQtMjQgIFRpbSBIb3J0b24gIDx0aW1v dGh5X2hvcnRvbkBhcHBsZS5jb20+CiAKICAgICAgICAgV2ViS2l0MiBWaWV3IEdlc3R1cmVzOiBV c2UgYSBzaW5nbGUgc3RydWN0IGZvciB0aGUgc25hcHNob3QsIGFuZCBwYXNzIGl0IGFyb3VuZApk aWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdDIvV2ViUHJvY2Vzcy9XZWJQYWdlL21hYy9SZW1vdGVM YXllclRyZWVEcmF3aW5nQXJlYS5tbSBiL1NvdXJjZS9XZWJLaXQyL1dlYlByb2Nlc3MvV2ViUGFn ZS9tYWMvUmVtb3RlTGF5ZXJUcmVlRHJhd2luZ0FyZWEubW0KaW5kZXggOTY3ZGNiMi4uY2YwYWUw OCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvV2ViUHJvY2Vzcy9XZWJQYWdlL21hYy9SZW1v dGVMYXllclRyZWVEcmF3aW5nQXJlYS5tbQorKysgYi9Tb3VyY2UvV2ViS2l0Mi9XZWJQcm9jZXNz L1dlYlBhZ2UvbWFjL1JlbW90ZUxheWVyVHJlZURyYXdpbmdBcmVhLm1tCkBAIC00Miw2ICs0Miw3 IEBACiAjaW1wb3J0IDxXZWJDb3JlL1JlbmRlclZpZXcuaD4KICNpbXBvcnQgPFdlYkNvcmUvU2V0 dGluZ3MuaD4KICNpbXBvcnQgPFdlYkNvcmUvVGlsZWRCYWNraW5nLmg+CisjaW1wb3J0IDxRdWFy dHpDb3JlL1F1YXJ0ekNvcmUuaD4KIAogdXNpbmcgbmFtZXNwYWNlIFdlYkNvcmU7CiAKQEAgLTI4 Miw2ICsyODMsMTAgQEAgdm9pZCBSZW1vdGVMYXllclRyZWVEcmF3aW5nQXJlYTo6Zmx1c2hMYXll cnMoKQogCiAgICAgaWYgKGhhZEFueUNoYW5nZWRCYWNraW5nU3RvcmUpCiAgICAgICAgIG1fcmVt b3RlTGF5ZXJUcmVlQ29udGV4dC0+YmFja2luZ1N0b3JlQ29sbGVjdGlvbigpLnNjaGVkdWxlUHVy Z2VhYmlsaXR5VGltZXIoKTsKKworICAgIC8vIFRoaXMgZW1wdHkgdHJhbnNhY3Rpb24gc2VydmVz IHRvIHRyaWdnZXIgQ0EncyBnYXJiYWdlIGNvbGxlY3Rpb24gb2YgSU9TdXJmYWNlcy4gU2VlIDxy ZGFyOi8vcHJvYmxlbS8xNjExMDY4Nz4KKyAgICBbQ0FUcmFuc2FjdGlvbiBiZWdpbl07CisgICAg W0NBVHJhbnNhY3Rpb24gY29tbWl0XTsKIH0KIAogdm9pZCBSZW1vdGVMYXllclRyZWVEcmF3aW5n QXJlYTo6ZGlkVXBkYXRlKCkK