Closed
Bug 18046
Opened 25 years ago
Closed 25 years ago
[DOGFOOD] nsISelectionController needs IDL and implementation finished
Categories
(Core :: DOM: Selection, defect, P3)
Core
DOM: Selection
Tracking
()
VERIFIED
FIXED
M12
People
(Reporter: mjudge, Assigned: mjudge)
References
Details
(Whiteboard: [PDT+] [by 12/15])
Attachments
(2 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
text/plain
|
Details |
nsPresShell implements this interface. interface now just a header and not IDL.
when implementation is actually progressing we will add IDL to generate the
nsISelectionController.h file.
Updated•25 years ago
|
Target Milestone: M12
Comment 1•25 years ago
|
||
I'm putting this on the M12 list -- adjust it if you think it needs to be
somewhere else Mike.
Updated•25 years ago
|
Whiteboard: [CODE cleanup]
Updated•25 years ago
|
Target Milestone: M12 → M14
Comment 2•25 years ago
|
||
setting this out to M14 -- cleanup time
Updated•25 years ago
|
Whiteboard: [CODE cleanup]
Comment 3•25 years ago
|
||
Beth, I'm removing the [CODE cleanup] designation you put on the status
whiteboard; I have a PDT+ bug which depends on this one, so it's more than just
a cleanup issue.
Updated•25 years ago
|
Target Milestone: M14 → M12
Comment 4•25 years ago
|
||
Moving back to M12.
Updated•25 years ago
|
Whiteboard: [18033 PDT+ dependent]
Comment 5•25 years ago
|
||
no problem -- so I won't do it again -- I'm putting in a PDT entry in whiteboard
-- sorry about that, next time I'll look at the dependency bugs.
Updated•25 years ago
|
Summary: nsISelectionController needs IDL and implementation finished → [DOGFOOD] nsISelectionController needs IDL and implementation finished
Comment 7•25 years ago
|
||
there is a PDT+ bug that is dependent on this getting fixed, putting on the
DOGFOOD radar
Updated•25 years ago
|
Whiteboard: [PDT+] 18033 pdt+ dependent → [PDT+] [dependent on 18033 getting resolved, which depends on 13378]
Updated•25 years ago
|
Whiteboard: [PDT+] [dependent on 18033 getting resolved, which depends on 13378] → [PDT+] [need input from saari]
Comment 8•25 years ago
|
||
I have an idl implementation of this file, attaching patches
Comment 9•25 years ago
|
||
Comment 10•25 years ago
|
||
Updated•25 years ago
|
Whiteboard: [PDT+] [need input from saari] → [PDT+] [dependent on bug 19981]
Comment 11•25 years ago
|
||
talked with Trudelle to see if saari could set 19981 as a top priority, Trudelle
will talk with saari and see what it will entail to resolve 19981, and they will
assign a fix by date. Once the fix by date is set on 19981, then mjudge can set
a fix by date on this bug.
Comment 12•25 years ago
|
||
Mike, can you give an estimate of how many days it will take to write the API if
bug 19981 was resolved?
Updated•25 years ago
|
Whiteboard: [PDT+] [dependent on bug 19981] → [PDT+] [19981 fix by date + 3 days]
Updated•25 years ago
|
Whiteboard: [PDT+] [19981 fix by date + 3 days] → [PDT+] [20225 fix by date + 3 days]
Comment 13•25 years ago
|
||
Mike split 19981 into 2 separate bugs, this is now dependent on 20225 being
resolved, updating summary
Comment 14•25 years ago
|
||
I have sford's IDL file working in my tree (I also fixed the spelling to
nsISelectionController since this seemed like the perfect time to do that) and
am awaiting code review by mjudge for that. Once that's in, I can proceed on
18033.
The implementations (and therefore the testing of 18033) are still waiting on
20255.
Updated•25 years ago
|
Whiteboard: [PDT+] [20225 fix by date + 3 days] → [PDT+] [by 12/10]
Updated•25 years ago
|
Whiteboard: [PDT+] [by 12/10] → [PDT+] [by 12/15]
Comment 15•25 years ago
|
||
updating fix by date
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 16•25 years ago
|
||
resolved enough for m12. will add more interfaces as we need them for m13. iam
resolving this one
Comment 17•25 years ago
|
||
This can't be verified by QA, of course.
mjudge, please feel free to rubber-stamp this one as verified yourself, or
designate a peer to do a code-level verification. Thanks!
Updated•25 years ago
|
Status: RESOLVED → VERIFIED
Comment 18•25 years ago
|
||
I'll verify it -- I was the one who was waiting for it, and it's working (well,
most of it) now.
You need to log in
before you can comment on or make changes to this bug.
Description
•