[WK2] [GTK] LayerTreeHostGtk: don't set m_isValid to false if glContext() returns...
authorberto@igalia.com <berto@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 11 Sep 2013 16:15:33 +0000 (16:15 +0000)
committerberto@igalia.com <berto@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 11 Sep 2013 16:15:33 +0000 (16:15 +0000)
https://bugs.webkit.org/show_bug.cgi?id=120892

Reviewed by Martin Robinson.

If the GL context cannot be created then m_isValid is set to false
during the initialization of LayerTreeHostGtk.

This is not really necessary since the rest of the code already
deals with that situation, so it doesn't have any effect other
than breaking the assertion in invalidate().

* WebProcess/WebPage/gtk/LayerTreeHostGtk.cpp:
(WebKit::LayerTreeHostGtk::initialize):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@155533 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit2/ChangeLog
Source/WebKit2/WebProcess/WebPage/gtk/LayerTreeHostGtk.cpp

index a534cc3..f8605f3 100644 (file)
@@ -1,3 +1,20 @@
+2013-09-11  Alberto Garcia  <berto@igalia.com>
+
+        [WK2] [GTK] LayerTreeHostGtk: don't set m_isValid to false if glContext() returns null
+        https://bugs.webkit.org/show_bug.cgi?id=120892
+
+        Reviewed by Martin Robinson.
+
+        If the GL context cannot be created then m_isValid is set to false
+        during the initialization of LayerTreeHostGtk.
+
+        This is not really necessary since the rest of the code already
+        deals with that situation, so it doesn't have any effect other
+        than breaking the assertion in invalidate().
+
+        * WebProcess/WebPage/gtk/LayerTreeHostGtk.cpp:
+        (WebKit::LayerTreeHostGtk::initialize):
+
 2013-09-11  Commit Queue  <commit-queue@webkit.org>
 
         Unreviewed, rolling out r154899.
index ea0061d..4fb0596 100644 (file)
@@ -116,10 +116,8 @@ void LayerTreeHostGtk::initialize()
     m_layerTreeContext.windowHandle = m_webPage->nativeWindowHandle();
 
     GLContext* context = glContext();
-    if (!context) {
-        m_isValid = false;
+    if (!context)
         return;
-    }
 
     // The creation of the TextureMapper needs an active OpenGL context.
     context->makeContextCurrent();