Bug 116119

Summary:

We should clear mainResource in DocumentLoader::cancelMainResourceLoad.

Product:

WebKit

Reporter:

Yongjun Zhang <yongjun_zhang>

Component:

Page Loading

Assignee:

Yongjun Zhang <yongjun_zhang>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ap, beidson, commit-queue, ddkilzer, japhet, joepeck, koivisto

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
clear m_mainResource in cancelMainResourceLoad.	none

Yongjun Zhang

Reported 2013-05-14 12:43:45 PDT

MainResourceLoader::clearResource() fell out when refactoring MainResourceLoader.cpp in http://trac.webkit.org/changeset/146239 for no apparent reasons. We should bring it back.

Attachments
clear m_mainResource in cancelMainResourceLoad. (3.39 KB, patch) 2013-05-15 10:33 PDT, Yongjun Zhang	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Yongjun Zhang

Comment 1 2013-05-14 12:44:48 PDT

<rdar://problem/13869068>

Yongjun Zhang

Comment 2 2013-05-15 10:33:24 PDT

Created attachment 201855 [details] clear m_mainResource in cancelMainResourceLoad.

WebKit Commit Bot

Comment 3 2013-05-15 14:50:41 PDT

Comment on attachment 201855 [details] clear m_mainResource in cancelMainResourceLoad. Clearing flags on attachment: 201855 Committed r150150: <http://trac.webkit.org/changeset/150150>

WebKit Commit Bot

Comment 4 2013-05-15 14:50:44 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.