nsFrameLoader uses incorrect base URI

RESOLVED FIXED in mozilla1.8beta2

Status

()

Core
Layout: HTML Frames
RESOLVED FIXED
13 years ago
13 years ago

People

(Reporter: Biesinger, Assigned: Biesinger)

Tracking

Trunk
mozilla1.8beta2
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug], URL)

Attachments

(2 attachments)

112   nsIURI *base_uri = doc->GetBaseURI();

Instead, that URI should be gotten off mOwnerContent directly, as each node can
have a different base.
(ie., mOwnerContent->GetBaseURI())
Whiteboard: [good first bug]
Created attachment 182209 [details] [diff] [review]
patch
Assignee: nobody → cbiesinger
Status: NEW → ASSIGNED
Attachment #182209 - Flags: superreview?(bzbarsky)
Attachment #182209 - Flags: review?(bzbarsky)
Attachment #182209 - Flags: superreview?(bzbarsky)
Attachment #182209 - Flags: superreview+
Attachment #182209 - Flags: review?(bzbarsky)
Attachment #182209 - Flags: review+
Comment on attachment 182209 [details] [diff] [review]
patch

low risk patch to use the correct base URI for frames
Attachment #182209 - Flags: approval1.8b2?
Target Milestone: --- → mozilla1.8beta2

Comment 5

13 years ago
Comment on attachment 182209 [details] [diff] [review]
patch

a=asa
Attachment #182209 - Flags: approval1.8b2? → approval1.8b2+
Checking in content/base/public/nsIContent.h;
/cvsroot/mozilla/content/base/public/nsIContent.h,v  <--  nsIContent.h
new revision: 3.108; previous revision: 3.107
done
Checking in content/base/src/nsFrameLoader.cpp;
/cvsroot/mozilla/content/base/src/nsFrameLoader.cpp,v  <--  nsFrameLoader.cpp
new revision: 1.53; previous revision: 1.52
done
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.