104116 2012-12-05 06:33:46 -0800 [Qt][Mac] QWebView disappears when the system tries to hide the scrollbars. 2012-12-11 03:22:20 -0800 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 zeno zeno hausmann mail pierre.rossi oldest_to_newest 783700 0 zeno 2012-12-05 06:33:46 -0800 [Qt][Mac] QWebView disappears when the system tries to hide the scrollbars. 783702 1 177738 zeno 2012-12-05 06:43:03 -0800 Created attachment 177738 Patch 783710 2 177738 hausmann 2012-12-05 07:00:01 -0800 Comment on attachment 177738 Patch Ohh, since the widget pointer in the style option was the webview, the animation of fading out the scrollbar actually operated on the webview? Hehehe LGTM. 783729 3 177738 zeno 2012-12-05 07:32:12 -0800 Comment on attachment 177738 Patch Clearing flags on attachment: 177738 Committed r136685: <http://trac.webkit.org/changeset/136685> 783730 4 zeno 2012-12-05 07:32:17 -0800 All reviewed patches have been landed. Closing bug. 788164 5 mail 2012-12-11 01:24:07 -0800 This is actually not really fixed, e.g. Qt Creator uses a custom style "ManhattenStyle" which is a QProxyStyle to the system style (which on Mac is QMacStyle), so it still happens there. 788266 6 pierre.rossi 2012-12-11 03:20:00 -0800 (In reply to comment #5) > This is actually not really fixed, e.g. Qt Creator uses a custom style "ManhattenStyle" which is a QProxyStyle to the system style (which on Mac is QMacStyle), so it still happens there. Seems like we would need an equivalent of inherits for style proxying in order for the condition used here to be more thorough. 788267 7 zeno 2012-12-11 03:22:20 -0800 (In reply to comment #6) > Seems like we would need an equivalent of inherits for style proxying in order for the condition used here to be more thorough. I think that could be achieved by querying QProxyStyle::baseStyle(), no? But it feels like yet another workaround. 177738 2012-12-05 06:43:03 -0800 2012-12-05 07:32:12 -0800 Patch bug-104116-20121205154027.patch text/plain 2922 zeno U3VidmVyc2lvbiBSZXZpc2lvbjogMTM2Njc1CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L3F0 L0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvcXQvQ2hhbmdlTG9nCmluZGV4IGViMDk3NTVlYWNh YTRlYjJhZDczN2M2MzliNjA2ODE0Njg0NTE4ZTcuLjMzNGU2Y2JjMzk1MmMyNWQxZGU2OTY4M2M3 OGVlYTdkYzQzNjJiOWEgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvcXQvQ2hhbmdlTG9nCisr KyBiL1NvdXJjZS9XZWJLaXQvcXQvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjMgQEAKKzIwMTItMTIt MDUgIFplbm8gQWxiaXNzZXIgIDx6ZW5vQHdlYmtpdC5vcmc+CisKKyAgICAgICAgW1F0XVtNYWNd IFFXZWJWaWV3IGRpc2FwcGVhcnMgd2hlbiB0aGUgc3lzdGVtIHRyaWVzIHRvIGhpZGUgdGhlIHNj cm9sbGJhcnMuCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9p ZD0xMDQxMTYKKworICAgICAgICBUaGlzIGlzIGEgd29ya2Fyb3VuZCBmb3IgYW4gaXNzdWUgaW4g UXQgdGhhdCB3YXMKKyAgICAgICAgY2F1c2VkIGJ5IENoYW5nZS1JZDogSTIwMDBmYTUwZDQ2YjE1 M2U5ODFjZWFmYzEyYTUzOTMyYTE5NjM4MmUKKyAgICAgICAgaW4gcXRiYXNlLgorICAgICAgICBT aW5jZSB3ZSBhcmUgZHJhd2luZyB0aGUgc2Nyb2xsYmFycyBieSBvdXJzZWx2ZXMsIHRoZXJlIGlz IG5vCisgICAgICAgIHdpZGdldCBhdmFpbGFibGUgdGhhdCBuZWVkcyB0byBiZSBoaWRkZW4gYnkg dGhlIHN0eWxlLgorICAgICAgICBUaGVyZWZvcmUgd2UgaGF2ZSB0byBkaXNhYmxlIHRyYW5zaWVu dCBzY3JvbGxiYXIKKyAgICAgICAgYW5pbWF0aW9ucyBvbiBNYWMuCisKKyAgICAgICAgUGF0Y2gg Ynk6IEotUCBOdXJtaSA8anBudXJtaUBkaWdpYS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkg Tk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBXaWRnZXRTdXBwb3J0L1FTdHlsZUZhY2FkZUlt cC5jcHA6CisgICAgICAgIChXZWJLaXQ6OlFTdHlsZUZhY2FkZUltcDo6cGFpbnRTY3JvbGxCYXIp OgorCiAyMDEyLTEyLTA0ICBQaWVycmUgUm9zc2kgIDxwaWVycmUucm9zc2lAZGlnaWEuY29tPgog CiAgICAgICAgIFtRdF0gWWV0IGFub3RoZXIgdW5yZXZpZXdlZCBzcGVjdWxhdGl2ZSB3aW5kb3dz IGJ1aWxkIGZpeC4gCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L3F0L1dpZGdldFN1cHBvcnQv UVN0eWxlRmFjYWRlSW1wLmNwcCBiL1NvdXJjZS9XZWJLaXQvcXQvV2lkZ2V0U3VwcG9ydC9RU3R5 bGVGYWNhZGVJbXAuY3BwCmluZGV4IGJkNmJlMWU0NzZhOTc0MzY3ZWJiMmEyOWJmOWQxZGRjODkw NmExMzQuLjdlMmI4Y2FiNzNhNDk1MTE1YTA0MGE1NWRiOWVlY2QzZTA5NWQ3ZTAgMTAwNjQ0Ci0t LSBhL1NvdXJjZS9XZWJLaXQvcXQvV2lkZ2V0U3VwcG9ydC9RU3R5bGVGYWNhZGVJbXAuY3BwCisr KyBiL1NvdXJjZS9XZWJLaXQvcXQvV2lkZ2V0U3VwcG9ydC9RU3R5bGVGYWNhZGVJbXAuY3BwCkBA IC00NTQsMjMgKzQ1NCwxOCBAQCB2b2lkIFFTdHlsZUZhY2FkZUltcDo6cGFpbnRTY3JvbGxCYXIo UVBhaW50ZXIgKnBhaW50ZXIsIGNvbnN0IFFTdHlsZUZhY2FkZU9wdGlvbgogCiAgICAgTWFwcGVk U3R5bGVPcHRpb248UVN0eWxlT3B0aW9uU2xpZGVyPiBvcHQod2lkZ2V0LCBwcm94eU9wdGlvbik7 CiAKLSNpZmRlZiBRX09TX01BQwotICAgIC8vIEZJWE1FOiBXZSBhbHNvIG5lZWQgdG8gY2hlY2sg dGhlIHdpZGdldCBzdHlsZSBidXQgdG9kYXkgU2Nyb2xsYmFyVGhlbWUgaXMgbm90IGF3YXJlIG9m IHRoZSBwYWdlIHNvIHdlCi0gICAgLy8gY2FuJ3QgZ2V0IHRoZSB3aWRnZXQuCi0gICAgaWYgKG1f c3R5bGUtPmluaGVyaXRzKCJRTWFjU3R5bGUiKSkKLSAgICAgICAgbV9zdHlsZS0+ZHJhd0NvbXBs ZXhDb250cm9sKFFTdHlsZTo6Q0NfU2Nyb2xsQmFyLCAmb3B0LCBwYWludGVyLCB3aWRnZXQpOwot ICAgIGVsc2UKLSNlbmRpZgotICAgIHsKLSAgICAgICAgLy8gVGhlIFFTdHlsZSBleHBlY3RzIHRo ZSBiYWNrZ3JvdW5kIHRvIGJlIGFscmVhZHkgZmlsbGVkLgotICAgICAgICBwYWludGVyLT5maWxs UmVjdChvcHQucmVjdCwgb3B0LnBhbGV0dGUuYmFja2dyb3VuZCgpKTsKLQotICAgICAgICBjb25z dCBRUG9pbnQgdG9wTGVmdCA9IG9wdC5yZWN0LnRvcExlZnQoKTsKLSAgICAgICAgcGFpbnRlci0+ dHJhbnNsYXRlKHRvcExlZnQpOwotICAgICAgICBvcHQucmVjdC5tb3ZlVG8oUVBvaW50KDAsIDAp KTsKLSAgICAgICAgc3R5bGUoKS0+ZHJhd0NvbXBsZXhDb250cm9sKFFTdHlsZTo6Q0NfU2Nyb2xs QmFyLCAmb3B0LCBwYWludGVyLCB3aWRnZXQpOwotICAgICAgICBvcHQucmVjdC5tb3ZlVG8odG9w TGVmdCk7CisgICAgaWYgKG1fc3R5bGUtPmluaGVyaXRzKCJRTWFjU3R5bGUiKSkgeworICAgICAg ICAvLyBGSVhNRTogRGlzYWJsZSB0cmFuc2llbnQgc2Nyb2xsYmFyIGFuaW1hdGlvbnMgb24gT1NY IHRvIGF2b2lkIGhpZGluZyB0aGUgd2hvbGUgd2VidmlldyB3aXRoIHRoZSBzY3JvbGxiYXIgZmFk ZSBvdXQgYW5pbWF0aW9uLgorICAgICAgICBvcHQuc3R5bGVPYmplY3QgPSAwOwogICAgIH0KKwor ICAgIHBhaW50ZXItPmZpbGxSZWN0KG9wdC5yZWN0LCBvcHQucGFsZXR0ZS5iYWNrZ3JvdW5kKCkp OworCisgICAgY29uc3QgUVBvaW50IHRvcExlZnQgPSBvcHQucmVjdC50b3BMZWZ0KCk7CisgICAg cGFpbnRlci0+dHJhbnNsYXRlKHRvcExlZnQpOworICAgIG9wdC5yZWN0Lm1vdmVUbyhRUG9pbnQo MCwgMCkpOworICAgIHN0eWxlKCktPmRyYXdDb21wbGV4Q29udHJvbChRU3R5bGU6OkNDX1Njcm9s bEJhciwgJm9wdCwgcGFpbnRlciwgd2lkZ2V0KTsKKyAgICBvcHQucmVjdC5tb3ZlVG8odG9wTGVm dCk7CiB9CiAKIFFPYmplY3QqIFFTdHlsZUZhY2FkZUltcDo6d2lkZ2V0Rm9yUGFpbnRlcihRUGFp bnRlciogcGFpbnRlcikK