Title: [140541] trunk/Source
Revision
140541
Author
[email protected]
Date
2013-01-23 10:04:07 -0800 (Wed, 23 Jan 2013)

Log Message

shadowAncestorNode() should be renamed to deprecatedShadowAncestorNode()
https://bugs.webkit.org/show_bug.cgi?id=107624

Reviewed by Dimitri Glazkov.

Since Node::shadowAncestorNode() is deprecated, we would like to rename it to prevent from further use.

Source/WebCore:

No new tests, simple refactoring.

* WebCore.order:
* accessibility/AccessibilityRenderObject.cpp:
(WebCore::AccessibilityRenderObject::accessibilityHitTest):
* dom/Document.cpp:
(WebCore::Document::handleZeroPadding):
* dom/Node.cpp:
(WebCore::Node::deprecatedShadowAncestorNode):
* dom/Node.h:
(Node):
* editing/Editor.cpp:
(WebCore::Editor::rangeOfString):
(WebCore::Editor::countMatchesForText):
* editing/ReplaceSelectionCommand.cpp:
(WebCore::ReplacementFragment::ReplacementFragment):
* page/DragController.cpp:
(WebCore::elementUnderMouse):
* page/EventHandler.cpp:
(WebCore::EventHandler::handleMouseReleaseEvent):
(WebCore::EventHandler::bestClickableNodeForTouchPoint):
* page/FocusController.cpp:
(WebCore::clearSelectionIfNeeded):
* platform/blackberry/RenderThemeBlackBerry.cpp:
(WebCore::RenderThemeBlackBerry::paintSearchFieldCancelButton):
* rendering/HitTestResult.cpp:
(WebCore::HitTestResult::setToNonShadowAncestor):
(WebCore::HitTestResult::addNodeToRectBasedTestResult):

Source/WebKit/blackberry:

* Api/WebPage.cpp:
(BlackBerry::WebKit::WebPagePrivate::webContext):
* WebKitSupport/FatFingers.h:
(BlackBerry::WebKit::FatFingersResult::node):
* WebKitSupport/InPageSearchManager.cpp:
(BlackBerry::WebKit::InPageSearchManager::scopeStringMatches):

Source/WebKit/chromium:

* src/WebFrameImpl.cpp:
(WebKit::WebFrameImpl::scopeStringMatches):
(WebKit::WebFrameImpl::setFindEndstateFocusAndSelection):

Source/WebKit/qt:

* WebCoreSupport/QWebPageAdapter.cpp:
(QWebPageAdapter::inputMethodEvent):
(QWebPageAdapter::inputMethodQuery):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (140540 => 140541)


--- trunk/Source/WebCore/ChangeLog	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebCore/ChangeLog	2013-01-23 18:04:07 UTC (rev 140541)
@@ -1,3 +1,41 @@
+2013-01-23  Shinya Kawanaka  <[email protected]>
+
+        shadowAncestorNode() should be renamed to deprecatedShadowAncestorNode()
+        https://bugs.webkit.org/show_bug.cgi?id=107624
+
+        Reviewed by Dimitri Glazkov.
+
+        Since Node::shadowAncestorNode() is deprecated, we would like to rename it to prevent from further use.
+
+        No new tests, simple refactoring.
+
+        * WebCore.order:
+        * accessibility/AccessibilityRenderObject.cpp:
+        (WebCore::AccessibilityRenderObject::accessibilityHitTest):
+        * dom/Document.cpp:
+        (WebCore::Document::handleZeroPadding):
+        * dom/Node.cpp:
+        (WebCore::Node::deprecatedShadowAncestorNode):
+        * dom/Node.h:
+        (Node):
+        * editing/Editor.cpp:
+        (WebCore::Editor::rangeOfString):
+        (WebCore::Editor::countMatchesForText):
+        * editing/ReplaceSelectionCommand.cpp:
+        (WebCore::ReplacementFragment::ReplacementFragment):
+        * page/DragController.cpp:
+        (WebCore::elementUnderMouse):
+        * page/EventHandler.cpp:
+        (WebCore::EventHandler::handleMouseReleaseEvent):
+        (WebCore::EventHandler::bestClickableNodeForTouchPoint):
+        * page/FocusController.cpp:
+        (WebCore::clearSelectionIfNeeded):
+        * platform/blackberry/RenderThemeBlackBerry.cpp:
+        (WebCore::RenderThemeBlackBerry::paintSearchFieldCancelButton):
+        * rendering/HitTestResult.cpp:
+        (WebCore::HitTestResult::setToNonShadowAncestor):
+        (WebCore::HitTestResult::addNodeToRectBasedTestResult):
+
 2013-01-22  Pavel Feldman  <[email protected]>
 
         Web Inspector: only allow evaluateForTestInFrontend for front-ends under test.

Modified: trunk/Source/WebCore/WebCore.order (140540 => 140541)


--- trunk/Source/WebCore/WebCore.order	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebCore/WebCore.order	2013-01-23 18:04:07 UTC (rev 140541)
@@ -841,7 +841,7 @@
 __ZN7WebCore11RenderLayer34clearClipRectsIncludingDescendantsEv
 __ZN7WebCore11RenderLayer12styleChangedENS_15StyleDifferenceEPKNS_11RenderStyleE
 __ZN7WebCore11RenderLayer23updateScrollCornerStyleEv
-__ZN7WebCore4Node18shadowAncestorNodeEv
+__ZN7WebCore4Node28deprecatedShadowAncestorNodeEv
 __ZNK7WebCore4Node15isSVGShadowRootEv
 __ZN7WebCore11RenderLayer18updateResizerStyleEv
 __ZN7WebCore11RenderLayer15updateTransformEv

Modified: trunk/Source/WebCore/accessibility/AccessibilityRenderObject.cpp (140540 => 140541)


--- trunk/Source/WebCore/accessibility/AccessibilityRenderObject.cpp	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebCore/accessibility/AccessibilityRenderObject.cpp	2013-01-23 18:04:07 UTC (rev 140541)
@@ -2194,7 +2194,7 @@
     layer->hitTest(request, hitTestResult);
     if (!hitTestResult.innerNode())
         return 0;
-    Node* node = hitTestResult.innerNode()->shadowAncestorNode();
+    Node* node = hitTestResult.innerNode()->deprecatedShadowAncestorNode();
 
     if (node->hasTagName(areaTag)) 
         return accessibilityImageMapHitTest(static_cast<HTMLAreaElement*>(node), point);

Modified: trunk/Source/WebCore/dom/Document.cpp (140540 => 140541)


--- trunk/Source/WebCore/dom/Document.cpp	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebCore/dom/Document.cpp	2013-01-23 18:04:07 UTC (rev 140541)
@@ -1395,7 +1395,7 @@
     if (!node)
         return 0;
 
-    node = node->shadowAncestorNode();
+    node = node->deprecatedShadowAncestorNode();
     ListHashSet<RefPtr<Node> > list;
     list.add(node);
     return StaticHashSetNodeList::adopt(list);

Modified: trunk/Source/WebCore/dom/Node.cpp (140540 => 140541)


--- trunk/Source/WebCore/dom/Node.cpp	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebCore/dom/Node.cpp	2013-01-23 18:04:07 UTC (rev 140541)
@@ -1229,7 +1229,7 @@
     return 0;
 }
 
-Node* Node::shadowAncestorNode() const
+Node* Node::deprecatedShadowAncestorNode() const
 {
     if (ShadowRoot* root = containingShadowRoot())
         return root->host();

Modified: trunk/Source/WebCore/dom/Node.h (140540 => 140541)


--- trunk/Source/WebCore/dom/Node.h	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebCore/dom/Node.h	2013-01-23 18:04:07 UTC (rev 140541)
@@ -267,7 +267,7 @@
     Element* shadowHost() const;
     // If this node is in a shadow tree, returns its shadow host. Otherwise, returns this.
     // Deprecated. Should use shadowHost() and check the return value.
-    Node* shadowAncestorNode() const;
+    Node* deprecatedShadowAncestorNode() const;
     ShadowRoot* containingShadowRoot() const;
     ShadowRoot* youngestShadowRoot() const;
 

Modified: trunk/Source/WebCore/editing/Editor.cpp (140540 => 140541)


--- trunk/Source/WebCore/editing/Editor.cpp	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebCore/editing/Editor.cpp	2013-01-23 18:04:07 UTC (rev 140541)
@@ -2709,9 +2709,9 @@
     if (resultRange->collapsed() && shadowTreeRoot) {
         searchRange = rangeOfContents(m_frame->document());
         if (forward)
-            searchRange->setStartAfter(shadowTreeRoot->shadowAncestorNode());
+            searchRange->setStartAfter(shadowTreeRoot->deprecatedShadowAncestorNode());
         else
-            searchRange->setEndBefore(shadowTreeRoot->shadowAncestorNode());
+            searchRange->setEndBefore(shadowTreeRoot->deprecatedShadowAncestorNode());
 
         resultRange = findPlainText(searchRange.get(), target, options);
     }
@@ -2768,7 +2768,7 @@
             if (!resultRange->startContainer()->isInShadowTree())
                 break;
 
-            searchRange->setStartAfter(resultRange->startContainer()->shadowAncestorNode(), exception);
+            searchRange->setStartAfter(resultRange->startContainer()->deprecatedShadowAncestorNode(), exception);
             searchRange->setEnd(originalEndContainer, originalEndOffset, exception);
             continue;
         }

Modified: trunk/Source/WebCore/editing/ReplaceSelectionCommand.cpp (140540 => 140541)


--- trunk/Source/WebCore/editing/ReplaceSelectionCommand.cpp	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebCore/editing/ReplaceSelectionCommand.cpp	2013-01-23 18:04:07 UTC (rev 140541)
@@ -156,7 +156,7 @@
     if (!editableRoot)
         return;
     
-    Node* shadowAncestorNode = editableRoot->shadowAncestorNode();
+    Node* shadowAncestorNode = editableRoot->deprecatedShadowAncestorNode();
     
     if (!editableRoot->getAttributeEventListener(eventNames().webkitBeforeTextInsertedEvent) &&
         // FIXME: Remove these checks once textareas and textfields actually register an event handler.

Modified: trunk/Source/WebCore/page/DragController.cpp (140540 => 140541)


--- trunk/Source/WebCore/page/DragController.cpp	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebCore/page/DragController.cpp	2013-01-23 18:04:07 UTC (rev 140541)
@@ -302,7 +302,7 @@
     while (n && !n->isElementNode())
         n = n->parentNode();
     if (n)
-        n = n->shadowAncestorNode();
+        n = n->deprecatedShadowAncestorNode();
 
     return static_cast<Element*>(n);
 }

Modified: trunk/Source/WebCore/page/EventHandler.cpp (140540 => 140541)


--- trunk/Source/WebCore/page/EventHandler.cpp	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebCore/page/EventHandler.cpp	2013-01-23 18:04:07 UTC (rev 140541)
@@ -1817,8 +1817,8 @@
 
     Node* clickTarget = mev.targetNode();
     if (clickTarget)
-        clickTarget = clickTarget->shadowAncestorNode();
-    Node* adjustedClickNode = m_clickNode ? m_clickNode->shadowAncestorNode() : 0;
+        clickTarget = clickTarget->deprecatedShadowAncestorNode();
+    Node* adjustedClickNode = m_clickNode ? m_clickNode->deprecatedShadowAncestorNode() : 0;
 
     bool contextMenuEvent = mouseEvent.button() == RightButton;
 #if PLATFORM(CHROMIUM) && OS(DARWIN)
@@ -2723,7 +2723,7 @@
     // handle targetNode being a shadow DOM node. 
     bool success = findBestClickableCandidate(targetNode, targetPoint, touchCenter, touchRect, *nodeList.get());
     if (success && targetNode)
-        targetNode = targetNode->shadowAncestorNode();
+        targetNode = targetNode->deprecatedShadowAncestorNode();
     return success;
 }
 

Modified: trunk/Source/WebCore/page/FocusController.cpp (140540 => 140541)


--- trunk/Source/WebCore/page/FocusController.cpp	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebCore/page/FocusController.cpp	2013-01-23 18:04:07 UTC (rev 140541)
@@ -558,7 +558,7 @@
         return;
 
     Node* selectionStartNode = s->selection().start().deprecatedNode();
-    if (selectionStartNode == newFocusedNode || selectionStartNode->isDescendantOf(newFocusedNode) || selectionStartNode->shadowAncestorNode() == newFocusedNode)
+    if (selectionStartNode == newFocusedNode || selectionStartNode->isDescendantOf(newFocusedNode) || selectionStartNode->deprecatedShadowAncestorNode() == newFocusedNode)
         return;
         
     if (Node* mousePressNode = newFocusedFrame->eventHandler()->mousePressNode()) {
@@ -568,7 +568,7 @@
             if (!root)
                 return;
 
-            if (Node* shadowAncestorNode = root->shadowAncestorNode()) {
+            if (Node* shadowAncestorNode = root->deprecatedShadowAncestorNode()) {
                 if (!shadowAncestorNode->hasTagName(inputTag) && !shadowAncestorNode->hasTagName(textareaTag))
                     return;
             }

Modified: trunk/Source/WebCore/platform/blackberry/RenderThemeBlackBerry.cpp (140540 => 140541)


--- trunk/Source/WebCore/platform/blackberry/RenderThemeBlackBerry.cpp	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebCore/platform/blackberry/RenderThemeBlackBerry.cpp	2013-01-23 18:04:07 UTC (rev 140541)
@@ -441,7 +441,7 @@
 
 bool RenderThemeBlackBerry::paintSearchFieldCancelButton(RenderObject* cancelButtonObject, const PaintInfo& paintInfo, const IntRect& r)
 {
-    Node* input = cancelButtonObject->node()->shadowAncestorNode();
+    Node* input = cancelButtonObject->node()->deprecatedShadowAncestorNode();
     if (!input->renderer()->isBox())
         return false;
 

Modified: trunk/Source/WebCore/rendering/HitTestResult.cpp (140540 => 140541)


--- trunk/Source/WebCore/rendering/HitTestResult.cpp	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebCore/rendering/HitTestResult.cpp	2013-01-23 18:04:07 UTC (rev 140541)
@@ -257,11 +257,11 @@
 {
     Node* node = innerNode();
     if (node)
-        node = node->shadowAncestorNode();
+        node = node->deprecatedShadowAncestorNode();
     setInnerNode(node);
     node = innerNonSharedNode();
     if (node)
-        node = node->shadowAncestorNode();
+        node = node->deprecatedShadowAncestorNode();
     setInnerNonSharedNode(node);
 }
 
@@ -699,7 +699,7 @@
         return true;
 
     if (!request.allowsShadowContent())
-        node = node->shadowAncestorNode();
+        node = node->deprecatedShadowAncestorNode();
 
     mutableRectBasedTestResult().add(node);
 
@@ -719,7 +719,7 @@
         return true;
 
     if (!request.allowsShadowContent())
-        node = node->shadowAncestorNode();
+        node = node->deprecatedShadowAncestorNode();
 
     mutableRectBasedTestResult().add(node);
 

Modified: trunk/Source/WebKit/blackberry/Api/WebPage.cpp (140540 => 140541)


--- trunk/Source/WebKit/blackberry/Api/WebPage.cpp	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebKit/blackberry/Api/WebPage.cpp	2013-01-23 18:04:07 UTC (rev 140541)
@@ -2346,7 +2346,7 @@
     bool canStartSelection = node->canStartSelection();
 
     if (node->isElementNode()) {
-        Element* element = static_cast<Element*>(node->shadowAncestorNode());
+        Element* element = static_cast<Element*>(node->deprecatedShadowAncestorNode());
 
         if (DOMSupport::isTextBasedContentEditableElement(element)) {
             if (!canStartSelection) {
@@ -2381,7 +2381,7 @@
     // Walk up the node tree looking for our custom webworks context attribute.
     while (node) {
         if (node->isElementNode()) {
-            Element* element = static_cast<Element*>(node->shadowAncestorNode());
+            Element* element = static_cast<Element*>(node->deprecatedShadowAncestorNode());
             String webWorksContext(DOMSupport::webWorksContext(element));
             if (!webWorksContext.stripWhiteSpace().isEmpty()) {
                 context.setFlag(Platform::WebContext::IsWebWorksContext);

Modified: trunk/Source/WebKit/blackberry/ChangeLog (140540 => 140541)


--- trunk/Source/WebKit/blackberry/ChangeLog	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebKit/blackberry/ChangeLog	2013-01-23 18:04:07 UTC (rev 140541)
@@ -1,3 +1,19 @@
+2013-01-23  Shinya Kawanaka  <[email protected]>
+
+        shadowAncestorNode() should be renamed to deprecatedShadowAncestorNode()
+        https://bugs.webkit.org/show_bug.cgi?id=107624
+
+        Reviewed by Dimitri Glazkov.
+
+        Since Node::shadowAncestorNode() is deprecated, we would like to rename it to prevent from further use.
+
+        * Api/WebPage.cpp:
+        (BlackBerry::WebKit::WebPagePrivate::webContext):
+        * WebKitSupport/FatFingers.h:
+        (BlackBerry::WebKit::FatFingersResult::node):
+        * WebKitSupport/InPageSearchManager.cpp:
+        (BlackBerry::WebKit::InPageSearchManager::scopeStringMatches):
+
 2013-01-22  Andrew Lo  <[email protected]>
 
         [BlackBerry] GIF & RAF animations sometimes don't resume after zoom

Modified: trunk/Source/WebKit/blackberry/WebKitSupport/FatFingers.h (140540 => 140541)


--- trunk/Source/WebKit/blackberry/WebKitSupport/FatFingers.h	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebKit/blackberry/WebKitSupport/FatFingers.h	2013-01-23 18:04:07 UTC (rev 140541)
@@ -159,7 +159,7 @@
 
         // Shadow trees can be nested.
         while (result->isInShadowTree())
-            result = toElement(result->shadowAncestorNode());
+            result = toElement(result->deprecatedShadowAncestorNode());
 
         if (!shouldUseRootEditableElement || !result->isElementNode())
             return result;

Modified: trunk/Source/WebKit/blackberry/WebKitSupport/InPageSearchManager.cpp (140540 => 140541)


--- trunk/Source/WebKit/blackberry/WebKitSupport/InPageSearchManager.cpp	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebKit/blackberry/WebKitSupport/InPageSearchManager.cpp	2013-01-23 18:04:07 UTC (rev 140541)
@@ -313,7 +313,7 @@
         if (resultRange->collapsed(ec)) {
             if (!resultRange->startContainer()->isInShadowTree())
                 break;
-            searchRange->setStartAfter(resultRange->startContainer()->shadowAncestorNode(), ec);
+            searchRange->setStartAfter(resultRange->startContainer()->deprecatedShadowAncestorNode(), ec);
             searchRange->setEnd(originalEndContainer, originalEndOffset, ec);
             continue;
         }

Modified: trunk/Source/WebKit/chromium/ChangeLog (140540 => 140541)


--- trunk/Source/WebKit/chromium/ChangeLog	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebKit/chromium/ChangeLog	2013-01-23 18:04:07 UTC (rev 140541)
@@ -1,3 +1,16 @@
+2013-01-23  Shinya Kawanaka  <[email protected]>
+
+        shadowAncestorNode() should be renamed to deprecatedShadowAncestorNode()
+        https://bugs.webkit.org/show_bug.cgi?id=107624
+
+        Reviewed by Dimitri Glazkov.
+
+        Since Node::shadowAncestorNode() is deprecated, we would like to rename it to prevent from further use.
+
+        * src/WebFrameImpl.cpp:
+        (WebKit::WebFrameImpl::scopeStringMatches):
+        (WebKit::WebFrameImpl::setFindEndstateFocusAndSelection):
+
 2013-01-22  Pavel Feldman  <[email protected]>
 
         Web Inspector: only allow evaluateForTestInFrontend for front-ends under test.

Modified: trunk/Source/WebKit/chromium/src/WebFrameImpl.cpp (140540 => 140541)


--- trunk/Source/WebKit/chromium/src/WebFrameImpl.cpp	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebKit/chromium/src/WebFrameImpl.cpp	2013-01-23 18:04:07 UTC (rev 140541)
@@ -1682,7 +1682,7 @@
                 break;
 
             searchRange->setStartAfter(
-                resultRange->startContainer()->shadowAncestorNode(), ec);
+                resultRange->startContainer()->deprecatedShadowAncestorNode(), ec);
             searchRange->setEnd(originalEndContainer, originalEndOffset, ec);
             continue;
         }
@@ -2336,7 +2336,7 @@
         // example, focus links if we have found text within the link.
         Node* node = m_activeMatch->firstNode();
         if (node && node->isInShadowTree()) {
-            Node* host = node->shadowAncestorNode();
+            Node* host = node->deprecatedShadowAncestorNode();
             if (host->hasTagName(HTMLNames::inputTag) || host->hasTagName(HTMLNames::textareaTag))
                 node = host;
         }

Modified: trunk/Source/WebKit/qt/ChangeLog (140540 => 140541)


--- trunk/Source/WebKit/qt/ChangeLog	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebKit/qt/ChangeLog	2013-01-23 18:04:07 UTC (rev 140541)
@@ -1,3 +1,16 @@
+2013-01-23  Shinya Kawanaka  <[email protected]>
+
+        shadowAncestorNode() should be renamed to deprecatedShadowAncestorNode()
+        https://bugs.webkit.org/show_bug.cgi?id=107624
+
+        Reviewed by Dimitri Glazkov.
+
+        Since Node::shadowAncestorNode() is deprecated, we would like to rename it to prevent from further use.
+
+        * WebCoreSupport/QWebPageAdapter.cpp:
+        (QWebPageAdapter::inputMethodEvent):
+        (QWebPageAdapter::inputMethodQuery):
+
 2013-01-22  Anders Carlsson  <[email protected]>
 
         Use a platforom strategy for local storage

Modified: trunk/Source/WebKit/qt/WebCoreSupport/QWebPageAdapter.cpp (140540 => 140541)


--- trunk/Source/WebKit/qt/WebCoreSupport/QWebPageAdapter.cpp	2013-01-23 18:02:28 UTC (rev 140540)
+++ trunk/Source/WebKit/qt/WebCoreSupport/QWebPageAdapter.cpp	2013-01-23 18:04:07 UTC (rev 140541)
@@ -647,7 +647,7 @@
 
     Node* node = 0;
     if (frame->selection()->rootEditableElement())
-        node = frame->selection()->rootEditableElement()->shadowAncestorNode();
+        node = frame->selection()->rootEditableElement()->deprecatedShadowAncestorNode();
 
     Vector<CompositionUnderline> underlines;
     bool hasSelection = false;
@@ -727,7 +727,7 @@
     RenderTextControl* renderTextControl = 0;
 
     if (frame->selection()->rootEditableElement())
-        renderer = frame->selection()->rootEditableElement()->shadowAncestorNode()->renderer();
+        renderer = frame->selection()->rootEditableElement()->deprecatedShadowAncestorNode()->renderer();
 
     if (renderer && renderer->isTextControl())
         renderTextControl = toRenderTextControl(renderer);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to