Closed
Bug 858044
Opened 12 years ago
Closed 11 years ago
Make addon startup code more resilient by using nsISessionStore.restored
Categories
(Add-on SDK Graveyard :: General, defect, P2)
Add-on SDK Graveyard
General
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 1042239
People
(Reporter: ochameau, Unassigned)
References
Details
Attachments
(1 file)
Bug 857602 is going to implement nsISessionStore.restored. A new flag to know if sessionstore-windows-restored event has already been fired or not -or- just know if the session has already been restored or not.
We should start using it in order to safely wait for sessionstore-windows-restored or not. We used to had weak assumptions based on addon startup reason, but it broke when we fixed a wrong reason code. We know have a weak workaround that checks if the hiddenWindow is ready. It can easily fail if the hidden window is initialized faster.
|
Reporter | |
Comment 1•12 years ago
|
||
Assignee: nobody → poirot.alex
Priority: -- → P2
|
||
Updated•11 years ago
|
Attachment #733446 -
Flags: review?(rFobic)
|
||
Updated•11 years ago
|
Attachment #733446 -
Flags: review?(rFobic) → review+
|
|
Reporter | |
Comment 2•11 years ago
|
||
Note that this patch depends on bug 857602, that isn't landed.
I do not have much time to figure out if these two patches still applies and works.
|
|
Reporter | |
Comment 3•11 years ago
|
||
That makes me sad to not drive this bug to the end and see this potential race still exists,
but given that in months I never had time to look at any sdk bug, I'm unassigning to all sdk-related bugs to keep a chance for someone to take them.
Assignee: poirot.alex → nobody
|
||
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•