258707 - fix base vs. sheet URI and line numbers for sub-sheet parsing

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

This is a followup to bug 255538 (and bug 240420).  I need to:
 * figure out what the correct sheet URI to use for the 4 XXX comments cited in
bug 255538 comment 4.
 * make those for which we don't have usable line numbers use line '0' instead
of line '1' and make line 0 be a magic non-incrementing line number (in case
scripts introduce line breaks)

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: patch

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: patch

Comment 3

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 158465 [details] [diff] [review]
patch

>Index: xul/content/src/nsXULElement.cpp
>+        // XXX Get correct Base URI (need GetBaseURI on *prototype* 

Could you please file a bug on this?  I don't think we have one...

r+sr=bzbarsky

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Fix checked in to trunk, 2004-09-10 16:31 -0700.

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Filed bug 258810 for comment 3.