Changeset 207431 in webkit

Timestamp:

Oct 17, 2016 2:17:20 PM (8 years ago)

Author:

andersca@apple.com

Message:

Remove an unused HistoryItem function
​https://bugs.webkit.org/show_bug.cgi?id=163565

Reviewed by Tim Horton.

• history/HistoryItem.cpp:

(WebCore::HistoryItem::isAncestorOf): Deleted.

• history/HistoryItem.h:

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• history/HistoryItem.cpp (modified) (1 diff)
• history/HistoryItem.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2016-10-17  Anders Carlsson  <andersca@apple.com>
+
+        Remove an unused HistoryItem function
+        https://bugs.webkit.org/show_bug.cgi?id=163565
+
+        Reviewed by Tim Horton.
+
+        * history/HistoryItem.cpp:
+        (WebCore::HistoryItem::isAncestorOf): Deleted.
+        * history/HistoryItem.h:
+
 2016-10-17  Nan Wang  <n_wang@apple.com>
 

trunk/Source/WebCore/history/HistoryItem.cpp

@@ -384 +384 @@
 }
 
-bool HistoryItem::isAncestorOf(const HistoryItem& item) const
-{
-    for (size_t i = 0; i < m_children.size(); ++i) {
-        auto& child = m_children[i].get();
-        if (&child == &item)
-            return true;
-        if (child.isAncestorOf(item))
-            return true;
-    }
-    return false;
-}
-
 // We do same-document navigation if going to a different item and if either of the following is true:
 // - The other item corresponds to the same document (for history entries created via pushState or fragment changes).

trunk/Source/WebCore/history/HistoryItem.h

@@ -145 +145 @@
     WEBCORE_EXPORT bool hasChildren() const;
     void clearChildren();
-    bool isAncestorOf(const HistoryItem&) const;
     
     bool shouldDoSameDocumentNavigationTo(HistoryItem& otherItem) const;