After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 353097 - URI wrongly resolved
URI wrongly resolved
Status: RESOLVED FIXED
Product: totem
Classification: Core
Component: Browser plugin (obsolete)
unspecified
Other Linux
: Normal normal
: ---
Assigned To: totem-browser-maint
totem-browser-maint
needs-testcase
Depends on: 350297 364022
Blocks:
 
 
Reported: 2006-08-27 13:36 UTC by Christian Persch
Modified: 2006-11-15 17:32 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Christian Persch 2006-08-27 13:36:12 UTC 
In totemPlugin.cpp we resolve relative URIs against the document URI, but I think we ought to resolve them against the <embed> tag's base URI instead. (See nsGenericHTMLElement::GetURIAttr)
Comment 1 Bastien Nocera 2006-08-28 11:19:35 UTC 
Do you have a website that shows the problem?
Comment 2 Elijah Newren 2006-10-06 22:41:04 UTC 
Closing this bug report as no further information has been provided. Please feel free to reopen this bug if you can provide the information asked for.
Thanks!
Comment 3 Christian Persch 2006-10-06 22:50:32 UTC 
I don't have a test site, but it should be simple to make a testcase, just use a <base> tag.
This is definitely a bug, reopening. I'll fix this soon.
Comment 4 Bastien Nocera 2006-10-21 20:16:15 UTC 
I will tuck on another instance of us not using the right base to resolve:
//FIXME
// http://developer.apple.com/documentation/QuickTime/Conceptual/QTScripting_HTML/QTScripting_HTML_Document/chapter_1000_section_3.html
// "Important: If you pass a relative URL in the HREF parameter, it must be relative to the currentlyloadedmovie, not relative to the current web page. If your movies are in a separate folder, specify URLs relative to the movies folder."

Christian, should we use gnome-vfs to resolve those text URIs?
Comment 5 Christian Persch 2006-10-21 21:05:40 UTC 
No, we should use necko for that. My patch (wip) for bug 350297 does that (using document base currently, but it'll be easy to fix to use the correct base).
Comment 6 Christian Persch 2006-11-15 17:32:12 UTC 
Fixed with the patch from bug 350297.