Keep certain about pages in-process

RESOLVED WONTFIX

Status

defect
RESOLVED WONTFIX
6 years ago
5 years ago

People

(Reporter: jimm, Unassigned)

Tracking

Trunk
x86_64
Windows 8.1
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Reporter

Description

6 years ago
about:start is a good candidate (which should fix bug 943379), about:support, about:blank, about:plugins, about:addons, among others.
Blocks: metrobacklog
Whiteboard: [feature] p=0
Matt, I'm curious about the rationale for making pages in-process. Jim mentioned that you would have insight into that.

If we do go ahead and make our "about:" pages load in-process, then we can resolve bug 943379 (and whatever similar bugs would exist for the other "about:" pages) as duplicates of this bug.
Flags: needinfo?(mbrubeck)
Whiteboard: [feature] p=0 → [feature] p=3
(In reply to Tim Abraldes [:TimAbraldes] [:tabraldes] from comment #1)
> Matt, I'm curious about the rationale for making pages in-process. Jim
> mentioned that you would have insight into that.

I believe the main reason in Fennec was that our out-of-process graphics and scrolling code at the time was not able to scroll XUL documents.  If that limitation is (or can be) fixed then I think we can WONTFIX this bug.

The Fennec approach did have the added benefit of making complex chrome pages a bit easier to implement, since they could communicate with the browser chrome directly, but this is not really a good enough reason to keep the special exception.  It also had a downside that it wasn't possible to navigate from an about: page to a remote page (or vice versa) in the same tab.
Flags: needinfo?(mbrubeck)
Bug 552832 is where this was originally implemented.  According to comments there, the main reason was actually that the about: protocol handler did not work in the child process.  That may have been fixed in more recent work -- it looks like desktop Firefox is able to load about: pages in the child process, for example.
Depends on: 552832
From comment 3 and bug 943379 comment 2 it sounds like we don't want to keep our about pages in-process. Re-open if that's inaccurate.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → WONTFIX
No longer blocks: metrobacklog, metrofxe10s, 943379
Whiteboard: [feature] p=3
You need to log in before you can comment on or make changes to this bug.