Bug 26601

Summary:

Remove disabled code for appcache dynamic entries

Product:

WebKit

Reporter:

Alexey Proskuryakov <ap>

Component:

WebCore Misc.

Assignee:

Alexey Proskuryakov <ap>

Status:

RESOLVED FIXED

Severity:

Normal

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
proposed patch	darin: review+

Alexey Proskuryakov

Reported 2009-06-22 01:43:06 PDT

We still have some unfinished code for application cache dynamic entries in trunk. It is based on DOMStringList code that we don't use anywhere else (even though it is used elsewhere in DOM 3 Core and in HTML5), which should probably be removed for now, as well.

Attachments
proposed patch (52.29 KB, patch) 2009-06-22 01:47 PDT, Alexey Proskuryakov	darin: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Alexey Proskuryakov

Comment 1 2009-06-22 01:47:39 PDT

Created attachment 31639 [details] proposed patch

Darin Adler

Comment 2 2009-06-22 08:38:02 PDT

Comment on attachment 31639 [details] proposed patch Diff seems to be confused about ApplicationCacheGroup.cpp. I see diffs there in code that hasn't changed at all. r=me

Alexey Proskuryakov

Comment 3 2009-06-22 11:23:25 PDT

(In reply to comment #2) > Diff seems to be confused about ApplicationCacheGroup.cpp. I see diffs there in > code that hasn't changed at all. Turns out Xcode corrected line endings there. Appcache files were missing svn:eol-style properties, which I now added.

Note You need to log in before you can comment on or make changes to this bug.