Profile mgr needs to put necko offline when switching profile

VERIFIED FIXED in mozilla0.9.6

Status

Core Graveyard
Profile: BackEnd
VERIFIED FIXED
17 years ago
2 years ago

People

(Reporter: Conrad Carlen (not reading bugmail), Assigned: Conrad Carlen (not reading bugmail))

Tracking

Trunk
mozilla0.9.6
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

17 years ago
This came from bug 101329 - summary says it all.

We need to put necko offline and then wait until receiving notification that it
really is offline before proceding with shutting down the profile. For that, see
bug 104020.
(Assignee)

Updated

17 years ago
Status: NEW → ASSIGNED
Depends on: 104020
Target Milestone: --- → mozilla0.9.6

Updated

17 years ago
Blocks: 101329

Comment 2

16 years ago
Comment on attachment 56684 [details] [diff] [review]
patch 1

how about just renumbering the Phases?  :-)
Attachment #56684 - Flags: review+

Updated

16 years ago
Attachment #56684 - Flags: superreview+

Comment 3

16 years ago
Comment on attachment 56684 [details] [diff] [review]
patch 1

rs=darin
(Assignee)

Comment 4

16 years ago
Kai - drivers have requested another review for this to go into 0.9.6. Can you
do it? Between this and bug 104020, all net activity should now be stopped when
the NSSComponent gets the profile-before-change notification.

Comment 5

16 years ago
You are only adding a new notification, that is currently not used anywhere.

But the code is required for the fix that I will produce for bug 104020.

You notification is currently not used and therefore can't cause any trouble for
096.

r=kaie
(Assignee)

Comment 6

16 years ago
Kai, I'm not sure what you mean. This notification will be used by the code
which was checked in for bug 104020. That patch is in place and the bug is closed.

Comment 7

16 years ago
I should have updated my tree before using grep to search it for the new
notification string you added...

Ok, your patch will already trigger necko to go off or back online. That's what
we want and your patch looks good.

r=kaie

Comment 8

16 years ago
a=asa (on behalf of drivers) for checkin to 0.9.6
Keywords: mozilla0.9.6+

Comment 9

16 years ago
I'm marking this fixed, as the code seems to have landed on the trunk.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
(Assignee)

Comment 10

16 years ago
Removing blockage of 104864 because this is in 0.9.6. It was checked in before
the branch was cut.
No longer blocks: 104864
Re-adding the 0.9.6 tracking bug since we need to keep track of all of the 0.9.6
bugs that were labeled as such. ( This is closed - it's OK. )
Blocks: 104864

Comment 12

16 years ago
Verified code fix
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.