Title: [207580] trunk/Source/WebCore
Revision
207580
Author
ander...@apple.com
Date
2016-10-19 17:31:56 -0700 (Wed, 19 Oct 2016)

Log Message

Remove m_redirectURLs from HistoryItem
https://bugs.webkit.org/show_bug.cgi?id=163704

Reviewed by Dan Bernstein.

* history/HistoryItem.cpp:
(WebCore::HistoryItem::HistoryItem):
(WebCore::HistoryItem::reset):
* history/HistoryItem.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (207579 => 207580)


--- trunk/Source/WebCore/ChangeLog	2016-10-19 23:55:30 UTC (rev 207579)
+++ trunk/Source/WebCore/ChangeLog	2016-10-20 00:31:56 UTC (rev 207580)
@@ -1,3 +1,15 @@
+2016-10-19  Anders Carlsson  <ander...@apple.com>
+
+        Remove m_redirectURLs from HistoryItem
+        https://bugs.webkit.org/show_bug.cgi?id=163704
+
+        Reviewed by Dan Bernstein.
+
+        * history/HistoryItem.cpp:
+        (WebCore::HistoryItem::HistoryItem):
+        (WebCore::HistoryItem::reset):
+        * history/HistoryItem.h:
+
 2016-10-19  Joone Hur  <joone....@intel.com>
 
         Add a plain space instead of &nbsp; between text nodes

Modified: trunk/Source/WebCore/history/HistoryItem.cpp (207579 => 207580)


--- trunk/Source/WebCore/history/HistoryItem.cpp	2016-10-19 23:55:30 UTC (rev 207579)
+++ trunk/Source/WebCore/history/HistoryItem.cpp	2016-10-20 00:31:56 UTC (rev 207580)
@@ -129,9 +129,6 @@
     m_children.reserveInitialCapacity(size);
     for (unsigned i = 0; i < size; ++i)
         m_children.uncheckedAppend(item.m_children[i]->copy());
-
-    if (item.m_redirectURLs)
-        m_redirectURLs = std::make_unique<Vector<String>>(*item.m_redirectURLs);
 }
 
 Ref<HistoryItem> HistoryItem::copy() const
@@ -153,8 +150,6 @@
     m_lastVisitWasFailure = false;
     m_isTargetItem = false;
 
-    m_redirectURLs = nullptr;
-
     m_itemSequenceNumber = generateSequenceNumber();
 
     m_stateObject = nullptr;

Modified: trunk/Source/WebCore/history/HistoryItem.h (207579 => 207580)


--- trunk/Source/WebCore/history/HistoryItem.h	2016-10-19 23:55:30 UTC (rev 207579)
+++ trunk/Source/WebCore/history/HistoryItem.h	2016-10-20 00:31:56 UTC (rev 207580)
@@ -227,8 +227,6 @@
     bool m_isTargetItem;
     bool m_wasRestoredFromSession { false };
 
-    std::unique_ptr<Vector<String>> m_redirectURLs;
-
     // If two HistoryItems have the same item sequence number, then they are
     // clones of one another.  Traversing history from one such HistoryItem to
     // another is a no-op.  HistoryItem clones are created for parent and
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to