192555 - nsProfileDirServiceProvider should send profile shutdown notifications

Status: VERIFIED FIXED

(Core Graveyard :: Profile: BackEnd, defect)

Description

[Conrad Carlen (not reading bugmail)]

See bug 188214. Notification that the profile is shutting down
(profile-before-change) should be sent out by nsProfileDirServiceProvider
instead of requiring the embedding app to do it.

Comment 1

[Conrad Carlen (not reading bugmail)]

Attachment: patch to add Shutdown()

Adds a Shutdown() method so that, if the requirements of shutting down this
directory service provider change, embedding apps don't break. Compiles on
Linux but not tested - my environment is wacked right now.

Comment 2

[Doug Turner (:dougt)]

Comment on attachment 114071 [details] [diff] [review]
patch to add Shutdown()

assuming that is the right notification topic, the patch looks fine. r=dougt

Comment 3

[Christopher Blizzard (:blizzard)]

Comment on attachment 114071 [details] [diff] [review]
patch to add Shutdown()

sr=blizzard

Looks good.

Comment 4

[Christopher Blizzard (:blizzard)]

Need to make sure this gets in before 1.4a.

Comment 5

[Christopher Blizzard (:blizzard)]

*** Bug 188214 has been marked as a duplicate of this bug. ***

Comment 6

[Conrad Carlen (not reading bugmail)]

Checked in - though with comment citing the wrong bug :-/

Comment 7

[Asa Dotzler [:asa]]

Comment on attachment 114071 [details] [diff] [review]
patch to add Shutdown()

a=asa (on behalf of drivers) for checkin to 1.3 branch.

Comment 8

[Christopher Blizzard (:blizzard)]

Checked into the 1.3 branch.

Comment 9

[Grace Bush]

verified all code fixes