10151 2006-07-29 05:27:57 -0700 ghostmap XHTML + SVG fails due to javascript undefined exception 2006-07-29 17:00:21 -0700 1 1 1 Unclassified WebKit SVG 420+ Mac OS X 10.4 RESOLVED FIXED http://www.logarithmic.net/ghost.xhtml P4 Normal --- 1 eric webkit-unassigned oldest_to_newest 51102 0 eric 2006-07-29 05:27:57 -0700 I'm not sure from looking at the JS console as to why this SVG + XHTML document is failing. The error shown in the console does not make any sense. This would be a cool demo of SVG + XHTML if we could make it work. 51116 1 rwlbuis 2006-07-29 09:24:19 -0700 Hi, (In reply to comment #0) > I'm not sure from looking at the JS console as to why this SVG + XHTML document > is failing. The error shown in the console does not make any sense. > > This would be a cool demo of SVG + XHTML if we could make it work. Actually there does not seem many problems. The main one is that the js contains document.URL, which AFAIK is a not supported IE extension. Note that FF must support it since this svg just works there. When I replaced it in the <script> with window.location.href (in 2 places) at least it rendered fine. After that the only problem is that the drawing area is unclipped, but I bet there is a seperate bug on that. Maybe someone with authority/experience can say how to handle document.URL, I have no idea how much it is used in real-life web pages. Cheers, Rob. 51118 2 rwlbuis 2006-07-29 10:00:45 -0700 Forget my earlier comment, document.URL *is* supported. Just in this particular testcase it evaluates to 'undefined'. Maybe something to do with being of xhtml+svg kind, further investigating... Cheers, Rob. (In reply to comment #1) > Hi, > > (In reply to comment #0) > > I'm not sure from looking at the JS console as to why this SVG + XHTML document > > is failing. The error shown in the console does not make any sense. > > > > This would be a cool demo of SVG + XHTML if we could make it work. > > Actually there does not seem many problems. The main one is that the js > contains document.URL, which AFAIK is a not supported IE extension. Note that > FF must support it since this svg just works there. When I replaced it in the > <script> > with window.location.href (in 2 places) at least it rendered fine. After that > the only problem is that the drawing area is unclipped, but I bet there is > a seperate bug on that. Maybe someone with authority/experience can say > how to handle document.URL, I have no idea how much it is used in real-life > web pages. > Cheers, > > Rob. > 51121 3 9755 rwlbuis 2006-07-29 12:53:42 -0700 Created attachment 9755 First attempt The problem in the end was not very svg specific. document.URL just only worked for html documents, this patch should fix that and the testcase. I also included the one changed result. Let me know if this requires an extra testcase, I don't think so since HTMLDocument04.xhtml tests it. Cheers, Rob. 51122 4 9755 ggaren 2006-07-29 12:57:09 -0700 Comment on attachment 9755 First attempt r=me 51131 5 darin 2006-07-29 17:00:21 -0700 Rob landed this as r15692. 9755 2006-07-29 12:53:42 -0700 2006-07-29 12:57:09 -0700 First attempt 10151.diff text/plain 4279 rwlbuis SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D aGFuZ2VMb2cJKHJldmlzaW9uIDE1NjkxKQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3Jr aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTMgQEAKKzIwMDYtMDctMjkgIFJvYiBCdWlzICA8YnVpc0Br ZGUub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAg IFRlc3QgY2FzZSBmb3IKKyAgICAgICAgPGh0dHA6Ly9idWd6aWxsYS5vcGVuZGFyd2luLm9yZy9z aG93X2J1Zy5jZ2k/aWQ9MTAxNTE+CisgICAgICAgIGdob3N0bWFwIFhIVE1MICsgU1ZHIGZhaWxz IGR1ZSB0byBqYXZhc2NyaXB0IHVuZGVmaW5lZCBleGNlcHRpb24KKworICAgICAgICAqIGRvbS94 aHRtbC9sZXZlbDIvaHRtbC9IVE1MRG9jdW1lbnQwNC1leHBlY3RlZC50eHQ6CisKIDIwMDYtMDct MjkgIERhcmluIEFkbGVyICA8ZGFyaW5AYXBwbGUuY29tPgogCiAgICAgICAgIC0gY2hhbmdlZCB0 aGlzIHRlc3QgdG8gYSBwbGFpbi10ZXh0IG9uZSBhbmQgYWRkZWQgdGV4dCBleHBsYWluaW5nIHdo YXQgaXQgdGVzdHMKSW5kZXg6IExheW91dFRlc3RzL2RvbS94aHRtbC9sZXZlbDIvaHRtbC9IVE1M RG9jdW1lbnQwNC1leHBlY3RlZC50eHQKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvZG9tL3ho dG1sL2xldmVsMi9odG1sL0hUTUxEb2N1bWVudDA0LWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMTU2 OTEpCisrKyBMYXlvdXRUZXN0cy9kb20veGh0bWwvbGV2ZWwyL2h0bWwvSFRNTERvY3VtZW50MDQt ZXhwZWN0ZWQudHh0CSh3b3JraW5nIGNvcHkpCkBAIC0xLDQgKzEsNCBAQAorQUxFUlQ6IGZpbGU6 Ly8vVXNlcnMvcndsYnVpcy9zdm4vdGVzdC9XZWJLaXQvTGF5b3V0VGVzdHMvZG9tL3hodG1sL2xl dmVsMi9odG1sL0hUTUxEb2N1bWVudDA0LnhodG1sCiBUZXN0CWh0dHA6Ly93d3cudzMub3JnLzIw MDEvRE9NLVRlc3QtU3VpdGUvbGV2ZWwyL2h0bWwvSFRNTERvY3VtZW50MDQKLVN0YXR1cwlmYWls dXJlCi1NZXNzYWdlCVVSTExpbms6IGFzc2VydE5vdE51bGwgZmFpbGVkCitTdGF0dXMJU3VjY2Vz cwogCkluZGV4OiBXZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL0NoYW5n ZUxvZwkocmV2aXNpb24gMTU2OTEpCisrKyBXZWJDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5 KQpAQCAtMSwzICsxLDE4IEBACisyMDA2LTA3LTI5ICBSb2IgQnVpcyAgPGJ1aXNAa2RlLm9yZz4K KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBodHRwOi8v YnVnemlsbGEub3BlbmRhcndpbi5vcmcvc2hvd19idWcuY2dpP2lkPTEwMTUxCisgICAgICAgIGdo b3N0bWFwIFhIVE1MICsgU1ZHIGZhaWxzIGR1ZSB0byBqYXZhc2NyaXB0IHVuZGVmaW5lZCBleGNl cHRpb24KKworICAgICAgICBNYWtlIHN1cmUgZG9jdW1lbnQuVVJMIHdvcmtzIHdoZW4gY2FsbGVk IGZyb20gYSBzdmcgZG9jdW1lbnQuCisKKyAgICAgICAgKiBiaW5kaW5ncy9qcy9ranNfaHRtbC5j cHA6CisgICAgICAgIChLSlM6OkpTSFRNTERvY3VtZW50OjpnZXRWYWx1ZVByb3BlcnR5KToKKyAg ICAgICAgKiBiaW5kaW5ncy9qcy9ranNfaHRtbC5oOgorICAgICAgICAoS0pTOjpKU0hUTUxEb2N1 bWVudDo6KToKKyAgICAgICAgKiBkb20vRG9jdW1lbnQuaWRsOgorCiAyMDA2LTA3LTI5ICBNaWtl IEVtbWVsICA8bWlrZS5lbW1lbEBnbWFpbC5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYW5kIHR3 ZWFrZWQgYnkgRGFyaW4uCkluZGV4OiBXZWJDb3JlL2JpbmRpbmdzL2pzL2tqc19odG1sLmNwcAo9 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09Ci0tLSBXZWJDb3JlL2JpbmRpbmdzL2pzL2tqc19odG1sLmNwcAkocmV2aXNpb24g MTU2OTEpCisrKyBXZWJDb3JlL2JpbmRpbmdzL2pzL2tqc19odG1sLmNwcAkod29ya2luZyBjb3B5 KQpAQCAtMTc0LDcgKzE3NCw2IEBAIGNvbnN0IENsYXNzSW5mbyBKU0hUTUxEb2N1bWVudDo6aW5m byA9CiAgIHRpdGxlICAgICAgICAgICAgICAgICBKU0hUTUxEb2N1bWVudDo6VGl0bGUgICAgICAg ICAgICAgRG9udERlbGV0ZQogICByZWZlcnJlciAgICAgICAgICAgICAgSlNIVE1MRG9jdW1lbnQ6 OlJlZmVycmVyICAgICAgICAgIERvbnREZWxldGV8UmVhZE9ubHkKICAgZG9tYWluICAgICAgICAg ICAgICAgIEpTSFRNTERvY3VtZW50OjpEb21haW4gICAgICAgICAgICBEb250RGVsZXRlCi0gIFVS TCAgICAgICAgICAgICAgICAgICBKU0hUTUxEb2N1bWVudDo6VVJMICAgICAgICAgICAgICAgRG9u dERlbGV0ZXxSZWFkT25seQogICBib2R5ICAgICAgICAgICAgICAgICAgSlNIVE1MRG9jdW1lbnQ6 OkJvZHkgICAgICAgICAgICAgIERvbnREZWxldGUKICAgbG9jYXRpb24gICAgICAgICAgICAgIEpT SFRNTERvY3VtZW50OjpMb2NhdGlvbiAgICAgICAgICBEb250RGVsZXRlCiAgIGNvb2tpZSAgICAg ICAgICAgICAgICBKU0hUTUxEb2N1bWVudDo6Q29va2llICAgICAgICAgICAgRG9udERlbGV0ZQpA QCAtMjQ5LDggKzI0OCw2IEBAIEpTVmFsdWUgKkpTSFRNTERvY3VtZW50OjpnZXRWYWx1ZVByb3Bl cnQKICAgICByZXR1cm4ganNTdHJpbmcoZG9jLnJlZmVycmVyKCkpOwogICBjYXNlIERvbWFpbjoK ICAgICByZXR1cm4ganNTdHJpbmcoZG9jLmRvbWFpbigpKTsKLSAgY2FzZSBVUkw6Ci0gICAgcmV0 dXJuIGpzU3RyaW5nKGRvYy5VUkwoKSk7CiAgIGNhc2UgQm9keToKICAgICByZXR1cm4gdG9KUyhl eGVjLCBib2R5KTsKICAgY2FzZSBMb2NhdGlvbjoKSW5kZXg6IFdlYkNvcmUvYmluZGluZ3MvanMv a2pzX2h0bWwuaAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL2JpbmRpbmdzL2pzL2tqc19odG1sLmgJ KHJldmlzaW9uIDE1NjkxKQorKysgV2ViQ29yZS9iaW5kaW5ncy9qcy9ranNfaHRtbC5oCSh3b3Jr aW5nIGNvcHkpCkBAIC01MSw3ICs1MSw3IEBAIG5hbWVzcGFjZSBLSlMgewogICAgIHZvaWQgcHV0 VmFsdWVQcm9wZXJ0eShFeGVjU3RhdGUqLCBpbnQgdG9rZW4sIEpTVmFsdWUqLCBpbnQgLyphdHRy Ki8pOwogICAgIHZpcnR1YWwgY29uc3QgQ2xhc3NJbmZvKiBjbGFzc0luZm8oKSBjb25zdCB7IHJl dHVybiAmaW5mbzsgfQogICAgIHN0YXRpYyBjb25zdCBDbGFzc0luZm8gaW5mbzsKLSAgICBlbnVt IHsgVGl0bGUsIFJlZmVycmVyLCBEb21haW4sIFVSTCwgQm9keSwgTG9jYXRpb24sIENvb2tpZSwK KyAgICBlbnVtIHsgVGl0bGUsIFJlZmVycmVyLCBEb21haW4sIEJvZHksIExvY2F0aW9uLCBDb29r aWUsCiAgICAgICAgICAgIEltYWdlcywgQXBwbGV0cywgRW1iZWRzLCBMaW5rcywgRm9ybXMsIEFu Y2hvcnMsIFNjcmlwdHMsIEFsbCwgQ2xlYXIsIE9wZW4sIENsb3NlLAogICAgICAgICAgICBXcml0 ZSwgV3JpdGVMbiwgR2V0RWxlbWVudHNCeU5hbWUsIENhcHR1cmVFdmVudHMsIFJlbGVhc2VFdmVu dHMsCiAgICAgICAgICAgIEJnQ29sb3IsIEZnQ29sb3IsIEFsaW5rQ29sb3IsIExpbmtDb2xvciwg VmxpbmtDb2xvciwgTGFzdE1vZGlmaWVkLCBIZWlnaHQsIFdpZHRoLCBEaXIsIERlc2lnbk1vZGUg fTsKSW5kZXg6IFdlYkNvcmUvZG9tL0RvY3VtZW50LmlkbAo9PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3Jl L2RvbS9Eb2N1bWVudC5pZGwJKHJldmlzaW9uIDE1NjkxKQorKysgV2ViQ29yZS9kb20vRG9jdW1l bnQuaWRsCSh3b3JraW5nIGNvcHkpCkBAIC0xNDgsNiArMTQ4LDggQEAgbW9kdWxlIGNvcmUgewog CiAgICAgICAgIC8vIElFIGV4dGVuc2lvbnMKIAorICAgICAgICByZWFkb25seSBhdHRyaWJ1dGUg RE9NU3RyaW5nIFVSTDsKKwogICAgICAgICBhdHRyaWJ1dGUgW0NvbnZlcnROdWxsU3RyaW5nVG89 VW5kZWZpbmVkXSBET01TdHJpbmcgY2hhcnNldDsKIAogICAgICAgICByZWFkb25seSBhdHRyaWJ1 dGUgW0NvbnZlcnROdWxsU3RyaW5nVG89VW5kZWZpbmVkXSBET01TdHJpbmcgZGVmYXVsdENoYXJz ZXQ7Cg==