Title: [121770] trunk/Source/WebKit2
Revision
121770
Author
[email protected]
Date
2012-07-03 08:40:02 -0700 (Tue, 03 Jul 2012)

Log Message

[EFL][WK2] Added missing WebPageProxy::ViewIsFocused flag to WebPageProxy::viewStateDidChange()
call in _ewk_view_smart_focus_out() function.
https://bugs.webkit.org/show_bug.cgi?id=89877

Patch by Piotr Roguski <[email protected]> on 2012-07-03
Reviewed by Andreas Kling.

Although name of the flag suggest it should be used only on focus in,
omitting it in case of focus out will prevent WebPageProxy::viewStateDidChange()
from sending Messages::WebPage::SetFocused(false).

* UIProcess/API/efl/ewk_view.cpp:
(_ewk_view_smart_focus_out):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (121769 => 121770)


--- trunk/Source/WebKit2/ChangeLog	2012-07-03 15:34:54 UTC (rev 121769)
+++ trunk/Source/WebKit2/ChangeLog	2012-07-03 15:40:02 UTC (rev 121770)
@@ -1,3 +1,18 @@
+2012-07-03  Piotr Roguski  <[email protected]>
+
+        [EFL][WK2] Added missing WebPageProxy::ViewIsFocused flag to WebPageProxy::viewStateDidChange()
+        call in _ewk_view_smart_focus_out() function.
+        https://bugs.webkit.org/show_bug.cgi?id=89877
+
+        Reviewed by Andreas Kling.
+
+        Although name of the flag suggest it should be used only on focus in,
+        omitting it in case of focus out will prevent WebPageProxy::viewStateDidChange()
+        from sending Messages::WebPage::SetFocused(false).
+
+        * UIProcess/API/efl/ewk_view.cpp:
+        (_ewk_view_smart_focus_out):
+
 2012-07-03  Viatcheslav Ostapenko  <[email protected]>
 
         [Qt][WK2] API tests randomly asserts in QQuickWebPage::setContentsScale(qreal)

Modified: trunk/Source/WebKit2/UIProcess/API/efl/ewk_view.cpp (121769 => 121770)


--- trunk/Source/WebKit2/UIProcess/API/efl/ewk_view.cpp	2012-07-03 15:34:54 UTC (rev 121769)
+++ trunk/Source/WebKit2/UIProcess/API/efl/ewk_view.cpp	2012-07-03 15:40:02 UTC (rev 121770)
@@ -123,7 +123,7 @@
 {
     EWK_VIEW_PRIV_GET_OR_RETURN(smartData, priv, false)
 
-    priv->pageClient->page()->viewStateDidChange(WebPageProxy::ViewWindowIsActive);
+    priv->pageClient->page()->viewStateDidChange(WebPageProxy::ViewIsFocused | WebPageProxy::ViewWindowIsActive);
     return true;
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to