Unreviewed, rolling out r111993.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 26 Mar 2012 10:51:41 +0000 (10:51 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 26 Mar 2012 10:51:41 +0000 (10:51 +0000)
http://trac.webkit.org/changeset/111993
https://bugs.webkit.org/show_bug.cgi?id=82184

It broke layout and API tests and made WTR crash (Requested by
Ossy on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-03-26

Source/WebKit2:

* UIProcess/qt/QtPageClient.cpp:
(QtPageClient::isViewWindowActive):

Tools:

* WebKitTestRunner/qt/PlatformWebViewQt.cpp:
(WTR::PlatformWebView::PlatformWebView):

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

Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/qt/QtPageClient.cpp
Tools/ChangeLog
Tools/WebKitTestRunner/qt/PlatformWebViewQt.cpp

index 7f841a9..9685fcc 100644 (file)
@@ -1,3 +1,15 @@
+2012-03-26  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r111993.
+        http://trac.webkit.org/changeset/111993
+        https://bugs.webkit.org/show_bug.cgi?id=82184
+
+        It broke layout and API tests and made WTR crash (Requested by
+        Ossy on #webkit).
+
+        * UIProcess/qt/QtPageClient.cpp:
+        (QtPageClient::isViewWindowActive):
+
 2012-03-25  Hayato Ito  <hayato@chromium.org>
 
         [Shadow DOM] Add Reified DOM Tree traversal internal APIs.
index af0797b..d7049cb 100644 (file)
@@ -29,7 +29,6 @@
 #include "qquickwebview_p.h"
 #include "qquickwebview_p_p.h"
 #include <QGuiApplication>
-#include <QQuickCanvas>
 #include <WebCore/Cursor.h>
 #include <WebCore/DragData.h>
 #include <WebCore/FloatRect.h>
@@ -256,9 +255,8 @@ WebCore::IntSize QtPageClient::viewSize()
 
 bool QtPageClient::isViewWindowActive()
 {
-    if (!m_webView || !m_webView->canvas())
-        return false;
-    return m_webView->canvas()->isActive();
+    // FIXME: The scene graph does not have the concept of being active or not when this was written.
+    return true;
 }
 
 bool QtPageClient::isViewFocused()
index ff1d5ad..0248853 100644 (file)
@@ -1,3 +1,15 @@
+2012-03-26  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r111993.
+        http://trac.webkit.org/changeset/111993
+        https://bugs.webkit.org/show_bug.cgi?id=82184
+
+        It broke layout and API tests and made WTR crash (Requested by
+        Ossy on #webkit).
+
+        * WebKitTestRunner/qt/PlatformWebViewQt.cpp:
+        (WTR::PlatformWebView::PlatformWebView):
+
 2012-03-25  Kevin Ollivier  <kevino@theolliviers.com>
 
         [wx] Unreviewed build fix. Move WTF to its own static lib build.
index f0fa640..7f51239 100644 (file)
@@ -76,12 +76,6 @@ PlatformWebView::PlatformWebView(WKContextRef contextRef, WKPageGroupRef pageGro
 {
     QQuickWebViewExperimental experimental(m_view);
     experimental.setRenderToOffscreenBuffer(true);
-
-    // QWindow does not create the underlying platform
-    // specific bits unless QWindow::setVisible() or QWindow::show() are called,
-    // and without it QWindow::isActive() will always return false.
-    // For these cases QWindow::create() is provided.
-    m_window->create();
 }
 
 PlatformWebView::~PlatformWebView()