Re: Intent to move Activity Stream into its own process

2018-06-21 Thread Dão Gottwald
This is about the code behind about:newtab and about:home, yes? Or some broader Activity Stream magic? Activity Stream has become kind of a fuzzy term, to the point where I'm not sure anymore what it means. 2018-06-18 21:58 GMT+02:00 Kris Maglione : > +1 > > This should also have some memory bene

Re: Intent to move Activity Stream into its own process

2018-06-19 Thread Kris Maglione
On Tue, Jun 19, 2018 at 06:26:11PM +0200, Dão Gottwald wrote: This is about the code behind about:newtab and about:home, yes? Or some broader Activity Stream magic? Activity Stream has become kind of a fuzzy term, to the point where I'm not sure anymore what it means. This is about about:home a

Re: Intent to move Activity Stream into its own process

2018-06-19 Thread Mike Conley
Hi Alex, Yes, I expect this to plug into the same process handling infrastructure that we already have for now. I also expect that we'll find some room for optimization while we're stomping around in this area. -Mike On Tue, 19 Jun 2018 at 09:13, Alex Gaynor wrote: > Do you have a sense of how

Re: Intent to move Activity Stream into its own process

2018-06-19 Thread Alex Gaynor
Do you have a sense of how this is going to be implemented? Is there going to be specialized code for this, or is it going to be handled by all the general navigation changes for process-switching when you change sites? Alex On Mon, Jun 18, 2018 at 5:02 PM Mike Conley wrote: > > > > I am not su

Re: Intent to move Activity Stream into its own process

2018-06-18 Thread Mike Conley
> > I am not sure about the specific AS work here, but for the general case, I > believe that Fission intends to cause a process switch anytime a tab > navigates to an unrelated origin, so this will also increase the frequency > of process switching. > Yep, this matches my understanding: we're goi

Re: Intent to move Activity Stream into its own process

2018-06-18 Thread J. Ryan Stinnett
On Mon, Jun 18, 2018 at 3:20 PM Gijs Kruitbosch wrote: > If it *would* mean a process switch, we may want to take another look at > some of the bugs relating to those (process switches are currently > relatively rare) and re-prioritize them. > I am not sure about the specific AS work here, but f

Re: Intent to move Activity Stream into its own process

2018-06-18 Thread Kris Maglione
On Mon, Jun 18, 2018 at 09:19:18PM +0100, Gijs Kruitbosch wrote: Hey, This sounds really interesting. However, wouldn't this mean that we will do a process switch for the tab's browser whenever we load a URL in the same tab that has AS in it? Or would you still intend to run the actual AS new

Re: Intent to move Activity Stream into its own process

2018-06-18 Thread Gijs Kruitbosch
Hey, This sounds really interesting. However, wouldn't this mean that we will do a process switch for the tab's browser whenever we load a URL in the same tab that has AS in it? Or would you still intend to run the actual AS new tab page in a "normal" tab process? If it *would* mean a proces

Re: Intent to move Activity Stream into its own process

2018-06-18 Thread Kris Maglione
+1 This should also have some memory benefits by preventing us from unnecessarily loading AS things into multiple content processes, like we do now. On Mon, Jun 18, 2018 at 03:35:01PM -0400, Mike Conley wrote: (posted on both dev-platform and firefox-dev. Please send replies to dev-platform)