RESOLVED FIXED 127202
Clean up PageCache classes 
https://bugs.webkit.org/show_bug.cgi?id=127202
Summary Clean up PageCache classes
Anders Carlsson
Reported 2014-01-17 16:00:45 PST
Clean up PageCache classes
Attachments
Patch (15.53 KB, patch)
2014-01-17 16:01 PST, Anders Carlsson 		no flags
DetailsFormatted DiffDiff
Patch (17.16 KB, patch)
2014-01-17 16:06 PST, Anders Carlsson 		kling: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Anders Carlsson
Comment 1 2014-01-17 16:01:53 PST
Created attachment 221503 [details] Patch
Anders Carlsson
Comment 2 2014-01-17 16:06:09 PST
Created attachment 221506 [details] Patch
Andreas Kling
Comment 3 2014-01-17 16:16:46 PST
Comment on attachment 221506 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=221506&action=review k > Source/WebKit/mac/WebCoreSupport/WebFrameLoaderClient.mm:1414 > - OwnPtr<WebCachedFramePlatformData> webPlatformData = adoptPtr(new WebCachedFramePlatformData([m_webFrame->_private->webFrameView documentView])); > - cachedFrame->setCachedFramePlatformData(webPlatformData.release()); > + auto webPlatformData = std::make_unique<WebCachedFramePlatformData>(m_webFrame->_private->webFrameView.documentView); > + cachedFrame->setCachedFramePlatformData(std::move(webPlatformData)); I'd get rid of the temporary here, since using it after this point would be an error anyway.
Anders Carlsson
Comment 4 2014-01-17 17:43:02 PST
Committed r162234: <http://trac.webkit.org/changeset/162234>
Note You need to log in before you can comment on or make changes to this bug.