After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 723302 - Add multi-webprocess support
Add multi-webprocess support
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: Backend
git master
Other Mac OS
: Normal normal
: ---
Assigned To: Epiphany Maintainers
Epiphany Maintainers
Depends on: 723087
Blocks:
 
 
Reported: 2014-01-30 16:56 UTC by Carlos Garcia Campos
Modified: 2014-02-05 17:38 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Add support for using multiple web processes (60.30 KB, patch)
2014-01-30 16:59 UTC, Carlos Garcia Campos 		committed Details | Review

Description Carlos Garcia Campos 2014-01-30 16:56:13 UTC 
There's WebKit API now to use one web process per web view. It requires to rework the web extensions handling, because ephy currently assumes a single web extension.
Comment 1 Carlos Garcia Campos 2014-01-30 16:59:19 UTC 
Created attachment 267662 [details] [review]
Add support for using multiple web processes

This depends on patch attached to bug #723087 and requires new API not yet released (only in trunk for now). It uses a gsetting to set the process model. The setting should be changed before launching ephy, because it's not possible to change the process model once the first web process is spawned. The single process model will be default for now, since multiple web process hasn't been tested enough.
Comment 2 Claudio Saavedra 2014-02-04 12:35:37 UTC 
Review of attachment 267662 [details] [review]:

I had a quick look at the patch and it seems fine, but I won't claim to have reviewed it :)

::: data/org.gnome.epiphany.gschema.xml
@@ +69,3 @@
+                <key name="process-model" enum="org.gnome.Epiphany.EphyPrefsProcessModel">
+                        <default>'shared-secondary-process'</default>
+                        <summary>Process model</summary>

We need to document this better.