No subject
Mon Jan 28 08:41:14 PST 2013
8800">r148800</a> by <tkent at chromium.org>:
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 <akling at apple.com>
+
+ Remove isPluginElement hack in Document::setFocusedNode()
+ <http://webkit.org/b/115171>
+
+ From Blink r148800 by <tkent at chromium.org>:
+
+ 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 <per-erik.brodi=
n at ericsson.com>
</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 && (newFocusedNode->isPlug=
inElement() || newFocusedNode->isFocusable())) {
</del><ins>+ if (newFocusedNode && newFocusedNode->isFocusa=
ble()) {
</ins><span class=3D"cx"> if (newFocusedNode->isRootEditableEl=
ement() && !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