Title: [113714] trunk
- Revision
- 113714
- Author
- [email protected]
- Date
- 2012-04-10 08:14:55 -0700 (Tue, 10 Apr 2012)
Log Message
[Qt][WK2] Implement PageClient::isViewWindowActive()
https://bugs.webkit.org/show_bug.cgi?id=81143
Reviewed by Kenneth Rohde Christiansen.
Source/WebKit2:
This patch implements PageClient::isViewWindowActive()
now that QQuickCanvas::isActive() is available (from QWindow).
* UIProcess/qt/QtPageClient.cpp:
(QtPageClient::isViewWindowActive):
Tools:
Fix the WrapperWindow from our PlatformWebView
so it correctly creates the platform related
stuff for QWindow, which is not created unless
QWindow::setVisible() or QWindow::show() are called.
* WebKitTestRunner/qt/PlatformWebViewQt.cpp:
(WTR::PlatformWebView::PlatformWebView):
Modified Paths
Diff
Modified: trunk/Source/WebKit2/ChangeLog (113713 => 113714)
--- trunk/Source/WebKit2/ChangeLog 2012-04-10 14:42:19 UTC (rev 113713)
+++ trunk/Source/WebKit2/ChangeLog 2012-04-10 15:14:55 UTC (rev 113714)
@@ -1,3 +1,16 @@
+2012-04-10 Jesus Sanchez-Palencia <[email protected]>
+
+ [Qt][WK2] Implement PageClient::isViewWindowActive()
+ https://bugs.webkit.org/show_bug.cgi?id=81143
+
+ Reviewed by Kenneth Rohde Christiansen.
+
+ This patch implements PageClient::isViewWindowActive()
+ now that QQuickCanvas::isActive() is available (from QWindow).
+
+ * UIProcess/qt/QtPageClient.cpp:
+ (QtPageClient::isViewWindowActive):
+
2012-04-10 Philippe Normand <[email protected]> and Carlos Garcia Campos <[email protected]>
[WK2][GTK] FullScreen signals
Modified: trunk/Source/WebKit2/UIProcess/qt/QtPageClient.cpp (113713 => 113714)
--- trunk/Source/WebKit2/UIProcess/qt/QtPageClient.cpp 2012-04-10 14:42:19 UTC (rev 113713)
+++ trunk/Source/WebKit2/UIProcess/qt/QtPageClient.cpp 2012-04-10 15:14:55 UTC (rev 113714)
@@ -256,8 +256,9 @@
bool QtPageClient::isViewWindowActive()
{
- // FIXME: The scene graph does not have the concept of being active or not when this was written.
- return true;
+ if (!m_webView || !m_webView->canvas())
+ return false;
+ return m_webView->canvas()->isActive();
}
bool QtPageClient::isViewFocused()
Modified: trunk/Tools/ChangeLog (113713 => 113714)
--- trunk/Tools/ChangeLog 2012-04-10 14:42:19 UTC (rev 113713)
+++ trunk/Tools/ChangeLog 2012-04-10 15:14:55 UTC (rev 113714)
@@ -1,3 +1,18 @@
+2012-03-23 Jesus Sanchez-Palencia <[email protected]>
+
+ [Qt][WK2] Implement PageClient::isViewWindowActive()
+ https://bugs.webkit.org/show_bug.cgi?id=81143
+
+ Reviewed by Kenneth Rohde Christiansen.
+
+ Fix the WrapperWindow from our PlatformWebView
+ so it correctly creates the platform related
+ stuff for QWindow, which is not created unless
+ QWindow::setVisible() or QWindow::show() are called.
+
+ * WebKitTestRunner/qt/PlatformWebViewQt.cpp:
+ (WTR::PlatformWebView::PlatformWebView):
+
2012-04-10 Balazs Kelemen <[email protected]>
[Qt][WK2] TestNetscapePlugin is broken
Modified: trunk/Tools/WebKitTestRunner/qt/PlatformWebViewQt.cpp (113713 => 113714)
--- trunk/Tools/WebKitTestRunner/qt/PlatformWebViewQt.cpp 2012-04-10 14:42:19 UTC (rev 113713)
+++ trunk/Tools/WebKitTestRunner/qt/PlatformWebViewQt.cpp 2012-04-10 15:14:55 UTC (rev 113714)
@@ -76,6 +76,11 @@
{
QQuickWebViewExperimental experimental(m_view);
experimental.setRenderToOffscreenBuffer(true);
+
+ // QWindow delays creating of the underlying platform specific bits until ::setVisible()
+ // or ::show() are called. This makes ::isActive() always return false. In order to force
+ // the creation ::create() exists, so we use it here.
+ m_window->create();
}
PlatformWebView::~PlatformWebView()
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
