Broken extensions initialization after landing 392251

NEW
Unassigned

Status

()

11 years ago
10 years ago

People

(Reporter: romaxa, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

11 years ago
Created attachment 293772 [details] [diff] [review]
Possible fix.

Problem mostly related to embedding api and initialization with XRE_InitEmbedding.

Before landing 392251 it was possible to initialize extensions with the next way:

Install extension - extension appears in some "extension" dir
extmgr->Start - extensions.ini updated
nsIComponentRegistrar->AutoRegister - Updated compreg.dat and components form extension folder preloaded...

After landing 392251 it is not possible to call LoadBundleDirectories (parse extensions.ini) twice in one session, without restarting.

I guess we need to check also extensions.ini modification time and reload it it was changed

In attachments proposed fix..
Attachment #293772 - Flags: review?(benjamin)
(Reporter)

Comment 1

11 years ago
Comment on attachment 293772 [details] [diff] [review]
Possible fix.

Ups, wrong version...
Attachment #293772 - Attachment is obsolete: true
Attachment #293772 - Flags: review?(benjamin)
(Reporter)

Comment 2

11 years ago
Created attachment 293778 [details] [diff] [review]
This should be better
Attachment #293778 - Flags: review?(benjamin)

Comment 3

11 years ago
Comment on attachment 293778 [details] [diff] [review]
This should be better

This is not the correct solution. If you wish, you can make LoadBundleDirecotories forward the request to mAppProvider before responding, but I don't want to institutionalize the extensions.ini file or make embedders write one.
Attachment #293778 - Flags: review?(benjamin) → review-

Updated

10 years ago
Component: XRE Startup → Startup and Profile System
QA Contact: xre.startup → startup
You need to log in before you can comment on or make changes to this bug.