No subject


Mon Jan 28 08:41:14 PST 2013


8800">r148800</a> by &lt;tkent at chromium.org&gt;:

This hack is no longer necessary because <a href=3D"http://trac.webkit.or=
g/projects/webkit/changeset/147591">r147591</a> made isFocusable for plug=
in
elements work correctly.

* dom/Document.cpp:
(WebCore::Document::setFocusedNode):</pre>

<h3>Modified Paths</h3>
<ul>
<li><a href=3D"#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeL=
og</a></li>
<li><a href=3D"#trunkSourceWebCoredomDocumentcpp">trunk/Source/WebCore/do=
m/Document.cpp</a></li>
</ul>

</div>
<div id=3D"patch">
<h3>Diff</h3>
<a id=3D"trunkSourceWebCoreChangeLog"></a>
<div class=3D"modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (1491=
00 =3D> 149101)</h4>
<pre class=3D"diff"><span>
<span class=3D"info">--- 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)
</span><span class=3D"lines">@@ -1,3 +1,16 @@
</span><ins>+2013-04-25  Andreas Kling  &lt;akling at apple.com&gt;
+
+        Remove isPluginElement hack in Document::setFocusedNode()
+        &lt;http://webkit.org/b/115171&gt;
+
+        From Blink r148800 by &lt;tkent at chromium.org&gt;:
+
+        This hack is no longer necessary because r147591 made isFocusabl=
e for plugin
+        elements work correctly.
+
+        * dom/Document.cpp:
+        (WebCore::Document::setFocusedNode):
+
</ins><span class=3D"cx"> 2013-04-25  Per-Erik Brodin  &lt;per-erik.brodi=
n at ericsson.com&gt;
</span><span class=3D"cx">=20
</span><span class=3D"cx">         EventSource: Synchronous loader callba=
ck not handled properly
</span></span></pre></div>
<a id=3D"trunkSourceWebCoredomDocumentcpp"></a>
<div class=3D"modfile"><h4>Modified: trunk/Source/WebCore/dom/Document.cp=
p (149100 =3D> 149101)</h4>
<pre class=3D"diff"><span>
<span class=3D"info">--- 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 14=
9101)
</span><span class=3D"lines">@@ -3419,7 +3419,7 @@
</span><span class=3D"cx">         }
</span><span class=3D"cx">     }
</span><span class=3D"cx">=20
</span><del>-    if (newFocusedNode &amp;&amp; (newFocusedNode-&gt;isPlug=
inElement() || newFocusedNode-&gt;isFocusable())) {
</del><ins>+    if (newFocusedNode &amp;&amp; newFocusedNode-&gt;isFocusa=
ble()) {
</ins><span class=3D"cx">         if (newFocusedNode-&gt;isRootEditableEl=
ement() &amp;&amp; !acceptsEditingFocus(newFocusedNode.get())) {
</span><span class=3D"cx">             // delegate blocks focus change
</span><span class=3D"cx">             focusChangeBlocked =3D true;
</span></span></pre>
</div>
</div>

</body>
</html>


More information about the webkit-changes mailing list