nsPermissionManager doesn't notify observers in content process

RESOLVED FIXED

Status

()

RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: jdm, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
The block guarded by IsChildProcess() returns before initializing the observer service, which is undesirable.
(Reporter)

Updated

8 years ago
Blocks: 621363
(Reporter)

Comment 1

8 years ago
Created attachment 499702 [details] [diff] [review]
Acquire observer service in content process.
Attachment #499702 - Flags: review?(dwitte)

Comment 2

8 years ago
Comment on attachment 499702 [details] [diff] [review]
Acquire observer service in content process.

Oh dear. r=dwitte
Attachment #499702 - Flags: review?(dwitte) → review+
(Reporter)

Comment 3

8 years ago
Comment on attachment 499702 [details] [diff] [review]
Acquire observer service in content process.

This is a low-risk fix for something that should be corrected.
Attachment #499702 - Flags: approval2.0?
Attachment #499702 - Flags: approval2.0? → approval2.0+
(Reporter)

Updated

8 years ago
Whiteboard: [needs patch with commit message]
(Reporter)

Comment 4

8 years ago
http://hg.mozilla.org/mozilla-central/rev/a4aadd1f8ce4
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Whiteboard: [needs patch with commit message]
I think this may have been intended here:

http://hg.mozilla.org/mozilla-central/rev/3249d9d62b58
You need to log in before you can comment on or make changes to this bug.