Move HistoryItem to std::unique_ptr
Created attachment 221638 [details] Patch
Created attachment 221720 [details] Patch
Comment on attachment 221720 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=221720&action=review Looks good, but is missing necessary changes in Mac and Windows code. r- for these build failures. > Source/WebCore/history/HistoryItem.cpp:225 > + m_redirectURLs = nullptr; You need to make a similar change in history/mac/HistoryItemMac.mm. > Source/WebCore/history/HistoryItem.h:183 > + void setRedirectURLs(std::unique_ptr<Vector<String>>); Some Windows code (WebKit/win/WebHistoryItem.cpp) still thinks this argument is a PassOwnPtr.
Created attachment 221994 [details] Patch
Created attachment 222003 [details] Patch Removed the HistoryItem::setRedirectURLs symbol that's listing PassOwnPtr<Vector> as the parameter, will likely have to replace it with an updated one.
Created attachment 222021 [details] Patch
Comment on attachment 222021 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=222021&action=review > Source/WebKit/mac/History/WebHistoryItem.mm:391 > + std::unique_ptr<Vector<String>> redirectURLsVector = std::make_unique<Vector<String>>(size); Seems like a good place to use auto: auto redirectURLsVector = std::make_unique<Vector<String>>(size);
Committed r162696: <http://trac.webkit.org/changeset/162696>
Comment on attachment 222021 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=222021&action=review >> Source/WebKit/mac/History/WebHistoryItem.mm:391 >> + std::unique_ptr<Vector<String>> redirectURLsVector = std::make_unique<Vector<String>>(size); > > Seems like a good place to use auto: > > auto redirectURLsVector = std::make_unique<Vector<String>>(size); Addressed this before landing.