35830 2010-03-06 11:52:52 -0800 Unable to use Hotmail.com mail features, as clicking on links seems not to work 2014-04-24 16:45:13 -0700 1 1 1 Unclassified WebKit JavaScriptCore 528+ (Nightly build) Other All RESOLVED WONTFIX Gtk, Qt, QtTriaged P1 Major --- 35784 1 pano_90 hausmann gustavo hausmann jedrzej.nowacki jturcotte kenneth kent.hansen koivisto laszlo.gombos nancy.piedra vikram.malaiya yael oldest_to_newest 196928 0 pano_90 2010-03-06 11:52:52 -0800 When arriving to hotmail.com's page which lists your e-mails, nothing works, clicking links doesn't do anything. It does not work in rekonq, nor arora. I created a test account on hotmail so that you don't have to :-P Username: weqtvy3@hotmail.com Password: 123456 After logging in, you might see a page stating that you should upgrade your webbrowser. Simply ignore this and browse to the mails by clicking on Mail in the upper part of the website. clicking on the mail does not work, the commands (on the lower part of the website "New, Delete, Junk …") do not work either. Original bug report: https://bugs.kde.org/show_bug.cgi?id=228475 Thanks :-) 196929 1 pano_90 2010-03-06 11:57:58 -0800 Forgot to mention: Tested here with Qt 4.6.1 198067 2 vestbo 2010-03-10 06:33:23 -0800 Please follow the QtWebKit bug reporting guidelines when reporting bugs. See http://trac.webkit.org/wiki/QtWebKitBugs Specifically: - The 'QtWebKit' component should only be used for bugs/features in the public QtWebKit API layer, not to signify that the bug is specific to the Qt port of WebKit http://trac.webkit.org/wiki/QtWebKitBugs#Component - Add the keyword 'Qt' to signal that it's a Qt-related bug http://trac.webkit.org/wiki/QtWebKitBugs#Keywords 198572 3 jturcotte 2010-03-11 05:45:56 -0800 Reproduced in trunk with Qt 4.6. We have to try fixing this for the next issue, depending on the work needed. 201323 4 jedrzej.nowacki 2010-03-18 08:50:21 -0700 Reproduced on Linux(32 bit Debian/testing): - with trunk of the Qt and the WebKit using QtLauncher. - with trunk of the WebKit using GtkLauncher. I tried to debug it, but it is rather difficult with obscured JavaScript code from that page. If someone know how to debug this kind of bugs, please share your knowledge :-). 209041 5 kent.hansen 2010-04-06 07:32:53 -0700 (In reply to comment #4) > I tried to debug it, but it is rather difficult with obscured JavaScript code > from that page. If someone know how to debug this kind of bugs, please share > your knowledge :-). I used http://jsbeautifier.org to unminify the JS. http://gfx6.hotmail.com/mail/15.1.3057.0331/i0a.mozilla.js seems to contain the relevant code, e.g. search for NewMessage / newMsg for code that should be triggered when clicking on the "New" item. My idea for debugging is to patch the network handler to deliver the unminified versions of the scripts, so that we can get line numbers and set breakpoints in the Inspector. We can even insert debug statements in the unminified script... :-) 210096 6 hausmann 2010-04-08 02:20:19 -0700 It appears that hotmail.com does not work with our default user agent string, which is Mozilla/5.0 (%Platform%; %Security%; %Subplatform%; %Locale%) AppleWebKit/%WebKitVersion% (KHTML, like Gecko) %AppVersion Safari/%WebKitVersion% However it works when inserting a Version/4.0 component, so that the resulting string is for example on Linux: Mozilla/5.0 (X11; U; Linux; en-US) AppleWebKit/533.4 (KHTML, like Gecko) Version/4.0 QtLauncher/0.1 Safari/533.4 What do you guys think about adding the "arbitrary" Version/4.0 compoment? Note that the order is significant: It only works if the Version component follows directly the AppleWebKit/... (KHTML...) part. 210116 7 yael 2010-04-08 05:17:35 -0700 (In reply to comment #6) > What do you guys think about adding the "arbitrary" Version/4.0 compoment? > > Note that the order is significant: It only works if the Version component > follows directly the AppleWebKit/... (KHTML...) part. A few years ago we updated webkit in S60, to the version of Safari 3.0. At the time, I did exactly the same thing in order to make hotmail.com work. (add version 3.0) 210117 8 kenneth 2010-04-08 05:40:28 -0700 (In reply to comment #6) > It appears that hotmail.com does not work with our default user agent string, > which is > > Mozilla/5.0 (%Platform%; %Security%; %Subplatform%; %Locale%) > AppleWebKit/%WebKitVersion% (KHTML, like Gecko) %AppVersion > Safari/%WebKitVersion% > > > However it works when inserting a Version/4.0 component, so that the resulting > string is for example on Linux: > > Mozilla/5.0 (X11; U; Linux; en-US) AppleWebKit/533.4 (KHTML, like Gecko) > Version/4.0 QtLauncher/0.1 Safari/533.4 > > > What do you guys think about adding the "arbitrary" Version/4.0 compoment? > > Note that the order is significant: It only works if the Version component > follows directly the AppleWebKit/... (KHTML...) part. Android adds it, and iPad adds Version/4.0.4. I think it would be fine for us to do that as well. 210118 9 kenneth 2010-04-08 05:44:12 -0700 > Android adds it, and iPad adds Version/4.0.4. I think it would be fine for us > to do that as well. The full iPad string is: Mozilla/5.0(iPad; U; CPU iPhone OS 3_2 like Mac OS X; en-us) AppleWebKit/531.21.10 (KHTML, like Gecko) Version/4.0.4 Mobile/7B314 Safari/531.21.10 Funny to see that they kept "iPhone" in the string :-) On the iPhone it doesn't say "CPU iPhone OS 3_2 like Mac OS X" but "CPU OS 3_2 like Mac OS X" 210120 10 52854 hausmann 2010-04-08 05:55:49 -0700 Created attachment 52854 Patch 210124 11 52854 kenneth 2010-04-08 06:17:19 -0700 Comment on attachment 52854 Patch Why not use 4.0.4 as the iPad? 210131 12 52854 gustavo 2010-04-08 06:28:01 -0700 Comment on attachment 52854 Patch Sounds good to me. rs=me on changing this to 4.0.4 if you feel the need, but I think we should be fine with the existing practice. 210136 13 52854 hausmann 2010-04-08 06:39:02 -0700 Comment on attachment 52854 Patch Thanks for the review Gustavo. I'm holding off the commit queue to give some more time for feedback before landing. 210139 14 kenneth 2010-04-08 06:41:19 -0700 (In reply to comment #13) > (From update of attachment 52854 [details]) > Thanks for the review Gustavo. I'm holding off the commit queue to give some > more time for feedback before landing. I would still like to know the advantage of using 4.0 instead of 4.0.4 which seems to be the latest version uses. Does it work with 4.0.4? 210155 15 hausmann 2010-04-08 07:08:32 -0700 (In reply to comment #14) > (In reply to comment #13) > > (From update of attachment 52854 [details] [details]) > > Thanks for the review Gustavo. I'm holding off the commit queue to give some > > more time for feedback before landing. > > I would still like to know the advantage of using 4.0 instead of 4.0.4 which > seems to be the latest version uses. Does it work with 4.0.4? I can confirm that it works with 4.0.4, but what's the advantage of a more specific number over a major one? :) 210157 16 kenneth 2010-04-08 07:13:08 -0700 Isn't it considered a minimum required version, thus 4.0.4 right be required by some services where 4.0 would fail? I'm not sure, but there must be a reason it was chosen for iPhone and iPad. 210164 17 kent.hansen 2010-04-08 07:29:32 -0700 (In reply to comment #16) > Isn't it considered a minimum required version, thus 4.0.4 right be required by > some services where 4.0 would fail? I'm not sure, but there must be a reason it > was chosen for iPhone and iPad. 4.0.4 is so oldschool; it's a "4.0.4 Not Found" waiting to happen. 4.1 works, as does 5.0. Might as well be future-proof (it should increase the time until this task must be reopened to bump the version again). :P 210168 18 kenneth 2010-04-08 07:35:49 -0700 (In reply to comment #17) > (In reply to comment #16) > > Isn't it considered a minimum required version, thus 4.0.4 right be required by > > some services where 4.0 would fail? I'm not sure, but there must be a reason it > > was chosen for iPhone and iPad. > > 4.0.4 is so oldschool; it's a "4.0.4 Not Found" waiting to happen. 4.1 works, > as does 5.0. Might as well be future-proof (it should increase the time until > this task must be reopened to bump the version again). :P Then use 4.0.5 which is the latest Safari version. The 4.0.4 or higher might be required to use some newer service being made for the iPad. 210223 19 laszlo.gombos 2010-04-08 08:45:10 -0700 According to this content - http://www.useragentstring.com/pages/Chrome/ Chrome does not add the "Version" part to the UA header. Taking a closer look it seems like Chrome had this for one single version and than its gone again (see Chrome 2.0.156.0). I do not know if this info helps in any way to make this decision. It would be good if we could only carry one cross-platform WebKit version around. With this patch it seems like we will have two version number (with the risk of the two being inconsistent). 210245 20 laszlo.gombos 2010-04-08 09:34:39 -0700 Just to be specific, I'm ok with the change but would be somehow better to pick up the number cleverly (if possible) than hard-coding it. It would also be useful to track down the JavaScript which check for the version (instead of doing object/feature detection). If it is a JavaScript library that is used in many places would be worthwhile to fix the JavaScript library (as well). 210250 21 kenneth 2010-04-08 09:38:41 -0700 kenne> aroben do you know if the Version/4.0.4 in the UA is the Safari version? or what does it represent? <aroben> kenne: yes, that is the Safari version 217124 22 hausmann 2010-04-26 08:40:24 -0700 (In reply to comment #19) > According to this content - http://www.useragentstring.com/pages/Chrome/ Chrome > does not add the "Version" part to the UA header. Taking a closer look it seems > like Chrome had this for one single version and than its gone again (see Chrome > 2.0.156.0). I do not know if this info helps in any way to make this decision. > > It would be good if we could only carry one cross-platform WebKit version > around. With this patch it seems like we will have two version number (with the > risk of the two being inconsistent). What's the other option? We have to include this tag in order for the web site to function. The tag/version does not come from WebKit but from an external third-party product ;( I guess the other option would be to pretend to be Chrome and not include a version... 217172 23 kenneth 2010-04-26 10:09:52 -0700 Or make a quirk that returns a different UA for the site? 222318 24 kenneth 2010-05-07 06:29:10 -0700 (In reply to comment #23) > Or make a quirk that returns a different UA for the site? We need to decide on this one. 224562 25 hausmann 2010-05-12 06:14:02 -0700 We decided to try to solve this on the server side and otherwise land it as-is on May the 26th. 226534 26 52854 laszlo.gombos 2010-05-16 13:55:50 -0700 Comment on attachment 52854 Patch cq- until we conclude investigation on the server side. 229303 27 kenneth 2010-05-22 12:25:43 -0700 (In reply to comment #26) > (From update of attachment 52854 [details]) > cq- until we conclude investigation on the server side. Any update on this Laszlo? We have almost waited long enough. 229378 28 laszlo.gombos 2010-05-22 18:05:02 -0700 This is not resolved yet but I got a very promising verbal promise that fix on the server side can be arrange and with that we were asked to not hack around this issue on the client/UA side. I suggest we close this bug without committing the patch. 229383 29 kenneth 2010-05-22 18:44:52 -0700 Ok, let's close it then. Thanks for looking into this. 1003848 30 darin 2014-04-24 16:45:13 -0700 Moving all JavaScriptGlue bugs to JavaScriptCore. The JavaScriptGlue framework itself is long gone. And most of the more recent bugs put in this component were put there by people who thought this was for some other aspect of “JavaScript glue” and have nothing to do with the actual original reason for the existence of this component, which was an OS-X-only framework named JavaScriptGlue. 52854 2010-04-08 05:55:49 -0700 2010-05-16 13:55:50 -0700 Patch bug-35830-20100408145547.patch text/plain 3568 hausmann ZGlmZiAtLWdpdCBhL1dlYktpdC9ndGsvQ2hhbmdlTG9nIGIvV2ViS2l0L2d0ay9DaGFuZ2VMb2cK aW5kZXggMzNhZTVmYS4uZmY2MDkxNCAxMDA2NDQKLS0tIGEvV2ViS2l0L2d0ay9DaGFuZ2VMb2cK KysrIGIvV2ViS2l0L2d0ay9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNiBAQAorMjAxMC0wNC0wOCAg U2ltb24gSGF1c21hbm4gIDxzaW1vbi5oYXVzbWFubkBub2tpYS5jb20+CisKKyAgICAgICAgUmV2 aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVW5hYmxlIHRvIHVzZSBIb3RtYWls LmNvbSBtYWlsIGZlYXR1cmVzLCBhcyBjbGlja2luZyBvbiBsaW5rcyBzZWVtcyBub3QgdG8gd29y aworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzU4MzAK KworICAgICAgICBIb3RtYWlsIGV4cGVjdHMgYSAiVmVyc2lvbiIgY29tcG9uZW50IGluIHRoZSB1 c2VyIGFnZW50IHN0cmluZywgaW4gb3JkZXIgdG8gZGVsaXZlcgorICAgICAgICBmdW5jdGlvbmFs IEhUTUwvSlMuCisKKyAgICAgICAgKiB3ZWJraXQvd2Via2l0d2Vic2V0dGluZ3MuY3BwOgorICAg ICAgICAod2Via2l0X2dldF91c2VyX2FnZW50KToKKwogMjAxMC0wNC0wNyAgQW5kcmV5IEtvc3lh a292ICA8Y2FzZXFAY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IFl1cnkgU2Vt aWtoYXRza3kuCmRpZmYgLS1naXQgYS9XZWJLaXQvZ3RrL3dlYmtpdC93ZWJraXR3ZWJzZXR0aW5n cy5jcHAgYi9XZWJLaXQvZ3RrL3dlYmtpdC93ZWJraXR3ZWJzZXR0aW5ncy5jcHAKaW5kZXggNWQy ZDY1OC4uMmM1ZmEwMiAxMDA2NDQKLS0tIGEvV2ViS2l0L2d0ay93ZWJraXQvd2Via2l0d2Vic2V0 dGluZ3MuY3BwCisrKyBiL1dlYktpdC9ndGsvd2Via2l0L3dlYmtpdHdlYnNldHRpbmdzLmNwcApA QCAtMjA2LDcgKzIwNiw3IEBAIHN0YXRpYyBTdHJpbmcgd2Via2l0X2dldF91c2VyX2FnZW50KCkK ICAgICAvLyBXZSByZS11c2UgdGhlIFdlYktpdCB2ZXJzaW9uLCB0aG91Z2ggaXQgZG9lc24ndCBz ZWVtIHRvIG1hdHRlciBtdWNoIGluIHByYWN0aWNlCiAKICAgICBERUZJTkVfU1RBVElDX0xPQ0FM KGNvbnN0IFN0cmluZywgdWFWZXJzaW9uLCAoU3RyaW5nOjpmb3JtYXQoIiVkLiVkKyIsIFdFQktJ VF9VU0VSX0FHRU5UX01BSk9SX1ZFUlNJT04sIFdFQktJVF9VU0VSX0FHRU5UX01JTk9SX1ZFUlNJ T04pKSk7Ci0gICAgREVGSU5FX1NUQVRJQ19MT0NBTChjb25zdCBTdHJpbmcsIHN0YXRpY1VBLCAo U3RyaW5nOjpmb3JtYXQoIk1vemlsbGEvNS4wICglczsgVTsgJXM7ICVzKSBBcHBsZVdlYktpdC8l cyAoS0hUTUwsIGxpa2UgR2Vja28pIFNhZmFyaS8lcyIsCisgICAgREVGSU5FX1NUQVRJQ19MT0NB TChjb25zdCBTdHJpbmcsIHN0YXRpY1VBLCAoU3RyaW5nOjpmb3JtYXQoIk1vemlsbGEvNS4wICgl czsgVTsgJXM7ICVzKSBBcHBsZVdlYktpdC8lcyAoS0hUTUwsIGxpa2UgR2Vja28pIFZlcnNpb24v NC4wIFNhZmFyaS8lcyIsCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgcGxhdGZvcm0sIG9zVmVyc2lvbiwgZGVmYXVsdExhbmd1 YWdlKCkudXRmOCgpLmRhdGEoKSwgdWFWZXJzaW9uLnV0ZjgoKS5kYXRhKCksIHVhVmVyc2lvbi51 dGY4KCkuZGF0YSgpKSkpOwogCiAgICAgZ19mcmVlKG9zVmVyc2lvbik7CmRpZmYgLS1naXQgYS9X ZWJLaXQvcXQvQXBpL3F3ZWJwYWdlLmNwcCBiL1dlYktpdC9xdC9BcGkvcXdlYnBhZ2UuY3BwCmlu ZGV4IDVkZTgxMzguLjU4OTk3NDcgMTAwNjQ0Ci0tLSBhL1dlYktpdC9xdC9BcGkvcXdlYnBhZ2Uu Y3BwCisrKyBiL1dlYktpdC9xdC9BcGkvcXdlYnBhZ2UuY3BwCkBAIC0zMTc5LDcgKzMxNzksNyBA QCBRV2ViUGx1Z2luRmFjdG9yeSAqUVdlYlBhZ2U6OnBsdWdpbkZhY3RvcnkoKSBjb25zdAogCiAg ICAgVGhlIGRlZmF1bHQgaW1wbGVtZW50YXRpb24gcmV0dXJucyB0aGUgZm9sbG93aW5nIHZhbHVl OgogCi0gICAgIk1vemlsbGEvNS4wICglUGxhdGZvcm0lOyAlU2VjdXJpdHklOyAlU3VicGxhdGZv cm0lOyAlTG9jYWxlJSkgQXBwbGVXZWJLaXQvJVdlYktpdFZlcnNpb24lIChLSFRNTCwgbGlrZSBH ZWNrbykgJUFwcFZlcnNpb24gU2FmYXJpLyVXZWJLaXRWZXJzaW9uJSIKKyAgICAiTW96aWxsYS81 LjAgKCVQbGF0Zm9ybSU7ICVTZWN1cml0eSU7ICVTdWJwbGF0Zm9ybSU7ICVMb2NhbGUlKSBBcHBs ZVdlYktpdC8lV2ViS2l0VmVyc2lvbiUgKEtIVE1MLCBsaWtlIEdlY2tvKSBWZXJzaW9uLzQuMCAl QXBwVmVyc2lvbiBTYWZhcmkvJVdlYktpdFZlcnNpb24lIgogCiAgICAgT24gbW9iaWxlIHBsYXRm b3JtcyBzdWNoIGFzIFN5bWJpYW4gUzYwIGFuZCBNYWVtbywgIk1vYmlsZSBTYWZhcmkiIGlzIHVz ZWQgaW5zdGVhZCBvZiAiU2FmYXJpIi4KIApAQCAtMzQwMSw2ICszNDAxLDkgQEAgUVN0cmluZyBR V2ViUGFnZTo6dXNlckFnZW50Rm9yVXJsKGNvbnN0IFFVcmwmIHVybCkgY29uc3QKICAgICB1YS5h cHBlbmQoUVN0cmluZyhRTGF0aW4xU3RyaW5nKCJBcHBsZVdlYktpdC8lMSAoS0hUTUwsIGxpa2Ug R2Vja28pICIpKQogICAgICAgICAgICAgICAgICAgICAgIC5hcmcoUVN0cmluZyhxV2ViS2l0VmVy c2lvbigpKSkpOwogCisgICAgLy8gbWFnaWMgdmVyc2lvbiBjb21wb25lbnQgKGh0dHBzOi8vYnVn cy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zNTgzMCkKKyAgICB1YS5hcHBlbmQoUUxhdGlu MVN0cmluZygiVmVyc2lvbi80LjAgIikpOworCiAgICAgLy8gQXBwbGljYXRpb24gbmFtZS92ZXJz aW9uCiAgICAgUVN0cmluZyBhcHBOYW1lID0gUUNvcmVBcHBsaWNhdGlvbjo6YXBwbGljYXRpb25O YW1lKCk7CiAgICAgaWYgKCFhcHBOYW1lLmlzRW1wdHkoKSkgewpkaWZmIC0tZ2l0IGEvV2ViS2l0 L3F0L0NoYW5nZUxvZyBiL1dlYktpdC9xdC9DaGFuZ2VMb2cKaW5kZXggOGZhNjYxNi4uMjc0Njhk MCAxMDA2NDQKLS0tIGEvV2ViS2l0L3F0L0NoYW5nZUxvZworKysgYi9XZWJLaXQvcXQvQ2hhbmdl TG9nCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTAtMDQtMDggIFNpbW9uIEhhdXNtYW5uICA8c2ltb24u aGF1c21hbm5Abm9raWEuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp LgorCisgICAgICAgIFVuYWJsZSB0byB1c2UgSG90bWFpbC5jb20gbWFpbCBmZWF0dXJlcywgYXMg Y2xpY2tpbmcgb24gbGlua3Mgc2VlbXMgbm90IHRvIHdvcmsKKyAgICAgICAgaHR0cHM6Ly9idWdz LndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTM1ODMwCisKKyAgICAgICAgSG90bWFpbCBleHBl Y3RzIGEgIlZlcnNpb24iIGNvbXBvbmVudCBpbiB0aGUgdXNlciBhZ2VudCBzdHJpbmcsIGluIG9y ZGVyIHRvIGRlbGl2ZXIKKyAgICAgICAgZnVuY3Rpb25hbCBIVE1ML0pTLgorCisgICAgICAgICog QXBpL3F3ZWJwYWdlLmNwcDoKKyAgICAgICAgKFFXZWJQYWdlOjp1c2VyQWdlbnRGb3JVcmwpOgor CiAyMDEwLTA0LTA4ICBKb2UgTGlnbWFuICA8am9zZXBoLmxpZ21hbkBub2tpYS5jb20+CiAKICAg ICAgICAgUmV2aWV3ZWQgYnkgU2ltb24gSGF1c21hbm4uCg==