Closed
Bug 1438024
Opened 6 years ago
Closed 6 years ago
Remove some shim-only XUL XPIDL interfaces
Categories
(Core :: XUL, enhancement)
Core
XUL
Tracking
()
RESOLVED
FIXED
mozilla60
Tracking | Status | |
---|---|---|
firefox60 | --- | fixed |
People
(Reporter: adrian17, Assigned: adrian17)
References
(Blocks 1 open bug)
Details
Attachments
(4 files, 1 obsolete file)
No description provided.
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → adrian.wielgosik
Status: NEW → ASSIGNED
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 4•6 years ago
|
||
There is one use of a removed interface (two, including a comment) in comm-central: https://searchfox.org/comm-central/search?q=nsIMenuBoxObject&case=false®exp=false&path= https://searchfox.org/comm-central/search?q=nsiscrollboxobject&case=false®exp=false&path= What's the general approach toward these?
Flags: needinfo?(bzbarsky)
Comment 5•6 years ago
|
||
General approach is to cc Jorg <jorgk@jorgk.com> on either this bug or a new bug you file on comm-central. In this case the fix is pretty simple: remove that whole .QueryInterface call.
Flags: needinfo?(bzbarsky) → needinfo?(jorgk)
Comment 6•6 years ago
|
||
Thanks, if you don't mind, I'll attach the patch here.
Flags: needinfo?(jorgk)
Comment 7•6 years ago
|
||
Philipp, how would you like to handle the comment? I've just remove the "since" clause.
Attachment #8950783 -
Flags: review?(philipp)
Attachment #8950783 -
Flags: review?(bzbarsky)
Comment 8•6 years ago
|
||
Comment on attachment 8950783 [details] [diff] [review] 1438024-comm-central.patch (v1) You could just say "doesn't implement ScrollBoxObject". r=me
Attachment #8950783 -
Flags: review?(bzbarsky) → review+
Comment 9•6 years ago
|
||
mozreview-review |
Comment on attachment 8950757 [details] Bug 1438024 - Remove nsIContainerBoxObject. https://reviewboard.mozilla.org/r/220000/#review225836
Attachment #8950757 -
Flags: review?(bzbarsky) → review+
Comment 10•6 years ago
|
||
mozreview-review |
Comment on attachment 8950758 [details] Bug 1438024 - Remove nsIScrollBoxObject. https://reviewboard.mozilla.org/r/220002/#review225838
Attachment #8950758 -
Flags: review?(bzbarsky) → review+
Comment 11•6 years ago
|
||
mozreview-review |
Comment on attachment 8950759 [details] Bug 1438024 - Remove nsIMenuBoxObject. https://reviewboard.mozilla.org/r/220004/#review225840
Attachment #8950759 -
Flags: review?(bzbarsky) → review+
Comment 12•6 years ago
|
||
OK, tweaked the comment to go with Boris suggestion :-) I forgot to say: Thanks for the head-up. Much nicer to fix it before it breaks.
Attachment #8950783 -
Attachment is obsolete: true
Attachment #8950783 -
Flags: review?(philipp)
Attachment #8950786 -
Flags: review+
Comment 13•6 years ago
|
||
Pushed by bzbarsky@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a72b573940c8 Remove nsIContainerBoxObject. r=bz https://hg.mozilla.org/integration/autoland/rev/cfa6d11fe286 Remove nsIScrollBoxObject. r=bz https://hg.mozilla.org/integration/autoland/rev/0457cb937b8e Remove nsIMenuBoxObject. r=bz
Comment 14•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a72b573940c8 https://hg.mozilla.org/mozilla-central/rev/cfa6d11fe286 https://hg.mozilla.org/mozilla-central/rev/0457cb937b8e
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox60:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
Comment 15•6 years ago
|
||
Pushed by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/752150efa9fb remove use of nsIMenuBoxObject and nsIScrollBoxObject (comm-central part). r=bz
You need to log in
before you can comment on or make changes to this bug.
Description
•