157499 2016-05-09 18:03:25 -0700 Allow some leeway after page load before throttling, to allow post load events to complete. 2016-05-11 14:56:40 -0700 1 1 1 Unclassified WebKit WebKit Misc. WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 barraclough barraclough oldest_to_newest 1191402 0 barraclough 2016-05-09 18:03:25 -0700 . 1191403 1 278466 barraclough 2016-05-09 18:06:16 -0700 Created attachment 278466 Fix 1191404 2 278466 rniwa 2016-05-09 18:28:57 -0700 Comment on attachment 278466 Fix View in context: https://bugs.webkit.org/attachment.cgi?id=278466&action=review > Source/WebCore/page/PageThrottler.cpp:34 > +static const double PageLoadHysteresisSeconds = 30; > + Maybe we should shorten this? 30s seems like a long time. 1192109 3 barraclough 2016-05-11 14:56:40 -0700 http://trac.webkit.org/changeset/200625 278466 2016-05-09 18:06:16 -0700 2016-05-09 18:28:57 -0700 Fix 157499.1.patch text/plain 3554 barraclough SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDIwMDQ5OCkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE4IEBACisyMDE2LTA1LTA1ICBHYXZpbiBC YXJyYWNsb3VnaCAgPGJhcnJhY2xvdWdoQGFwcGxlLmNvbT4KKworICAgICAgICBBbGxvdyBzb21l IGxlZXdheSBhZnRlciBwYWdlIGxvYWQgYmVmb3JlIHRocm90dGxpbmcsIHRvIGFsbG93IHBvc3Qg bG9hZCBldmVudHMgdG8gY29tcGxldGUuCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn L3Nob3dfYnVnLmNnaT9pZD0xNTc0OTkKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9P UFMhKS4KKworICAgICAgICAqIHBhZ2UvUGFnZVRocm90dGxlci5jcHA6CisgICAgICAgIChXZWJD b3JlOjpQYWdlVGhyb3R0bGVyOjpQYWdlVGhyb3R0bGVyKToKKyAgICAgICAgICAgIC0gbV9wYWdl TG9hZEFjdGl2aXR5Q291bnRlciBub3cgYWZmZWN0cyBhY3Rpdml0eSBzdGF0ZSBpbmRpcmVjdHks IHZpYSBtX3BhZ2VMb2FkQWN0aXZpdHlIeXN0ZXJlc2lzLgorICAgICAgICAoV2ViQ29yZTo6UGFn ZVRocm90dGxlcjo6cGFnZUxvYWRBY3Rpdml0eUNvdW50ZXJDaGFuZ2VkKToKKyAgICAgICAgICAg IC0gY291bnRlciB1cGRhdGVzIGh5c3RlcmVzaXMgb2JqZWN0LCB3aGljaCBpbiB0dXJuIHdpbGwg YWZmZWN0IGFjdGl2aXR5IHN0YXRlLgorICAgICAgICAqIHBhZ2UvUGFnZVRocm90dGxlci5oOgor ICAgICAgICAgICAgLSBhZGRlZCBtX3BhZ2VMb2FkQWN0aXZpdHlIeXN0ZXJlc2lzLCBwYWdlTG9h ZEFjdGl2aXR5Q291bnRlckNoYW5nZWQuCisKIDIwMTYtMDUtMDUgIEJyYWR5IEVpZHNvbiAgPGJl aWRzb25AYXBwbGUuY29tPgogCiAgICAgICAgIE1vZGVybiBJREIgKFdvcmtlcnMpOiBSZW1vdmUg dGhlIG5lZWQgZm9yIElEQkNvbm5lY3Rpb25Qcm94eSB0byBleHBvc2UgaXRzIElEQkNvbm5lY3Rp b25Ub1NlcnZlci4KSW5kZXg6IFNvdXJjZS9XZWJDb3JlL3BhZ2UvUGFnZVRocm90dGxlci5jcHAK PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUvcGFnZS9QYWdlVGhyb3R0bGVyLmNwcAkocmV2 aXNpb24gMjAwMTM0KQorKysgU291cmNlL1dlYkNvcmUvcGFnZS9QYWdlVGhyb3R0bGVyLmNwcAko d29ya2luZyBjb3B5KQpAQCAtMzAsMTIgKzMwLDE1IEBACiAKIG5hbWVzcGFjZSBXZWJDb3JlIHsK IAorc3RhdGljIGNvbnN0IGRvdWJsZSBQYWdlTG9hZEh5c3RlcmVzaXNTZWNvbmRzID0gMzA7CisK IFBhZ2VUaHJvdHRsZXI6OlBhZ2VUaHJvdHRsZXIoUGFnZSYgcGFnZSkKICAgICA6IG1fcGFnZShw YWdlKQogICAgICwgbV91c2VySW5wdXRIeXN0ZXJlc2lzKFt0aGlzXShIeXN0ZXJlc2lzU3RhdGUg c3RhdGUpIHsgc2V0QWN0aXZpdHlGbGFnKFBhZ2VBY3Rpdml0eVN0YXRlOjpVc2VySW5wdXRBY3Rp dml0eSwgc3RhdGUgPT0gSHlzdGVyZXNpc1N0YXRlOjpTdGFydGVkKTsgfSkKICAgICAsIG1fbWVk aWFBY3Rpdml0eUh5c3RlcmVzaXMoW3RoaXNdKEh5c3RlcmVzaXNTdGF0ZSBzdGF0ZSkgeyBzZXRB Y3Rpdml0eUZsYWcoUGFnZUFjdGl2aXR5U3RhdGU6Ok1lZGlhQWN0aXZpdHksIHN0YXRlID09IEh5 c3RlcmVzaXNTdGF0ZTo6U3RhcnRlZCk7IH0pCisgICAgLCBtX3BhZ2VMb2FkQWN0aXZpdHlIeXN0 ZXJlc2lzKFt0aGlzXShIeXN0ZXJlc2lzU3RhdGUgc3RhdGUpIHsgc2V0QWN0aXZpdHlGbGFnKFBh Z2VBY3Rpdml0eVN0YXRlOjpQYWdlTG9hZEFjdGl2aXR5LCBzdGF0ZSA9PSBIeXN0ZXJlc2lzU3Rh dGU6OlN0YXJ0ZWQpOyB9LCBQYWdlTG9hZEh5c3RlcmVzaXNTZWNvbmRzKQogICAgICwgbV9tZWRp YUFjdGl2aXR5Q291bnRlcihbdGhpc10oUmVmQ291bnRlckV2ZW50KSB7IG1lZGlhQWN0aXZpdHlD b3VudGVyQ2hhbmdlZCgpOyB9KQotICAgICwgbV9wYWdlTG9hZEFjdGl2aXR5Q291bnRlcihbdGhp c10oUmVmQ291bnRlckV2ZW50KSB7IHNldEFjdGl2aXR5RmxhZyhQYWdlQWN0aXZpdHlTdGF0ZTo6 UGFnZUxvYWRBY3Rpdml0eSwgbV9wYWdlTG9hZEFjdGl2aXR5Q291bnRlci52YWx1ZSgpKTsgfSkK KyAgICAsIG1fcGFnZUxvYWRBY3Rpdml0eUNvdW50ZXIoW3RoaXNdKFJlZkNvdW50ZXJFdmVudCkg eyBwYWdlTG9hZEFjdGl2aXR5Q291bnRlckNoYW5nZWQoKTsgfSkKIHsKIH0KIApAQCAtNTcsNiAr NjAsMTQgQEAgdm9pZCBQYWdlVGhyb3R0bGVyOjptZWRpYUFjdGl2aXR5Q291bnRlcgogICAgICAg ICBtX21lZGlhQWN0aXZpdHlIeXN0ZXJlc2lzLnN0b3AoKTsKIH0KIAordm9pZCBQYWdlVGhyb3R0 bGVyOjpwYWdlTG9hZEFjdGl2aXR5Q291bnRlckNoYW5nZWQoKQoreworICAgIGlmIChtX3BhZ2VM b2FkQWN0aXZpdHlDb3VudGVyLnZhbHVlKCkpCisgICAgICAgIG1fcGFnZUxvYWRBY3Rpdml0eUh5 c3RlcmVzaXMuc3RhcnQoKTsKKyAgICBlbHNlCisgICAgICAgIG1fcGFnZUxvYWRBY3Rpdml0eUh5 c3RlcmVzaXMuc3RvcCgpOworfQorCiB2b2lkIFBhZ2VUaHJvdHRsZXI6OnNldEFjdGl2aXR5Rmxh ZyhQYWdlQWN0aXZpdHlTdGF0ZTo6RmxhZ3MgZmxhZywgYm9vbCB2YWx1ZSkKIHsKICAgICBQYWdl QWN0aXZpdHlTdGF0ZTo6RmxhZ3MgYWN0aXZpdHlTdGF0ZSA9IG1fYWN0aXZpdHlTdGF0ZTsKSW5k ZXg6IFNvdXJjZS9XZWJDb3JlL3BhZ2UvUGFnZVRocm90dGxlci5oCj09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNv dXJjZS9XZWJDb3JlL3BhZ2UvUGFnZVRocm90dGxlci5oCShyZXZpc2lvbiAyMDAxMzQpCisrKyBT b3VyY2UvV2ViQ29yZS9wYWdlL1BhZ2VUaHJvdHRsZXIuaAkod29ya2luZyBjb3B5KQpAQCAtNjYs MTIgKzY2LDE0IEBAIHB1YmxpYzoKIAogcHJpdmF0ZToKICAgICB2b2lkIG1lZGlhQWN0aXZpdHlD b3VudGVyQ2hhbmdlZCgpOworICAgIHZvaWQgcGFnZUxvYWRBY3Rpdml0eUNvdW50ZXJDaGFuZ2Vk KCk7CiAgICAgdm9pZCBzZXRBY3Rpdml0eUZsYWcoUGFnZUFjdGl2aXR5U3RhdGU6OkZsYWdzLCBi b29sKTsKIAogICAgIFBhZ2UmIG1fcGFnZTsKICAgICBQYWdlQWN0aXZpdHlTdGF0ZTo6RmxhZ3Mg bV9hY3Rpdml0eVN0YXRlIHsgUGFnZUFjdGl2aXR5U3RhdGU6Ok5vRmxhZ3MgfTsKICAgICBIeXN0 ZXJlc2lzQWN0aXZpdHkgbV91c2VySW5wdXRIeXN0ZXJlc2lzOwogICAgIEh5c3RlcmVzaXNBY3Rp dml0eSBtX21lZGlhQWN0aXZpdHlIeXN0ZXJlc2lzOworICAgIEh5c3RlcmVzaXNBY3Rpdml0eSBt X3BhZ2VMb2FkQWN0aXZpdHlIeXN0ZXJlc2lzOwogICAgIFBhZ2VBY3Rpdml0eUNvdW50ZXIgbV9t ZWRpYUFjdGl2aXR5Q291bnRlcjsKICAgICBQYWdlQWN0aXZpdHlDb3VudGVyIG1fcGFnZUxvYWRB Y3Rpdml0eUNvdW50ZXI7CiB9Owo=