<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[163617] trunk/Source/WebCore</title>
</head>
<body>
<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/163617">163617</a></dd>
<dt>Author</dt> <dd>commit-queue@webkit.org</dd>
<dt>Date</dt> <dd>2014-02-07 02:21:15 -0800 (Fri, 07 Feb 2014)</dd>
</dl>
<h3>Log Message</h3>
<pre>Renaming isTableElement() to isRenderedTable() as per the FIXME comment
https://bugs.webkit.org/show_bug.cgi?id=127769
Patch by Laszlo Vidacs <lvidacs.u-szeged@partner.samsung.com> on 2014-02-07
Reviewed by Andreas Kling.
Rename method and use IsTable() instead of local type checking.
* dom/Position.cpp:
(WebCore::Position::parentAnchoredEquivalent):
(WebCore::Position::upstream):
(WebCore::Position::downstream):
(WebCore::Position::isCandidate):
* dom/PositionIterator.cpp:
(WebCore::PositionIterator::isCandidate):
* editing/CompositeEditCommand.cpp:
(WebCore::CompositeEditCommand::cloneParagraphUnderNewElement):
(WebCore::CompositeEditCommand::moveParagraphWithClones):
* editing/FrameSelection.cpp:
(WebCore::caretRendersInsideNode):
* editing/VisibleUnits.cpp:
(WebCore::startOfParagraph):
(WebCore::endOfParagraph):
* editing/htmlediting.cpp:
(WebCore::firstInSpecialElement):
(WebCore::lastInSpecialElement):
(WebCore::isRenderedTable):
* editing/htmlediting.h:
* rendering/RenderBox.cpp:
(WebCore::RenderBox::localCaretRect):</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>
<li><a href="#trunkSourceWebCoredomPositioncpp">trunk/Source/WebCore/dom/Position.cpp</a></li>
<li><a href="#trunkSourceWebCoredomPositionIteratorcpp">trunk/Source/WebCore/dom/PositionIterator.cpp</a></li>
<li><a href="#trunkSourceWebCoreeditingCompositeEditCommandcpp">trunk/Source/WebCore/editing/CompositeEditCommand.cpp</a></li>
<li><a href="#trunkSourceWebCoreeditingFrameSelectioncpp">trunk/Source/WebCore/editing/FrameSelection.cpp</a></li>
<li><a href="#trunkSourceWebCoreeditingVisibleUnitscpp">trunk/Source/WebCore/editing/VisibleUnits.cpp</a></li>
<li><a href="#trunkSourceWebCoreeditinghtmleditingcpp">trunk/Source/WebCore/editing/htmlediting.cpp</a></li>
<li><a href="#trunkSourceWebCoreeditinghtmleditingh">trunk/Source/WebCore/editing/htmlediting.h</a></li>
<li><a href="#trunkSourceWebCorerenderingRenderBoxcpp">trunk/Source/WebCore/rendering/RenderBox.cpp</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkSourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (163616 => 163617)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ChangeLog 2014-02-07 10:10:55 UTC (rev 163616)
+++ trunk/Source/WebCore/ChangeLog 2014-02-07 10:21:15 UTC (rev 163617)
</span><span class="lines">@@ -1,3 +1,35 @@
</span><ins>+2014-02-07 Laszlo Vidacs <lvidacs.u-szeged@partner.samsung.com>
+
+ Renaming isTableElement() to isRenderedTable() as per the FIXME comment
+ https://bugs.webkit.org/show_bug.cgi?id=127769
+
+ Reviewed by Andreas Kling.
+
+ Rename method and use IsTable() instead of local type checking.
+
+ * dom/Position.cpp:
+ (WebCore::Position::parentAnchoredEquivalent):
+ (WebCore::Position::upstream):
+ (WebCore::Position::downstream):
+ (WebCore::Position::isCandidate):
+ * dom/PositionIterator.cpp:
+ (WebCore::PositionIterator::isCandidate):
+ * editing/CompositeEditCommand.cpp:
+ (WebCore::CompositeEditCommand::cloneParagraphUnderNewElement):
+ (WebCore::CompositeEditCommand::moveParagraphWithClones):
+ * editing/FrameSelection.cpp:
+ (WebCore::caretRendersInsideNode):
+ * editing/VisibleUnits.cpp:
+ (WebCore::startOfParagraph):
+ (WebCore::endOfParagraph):
+ * editing/htmlediting.cpp:
+ (WebCore::firstInSpecialElement):
+ (WebCore::lastInSpecialElement):
+ (WebCore::isRenderedTable):
+ * editing/htmlediting.h:
+ * rendering/RenderBox.cpp:
+ (WebCore::RenderBox::localCaretRect):
+
</ins><span class="cx"> 2014-02-06 Brady Eidson <beidson@apple.com>
</span><span class="cx">
</span><span class="cx"> IDB: Remove the entirely unnecessary "Value Key" concept
</span></span></pre></div>
<a id="trunkSourceWebCoredomPositioncpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/dom/Position.cpp (163616 => 163617)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/dom/Position.cpp 2014-02-07 10:10:55 UTC (rev 163616)
+++ trunk/Source/WebCore/dom/Position.cpp 2014-02-07 10:21:15 UTC (rev 163617)
</span><span class="lines">@@ -236,13 +236,13 @@
</span><span class="cx">
</span><span class="cx"> // FIXME: This should only be necessary for legacy positions, but is also needed for positions before and after Tables
</span><span class="cx"> if (m_offset <= 0 && (m_anchorType != PositionIsAfterAnchor && m_anchorType != PositionIsAfterChildren)) {
</span><del>- if (findParent(m_anchorNode.get()) && (editingIgnoresContent(m_anchorNode.get()) || isTableElement(m_anchorNode.get())))
</del><ins>+ if (findParent(m_anchorNode.get()) && (editingIgnoresContent(m_anchorNode.get()) || isRenderedTable(m_anchorNode.get())))
</ins><span class="cx"> return positionInParentBeforeNode(m_anchorNode.get());
</span><span class="cx"> return Position(m_anchorNode.get(), 0, PositionIsOffsetInAnchor);
</span><span class="cx"> }
</span><span class="cx"> if (!m_anchorNode->offsetInCharacters()
</span><span class="cx"> && (m_anchorType == PositionIsAfterAnchor || m_anchorType == PositionIsAfterChildren || static_cast<unsigned>(m_offset) == m_anchorNode->childNodeCount())
</span><del>- && (editingIgnoresContent(m_anchorNode.get()) || isTableElement(m_anchorNode.get()))
</del><ins>+ && (editingIgnoresContent(m_anchorNode.get()) || isRenderedTable(m_anchorNode.get()))
</ins><span class="cx"> && containerNode()) {
</span><span class="cx"> return positionInParentAfterNode(m_anchorNode.get());
</span><span class="cx"> }
</span><span class="lines">@@ -634,7 +634,7 @@
</span><span class="cx"> return lastVisible;
</span><span class="cx">
</span><span class="cx"> // Return position after tables and nodes which have content that can be ignored.
</span><del>- if (editingIgnoresContent(currentNode) || isTableElement(currentNode)) {
</del><ins>+ if (editingIgnoresContent(currentNode) || isRenderedTable(currentNode)) {
</ins><span class="cx"> if (currentPos.atEndOfNode())
</span><span class="cx"> return positionAfterNode(currentNode);
</span><span class="cx"> continue;
</span><span class="lines">@@ -766,7 +766,7 @@
</span><span class="cx"> lastVisible = currentPos;
</span><span class="cx">
</span><span class="cx"> // Return position before tables and nodes which have content that can be ignored.
</span><del>- if (editingIgnoresContent(currentNode) || isTableElement(currentNode)) {
</del><ins>+ if (editingIgnoresContent(currentNode) || isRenderedTable(currentNode)) {
</ins><span class="cx"> if (currentPos.offsetInLeafNode() <= renderer->caretMinOffset())
</span><span class="cx"> return createLegacyEditingPosition(currentNode, renderer->caretMinOffset());
</span><span class="cx"> continue;
</span><span class="lines">@@ -928,7 +928,7 @@
</span><span class="cx"> if (renderer->isText())
</span><span class="cx"> return !nodeIsUserSelectNone(deprecatedNode()) && toRenderText(renderer)->containsCaretOffset(m_offset);
</span><span class="cx">
</span><del>- if (isTableElement(deprecatedNode()) || editingIgnoresContent(deprecatedNode()))
</del><ins>+ if (isRenderedTable(deprecatedNode()) || editingIgnoresContent(deprecatedNode()))
</ins><span class="cx"> return (atFirstEditingPositionForNode() || atLastEditingPositionForNode()) && !nodeIsUserSelectNone(deprecatedNode()->parentNode());
</span><span class="cx">
</span><span class="cx"> if (m_anchorNode->hasTagName(htmlTag))
</span></span></pre></div>
<a id="trunkSourceWebCoredomPositionIteratorcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/dom/PositionIterator.cpp (163616 => 163617)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/dom/PositionIterator.cpp 2014-02-07 10:10:55 UTC (rev 163616)
+++ trunk/Source/WebCore/dom/PositionIterator.cpp 2014-02-07 10:21:15 UTC (rev 163617)
</span><span class="lines">@@ -157,7 +157,7 @@
</span><span class="cx"> if (renderer->isText())
</span><span class="cx"> return !Position::nodeIsUserSelectNone(m_anchorNode) && toRenderText(renderer)->containsCaretOffset(m_offsetInAnchor);
</span><span class="cx">
</span><del>- if (isTableElement(m_anchorNode) || editingIgnoresContent(m_anchorNode))
</del><ins>+ if (isRenderedTable(m_anchorNode) || editingIgnoresContent(m_anchorNode))
</ins><span class="cx"> return (atStartOfNode() || atEndOfNode()) && !Position::nodeIsUserSelectNone(m_anchorNode->parentNode());
</span><span class="cx">
</span><span class="cx"> if (!m_anchorNode->hasTagName(htmlTag) && renderer->isRenderBlockFlow()) {
</span></span></pre></div>
<a id="trunkSourceWebCoreeditingCompositeEditCommandcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/editing/CompositeEditCommand.cpp (163616 => 163617)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/editing/CompositeEditCommand.cpp 2014-02-07 10:10:55 UTC (rev 163616)
+++ trunk/Source/WebCore/editing/CompositeEditCommand.cpp 2014-02-07 10:21:15 UTC (rev 163617)
</span><span class="lines">@@ -1057,7 +1057,7 @@
</span><span class="cx"> if (outerNode->isRootEditableElement()) {
</span><span class="cx"> lastNode = blockElement;
</span><span class="cx"> } else {
</span><del>- lastNode = outerNode->cloneNode(isTableElement(outerNode.get()));
</del><ins>+ lastNode = outerNode->cloneNode(isRenderedTable(outerNode.get()));
</ins><span class="cx"> appendNode(lastNode, blockElement);
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -1072,7 +1072,7 @@
</span><span class="cx">
</span><span class="cx"> for (size_t i = ancestors.size(); i != 0; --i) {
</span><span class="cx"> Node* item = ancestors[i - 1].get();
</span><del>- RefPtr<Node> child = item->cloneNode(isTableElement(item));
</del><ins>+ RefPtr<Node> child = item->cloneNode(isRenderedTable(item));
</ins><span class="cx"> appendNode(child, toElement(lastNode.get()));
</span><span class="cx"> lastNode = child.release();
</span><span class="cx"> }
</span><span class="lines">@@ -1190,7 +1190,7 @@
</span><span class="cx"> beforeParagraph = VisiblePosition(beforeParagraph.deepEquivalent());
</span><span class="cx"> afterParagraph = VisiblePosition(afterParagraph.deepEquivalent());
</span><span class="cx">
</span><del>- if (beforeParagraph.isNotNull() && !isTableElement(beforeParagraph.deepEquivalent().deprecatedNode())
</del><ins>+ if (beforeParagraph.isNotNull() && !isRenderedTable(beforeParagraph.deepEquivalent().deprecatedNode())
</ins><span class="cx"> && ((!isEndOfParagraph(beforeParagraph) && !isStartOfParagraph(beforeParagraph)) || beforeParagraph == afterParagraph)) {
</span><span class="cx"> // FIXME: Trim text between beforeParagraph and afterParagraph if they aren't equal.
</span><span class="cx"> insertNodeAt(createBreakElement(document()), beforeParagraph.deepEquivalent());
</span></span></pre></div>
<a id="trunkSourceWebCoreeditingFrameSelectioncpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/editing/FrameSelection.cpp (163616 => 163617)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/editing/FrameSelection.cpp 2014-02-07 10:10:55 UTC (rev 163616)
+++ trunk/Source/WebCore/editing/FrameSelection.cpp 2014-02-07 10:21:15 UTC (rev 163617)
</span><span class="lines">@@ -1288,7 +1288,7 @@
</span><span class="cx">
</span><span class="cx"> static inline bool caretRendersInsideNode(Node* node)
</span><span class="cx"> {
</span><del>- return node && !isTableElement(node) && !editingIgnoresContent(node);
</del><ins>+ return node && !isRenderedTable(node) && !editingIgnoresContent(node);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> static RenderObject* caretRenderer(Node* node)
</span></span></pre></div>
<a id="trunkSourceWebCoreeditingVisibleUnitscpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/editing/VisibleUnits.cpp (163616 => 163617)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/editing/VisibleUnits.cpp 2014-02-07 10:10:55 UTC (rev 163616)
+++ trunk/Source/WebCore/editing/VisibleUnits.cpp 2014-02-07 10:21:15 UTC (rev 163617)
</span><span class="lines">@@ -1159,7 +1159,7 @@
</span><span class="cx"> node = n;
</span><span class="cx"> offset = 0;
</span><span class="cx"> n = NodeTraversal::previousPostOrder(n, startBlock);
</span><del>- } else if (editingIgnoresContent(n) || isTableElement(n)) {
</del><ins>+ } else if (editingIgnoresContent(n) || isRenderedTable(n)) {
</ins><span class="cx"> node = n;
</span><span class="cx"> type = Position::PositionIsBeforeAnchor;
</span><span class="cx"> n = n->previousSibling() ? n->previousSibling() : NodeTraversal::previousPostOrder(n, startBlock);
</span><span class="lines">@@ -1239,7 +1239,7 @@
</span><span class="cx"> node = n;
</span><span class="cx"> offset = r->caretMaxOffset();
</span><span class="cx"> n = NodeTraversal::next(n, stayInsideBlock);
</span><del>- } else if (editingIgnoresContent(n) || isTableElement(n)) {
</del><ins>+ } else if (editingIgnoresContent(n) || isRenderedTable(n)) {
</ins><span class="cx"> node = n;
</span><span class="cx"> type = Position::PositionIsAfterAnchor;
</span><span class="cx"> n = NodeTraversal::nextSkippingChildren(n, stayInsideBlock);
</span></span></pre></div>
<a id="trunkSourceWebCoreeditinghtmleditingcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/editing/htmlediting.cpp (163616 => 163617)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/editing/htmlediting.cpp 2014-02-07 10:10:55 UTC (rev 163616)
+++ trunk/Source/WebCore/editing/htmlediting.cpp 2014-02-07 10:21:15 UTC (rev 163617)
</span><span class="lines">@@ -423,7 +423,7 @@
</span><span class="cx"> if (isSpecialElement(n)) {
</span><span class="cx"> VisiblePosition vPos = VisiblePosition(pos, DOWNSTREAM);
</span><span class="cx"> VisiblePosition firstInElement = VisiblePosition(firstPositionInOrBeforeNode(n), DOWNSTREAM);
</span><del>- if (isTableElement(n) && vPos == firstInElement.next())
</del><ins>+ if (isRenderedTable(n) && vPos == firstInElement.next())
</ins><span class="cx"> return n;
</span><span class="cx"> if (vPos == firstInElement)
</span><span class="cx"> return n;
</span><span class="lines">@@ -438,7 +438,7 @@
</span><span class="cx"> if (isSpecialElement(n)) {
</span><span class="cx"> VisiblePosition vPos = VisiblePosition(pos, DOWNSTREAM);
</span><span class="cx"> VisiblePosition lastInElement = VisiblePosition(lastPositionInOrAfterNode(n), DOWNSTREAM);
</span><del>- if (isTableElement(n) && vPos == lastInElement.previous())
</del><ins>+ if (isRenderedTable(n) && vPos == lastInElement.previous())
</ins><span class="cx"> return n;
</span><span class="cx"> if (vPos == lastInElement)
</span><span class="cx"> return n;
</span><span class="lines">@@ -807,14 +807,14 @@
</span><span class="cx"> return 0;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-// FIXME: do not require renderer, so that this can be used within fragments, or rename to isRenderedTable()
-bool isTableElement(Node* n)
</del><ins>+// FIXME: do not require renderer, so that this can be used within fragments
+bool isRenderedTable(const Node* n)
</ins><span class="cx"> {
</span><span class="cx"> if (!n || !n->isElementNode())
</span><span class="cx"> return false;
</span><span class="cx">
</span><span class="cx"> RenderObject* renderer = n->renderer();
</span><del>- return (renderer && (renderer->style().display() == TABLE || renderer->style().display() == INLINE_TABLE));
</del><ins>+ return (renderer && renderer->isTable());
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> bool isTableCell(const Node* node)
</span></span></pre></div>
<a id="trunkSourceWebCoreeditinghtmleditingh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/editing/htmlediting.h (163616 => 163617)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/editing/htmlediting.h 2014-02-07 10:10:55 UTC (rev 163616)
+++ trunk/Source/WebCore/editing/htmlediting.h 2014-02-07 10:21:15 UTC (rev 163617)
</span><span class="lines">@@ -108,7 +108,7 @@
</span><span class="cx"> bool isTabSpanNode(const Node*);
</span><span class="cx"> bool isTabSpanTextNode(const Node*);
</span><span class="cx"> bool isMailBlockquote(const Node*);
</span><del>-bool isTableElement(Node*);
</del><ins>+bool isRenderedTable(const Node*);
</ins><span class="cx"> bool isTableCell(const Node*);
</span><span class="cx"> bool isEmptyTableCell(const Node*);
</span><span class="cx"> bool isTableStructureNode(const Node*);
</span></span></pre></div>
<a id="trunkSourceWebCorerenderingRenderBoxcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/RenderBox.cpp (163616 => 163617)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/rendering/RenderBox.cpp 2014-02-07 10:10:55 UTC (rev 163616)
+++ trunk/Source/WebCore/rendering/RenderBox.cpp 2014-02-07 10:21:15 UTC (rev 163617)
</span><span class="lines">@@ -4116,7 +4116,7 @@
</span><span class="cx"> // FIXME: Border/padding should be added for all elements but this workaround
</span><span class="cx"> // is needed because we use offsets inside an "atomic" element to represent
</span><span class="cx"> // positions before and after the element in deprecated editing offsets.
</span><del>- if (element() && !(editingIgnoresContent(element()) || isTableElement(element()))) {
</del><ins>+ if (element() && !(editingIgnoresContent(element()) || isRenderedTable(element()))) {
</ins><span class="cx"> rect.setX(rect.x() + borderLeft() + paddingLeft());
</span><span class="cx"> rect.setY(rect.y() + paddingTop() + borderTop());
</span><span class="cx"> }
</span></span></pre>
</div>
</div>
</body>
</html>