42626 2010-07-20 03:50:07 -0700 Web Inspector: Incorrect absolute URLs in tooltips of links in the ElementsTreeOutline 2010-07-20 05:02:10 -0700 1 1 1 Unclassified WebKit Web Inspector (Deprecated) 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 1 apavlov apavlov bweinstein joepeck keishi pfeldman pmuellr rik timothy yurys oldest_to_newest 253644 0 apavlov 2010-07-20 03:50:07 -0700 1. Go to any page that links to a script, style, image or hyperlink 2. Open Developer Tools window, select Elements tab 3. Hover over a link (shown by the underline) and note the tooltip 4. Drag the link to a new tab and note the url What is the expected result? For example the <link rel="icon" type="image/vnd.microsoft.icon" href="http://www.gstatic.com/codesite/ph/images/phosting.ico"> in the head of this page, Hovering should show http://www.gstatic.com/codesite/ph/images/phosting.ico Dragging the link should open a new tab for the icon. What happens instead? Hovering shows "http://code.google.com/p/chromium/issues/entry/http:​//www.gstatic.com/​codesite/​ph/​images/​hosting.ico" (the text includes square characters) Dragging the link shows the same incorrect url as above, and as a result the server sends a 404 Error page (or in google code's case redirects you to the home page). This is an upstreaming of http://code.google.com/p/chromium/issues/detail?id=48932 253647 1 62053 apavlov 2010-07-20 04:00:49 -0700 Created attachment 62053 [PATCH] Suggested solution 253652 2 apavlov 2010-07-20 05:02:10 -0700 Committing to http://svn.webkit.org/repository/webkit/trunk ... M WebCore/ChangeLog M WebCore/inspector/front-end/ElementsTreeOutline.js Committed r63739 62053 2010-07-20 04:00:49 -0700 2010-07-20 04:54:51 -0700 [PATCH] Suggested solution rewritehrefs.patch text/plain 1617 apavlov ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg Zjc5NmFiZC4uNGU3MjgzOCAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi Q29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxMyBAQAorMjAxMC0wNy0yMCAgQWxleGFuZGVyIFBh dmxvdiAgPGFwYXZsb3ZAY2hyb21pdW0ub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9E WSAoT09QUyEpLgorCisgICAgICAgIFdlYiBJbnNwZWN0b3I6IEluY29ycmVjdCBhYnNvbHV0ZSBV UkxzIGluIHRvb2x0aXBzIG9mIGxpbmtzIGluIHRoZSBFbGVtZW50c1RyZWVPdXRsaW5lCisgICAg ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD00MjYyNgorCisgICAg ICAgICogaW5zcGVjdG9yL2Zyb250LWVuZC9FbGVtZW50c1RyZWVPdXRsaW5lLmpzOgorICAgICAg ICAoV2ViSW5zcGVjdG9yLkVsZW1lbnRzVHJlZUVsZW1lbnQucHJvdG90eXBlLl9hdHRyaWJ1dGVI VE1MKToKKwogMjAxMC0wNy0yMCAgQWRhbSBCYXJ0aCAgPGFiYXJ0aEB3ZWJraXQub3JnPgogCiAg ICAgICAgIFJldmlld2VkIGJ5IERhcmluIEFkbGVyLgpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9pbnNw ZWN0b3IvZnJvbnQtZW5kL0VsZW1lbnRzVHJlZU91dGxpbmUuanMgYi9XZWJDb3JlL2luc3BlY3Rv ci9mcm9udC1lbmQvRWxlbWVudHNUcmVlT3V0bGluZS5qcwppbmRleCAxYjg0YjgzLi43MjQ3YmEw IDEwMDY0NAotLS0gYS9XZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvRWxlbWVudHNUcmVlT3V0 bGluZS5qcworKysgYi9XZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvRWxlbWVudHNUcmVlT3V0 bGluZS5qcwpAQCAtMTIwNSw4ICsxMjA1LDkgQEAgV2ViSW5zcGVjdG9yLkVsZW1lbnRzVHJlZUVs ZW1lbnQucHJvdG90eXBlID0gewogICAgICAgICAgICAgaHRtbCArPSAiPSYjODIwMztcIiI7CiAK ICAgICAgICAgaWYgKGxpbmtpZnkgJiYgKG5hbWUgPT09ICJzcmMiIHx8IG5hbWUgPT09ICJocmVm IikpIHsKKyAgICAgICAgICAgIHZhciByZXdyaXR0ZW5IcmVmID0gdGhpcy5fcmV3cml0ZUF0dHJI cmVmKG5vZGUsIHZhbHVlKTsKICAgICAgICAgICAgIHZhbHVlID0gdmFsdWUucmVwbGFjZSgvKFtc Lzs6XClcXVx9XSkvZywgIiQxXHUyMDBCIik7Ci0gICAgICAgICAgICBodG1sICs9IGxpbmtpZnko dGhpcy5fcmV3cml0ZUF0dHJIcmVmKG5vZGUsIHZhbHVlKSwgdmFsdWUsICJ3ZWJraXQtaHRtbC1h dHRyaWJ1dGUtdmFsdWUiLCBub2RlLm5vZGVOYW1lLnRvTG93ZXJDYXNlKCkgPT09ICJhIik7Cisg ICAgICAgICAgICBodG1sICs9IGxpbmtpZnkocmV3cml0dGVuSHJlZiwgdmFsdWUsICJ3ZWJraXQt aHRtbC1hdHRyaWJ1dGUtdmFsdWUiLCBub2RlLm5vZGVOYW1lLnRvTG93ZXJDYXNlKCkgPT09ICJh Iik7CiAgICAgICAgIH0gZWxzZSB7CiAgICAgICAgICAgICB2YWx1ZSA9IHZhbHVlLmVzY2FwZUhU TUwoKS5yZXBsYWNlKC8oW1wvOzpcKVxdXH1dKS9nLCAiJDEmIzgyMDM7Iik7CiAgICAgICAgICAg ICBodG1sICs9ICI8c3BhbiBjbGFzcz1cIndlYmtpdC1odG1sLWF0dHJpYnV0ZS12YWx1ZVwiPiIg KyB2YWx1ZSArICI8L3NwYW4+IjsK