86246 2012-05-11 13:06:42 -0700 FrameView->m_lastPaintTime is not updated in the tiled drawing case 2012-05-11 13:16:51 -0700 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 thorton webkit-unassigned andersca eric koivisto simon.fraser webkit.review.bot oldest_to_newest 621914 0 thorton 2012-05-11 13:06:42 -0700 FrameView->m_lastPaintTime is only updated from FrameView::paintContents, not RenderLayerBacking::paintContents. This causes m_lastPaintTime to never get updated in the tiled drawing case. This can cause horrible things, like the HTML parser incessantly yielding even after the page has always painted, on a page with lots of inline <script>s, etc. 621917 1 141481 thorton 2012-05-11 13:11:49 -0700 Created attachment 141481 patch 621921 2 thorton 2012-05-11 13:16:30 -0700 http://trac.webkit.org/changeset/116799 621922 3 thorton 2012-05-11 13:16:46 -0700 <rdar://problem/11248475> 141481 2012-05-11 13:11:49 -0700 2012-05-11 13:12:29 -0700 patch painttime.diff text/plain 2655 thorton ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZwppbmRleCAzMmY5MGY2Li44YzEzYjY1IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjUg QEAKKzIwMTItMDUtMTEgIFRpbSBIb3J0b24gIDx0aW1vdGh5X2hvcnRvbkBhcHBsZS5jb20+CisK KyAgICAgICAgRnJhbWVWaWV3LT5tX2xhc3RQYWludFRpbWUgaXMgbm90IHVwZGF0ZWQgaW4gdGhl IHRpbGVkIGRyYXdpbmcgY2FzZQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93 X2J1Zy5jZ2k/aWQ9ODYyNDYKKyAgICAgICAgPHJkYXI6Ly9wcm9ibGVtLzExMjQ4NDc1PgorCisg ICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFVwZGF0ZSBGcmFt ZVZpZXcncyBtX2xhc3RQYWludFRpbWUgZnJvbSBSZW5kZXJMYXllckJhY2tpbmc6OnBhaW50Q29u dGVudHMKKyAgICAgICAgaWYgdGhlIFJlbmRlckxheWVyQmFja2luZyBpcyBiYWNraW5nIGEgdGls ZWQgZHJhd2luZyBsYXllci4KKworICAgICAgICBJbiB0aGUgZnV0dXJlIHdlIG1pZ2h0IHdhbnQg dG8gY29uc2lkZXIgdXBkYXRpbmcgbV9sYXN0UGFpbnRUaW1lIHdoZW4gYW55CisgICAgICAgIGNv bXBvc2l0aW5nIGxheWVyIGlzIHBhaW50ZWQgaW50bywgYnV0IHRoaXMgY2hhbmdlIGdldHMgdXMg b24gcGFyIHdpdGggdGhlCisgICAgICAgIG5vbi10aWxlZC1kcmF3aW5nIGNhc2UgYXMgaXQgc3Rh bmRzIG5vdy4KKworICAgICAgICBObyBuZXcgdGVzdHMuCisKKyAgICAgICAgKiBwYWdlL0ZyYW1l Vmlldy5oOgorICAgICAgICAoV2ViQ29yZTo6RnJhbWVWaWV3OjpzZXRMYXN0UGFpbnRUaW1lKToK KyAgICAgICAgKiByZW5kZXJpbmcvUmVuZGVyTGF5ZXJCYWNraW5nLmNwcDoKKyAgICAgICAgKFdl YkNvcmU6OlJlbmRlckxheWVyQmFja2luZzo6cGFpbnRDb250ZW50cyk6CisKIDIwMTItMDUtMTAg IEFiaGlzaGVrIEFyeWEgIDxpbmZlcm5vQGNocm9taXVtLm9yZz4KIAogICAgICAgICBDcmFzaCBp biBJbnNlcnRQYXJhZ3JhcGhTZXBhcmF0b3JDb21tYW5kOjpkb0FwcGx5LgpkaWZmIC0tZ2l0IGEv U291cmNlL1dlYkNvcmUvcGFnZS9GcmFtZVZpZXcuaCBiL1NvdXJjZS9XZWJDb3JlL3BhZ2UvRnJh bWVWaWV3LmgKaW5kZXggNTAyMzI3NS4uMjVkOTYyYiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNv cmUvcGFnZS9GcmFtZVZpZXcuaAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wYWdlL0ZyYW1lVmlldy5o CkBAIC0yNDIsNiArMjQyLDcgQEAgcHVibGljOgogICAgIFBhaW50QmVoYXZpb3IgcGFpbnRCZWhh dmlvcigpIGNvbnN0OwogICAgIGJvb2wgaXNQYWludGluZygpIGNvbnN0OwogICAgIGJvb2wgaGFz RXZlclBhaW50ZWQoKSBjb25zdCB7IHJldHVybiBtX2xhc3RQYWludFRpbWU7IH0KKyAgICB2b2lk IHNldExhc3RQYWludFRpbWUoZG91YmxlIGxhc3RQYWludFRpbWUpIHsgbV9sYXN0UGFpbnRUaW1l ID0gbGFzdFBhaW50VGltZTsgfQogICAgIHZvaWQgc2V0Tm9kZVRvRHJhdyhOb2RlKik7CiAKICAg ICB2aXJ0dWFsIHZvaWQgcGFpbnRPdmVyaGFuZ0FyZWFzKEdyYXBoaWNzQ29udGV4dCosIGNvbnN0 IEludFJlY3QmIGhvcml6b250YWxPdmVyaGFuZ0FyZWEsIGNvbnN0IEludFJlY3QmIHZlcnRpY2Fs T3ZlcmhhbmdBcmVhLCBjb25zdCBJbnRSZWN0JiBkaXJ0eVJlY3QpOwpkaWZmIC0tZ2l0IGEvU291 cmNlL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckxheWVyQmFja2luZy5jcHAgYi9Tb3VyY2UvV2Vi Q29yZS9yZW5kZXJpbmcvUmVuZGVyTGF5ZXJCYWNraW5nLmNwcAppbmRleCBhMmQ0YzUxLi45NjEz YjRiIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyTGF5ZXJCYWNr aW5nLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyTGF5ZXJCYWNraW5n LmNwcApAQCAtMTIwMyw2ICsxMjAzLDkgQEAgdm9pZCBSZW5kZXJMYXllckJhY2tpbmc6OnBhaW50 Q29udGVudHMoY29uc3QgR3JhcGhpY3NMYXllciogZ3JhcGhpY3NMYXllciwgR3JhcGgKICAgICAg ICAgLy8gV2UgaGF2ZSB0byB1c2UgdGhlIHNhbWUgcm9vdCBhcyBmb3IgaGl0IHRlc3RpbmcsIGJl Y2F1c2UgYm90aCBtZXRob2RzIGNhbiBjb21wdXRlIGFuZCBjYWNoZSBjbGlwUmVjdHMuCiAgICAg ICAgIHBhaW50SW50b0xheWVyKG1fb3duaW5nTGF5ZXIsICZjb250ZXh0LCBkaXJ0eVJlY3QsIFBh aW50QmVoYXZpb3JOb3JtYWwsIHBhaW50aW5nUGhhc2UsIHJlbmRlcmVyKCkpOwogCisgICAgICAg IGlmIChtX3VzaW5nVGlsZWRDYWNoZUxheWVyKQorICAgICAgICAgICAgbV9vd25pbmdMYXllci0+ cmVuZGVyZXIoKS0+ZnJhbWUoKS0+dmlldygpLT5zZXRMYXN0UGFpbnRUaW1lKGN1cnJlbnRUaW1l KCkpOworCiAgICAgICAgIEluc3BlY3Rvckluc3RydW1lbnRhdGlvbjo6ZGlkUGFpbnQoY29va2ll KTsKICAgICB9IGVsc2UgaWYgKGdyYXBoaWNzTGF5ZXIgPT0gbGF5ZXJGb3JIb3Jpem9udGFsU2Ny b2xsYmFyKCkpIHsKICAgICAgICAgcGFpbnRTY3JvbGxiYXIobV9vd25pbmdMYXllci0+aG9yaXpv bnRhbFNjcm9sbGJhcigpLCBjb250ZXh0LCBjbGlwKTsK