Well, the issue with loading views first is that when you open a new window, that would create new views. In the extension we've implemented, the background page iterates through the views whenever it needs to update them. Alternatively, we may want to implement onViewCreated events so it's easier to keep track of them (but that might be more brittle). -Nick
On Thu, Jun 11, 2009 at 9:16 AM, Nebojša Ćirić <[email protected]> wrote: > I am somewhat confused wrt. loading sequence. If background page main task > is to load data once for all views, and then to update them, shouldn't then > views be loaded first (toolstrips etc.)? > *background_page*: > ... > 1. gather data once > ... > 2. update all views // if loads first, no other views are going to be in > the list > ... > > *all other views*: > ... > o. have a function that would update view once called from background page > ... > > Or do you expect views to poll background page? > > Cira > > 2009/6/10 Aaron Boodman <[email protected]> > >> I agree the background page should load first. >> >> What's happening right now is that ExtensionProcessManager monitors >> EXTENSIONS_LOADED, just like everyone else (including Browser) so they >> get loaded about the same time. It looks like currently the background >> page starts first, because it was registered first, but it's basically >> a race. >> >> - a >> >> 2009/6/10 Nick Baum <[email protected]>: >> > This might be the same problem I ran into when writing my extension: >> because >> > the bg page and toolstrips don't load in a predictable order, you're not >> > guaranteed to catch all views if you do getViews() right as the page >> loads. >> > If the background page was guaranteed to load first, the toolstrips >> could >> > all make sure to find it when they loaded. >> > -Nick >> > >> > 2009/6/10 Aaron Boodman <[email protected]> >> >> >> >> On Wed, Jun 10, 2009 at 3:17 PM, Nebojša Ćirić<[email protected]> >> wrote: >> >> > What about the second issue? Shouldn't iterating over background page >> >> > views >> >> > return toolstrip views, instead of background page itself (and no >> >> > toolstrip >> >> > views)? I'll try it again just to make sure I didn't make an obvious >> >> > mistake... >> >> >> >> This method should return all running HTML pages in your extension -- >> >> this includes the background page, any toolstrips, and any tabs that >> >> are showing HTML from the extension. If it's not doing that, it's a >> >> bug. >> >> >> >> - a >> >> >> >> >> >> >> > >> > >> > > --~--~---------~--~----~------------~-------~--~----~ Chromium Developers mailing list: [email protected] View archives, change email options, or unsubscribe: http://groups.google.com/group/chromium-dev -~----------~----~----~----~------~----~------~--~---
