Bug 141568

Summary:

CachedResource::clearLoader() should self-destruct if nothing else retains the CachedResource.

Product:

WebKit

Reporter:

Andreas Kling <kling>

Component:

Page Loading

Assignee:

Andreas Kling <kling>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, japhet

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	koivisto: review+
Patch for landing	none

Andreas Kling

Reported 2015-02-13 10:47:25 PST

Anyone who may change the return value of CachedResource::canDelete() is responsible for calling deleteIfPossible() afterwards.

Attachments
Patch (1.73 KB, patch) 2015-02-13 10:50 PST, Andreas Kling	koivisto: review+	Details Formatted Diff Diff
Patch for landing (1.75 KB, patch) 2015-02-13 10:54 PST, Andreas Kling	no flags	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Andreas Kling

Comment 1 2015-02-13 10:50:22 PST

Andreas Kling

Comment 2 2015-02-13 10:54:02 PST

Created attachment 246535 [details] Patch for landing

WebKit Commit Bot

Comment 3 2015-02-13 12:30:40 PST

Comment on attachment 246535 [details] Patch for landing Clearing flags on attachment: 246535 Committed r180068: <http://trac.webkit.org/changeset/180068>

WebKit Commit Bot

Comment 4 2015-02-13 12:30:44 PST

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.