If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

bad cast in appcore shutdown -> change appcores into components

VERIFIED FIXED in M16

Status

Core Graveyard
Tracking
P3
normal
VERIFIED FIXED
19 years ago
a year ago

People

(Reporter: Dan M, Assigned: Scott Collins)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
nsAppCoresManager:::Shutdown static casts each of the appcores in its list of active cores to an
nsIDOMBaseAppCore, then calls Release() on the resulting pointer.  Of all our appcores, only one
of them (nsBaseAppCore) actually inherits from nsIDOMBaseAppCore.  Oddly enough, Release()
seems to be called correctly on Windows, at least for some appcores.  Other platforms are not so ...
forgiving.  They seem to just hang up after getting a wrong number.  How civilized.  Still, seems like
there's a problem here.

I'm making this a bug, rather than just fixing it, because Scott's work to change appcores into
components should neatly sidestep this problem.  So I'm ignoring it for now, but making a note...

Updated

19 years ago
QA Contact: leger → gerardok
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
Target Milestone: M7
(Assignee)

Comment 1

19 years ago
I'll take care of this as part of converting AppCores to components

Comment 2

19 years ago
move to later milestone?

Updated

19 years ago
Summary: bad cast in appcore shutdown → bad cast in appcore shutdown -> change appcores into components
Target Milestone: M7 → M8

Comment 3

19 years ago
this has moved to m8

Comment 4

19 years ago
Moving all Apprunner bugs past and present to Other component temporarily whilst
don and I set correct component.  Apprunner component will be deleted/retired
shortly.
(Assignee)

Updated

19 years ago
Target Milestone: M8 → M9
(Assignee)

Comment 5

19 years ago
Moving to M9, when the rest of this change can be checked in.
(Assignee)

Updated

18 years ago
Target Milestone: M9 → M10

Updated

18 years ago
Target Milestone: M10 → M11

Comment 6

18 years ago
Moving to M11 per today's bug triage.
(Assignee)

Updated

18 years ago
Depends on: 13021
Target Milestone: M11 → M16
(Assignee)

Comment 7

18 years ago
following bug #13021, as usual
(Assignee)

Updated

18 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
(Assignee)

Comment 8

18 years ago
According to Bill Law in bug #13021, AppCores are no longer used (though they are
still part of the build till he gets some makefile stuff checked in).  Therefore,
this bad code in |nsAppCoresManager::Shutdown| is now dead code and so no longer
a problem.

Updated

18 years ago
Status: RESOLVED → VERIFIED

Comment 9

18 years ago
According to developer's comments this bug is marked as verified. If you think
the resolution is wrong please reopen.
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.