Title: [149101] trunk/Source/WebCore
Revision
149101
Author
[email protected]
Date
2013-04-25 05:32:49 -0700 (Thu, 25 Apr 2013)

Log Message

Remove isPluginElement hack in Document::setFocusedNode()
<http://webkit.org/b/115171>

>From Blink r148800 by <[email protected]>:

This hack is no longer necessary because r147591 made isFocusable for plugin
elements work correctly.

* dom/Document.cpp:
(WebCore::Document::setFocusedNode):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (149100 => 149101)


--- trunk/Source/WebCore/ChangeLog	2013-04-25 12:13:33 UTC (rev 149100)
+++ trunk/Source/WebCore/ChangeLog	2013-04-25 12:32:49 UTC (rev 149101)
@@ -1,3 +1,16 @@
+2013-04-25  Andreas Kling  <[email protected]>
+
+        Remove isPluginElement hack in Document::setFocusedNode()
+        <http://webkit.org/b/115171>
+
+        From Blink r148800 by <[email protected]>:
+
+        This hack is no longer necessary because r147591 made isFocusable for plugin
+        elements work correctly.
+
+        * dom/Document.cpp:
+        (WebCore::Document::setFocusedNode):
+
 2013-04-25  Per-Erik Brodin  <[email protected]>
 
         EventSource: Synchronous loader callback not handled properly

Modified: trunk/Source/WebCore/dom/Document.cpp (149100 => 149101)


--- trunk/Source/WebCore/dom/Document.cpp	2013-04-25 12:13:33 UTC (rev 149100)
+++ trunk/Source/WebCore/dom/Document.cpp	2013-04-25 12:32:49 UTC (rev 149101)
@@ -3419,7 +3419,7 @@
         }
     }
 
-    if (newFocusedNode && (newFocusedNode->isPluginElement() || newFocusedNode->isFocusable())) {
+    if (newFocusedNode && newFocusedNode->isFocusable()) {
         if (newFocusedNode->isRootEditableElement() && !acceptsEditingFocus(newFocusedNode.get())) {
             // delegate blocks focus change
             focusChangeBlocked = true;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to