38597 2010-05-05 11:43:49 -0700 [Qt] QtTestBrowser does not have a "Load" button ; therefore, unable to load pages on touch only symbian devices (portrait mode). 2011-04-19 05:15:29 -0700 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) S60 Hardware S60 3rd edition CLOSED FIXED Qt, QtTriaged P3 Normal --- 1 alexandra.1.santos webkit-qt-unassigned commit-queue hausmann jesus kenneth tonikitoo oldest_to_newest 221211 0 alexandra.1.santos 2010-05-05 11:43:49 -0700 Since QtLauncher does not have a "load" button, on touch only symbian devices like N8, it is not possible to load pages. Steps to reproduce: 1.- Install qt.sis (4.6.2), qtwebkit.sis and qtlauncher on the device. 2.- Launch qtlauncher 3.- Enter a url address and try to load the page. Actual Results: It is not possible to load pages; there is no "load" or "enter" button. Expected results: qtlauncher would have a "Load" button to be pressed in order to load a page. 224618 1 jesus 2010-05-12 08:15:53 -0700 Good point! Blocking the 2.0 release as this is simple to solve and quite useful to have. 224629 2 kenneth 2010-05-12 08:29:57 -0700 Shouldn't enter from the virtual keyboard close the keyboard and start the load? What does the iphone and the ipad do? 227482 3 hausmann 2010-05-18 13:44:12 -0700 Hmm, on my N8 I'm using the "Enter" key (the little arrow) on the virtual keyboard to initiate the load. I'm moving this issue from the release blocker to the should-include-if-patch-is-available tracker. It would be nice to fix QtTestBrowser (the new name :-), but it's not critical to the release I'd say. 242532 4 59658 jesus 2010-06-24 08:48:14 -0700 Created attachment 59658 patch Simple patch that makes the reload button act as a load button as well. Solves the problem and we don't need to add another button. :) 243290 5 59658 commit-queue 2010-06-25 17:58:31 -0700 Comment on attachment 59658 patch Clearing flags on attachment: 59658 Committed r61926: <http://trac.webkit.org/changeset/61926> 243291 6 commit-queue 2010-06-25 17:58:35 -0700 All reviewed patches have been landed. Closing bug. 243639 7 hausmann 2010-06-28 03:16:59 -0700 Revision r61926 cherry-picked into qtwebkit-2.0 with commit 38d650efc92427cd6452f6685d3c40d22428cdb7 243717 8 jesus 2010-06-28 08:51:03 -0700 (In reply to comment #7) > Revision r61926 cherry-picked into qtwebkit-2.0 with commit 38d650efc92427cd6452f6685d3c40d22428cdb7 Nice! But I had to upload a follow-up patch because the behavior was wrong. It wasn't loading a new web page if the url in the line edit had been changed. Simon, could you please cherry-pick this revision as well? http://trac.webkit.org/changeset/62009 Changing back from CLOSED to RESOLVED so we don't forget it... 244769 9 hausmann 2010-06-30 07:43:18 -0700 Revision r61926 cherry-picked into qtwebkit-2.0 with commit 38d650efc92427cd6452f6685d3c40d22428cdb7 244770 10 hausmann 2010-06-30 07:43:46 -0700 <cherry-pick-for-backport: r62009> 245199 11 hausmann 2010-07-01 01:32:24 -0700 Revision r62009 cherry-picked into qtwebkit-2.0 with commit 0eee8df53d1873669a9dda8f9c0340543258a316 59658 2010-06-24 08:48:14 -0700 2010-06-25 17:58:30 -0700 patch 0001-Make-the-reload-button-of-QtTestBrowser-act-as-a-loa.patch text/plain 2527 jesus RnJvbSAxMDBlYzZmM2Q0NzU4NzBiMzNiOTIzYzE4YTQyN2M1YTk1OGQ3MmE1IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBKZXN1cyBTYW5jaGV6LVBhbGVuY2lhIDxqZXN1cy5wYWxlbmNp YUBvcGVuYm9zc2Eub3JnPgpEYXRlOiBUaHUsIDI0IEp1biAyMDEwIDEyOjQ3OjExIC0wMzAwClN1 YmplY3Q6IFtQQVRDSF0gTWFrZSB0aGUgcmVsb2FkIGJ1dHRvbiBvZiBRdFRlc3RCcm93c2VyIGFj dCBhcyBhIGxvYWQgYnV0dG9uIGFzIHdlbGwuCgpbUXRdIFF0VGVzdEJyb3dzZXIgZG9lcyBub3Qg aGF2ZSBhICJMb2FkIiBidXR0b24gOyB0aGVyZWZvcmUsIHVuYWJsZSB0byBsb2FkIHBhZ2VzIG9u IHRvdWNoIG9ubHkgc3ltYmlhbiBkZXZpY2VzIChwb3J0cmFpdCBtb2RlKS4KaHR0cHM6Ly9idWdz LndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTM4NTk3CgpSZXZpZXdlZCBieSBOT0JPRFkgKE9P UFMhKS4KCiogUXRUZXN0QnJvd3Nlci9tYWlud2luZG93LmNwcDoKKE1haW5XaW5kb3c6OmJ1aWxk VUkpOgooTWFpbldpbmRvdzo6Y2hhbmdlTG9jYXRpb24pOgotLS0KIFdlYktpdFRvb2xzL0NoYW5n ZUxvZyAgICAgICAgICAgICAgICAgICAgfCAgIDEzICsrKysrKysrKysrKysKIFdlYktpdFRvb2xz L1F0VGVzdEJyb3dzZXIvbWFpbndpbmRvdy5jcHAgfCAgIDEwICsrKysrKysrKy0KIDIgZmlsZXMg Y2hhbmdlZCwgMjIgaW5zZXJ0aW9ucygrKSwgMSBkZWxldGlvbnMoLSkKCmRpZmYgLS1naXQgYS9X ZWJLaXRUb29scy9DaGFuZ2VMb2cgYi9XZWJLaXRUb29scy9DaGFuZ2VMb2cKaW5kZXggNDBhNTEx OC4uMmU5YTkxOCAxMDA2NDQKLS0tIGEvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCisrKyBiL1dlYktp dFRvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDEwLTA2LTI0ICBKZXN1cyBTYW5j aGV6LVBhbGVuY2lhICA8amVzdXMucGFsZW5jaWFAb3BlbmJvc3NhLm9yZz4KKworICAgICAgICBS ZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBbUXRdIFF0VGVzdEJyb3dzZXIg ZG9lcyBub3QgaGF2ZSBhICJMb2FkIiBidXR0b24gOyB0aGVyZWZvcmUsIHVuYWJsZSB0byBsb2Fk IHBhZ2VzIG9uIHRvdWNoIG9ubHkgc3ltYmlhbiBkZXZpY2VzIChwb3J0cmFpdCBtb2RlKS4KKyAg ICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTM4NTk3CisKKyAg ICAgICAgTWFrZSB0aGUgcmVsb2FkIGJ1dHRvbiBhY3QgYXMgYSBsb2FkIGJ1dHRvbiBhcyB3ZWxs LgorCisgICAgICAgICogUXRUZXN0QnJvd3Nlci9tYWlud2luZG93LmNwcDoKKyAgICAgICAgKE1h aW5XaW5kb3c6OmJ1aWxkVUkpOgorICAgICAgICAoTWFpbldpbmRvdzo6Y2hhbmdlTG9jYXRpb24p OgorCiAyMDEwLTA2LTIxICBKb2NlbHluIFR1cmNvdHRlICA8am9jZWx5bi50dXJjb3R0ZUBub2tp YS5jb20+CiAKICAgICAgICAgVW5yZXZpZXdlZC4KZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL1F0 VGVzdEJyb3dzZXIvbWFpbndpbmRvdy5jcHAgYi9XZWJLaXRUb29scy9RdFRlc3RCcm93c2VyL21h aW53aW5kb3cuY3BwCmluZGV4IDk0MWVmNzkuLjBjOWZjYzEgMTAwNjQ0Ci0tLSBhL1dlYktpdFRv b2xzL1F0VGVzdEJyb3dzZXIvbWFpbndpbmRvdy5jcHAKKysrIGIvV2ViS2l0VG9vbHMvUXRUZXN0 QnJvd3Nlci9tYWlud2luZG93LmNwcApAQCAtNTEsOSArNTEsMTIgQEAgdm9pZCBNYWluV2luZG93 OjpidWlsZFVJKCkKICNpZiBkZWZpbmVkKFFfV1NfUzYwKQogICAgIGJhci0+c2V0SWNvblNpemUo UVNpemUoMTYsIDE2KSk7CiAjZW5kaWYKKyAgICBRQWN0aW9uKiByZWxvYWRBY3Rpb24gPSBwYWdl KCktPmFjdGlvbihRV2ViUGFnZTo6UmVsb2FkKTsKKyAgICBjb25uZWN0KHJlbG9hZEFjdGlvbiwg U0lHTkFMKHRyaWdnZXJlZCgpKSwgdGhpcywgU0xPVChjaGFuZ2VMb2NhdGlvbigpKSk7CisKICAg ICBiYXItPmFkZEFjdGlvbihwYWdlKCktPmFjdGlvbihRV2ViUGFnZTo6QmFjaykpOwogICAgIGJh ci0+YWRkQWN0aW9uKHBhZ2UoKS0+YWN0aW9uKFFXZWJQYWdlOjpGb3J3YXJkKSk7Ci0gICAgYmFy LT5hZGRBY3Rpb24ocGFnZSgpLT5hY3Rpb24oUVdlYlBhZ2U6OlJlbG9hZCkpOworICAgIGJhci0+ YWRkQWN0aW9uKHJlbG9hZEFjdGlvbik7CiAgICAgYmFyLT5hZGRBY3Rpb24ocGFnZSgpLT5hY3Rp b24oUVdlYlBhZ2U6OlN0b3ApKTsKIAogICAgIHVybEVkaXQgPSBuZXcgTG9jYXRpb25FZGl0KHRo aXMpOwpAQCAtMTQxLDYgKzE0NCwxMSBAQCB2b2lkIE1haW5XaW5kb3c6OmxvYWQoY29uc3QgUVVy bCYgdXJsKQogCiB2b2lkIE1haW5XaW5kb3c6OmNoYW5nZUxvY2F0aW9uKCkKIHsKKyAgICBpZiAo cGFnZSgpLT5tYWluRnJhbWUoKS0+dXJsKCkuaXNWYWxpZCgpKSB7CisgICAgICAgIHBhZ2UoKS0+ dHJpZ2dlckFjdGlvbihRV2ViUGFnZTo6UmVsb2FkKTsKKyAgICAgICAgcmV0dXJuOworICAgIH0K KwogICAgIFFTdHJpbmcgc3RyaW5nID0gdXJsRWRpdC0+dGV4dCgpOwogICAgIGxvYWQoc3RyaW5n KTsKIH0KLS0gCjEuNy4wLjMKCg==