104602 2012-12-10 15:15:18 -0800 DidHitRelevantRepaintedObjectsAreaThreshold LayoutMilestone fires too early on some pages with iframes 2012-12-10 16:33:34 -0800 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 bdakin bdakin bdakin simon.fraser webkit-bug-importer oldest_to_newest 787753 0 bdakin 2012-12-10 15:15:18 -0800 DidHitRelevantRepaintedObjectsAreaThreshold LayoutMilestone fires too early on some pages with iframes. This site is a great example of this bug: http://www2.macleans.ca/2012/11/04/tarnished-silver-assessing-the-new-king-of-stats/ 787754 1 webkit-bug-importer 2012-12-10 15:16:01 -0800 <rdar://problem/12850520> 787762 2 178646 bdakin 2012-12-10 15:20:17 -0800 Created attachment 178646 Patch 787841 3 bdakin 2012-12-10 16:33:34 -0800 Thanks, Simon! I committed this: http://trac.webkit.org/changeset/137224 178646 2012-12-10 15:20:17 -0800 2012-12-10 16:31:22 -0800 Patch for-review.txt text/plain 2125 bdakin SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDEzNzIxMikKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE2IEBACisyMDEyLTEyLTEwICBCZXRoIERh a2luICA8YmRha2luQGFwcGxlLmNvbT4KKworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9y Zy9zaG93X2J1Zy5jZ2k/aWQ9MTA0NjAyCisgICAgICAgIERpZEhpdFJlbGV2YW50UmVwYWludGVk T2JqZWN0c0FyZWFUaHJlc2hvbGQgTGF5b3V0TWlsZXN0b25lIGZpcmVzIHRvbyAKKyAgICAgICAg ZWFybHkgb24gc29tZSBwYWdlcyB3aXRoIGlmcmFtZXMKKworICAgICAgICBSZXZpZXdlZCBieSBO T0JPRFkgKE9PUFMhKS4KKworICAgICAgICBUaGlzIGNoYW5nZSBtYWtlcyBpdCBzbyB0aGF0IGFu eSBSZW5kZXJPYmplY3RzIGluc2lkZSBpZnJhbWVzIGFyZSBub3QgCisgICAgICAgIGNvbnNpZGVy ZWQgcmVsZXZhbnQuCisgICAgICAgICogcGFnZS9QYWdlLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6 OlBhZ2U6OmFkZFJlbGV2YW50UmVwYWludGVkT2JqZWN0KToKKwogMjAxMi0xMi0xMCAgS29uZGFw YWxseSBLYWx5YW4gIDxrYWx5YW4ua29uZGFwYWxseUBpbnRlbC5jb20+CiAKICAgICAgICAgW0VG TF1bV2ViR0xdIE1pbm9yIFJlZmFjdG9yaW5nIGluIEdyYXBoaWNzQ29udGV4dDNERWZsLgpJbmRl eDogU291cmNlL1dlYkNvcmUvcGFnZS9QYWdlLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9wYWdlL1BhZ2UuY3BwCShyZXZpc2lvbiAxMzcxOTYpCisrKyBTb3VyY2UvV2ViQ29yZS9w YWdlL1BhZ2UuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0xMTgxLDExICsxMTgxLDE3IEBAIHZvaWQg UGFnZTo6YWRkUmVsZXZhbnRSZXBhaW50ZWRPYmplY3QoUmUKICAgICBpZiAoIWlzQ291bnRpbmdS ZWxldmFudFJlcGFpbnRlZE9iamVjdHMoKSkKICAgICAgICAgcmV0dXJuOwogCisgICAgLy8gT2Jq ZWN0cyBpbnNpZGUgc3ViLWZyYW1lcyBhcmUgbm90IGNvbnNpZGVyZWQgdG8gYmUgcmVsZXZhbnQu CisgICAgaWYgKG9iamVjdC0+ZG9jdW1lbnQoKS0+ZnJhbWUoKSAhPSBtYWluRnJhbWUoKSkKKyAg ICAgICAgcmV0dXJuOworCisgICAgUmVuZGVyVmlldyogdmlldyA9IG9iamVjdC0+dmlldygpOwor ICAgIGlmICghdmlldykKKyAgICAgICAgcmV0dXJuOworCiAgICAgLy8gVGhlIG9iamVjdHMgYXJl IG9ubHkgcmVsZXZhbnQgaWYgdGhleSBhcmUgYmVpbmcgcGFpbnRlZCB3aXRoaW4gdGhlIHZpZXdS ZWN0KCkuCi0gICAgaWYgKFJlbmRlclZpZXcqIHZpZXcgPSBvYmplY3QtPnZpZXcoKSkgewotICAg ICAgICBpZiAoIW9iamVjdFBhaW50UmVjdC5pbnRlcnNlY3RzKHBpeGVsU25hcHBlZEludFJlY3Qo dmlldy0+dmlld1JlY3QoKSkpKQotICAgICAgICAgICAgcmV0dXJuOwotICAgIH0KKyAgICBpZiAo IW9iamVjdFBhaW50UmVjdC5pbnRlcnNlY3RzKHBpeGVsU25hcHBlZEludFJlY3Qodmlldy0+dmll d1JlY3QoKSkpKQorICAgICAgICByZXR1cm47CiAKICAgICBJbnRSZWN0IHNuYXBwZWRQYWludFJl Y3QgPSBwaXhlbFNuYXBwZWRJbnRSZWN0KG9iamVjdFBhaW50UmVjdCk7CiAKQEAgLTExOTgsMTAg KzEyMDQsNiBAQCB2b2lkIFBhZ2U6OmFkZFJlbGV2YW50UmVwYWludGVkT2JqZWN0KFJlCiAgICAg fQogCiAgICAgbV9yZWxldmFudFBhaW50ZWRSZWdpb24udW5pdGUoc25hcHBlZFBhaW50UmVjdCk7 Ci0KLSAgICBSZW5kZXJWaWV3KiB2aWV3ID0gb2JqZWN0LT52aWV3KCk7Ci0gICAgaWYgKCF2aWV3 KQotICAgICAgICByZXR1cm47CiAgICAgCiAgICAgZmxvYXQgdmlld0FyZWEgPSB2aWV3LT52aWV3 UmVjdCgpLndpZHRoKCkgKiB2aWV3LT52aWV3UmVjdCgpLmhlaWdodCgpOwogICAgIGZsb2F0IHJh dGlvT2ZWaWV3VGhhdElzUGFpbnRlZCA9IG1fcmVsZXZhbnRQYWludGVkUmVnaW9uLnRvdGFsQXJl YSgpIC8gdmlld0FyZWE7Cg==