21024 2008-09-23 06:28:44 -0700 Scrollbar not painted until hovered 2008-09-25 00:16:30 -0700 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) PC Windows Vista RESOLVED FIXED https://bugs.webkit.org/attachment.cgi?id=23685&action=edit InRadar, NeedsReduction, PlatformOnly, Regression P2 Normal --- 1 dev+webkit webkit-unassigned hyatt oldest_to_newest 92327 0 dev+webkit 2008-09-23 06:28:44 -0700 Open <https://bugs.webkit.org/attachment.cgi?id=23685&action=edit> and look at the patch <iframe> - the scrollbar is not displayed until hovered or a repaint is forced on Windows with Safari 4. 92387 1 mrowe 2008-09-23 14:13:51 -0700 <rdar://problem/6240821> 92409 2 dev+webkit 2008-09-23 15:10:00 -0700 I just reproduced this in regular Safari 4, so it's not a regression from Hyatt's recent work. Checking to see if it's a Safari 3 -> 4 regression now. 92669 3 23781 mitz 2008-09-24 23:04:10 -0700 Created attachment 23781 Position the widget correctly in RenderWidget::setWidget to ensure proper invalidation of scrollbars 92675 4 23781 hyatt 2008-09-25 00:07:18 -0700 Comment on attachment 23781 Position the widget correctly in RenderWidget::setWidget to ensure proper invalidation of scrollbars Awesome! r=me 92677 5 mitz 2008-09-25 00:16:30 -0700 Fixed in r36885 <http://trac.webkit.org/changeset/36885>. 23781 2008-09-24 23:04:10 -0700 2008-09-25 00:07:18 -0700 Position the widget correctly in RenderWidget::setWidget to ensure proper invalidation of scrollbars 21024_r1.diff text/plain 3094 mitz SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiAzNjg4NCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTkgQEAKKzIwMDgtMDktMjQgIERhbiBCZXJuc3RlaW4gIDxtaXR6QGFwcGxlLmNv bT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAtIGZp eCBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjEwMjQKKyAgICAgICAg ICA8cmRhcjovL3Byb2JsZW0vNjI0MDgyMT4gU2Nyb2xsYmFyIG5vdCBwYWludGVkIHVudGlsIGhv dmVyZWQKKworICAgICAgICAqIHJlbmRlcmluZy9SZW5kZXJXaWRnZXQuY3BwOgorICAgICAgICAo V2ViQ29yZTo6UmVuZGVyV2lkZ2V0OjpzZXRXaWRnZXRHZW9tZXRyeSk6IFJlcGxhY2VkIHJlc2l6 ZVdpZGdldCB3aXRoCisgICAgICAgIHRoaXMgbWV0aG9kLCB3aGljaCBzZXRzIGJvdGggdGhlIGxv Y2F0aW9uIGFuZCB0aGUgc2l6ZS4KKyAgICAgICAgKFdlYkNvcmU6OlJlbmRlcldpZGdldDo6c2V0 V2lkZ2V0KTogUmVwbGFjZWQgdGhlIGNhbGwgdG8gcmVzaXplV2lkZ2V0CisgICAgICAgIHdpdGgg YSBjYWxsIHRvIHRoZSBuZXcgbWV0aG9kIHNldFdpZGdldEdlb21ldHJ5LiBQb3NpdGlvbmluZyB0 aGUKKyAgICAgICAgd2lkZ2V0IGNvcnJlY3RseSBlbnN1cmVzIHRoYXQgdGhlIHNjcm9sbCBiYXJz IGludmFsaWRhdGUgY29ycmVjdGx5IHdoZW4KKyAgICAgICAgdGhleSBhcmUgY3JlYXRlZCBhbmQg cmVzaXplZC4KKyAgICAgICAgKiByZW5kZXJpbmcvUmVuZGVyV2lkZ2V0Lmg6CisKIDIwMDgtMDkt MjQgIERhbiBCZXJuc3RlaW4gIDxtaXR6QGFwcGxlLmNvbT4KIAogICAgICAgICBSdWJiZXItc3Rh bXBlZCBieSBTYW0gV2VpbmlnLgpJbmRleDogV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyV2lkZ2V0 LmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJXaWRnZXQuY3BwCShy ZXZpc2lvbiAzNjg4MikKKysrIFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlcldpZGdldC5jcHAJKHdv cmtpbmcgY29weSkKQEAgLTExMiwxMyArMTEyLDEyIEBAIFJlbmRlcldpZGdldDo6flJlbmRlcldp ZGdldCgpCiAgICAgZGVsZXRlV2lkZ2V0KCk7CiB9CiAKLXZvaWQgUmVuZGVyV2lkZ2V0OjpyZXNp emVXaWRnZXQoV2lkZ2V0KiB3aWRnZXQsIGludCB3LCBpbnQgaCkKK3ZvaWQgUmVuZGVyV2lkZ2V0 OjpzZXRXaWRnZXRHZW9tZXRyeShjb25zdCBJbnRSZWN0JiBmcmFtZSkKIHsKLSAgICBpZiAoZWxl bWVudCgpICYmICh3aWRnZXQtPndpZHRoKCkgIT0gdyB8fCB3aWRnZXQtPmhlaWdodCgpICE9IGgp KSB7CisgICAgaWYgKGVsZW1lbnQoKSAmJiBtX3dpZGdldC0+ZnJhbWVHZW9tZXRyeSgpICE9IGZy YW1lKSB7CiAgICAgICAgIFJlbmRlckFyZW5hKiBhcmVuYSA9IHJlZigpOwotICAgICAgICBlbGVt ZW50KCktPnJlZigpOwotICAgICAgICB3aWRnZXQtPnJlc2l6ZSh3LCBoKTsKLSAgICAgICAgZWxl bWVudCgpLT5kZXJlZigpOworICAgICAgICBSZWZQdHI8Tm9kZT4gcHJvdGVjdGVkRWxlbWVudChl bGVtZW50KCkpOworICAgICAgICBtX3dpZGdldC0+c2V0RnJhbWVHZW9tZXRyeShmcmFtZSk7CiAg ICAgICAgIGRlcmVmKGFyZW5hKTsKICAgICB9CiB9CkBAIC0xMzgsMTEgKzEzNyw5IEBAIHZvaWQg UmVuZGVyV2lkZ2V0OjpzZXRXaWRnZXQoV2lkZ2V0KiB3aWQKICAgICAgICAgICAgIC8vIGlmIHdl J3ZlIGFscmVhZHkgcmVjZWl2ZWQgYSBsYXlvdXQsIGFwcGx5IHRoZSBjYWxjdWxhdGVkIHNwYWNl IHRvIHRoZQogICAgICAgICAgICAgLy8gd2lkZ2V0IGltbWVkaWF0ZWx5LCBidXQgd2UgaGF2ZSB0 byBoYXZlIHJlYWxseSBiZWVuIGZ1bGwgY29uc3RydWN0ZWQgKHdpdGggYSBub24tbnVsbAogICAg ICAgICAgICAgLy8gc3R5bGUgcG9pbnRlcikuCi0gICAgICAgICAgICBpZiAoIW5lZWRzTGF5b3V0 KCkgJiYgc3R5bGUoKSkKLSAgICAgICAgICAgICAgICByZXNpemVXaWRnZXQobV93aWRnZXQsCi0g ICAgICAgICAgICAgICAgICAgIG1fd2lkdGggLSBib3JkZXJMZWZ0KCkgLSBib3JkZXJSaWdodCgp IC0gcGFkZGluZ0xlZnQoKSAtIHBhZGRpbmdSaWdodCgpLAotICAgICAgICAgICAgICAgICAgICBt X2hlaWdodCAtIGJvcmRlclRvcCgpIC0gYm9yZGVyQm90dG9tKCkgLSBwYWRkaW5nVG9wKCkgLSBw YWRkaW5nQm90dG9tKCkpOwogICAgICAgICAgICAgaWYgKHN0eWxlKCkpIHsKKyAgICAgICAgICAg ICAgICBpZiAoIW5lZWRzTGF5b3V0KCkpCisgICAgICAgICAgICAgICAgICAgIHNldFdpZGdldEdl b21ldHJ5KGFic29sdXRlQ29udGVudEJveCgpKTsKICAgICAgICAgICAgICAgICBpZiAoc3R5bGUo KS0+dmlzaWJpbGl0eSgpICE9IFZJU0lCTEUpCiAgICAgICAgICAgICAgICAgICAgIG1fd2lkZ2V0 LT5oaWRlKCk7CiAgICAgICAgICAgICAgICAgZWxzZQpJbmRleDogV2ViQ29yZS9yZW5kZXJpbmcv UmVuZGVyV2lkZ2V0LmgKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyV2lk Z2V0LmgJKHJldmlzaW9uIDM2ODgyKQorKysgV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyV2lkZ2V0 LmgJKHdvcmtpbmcgY29weSkKQEAgLTU5LDcgKzU5LDcgQEAgcHVibGljOgogICAgIHZpcnR1YWwg Ym9vbCBub2RlQXRQb2ludChjb25zdCBIaXRUZXN0UmVxdWVzdCYsIEhpdFRlc3RSZXN1bHQmLCBp bnQgeCwgaW50IHksIGludCB0eCwgaW50IHR5LCBIaXRUZXN0QWN0aW9uKTsKIAogcHJpdmF0ZToK LSAgICB2b2lkIHJlc2l6ZVdpZGdldChXaWRnZXQqLCBpbnQgdywgaW50IGgpOworICAgIHZvaWQg c2V0V2lkZ2V0R2VvbWV0cnkoY29uc3QgSW50UmVjdCYpOwogCiAgICAgdmlydHVhbCB2b2lkIGRl bGV0ZVdpZGdldCgpOwogCg==