Bug 158076

Summary: Simplify a few lambda captures in the network cache code
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebKit2Assignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: cgarcia, commit-queue, koivisto
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Chris Dumez
Reported 2016-05-25 10:50:28 PDT
Simplify a few lambda captures in the network cache code by WTFMoving upon capture.
Attachments
Patch (13.46 KB, patch)
2016-05-25 10:52 PDT, Chris Dumez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2016-05-25 10:52:41 PDT
Created attachment 279785 [details] Patch
WebKit Commit Bot
Comment 2 2016-05-25 10:54:44 PDT
Attachment 279785 [details] did not pass style-queue: ERROR: Source/WebKit2/NetworkProcess/cache/NetworkCacheSpeculativeLoadManager.h:65: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebKit2/NetworkProcess/cache/NetworkCacheSpeculativeLoadManager.cpp:351: More than one command on the same line [whitespace/newline] [4] ERROR: Source/WebKit2/NetworkProcess/cache/NetworkCacheSpeculativeLoadManager.cpp:531: Extra space before ( in function call [whitespace/parens] [4] Total errors found: 3 in 5 files If any of these errors are false positives, please file a bug against check-webkit-style.
Chris Dumez
Comment 3 2016-05-25 12:16:50 PDT
Comment on attachment 279785 [details] Patch Clearing flags on attachment: 279785 Committed r201395: <http://trac.webkit.org/changeset/201395>
Chris Dumez
Comment 4 2016-05-25 12:16:56 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.