38752 2010-05-07 08:15:34 -0700 Strange behaviour after js insertion into a page pointing directly to an object (image, swf, etc) 2010-05-12 12:21:10 -0700 1 1 1 Unclassified WebKit DOM 528+ (Nightly build) PC OS X 10.5 RESOLVED FIXED P2 Normal --- 1 avitte ap ap oldest_to_newest 222383 0 avitte 2010-05-07 08:15:34 -0700 The problem was discovered with this url : http://www.villiard.com/images/voitures/voiture.jpg Go on this url and put in the address bar following js insertion (that we call "Extract Widget"): javascript:(function(){var d=document;var h=d.getElementsByTagName("head");var s=h.length===0?d.getElementsByTagName("html")[0].insertBefore(d.createElement("HEAD"),d.body):h[0];if(typeof JSON!='object'){var j=d.createElement("SCRIPT");j.charset="utf-8";j.id="jsonscript";j.src="http://www.blimpme.com/bbrowser/json2.js";j=s.appendChild(j);};var b=d.createElement("SCRIPT");b.charset="utf-8";b.src="http://www.blimpme.com/bbrowser/injectold.php?url="+encodeURIComponent(d.location.href);b=s.appendChild(b);})(); Then reload the first url and put in the address bar : javascript:(function(){var d=document;var h=d.getElementsByTagName("head");var s=h.length===0?d.getElementsByTagName("html")[0].insertBefore(d.createElement("HEAD"),d.body):h[0];if(typeof JSON!='object'){var j=d.createElement("SCRIPT");j.charset="utf-8";j.id="jsonscript";j.src="http://www.blimpme.com/bbrowser/json2.js";j=s.appendChild(j);};var b=d.createElement("SCRIPT");b.charset="utf-8";b.src="http://www.blimpme.com/bbrowser/inject.php?url="+encodeURIComponent(d.location.href);b=s.appendChild(b);})(); As you can see the result is different, in the first case we do not see the background images although it's there, we have checked with the debugger and the objects look correct in the page. The second case is a palliative to correct that, we do reload the url inside an iframe and then the extract widget js insertion works correctly. We know it's a very particular case but still needs to be investigated, it behaves as if everything was not working in the dom while loading an url pointing directly to an object. 222531 1 ap 2010-05-07 12:14:40 -0700 There is a lot of obfuscated code in inject.php. Could you make a reduced test case that would be easier to understand? I suspect that the issue here is that ImageDocument is a separate Document subclass, different from HTMLDocument. That's a bug in WebKit, it should not be separate. But it's hard to tell whether this is the issue without a reduced test case. See also: bug 25397. 222881 2 avitte 2010-05-08 07:52:35 -0700 Hi, Please try : javascript:(function(){var d=document;var h=d.getElementsByTagName("head");var s=h.length===0?d.getElementsByTagName("html")[0].insertBefore(d.createElement("HEAD"),d.body):h[0];var b=d.createElement("SCRIPT");b.charset="utf-8";b.src="http://www.blimpme.com/bbrowser/bug.js";b=s.appendChild(b);})(); You will see on the top left corner of the page 'Test Bug' but without the background image that we include in bug.js which just insert a div in the page with a background image and a text 'Test Bug' 223591 3 ap 2010-05-10 16:53:12 -0700 This happens because DocLoader::m_autoLoadImages is false for ImageDocument documents. 224017 4 55721 ap 2010-05-11 10:58:54 -0700 Created attachment 55721 proposed fix 224795 5 ap 2010-05-12 12:21:10 -0700 Committed <http://trac.webkit.org/changeset/59250>. I'm not 100% sure if this also fixed .swf - please test with the next nightly build, and file a new bug report if it didn't. 55721 2010-05-11 10:58:54 -0700 2010-05-12 12:02:46 -0700 proposed fix ImageImage.txt text/plain 6805 ap SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA1OTE1NykKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTUgQEAKKzIwMTAtMDUtMTEgIEFsZXhleSBQcm9za3VyeWFrb3YgIDxhcEBhcHBs ZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTM4NzUyCisgICAgICAgIFN0 cmFuZ2UgYmVoYXZpb3VyIGFmdGVyIGpzIGluc2VydGlvbiBpbnRvIGEgcGFnZSBwb2ludGluZyBk aXJlY3RseSB0byBhbiBvYmplY3QgKGltYWdlLCBzd2YsIGV0YykKKworICAgICAgICBUZXN0OiBm YXN0L2ltYWdlcy9leHRyYS1pbWFnZS1pbi1pbWFnZS1kb2N1bWVudC5odG1sCisKKyAgICAgICAg KiBsb2FkZXIvSW1hZ2VMb2FkZXIuY3BwOiAoV2ViQ29yZTo6SW1hZ2VMb2FkZXI6OnVwZGF0ZUZy b21FbGVtZW50KTogUmVzZXQgc2V0QXV0b0xvYWRJbWFnZXMKKyAgICAgICAgYWZ0ZXIgbWFudWFs bHkgZmVlZGluZyBhbiBpbWFnZSAtIHRoZXJlIG1heSBiZSBtb3JlIGNvbWluZy4KKwogMjAxMC0w NS0xMSAgWGFuIExvcGV6ICA8eGxvcGV6QGlnYWxpYS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQg YnkgR3VzdGF2byBOb3JvbmhhLgpJbmRleDogV2ViQ29yZS9sb2FkZXIvSW1hZ2VMb2FkZXIuY3Bw Cj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT0KLS0tIFdlYkNvcmUvbG9hZGVyL0ltYWdlTG9hZGVyLmNwcAkocmV2aXNpb24g NTg5MDgpCisrKyBXZWJDb3JlL2xvYWRlci9JbWFnZUxvYWRlci5jcHAJKHdvcmtpbmcgY29weSkK QEAgLTE1NSwxMSArMTU1LDEzIEBAIHZvaWQgSW1hZ2VMb2FkZXI6OnVwZGF0ZUZyb21FbGVtZW50 KCkKICAgICBDYWNoZWRJbWFnZSogbmV3SW1hZ2UgPSAwOwogICAgIGlmICghKGF0dHIuaXNOdWxs KCkgfHwgKGF0dHIuaXNFbXB0eSgpICYmIGRvY3VtZW50LT5iYXNlVVJJKCkuaXNMb2NhbEZpbGUo KSkpKSB7CiAgICAgICAgIGlmIChtX2xvYWRNYW51YWxseSkgeworICAgICAgICAgICAgYm9vbCBh dXRvTG9hZE90aGVySW1hZ2VzID0gZG9jdW1lbnQtPmRvY0xvYWRlcigpLT5hdXRvTG9hZEltYWdl cygpOwogICAgICAgICAgICAgZG9jdW1lbnQtPmRvY0xvYWRlcigpLT5zZXRBdXRvTG9hZEltYWdl cyhmYWxzZSk7CiAgICAgICAgICAgICBuZXdJbWFnZSA9IG5ldyBDYWNoZWRJbWFnZShzb3VyY2VV UkkoYXR0cikpOwogICAgICAgICAgICAgbmV3SW1hZ2UtPnNldExvYWRpbmcodHJ1ZSk7CiAgICAg ICAgICAgICBuZXdJbWFnZS0+c2V0RG9jTG9hZGVyKGRvY3VtZW50LT5kb2NMb2FkZXIoKSk7CiAg ICAgICAgICAgICBkb2N1bWVudC0+ZG9jTG9hZGVyKCktPm1fZG9jdW1lbnRSZXNvdXJjZXMuc2V0 KG5ld0ltYWdlLT51cmwoKSwgbmV3SW1hZ2UpOworICAgICAgICAgICAgZG9jdW1lbnQtPmRvY0xv YWRlcigpLT5zZXRBdXRvTG9hZEltYWdlcyhhdXRvTG9hZE90aGVySW1hZ2VzKTsKICAgICAgICAg fSBlbHNlCiAgICAgICAgICAgICBuZXdJbWFnZSA9IGRvY3VtZW50LT5kb2NMb2FkZXIoKS0+cmVx dWVzdEltYWdlKHNvdXJjZVVSSShhdHRyKSk7CiAKSW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxv Zwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJKHJldmlzaW9uIDU5MTU3KQor KysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTQgQEAK KzIwMTAtMDUtMTEgIEFsZXhleSBQcm9za3VyeWFrb3YgIDxhcEBhcHBsZS5jb20+CisKKyAgICAg ICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgaHR0cHM6Ly9idWdzLndl YmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTM4NzUyCisgICAgICAgIFN0cmFuZ2UgYmVoYXZpb3Vy IGFmdGVyIGpzIGluc2VydGlvbiBpbnRvIGEgcGFnZSBwb2ludGluZyBkaXJlY3RseSB0byBhbiBv YmplY3QgKGltYWdlLCBzd2YsIGV0YykKKworICAgICAgICAqIGZhc3QvaW1hZ2VzL2V4dHJhLWlt YWdlLWluLWltYWdlLWRvY3VtZW50LWV4cGVjdGVkLnR4dDogQWRkZWQuCisgICAgICAgICogZmFz dC9pbWFnZXMvZXh0cmEtaW1hZ2UtaW4taW1hZ2UtZG9jdW1lbnQuaHRtbDogQWRkZWQuCisgICAg ICAgICogZmFzdC9pbWFnZXMvcmVzb3VyY2VzL2dyZWVuLmpwZzogQWRkZWQuCisKIDIwMTAtMDUt MTEgIEtlbnQgVGFtdXJhICA8dGtlbnRAY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFJldmlld2Vk IGJ5IFNoaW5pY2hpcm8gSGFtYWppLgpJbmRleDogTGF5b3V0VGVzdHMvZmFzdC9pbWFnZXMvZXh0 cmEtaW1hZ2UtaW4taW1hZ2UtZG9jdW1lbnQtZXhwZWN0ZWQudHh0Cj09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExh eW91dFRlc3RzL2Zhc3QvaW1hZ2VzL2V4dHJhLWltYWdlLWluLWltYWdlLWRvY3VtZW50LWV4cGVj dGVkLnR4dAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL2Zhc3QvaW1hZ2VzL2V4dHJhLWlt YWdlLWluLWltYWdlLWRvY3VtZW50LWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMCkKQEAgLTAsMCAr MSw1IEBACitUZXN0IGZvciBidWcgMzg3NTI6IEltYWdlcyBjYW5ub3QgYmUgZHluYW1pY2FsbHkg YWRkZWQgdG8gaW1hZ2UgZG9jdW1lbnRzLgorCitQQVNTCisKKwoKUHJvcGVydHkgY2hhbmdlcyBv bjogTGF5b3V0VGVzdHMvZmFzdC9pbWFnZXMvZXh0cmEtaW1hZ2UtaW4taW1hZ2UtZG9jdW1lbnQt ZXhwZWN0ZWQudHh0Cl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX18KTmFtZTogc3ZuOm1pbWUtdHlwZQogICArIHRleHQvcGxh aW4KTmFtZTogc3ZuOmVvbC1zdHlsZQogICArIG5hdGl2ZQoKSW5kZXg6IExheW91dFRlc3RzL2Zh c3QvaW1hZ2VzL2V4dHJhLWltYWdlLWluLWltYWdlLWRvY3VtZW50Lmh0bWwKPT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQot LS0gTGF5b3V0VGVzdHMvZmFzdC9pbWFnZXMvZXh0cmEtaW1hZ2UtaW4taW1hZ2UtZG9jdW1lbnQu aHRtbAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL2Zhc3QvaW1hZ2VzL2V4dHJhLWltYWdl LWluLWltYWdlLWRvY3VtZW50Lmh0bWwJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEsNTEgQEAKKzxi b2R5IG9ubG9hZD0idGVzdCgpIj4KKzxwPlRlc3QgZm9yIDxhIGhyZWY9Imh0dHBzOi8vYnVncy53 ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zODc1MiI+YnVnIDM4NzUyPC9hPjoKK0ltYWdlcyBj YW5ub3QgYmUgZHluYW1pY2FsbHkgYWRkZWQgdG8gaW1hZ2UgZG9jdW1lbnRzLjwvcD4KKzxwIGlk PXJlc3VsdD5PbmxvYWQgd2FzIG5vdCBjYWxsZWQsIHdoaWNoIGlzIG5vdCBnb29kLiBCdXQgaXMg dGhlIHNxdWFyZSBiZWxvdyBncmVlbj88L3A+Cis8aWZyYW1lIHNyYz0icmVzb3VyY2VzL3Rlc3Qt bG9hZC5qcGciIHdpZHRoPTEwMCBoZWlnaHQ9MTAwIGZyYW1lYm9yZGVyPTA+PC9pZnJhbWU+Cis8 c2NyaXB0PgoraWYgKHdpbmRvdy5sYXlvdXRUZXN0Q29udHJvbGxlcikgeworICAgIGxheW91dFRl c3RDb250cm9sbGVyLndhaXRVbnRpbERvbmUoKTsKKyAgICBsYXlvdXRUZXN0Q29udHJvbGxlci5k dW1wQXNUZXh0KCk7Cit9CisKK2Z1bmN0aW9uIGxvYWRlZCgpCit7CisgICAgaWYgKGZyYW1lc1sw XS5kb2N1bWVudC5nZXRFbGVtZW50QnlJZCgiZ3JlZW4iKS5uYXR1cmFsV2lkdGggPT0gMTYpCisg ICAgICAgIGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJyZXN1bHQiKS5pbm5lckhUTUwgPSAiUEFT UyI7CisgICAgaWYgKHdpbmRvdy5sYXlvdXRUZXN0Q29udHJvbGxlcikKKyAgICAgICAgbGF5b3V0 VGVzdENvbnRyb2xsZXIubm90aWZ5RG9uZSgpOworfQorCitmdW5jdGlvbiB0ZXN0KCkKK3sKKyAg ICB2YXIgZCA9IGZyYW1lc1swXS5kb2N1bWVudDsKKyAgICB2YXIgc2NyaXB0ID0gZC5jcmVhdGVF bGVtZW50KCJTQ1JJUFQiKTsKKyAgICBzY3JpcHQuc3JjID0gImRhdGE6dGV4dC9wbGFpbiwiICsg CisgICAgICAgICJ2YXIgYmcgPSBkb2N1bWVudC5jcmVhdGVFbGVtZW50KCdESVYnKTsiICsKKyAg ICAgICAgImJnLnN0eWxlLndpZHRoID0gJzEwMHB4JzsiICsgCisgICAgICAgICJiZy5zdHlsZS5o ZWlnaHQgPSAnMTAwcHgnOyIgKyAKKyAgICAgICAgImJnLnN0eWxlLnBvc2l0aW9uID0gJ2Fic29s dXRlJzsiICsgCisgICAgICAgICJiZy5zdHlsZS50b3AgPSAwOyIgKyAKKyAgICAgICAgImJnLnN0 eWxlLmxlZnQgPSAwOyIgKyAKKyAgICAgICAgImJnLnN0eWxlLmJhY2tncm91bmQgPSAncmVkIG5v LXJlcGVhdCc7IiArIAorICAgICAgICAiYmcuc3R5bGUub3ZlcmZsb3cgPSAndmlzaWJsZSc7IiAr IAorICAgICAgICAiYmcuc3R5bGUuekluZGV4ID0gJzEwMDAwMDAnOyIgKyAKKyAgICAgICAgImRv Y3VtZW50LmJvZHkuYXBwZW5kQ2hpbGQoYmcpOyIgKworCisgICAgICAgICJ2YXIgaW1nID0gZG9j dW1lbnQuY3JlYXRlRWxlbWVudCgnSU1HJyk7IiArCisgICAgICAgICJpbWcuaWQgPSAnZ3JlZW4n OyIgKworICAgICAgICAiaW1nLnNyYyA9ICdncmVlbi5qcGcnOyIgKworICAgICAgICAiaW1nLnN0 eWxlLndpZHRoID0gJzEwMHB4JzsiICsKKyAgICAgICAgImltZy5zdHlsZS5oZWlnaHQgPSAnMTAw cHgnOyIgKworICAgICAgICAiaW1nLnN0eWxlLnBvc2l0aW9uID0gJ2Fic29sdXRlJzsiICsKKyAg ICAgICAgImltZy5zdHlsZS50b3AgPSAwOyIgKworICAgICAgICAiaW1nLnN0eWxlLmxlZnQgPSAw OyIgKworICAgICAgICAiaW1nLnN0eWxlLnpJbmRleCA9ICcyMDAwMDAwJzsiICsKKyAgICAgICAg ImltZy5vbmxvYWQgPSBmdW5jdGlvbigpIHsgdG9wLmxvYWRlZCgpIH07IiArIAorICAgICAgICAi ZG9jdW1lbnQuYm9keS5hcHBlbmRDaGlsZChpbWcpOyIKKyAgICBkLmJvZHkuYXBwZW5kQ2hpbGQo c2NyaXB0KTsKK30KKworPC9zY3JpcHQ+Cis8L2JvZHk+CgpQcm9wZXJ0eSBjaGFuZ2VzIG9uOiBM YXlvdXRUZXN0cy9mYXN0L2ltYWdlcy9leHRyYS1pbWFnZS1pbi1pbWFnZS1kb2N1bWVudC5odG1s Cl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX18KTmFtZTogc3ZuOm1pbWUtdHlwZQogICArIHRleHQvaHRtbAoKSW5kZXg6IExh eW91dFRlc3RzL2Zhc3QvaW1hZ2VzL3Jlc291cmNlcy9ncmVlbi5qcGcKPT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQpDYW5u b3QgZGlzcGxheTogZmlsZSBtYXJrZWQgYXMgYSBiaW5hcnkgdHlwZS4Kc3ZuOm1pbWUtdHlwZSA9 IGltYWdlL2pwZWcKClByb3BlcnR5IGNoYW5nZXMgb246IExheW91dFRlc3RzL2Zhc3QvaW1hZ2Vz L3Jlc291cmNlcy9ncmVlbi5qcGcKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpOYW1lOiBzdm46bWltZS10eXBlCiAgICsg aW1hZ2UvanBlZwoKCi85ai80QUFRU2taSlJnQUJBUUVBWUFCZ0FBRC80UUNBUlhocFpnQUFUVTBB S2dBQUFBZ0FCUUVTQUFNQUFBQUJBQUVBQUFFYUFBVUEKQUFBQkFBQUFTZ0ViQUFVQUFBQUJBQUFB VWdFb0FBTUFBQUFCQUFJQUFJZHBBQVFBQUFBQkFBQUFXZ0FBQUFBQUFBQmdBQUFBQVFBQQpBR0FB QUFBQkFBS2dBZ0FFQUFBQUFRQUFBQkNnQXdBRUFBQUFBUUFBQUJBQUFBQUEvOXNBUXdBQ0FRRUNB UUVDQWdFQ0FnSUNBZ01GCkF3TURBd01HQkFRREJRY0dCd2NIQmdZR0J3Z0xDUWNJQ2dnR0Jna05D UW9MQ3d3TURBY0pEUTROREE0TERBd0wvOXNBUXdFQ0FnSUQKQWdNRkF3TUZDd2dHQ0FzTEN3c0xD d3NMQ3dzTEN3c0xDd3NMQ3dzTEN3c0xDd3NMQ3dzTEN3c0xDd3NMQ3dzTEN3c0xDd3NMQ3dzTApD d3NMLzhBQUVRZ0FFQUFRQXdFaUFBSVJBUU1SQWYvRUFCOEFBQUVGQVFFQkFRRUJBQUFBQUFBQUFB QUJBZ01FQlFZSENBa0tDLy9FCkFMVVFBQUlCQXdNQ0JBTUZCUVFFQUFBQmZRRUNBd0FFRVFVU0lU RkJCaE5SWVFjaWNSUXlnWkdoQ0NOQ3NjRVZVdEh3SkROaWNvSUoKQ2hZWEdCa2FKU1luS0NrcU5E VTJOemc1T2tORVJVWkhTRWxLVTFSVlZsZFlXVnBqWkdWbVoyaHBhbk4wZFhaM2VIbDZnNFNGaG9l SQppWXFTazVTVmxwZVltWnFpbzZTbHBxZW9xYXF5czdTMXRyZTR1YnJDdzhURnhzZkl5Y3JTMDlU VjF0ZlkyZHJoNHVQazVlYm42T25xCjhmTHo5UFgyOS9qNSt2L0VBQjhCQUFNQkFRRUJBUUVCQVFF QUFBQUFBQUFCQWdNRUJRWUhDQWtLQy8vRUFMVVJBQUlCQWdRRUF3UUgKQlFRRUFBRUNkd0FCQWdN UkJBVWhNUVlTUVZFSFlYRVRJaktCQ0JSQ2thR3h3UWtqTTFMd0ZXSnkwUW9XSkRUaEpmRVhHQmth SmljbwpLU28xTmpjNE9UcERSRVZHUjBoSlNsTlVWVlpYV0ZsYVkyUmxabWRvYVdwemRIVjJkM2g1 ZW9LRGhJV0doNGlKaXBLVGxKV1dsNWlaCm1xS2pwS1dtcDZpcHFyS3p0TFcydDdpNXVzTER4TVhH eDhqSnl0TFQxTlhXMTlqWjJ1TGo1T1htNStqcDZ2THo5UFgyOS9qNSt2L2EKQUF3REFRQUNFUU1S QUQ4QTkwb29vcitLei9EOC85az0KCg==