Bug 175109

Summary: NetworkResourceLoader::setDefersLoading() may cause start() to be called multiple times
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebKit2Assignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, aestes, beidson, commit-queue, ggaren, kling, koivisto, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Chris Dumez
Reported 2017-08-02 19:30:35 PDT
NetworkResourceLoader::setDefersLoading() may cause start() to be called multiple times.
Attachments
Patch (14.85 KB, patch)
2017-08-02 19:56 PDT, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Patch (14.85 KB, patch)
2017-08-02 20:28 PDT, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2017-08-02 19:30:52 PDT
<rdar://problem/33363169>
Chris Dumez
Comment 2 2017-08-02 19:56:16 PDT
Created attachment 317070 [details] Patch
Brady Eidson
Comment 3 2017-08-02 20:12:27 PDT
Comment on attachment 317070 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=317070&action=review > Source/WebKit/ChangeLog:16 > + Normally, setDefersLoading(false) returns without calling start() is m_networkLoad is not null. It typo "...if m_networkLoad is not..."
Chris Dumez
Comment 4 2017-08-02 20:28:57 PDT
Created attachment 317078 [details] Patch
WebKit Commit Bot
Comment 5 2017-08-02 22:57:48 PDT
Comment on attachment 317078 [details] Patch Clearing flags on attachment: 317078 Committed r220185: <http://trac.webkit.org/changeset/220185>
WebKit Commit Bot
Comment 6 2017-08-02 22:57:50 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.