38641 2010-05-06 06:46:51 -0700 [Qt] Nested overflow div does not scroll 2010-05-21 05:48:12 -0700 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) S60 Hardware S60 3rd edition CLOSED FIXED PlatformOnly, Qt P1 Normal --- 35784 1 joseph.ligman webkit-unassigned arho.huttunen christian.webkit commit-queue hausmann kenneth laszlo.gombos oldest_to_newest 221650 0 joseph.ligman 2010-05-06 06:46:51 -0700 A div with CSS overflow:hidden nested inside a div with CSS overflow:scroll does not pan scroll. A div with CSS overflow:scroll nested inside another div with overflow:scroll does not pan scroll. etc... Also, I would like to export webframe_scrollOverflow so I can use it independently of qtwebkit_webframe_scrollRecursively 221652 1 55232 joseph.ligman 2010-05-06 06:50:54 -0700 Created attachment 55232 patch 221663 2 arho.huttunen 2010-05-06 07:16:17 -0700 Is there any chance this could be made a public API? 223609 3 laszlo.gombos 2010-05-10 17:11:00 -0700 This looks good to me for an internal API. Any objections ? Kenneth, Simon ? Note that we also have a need to merge this back to the 4.6.x branch. 223850 4 hausmann 2010-05-11 03:25:41 -0700 (In reply to comment #3) > This looks good to me for an internal API. Any objections ? Kenneth, Simon ? > > Note that we also have a need to merge this back to the 4.6.x branch. Fine with me. Please also update the eabi/wins def files. 223873 5 kenneth 2010-05-11 05:13:30 -0700 (In reply to comment #4) > (In reply to comment #3) > > This looks good to me for an internal API. Any objections ? Kenneth, Simon ? > > > > Note that we also have a need to merge this back to the 4.6.x branch. > > Fine with me. Please also update the eabi/wins def files. Also ok with me. 223926 6 55232 laszlo.gombos 2010-05-11 08:12:00 -0700 Comment on attachment 55232 patch looks good, r+. I will update the def files after the patch is landed. 225057 7 55232 commit-queue 2010-05-12 19:38:08 -0700 Comment on attachment 55232 patch Clearing flags on attachment: 55232 Committed r59335: <http://trac.webkit.org/changeset/59335> 225058 8 commit-queue 2010-05-12 19:38:14 -0700 All reviewed patches have been landed. Closing bug. 227195 9 hausmann 2010-05-18 02:01:51 -0700 (In reply to comment #6) > (From update of attachment 55232 [details]) > looks good, r+. I will update the def files after the patch is landed. Re-opening this issue, as the new export is still missing from the def files. 227528 10 laszlo.gombos 2010-05-18 15:49:55 -0700 Fixed the def file in http://trac.webkit.org/changeset/59717. 228372 11 hausmann 2010-05-20 07:56:35 -0700 Revision r59335 cherry-picked into qtwebkit-2.0 with commit 92655349c9c0299079107fdda847673a0024cf2f 228871 12 hausmann 2010-05-21 05:47:57 -0700 Cherry-picked into qtwebkit-4.6 with commit d85b7b520ec57bfd3b4d25482ab62520f6736289 55232 2010-05-06 06:50:54 -0700 2010-05-12 19:38:08 -0700 patch bug36481.patch text/plain 3680 joseph.ligman SW5kZXg6IFdlYktpdC9xdC9DaGFuZ2VMb2cNCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0NCi0tLSBXZWJLaXQvcXQvQ2hh bmdlTG9nCShyZXZpc2lvbiA1ODgzNykNCisrKyBXZWJLaXQvcXQvQ2hhbmdlTG9nCSh3b3JraW5n IGNvcHkpDQpAQCAtMSwzICsxLDE5IEBADQorMjAxMC0wNS0wNSAgSm9lIExpZ21hbiAgPGpvc2Vw aC5saWdtYW5Abm9raWEuY29tPg0KKw0KKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT ISkuDQorDQorICAgICAgICBbUXRdIE5lc3RlZCBvdmVyZmxvdyBkaXYgZG9lcyBub3Qgc2Nyb2xs DQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Mzg2NDEN CisNCisgICAgICAgIE1vZGlmeSBxdHdlYmtpdF93ZWJmcmFtZV9zY3JvbGxPdmVyZmxvdywgaWYg dGhlIGN1cnJlbnQgbm9kZSdzIHJlbmRlciBsYXllcg0KKyAgICAgICAgZG9lcyBub3Qgc2Nyb2xs IGl0IHdpbGwgdHJ5IGFuZCBzY3JvbGwgdGhlIHBhcmVudCdzIHJlbmRlciBsYXllci4gQWxzbyBl eHBvcnQNCisgICAgICAgIHF0d2Via2l0X3dlYmZyYW1lX3Njcm9sbE92ZXJmbG93IHNvIHdlIGNh biB1c2UgaXQgaW5kZXBlbmRlbnRseSBvZg0KKyAgICAgICAgcXR3ZWJraXRfd2ViZnJhbWVfc2Ny b2xsUmVjdXJzaXZlbHkNCisNCisgICAgICAgICogQXBpL3F3ZWJmcmFtZS5jcHA6DQorICAgICAg ICAocXR3ZWJraXRfd2ViZnJhbWVfc2Nyb2xsT3ZlcmZsb3cpOg0KKyAgICAgICAgKHF0d2Via2l0 X3dlYmZyYW1lX3Njcm9sbFJlY3Vyc2l2ZWx5KToNCisNCiAyMDEwLTA1LTA0ICBMYXN6bG8gR29t Ym9zICA8bGFzemxvLjEuZ29tYm9zQG5va2lhLmNvbT4NCiANCiAgICAgICAgIFJldmlld2VkIGJ5 IEtlbm5ldGggUm9oZGUgQ2hyaXN0aWFuc2VuLg0KSW5kZXg6IFdlYktpdC9xdC9BcGkvcXdlYmZy YW1lLmNwcA0KPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PQ0KLS0tIFdlYktpdC9xdC9BcGkvcXdlYmZyYW1lLmNwcAkocmV2 aXNpb24gNTg4MzcpDQorKysgV2ViS2l0L3F0L0FwaS9xd2ViZnJhbWUuY3BwCSh3b3JraW5nIGNv cHkpDQpAQCAtODgsOCArODgsOSBAQCBRVF9CRUdJTl9OQU1FU1BBQ0UNCiBleHRlcm4gUV9HVUlf RVhQT1JUIGludCBxdF9kZWZhdWx0RHBpKCk7DQogUVRfRU5EX05BTUVTUEFDRQ0KIA0KLXN0YXRp YyBib29sIHdlYmZyYW1lX3Njcm9sbE92ZXJmbG93KFdlYkNvcmU6OkZyYW1lKiBmcmFtZSwgaW50 IGR4LCBpbnQgZHksIGNvbnN0IFFQb2ludCYgcG9zKQ0KK2Jvb2wgUVdFQktJVF9FWFBPUlQgcXR3 ZWJraXRfd2ViZnJhbWVfc2Nyb2xsT3ZlcmZsb3coUVdlYkZyYW1lKiBxRnJhbWUsIGludCBkeCwg aW50IGR5LCBjb25zdCBRUG9pbnQmIHBvcykNCiB7DQorICAgIFdlYkNvcmU6OkZyYW1lKiBmcmFt ZSA9IFFXZWJGcmFtZVByaXZhdGU6OmNvcmUocUZyYW1lKTsNCiAgICAgaWYgKCFmcmFtZSB8fCAh ZnJhbWUtPmRvY3VtZW50KCkgfHwgIWZyYW1lLT52aWV3KCkgfHwgIWZyYW1lLT5ldmVudEhhbmRs ZXIoKSkNCiAgICAgICAgIHJldHVybiBmYWxzZTsNCiANCkBAIC0xMTIsMTcgKzExMywyNCBAQCBz dGF0aWMgYm9vbCB3ZWJmcmFtZV9zY3JvbGxPdmVyZmxvdyhXZWJDDQogICAgIGJvb2wgc2Nyb2xs ZWRIb3Jpem9udGFsID0gZmFsc2U7DQogICAgIGJvb2wgc2Nyb2xsZWRWZXJ0aWNhbCA9IGZhbHNl Ow0KIA0KLSAgICBpZiAoZHggPiAwKQ0KLSAgICAgICAgc2Nyb2xsZWRIb3Jpem9udGFsID0gcmVu ZGVyTGF5ZXItPnNjcm9sbChTY3JvbGxSaWdodCwgU2Nyb2xsQnlQaXhlbCwgZHgpOw0KLSAgICBl bHNlIGlmIChkeCA8IDApDQotICAgICAgICBzY3JvbGxlZEhvcml6b250YWwgPSByZW5kZXJMYXll ci0+c2Nyb2xsKFNjcm9sbExlZnQsIFNjcm9sbEJ5UGl4ZWwsIHFBYnMoZHgpKTsNCi0NCi0gICAg aWYgKGR5ID4gMCkNCi0gICAgICAgIHNjcm9sbGVkVmVydGljYWwgPSByZW5kZXJMYXllci0+c2Ny b2xsKFNjcm9sbERvd24sIFNjcm9sbEJ5UGl4ZWwsIGR5KTsNCi0gICAgZWxzZSBpZiAoZHkgPCAw KQ0KLSAgICAgICAgc2Nyb2xsZWRWZXJ0aWNhbCA9IHJlbmRlckxheWVyLT5zY3JvbGwoU2Nyb2xs VXAsIFNjcm9sbEJ5UGl4ZWwsIHFBYnMoZHkpKTsNCisgICAgZG8gew0KKyAgICAgICAgaWYgKGR4 ID4gMCkNCisgICAgICAgICAgICBzY3JvbGxlZEhvcml6b250YWwgPSByZW5kZXJMYXllci0+c2Ny b2xsKFNjcm9sbFJpZ2h0LCBTY3JvbGxCeVBpeGVsLCBkeCk7DQorICAgICAgICBlbHNlIGlmIChk eCA8IDApDQorICAgICAgICAgICAgc2Nyb2xsZWRIb3Jpem9udGFsID0gcmVuZGVyTGF5ZXItPnNj cm9sbChTY3JvbGxMZWZ0LCBTY3JvbGxCeVBpeGVsLCBxQWJzKGR4KSk7DQorDQorICAgICAgICBp ZiAoZHkgPiAwKQ0KKyAgICAgICAgICAgIHNjcm9sbGVkVmVydGljYWwgPSByZW5kZXJMYXllci0+ c2Nyb2xsKFNjcm9sbERvd24sIFNjcm9sbEJ5UGl4ZWwsIGR5KTsNCisgICAgICAgIGVsc2UgaWYg KGR5IDwgMCkNCisgICAgICAgICAgICBzY3JvbGxlZFZlcnRpY2FsID0gcmVuZGVyTGF5ZXItPnNj cm9sbChTY3JvbGxVcCwgU2Nyb2xsQnlQaXhlbCwgcUFicyhkeSkpOw0KKw0KKyAgICAgICAgaWYg KHNjcm9sbGVkSG9yaXpvbnRhbCB8fCBzY3JvbGxlZFZlcnRpY2FsKQ0KKyAgICAgICAgICAgIHJl dHVybiB0cnVlOw0KKw0KKyAgICAgICAgcmVuZGVyTGF5ZXIgPSByZW5kZXJMYXllci0+cGFyZW50 KCk7DQorICAgIH0gd2hpbGUgKHJlbmRlckxheWVyKTsNCiANCi0gICAgcmV0dXJuIChzY3JvbGxl ZEhvcml6b250YWwgfHwgc2Nyb2xsZWRWZXJ0aWNhbCk7DQorICAgIHJldHVybiBmYWxzZTsNCiB9 DQogDQogDQpAQCAtMTM4LDcgKzE0Niw3IEBAIHZvaWQgUVdFQktJVF9FWFBPUlQgcXR3ZWJraXRf d2ViZnJhbWVfc2MNCiAgICAgaWYgKCFxRnJhbWUpDQogICAgICAgICByZXR1cm47DQogDQotICAg IGlmICh3ZWJmcmFtZV9zY3JvbGxPdmVyZmxvdyhRV2ViRnJhbWVQcml2YXRlOjpjb3JlKHFGcmFt ZSksIGR4LCBkeSwgcG9zKSkNCisgICAgaWYgKHF0d2Via2l0X3dlYmZyYW1lX3Njcm9sbE92ZXJm bG93KHFGcmFtZSwgZHgsIGR5LCBwb3MpKQ0KICAgICAgICAgcmV0dXJuOw0KIA0KICAgICBib29s IHNjcm9sbEhvcml6b250YWwgPSBmYWxzZTsNCkBAIC0xNTIsNyArMTYwLDcgQEAgdm9pZCBRV0VC S0lUX0VYUE9SVCBxdHdlYmtpdF93ZWJmcmFtZV9zYw0KIA0KICAgICAgICAgaWYgKGR5ID4gMCkg IC8vIHNjcm9sbCBkb3duDQogICAgICAgICAgICAgc2Nyb2xsVmVydGljYWwgPSBxRnJhbWUtPnNj cm9sbEJhclZhbHVlKFF0OjpWZXJ0aWNhbCkgPCBxRnJhbWUtPnNjcm9sbEJhck1heGltdW0oUXQ6 OlZlcnRpY2FsKTsNCi0gICAgICAgICAgICBlbHNlIGlmIChkeSA8IDApIC8vc2Nyb2xsIHVwDQor ICAgICAgICBlbHNlIGlmIChkeSA8IDApIC8vc2Nyb2xsIHVwDQogICAgICAgICAgICAgc2Nyb2xs VmVydGljYWwgPSBxRnJhbWUtPnNjcm9sbEJhclZhbHVlKFF0OjpWZXJ0aWNhbCkgPiBxRnJhbWUt PnNjcm9sbEJhck1pbmltdW0oUXQ6OlZlcnRpY2FsKTsNCiANCiAgICAgICAgIGlmIChzY3JvbGxI b3Jpem9udGFsIHx8IHNjcm9sbFZlcnRpY2FsKSB7DQo=