Splitting off from bug #173564. Have the FrameLoadRequest constructor that takes a frame take it by reference instead of by pointer as we assume the frame is non-null.
Created attachment 313423 [details] Patch
Created attachment 313424 [details] Patch
Attachment 313424 [details] did not pass style-queue: ERROR: Source/WebCore/loader/FrameLoadRequest.cpp:41: Code inside a namespace should not be indented. [whitespace/indent] [4] Total errors found: 1 in 14 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 313425 [details] Patch
Attachment 313425 [details] did not pass style-queue: ERROR: Source/WebCore/loader/FrameLoadRequest.cpp:41: Code inside a namespace should not be indented. [whitespace/indent] [4] Total errors found: 1 in 14 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 313435 [details] Patch
Attachment 313435 [details] did not pass style-queue: ERROR: Source/WebCore/loader/FrameLoadRequest.cpp:41: Code inside a namespace should not be indented. [whitespace/indent] [4] Total errors found: 1 in 14 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 313435 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=313435&action=review r=me > Source/WebCore/ChangeLog:4 > + https://bugs.webkit.org/show_bug.cgi?id=173614 Please import to Radar.
<rdar://problem/32884890>
Committed r218611: <http://trac.webkit.org/changeset/218611>