Drop HistoryItem's m_prev / m_next
authorcdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 31 Jan 2015 03:06:11 +0000 (03:06 +0000)
committercdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 31 Jan 2015 03:06:11 +0000 (03:06 +0000)
https://bugs.webkit.org/show_bug.cgi?id=141105

Reviewed by Zalan Bujtas.

Drop HistoryItem's m_prev / m_next. Those are no longer needed after
the PageCache refactoring in <http://trac.webkit.org/r179347>.

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

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179434 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/history/HistoryItem.cpp
Source/WebCore/history/HistoryItem.h

index 81c33eb..56c9844 100644 (file)
@@ -1,3 +1,17 @@
+2015-01-30  Chris Dumez  <cdumez@apple.com>
+
+        Drop HistoryItem's m_prev / m_next
+        https://bugs.webkit.org/show_bug.cgi?id=141105
+
+        Reviewed by Zalan Bujtas.
+
+        Drop HistoryItem's m_prev / m_next. Those are no longer needed after
+        the PageCache refactoring in <http://trac.webkit.org/r179347>.
+
+        * history/HistoryItem.cpp:
+        (WebCore::HistoryItem::HistoryItem):
+        * history/HistoryItem.h:
+
 2015-01-30  Jeongmin Kim  <jm86.kim@lge.com>
 
         Update the comment in RenderLayer for clarity as related code location is changed
index c3c367f..3c52fd6 100644 (file)
@@ -61,8 +61,6 @@ HistoryItem::HistoryItem()
     , m_isTargetItem(false)
     , m_itemSequenceNumber(generateSequenceNumber())
     , m_documentSequenceNumber(generateSequenceNumber())
-    , m_next(nullptr)
-    , m_prev(nullptr)
     , m_pruningReason(PruningReason::None)
 #if PLATFORM(IOS)
     , m_scale(0)
@@ -81,8 +79,6 @@ HistoryItem::HistoryItem(const String& urlString, const String& title)
     , m_isTargetItem(false)
     , m_itemSequenceNumber(generateSequenceNumber())
     , m_documentSequenceNumber(generateSequenceNumber())
-    , m_next(nullptr)
-    , m_prev(nullptr)
     , m_pruningReason(PruningReason::None)
 #if PLATFORM(IOS)
     , m_scale(0)
@@ -103,8 +99,6 @@ HistoryItem::HistoryItem(const String& urlString, const String& title, const Str
     , m_isTargetItem(false)
     , m_itemSequenceNumber(generateSequenceNumber())
     , m_documentSequenceNumber(generateSequenceNumber())
-    , m_next(nullptr)
-    , m_prev(nullptr)
     , m_pruningReason(PruningReason::None)
 #if PLATFORM(IOS)
     , m_scale(0)
@@ -126,8 +120,6 @@ HistoryItem::HistoryItem(const URL& url, const String& target, const String& par
     , m_isTargetItem(false)
     , m_itemSequenceNumber(generateSequenceNumber())
     , m_documentSequenceNumber(generateSequenceNumber())
-    , m_next(nullptr)
-    , m_prev(nullptr)
     , m_pruningReason(PruningReason::None)
 #if PLATFORM(IOS)
     , m_scale(0)
@@ -160,8 +152,6 @@ inline HistoryItem::HistoryItem(const HistoryItem& item)
     , m_itemSequenceNumber(item.m_itemSequenceNumber)
     , m_documentSequenceNumber(item.m_documentSequenceNumber)
     , m_formContentType(item.m_formContentType)
-    , m_next(nullptr)
-    , m_prev(nullptr)
     , m_pruningReason(PruningReason::None)
 #if PLATFORM(IOS)
     , m_scale(item.m_scale)
index 3800191..64c04be 100644 (file)
@@ -256,8 +256,6 @@ private:
     String m_formContentType;
 
     // PageCache controls these fields.
-    HistoryItem* m_next;
-    HistoryItem* m_prev;
     std::unique_ptr<CachedPage> m_cachedPage;
     PruningReason m_pruningReason;