408919 - Broken extensions initialization after landing 392251

Status: NEW

(Toolkit :: Startup and Profile System, defect)

Description

[Oleg Romashin (:romaxa)]

Attachment: 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..

Comment 1

[Oleg Romashin (:romaxa)]

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

Ups, wrong version...

Comment 2

[Oleg Romashin (:romaxa)]

Attachment: This should be better

Comment 3

[Benjamin Smedberg]

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.