Title: [211277] trunk/Source/WebKit2
- Revision
- 211277
- Author
- carlo...@webkit.org
- Date
- 2017-01-27 00:54:23 -0800 (Fri, 27 Jan 2017)
Log Message
[GTK] Stop forcing accelerated compositing by default
https://bugs.webkit.org/show_bug.cgi?id=167492
Reviewed by Žan Doberšek.
We are now ready to bring back the on demand accelerated compositing mode with the threaded
compositor. Unfortunately, having AC always enabled brought a lot of issues for many people. Not only it
requires a lot more memory but also caused rendering issues (or even nothing rendered at all) with some graphics
drivers. People started to use WEBKIT_DISABLE_COMPOSITING_MODE as a workaround, but that was not expected to be
used by users. So, entering/leaving AC when required by web contents is not actually the solution but a huge
improvement for many people. If we are eventually ready to force AC mode again, we'll change this again. Note
that we still have WEBKIT_DISABLE_COMPOSITING_MODE, but now also WEBKIT_FORCE_COMPOSITING_MODE for testing and debugging.
* UIProcess/gtk/WebPreferencesGtk.cpp:
(WebKit::WebPreferences::platformInitializeStore):
Modified Paths
Diff
Modified: trunk/Source/WebKit2/ChangeLog (211276 => 211277)
--- trunk/Source/WebKit2/ChangeLog 2017-01-27 08:26:40 UTC (rev 211276)
+++ trunk/Source/WebKit2/ChangeLog 2017-01-27 08:54:23 UTC (rev 211277)
@@ -1,3 +1,21 @@
+2017-01-27 Carlos Garcia Campos <cgar...@igalia.com>
+
+ [GTK] Stop forcing accelerated compositing by default
+ https://bugs.webkit.org/show_bug.cgi?id=167492
+
+ Reviewed by Žan Doberšek.
+
+ We are now ready to bring back the on demand accelerated compositing mode with the threaded
+ compositor. Unfortunately, having AC always enabled brought a lot of issues for many people. Not only it
+ requires a lot more memory but also caused rendering issues (or even nothing rendered at all) with some graphics
+ drivers. People started to use WEBKIT_DISABLE_COMPOSITING_MODE as a workaround, but that was not expected to be
+ used by users. So, entering/leaving AC when required by web contents is not actually the solution but a huge
+ improvement for many people. If we are eventually ready to force AC mode again, we'll change this again. Note
+ that we still have WEBKIT_DISABLE_COMPOSITING_MODE, but now also WEBKIT_FORCE_COMPOSITING_MODE for testing and debugging.
+
+ * UIProcess/gtk/WebPreferencesGtk.cpp:
+ (WebKit::WebPreferences::platformInitializeStore):
+
2017-01-26 Chris Dumez <cdu...@apple.com>
Crash when navigating back to a page in PacheCache when one of its frames has been removed
Modified: trunk/Source/WebKit2/UIProcess/gtk/WebPreferencesGtk.cpp (211276 => 211277)
--- trunk/Source/WebKit2/UIProcess/gtk/WebPreferencesGtk.cpp 2017-01-27 08:26:40 UTC (rev 211276)
+++ trunk/Source/WebKit2/UIProcess/gtk/WebPreferencesGtk.cpp 2017-01-27 08:54:23 UTC (rev 211277)
@@ -44,16 +44,12 @@
#if !ENABLE(OPENGL)
setAcceleratedCompositingEnabled(false);
#else
-#if USE(COORDINATED_GRAPHICS_THREADED)
- setForceCompositingMode(true);
-#else
- const char* force_compositing = getenv("WEBKIT_FORCE_COMPOSITING_MODE");
- if (force_compositing && strcmp(force_compositing, "0"))
+ const char* forceCompositing = getenv("WEBKIT_FORCE_COMPOSITING_MODE");
+ if (forceCompositing && strcmp(forceCompositing, "0"))
setForceCompositingMode(true);
-#endif
- const char* disable_compositing = getenv("WEBKIT_DISABLE_COMPOSITING_MODE");
- if (disable_compositing && strcmp(disable_compositing, "0")) {
+ const char* disableCompositing = getenv("WEBKIT_DISABLE_COMPOSITING_MODE");
+ if (disableCompositing && strcmp(disableCompositing, "0")) {
setAcceleratedCompositingEnabled(false);
return;
}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes