Bug 11663

Summary:

[S60]Cache not cleared when it gets filled

Product:

WebKit

Reporter:

Shyam Sareen <shyam.sareen>

Component:

WebKit Misc.

Assignee:

Nobody <webkit-unassigned>

Status:

CLOSED FIXED

Severity:

Normal

CC:

shyam.sareen

Priority:

Version:

420+

Hardware:

S60 Emulator

OS:

S60 3rd edition

Attachments:

Description	Flags
Common brach patch	Sachin.Padma: review-
3.1m branch patch	Sachin.Padma: review-
Check for 304 response.	Sachin.Padma: review-
3.1m	Sachin.Padma: review-
Setting the cache handler in RequestL(..)	Sachin.Padma: review+
3.1m: Move code to cache handler.	Sachin.Padma: review+

Shyam Sareen

Reported 2006-11-21 07:02:24 PST

TSW: AHUN-6VBVP8

Attachments
Common brach patch (1.65 KB, patch) 2006-11-21 07:19 PST, Shyam Sareen	Sachin.Padma: review-	Details Formatted Diff Diff
3.1m branch patch (1.66 KB, patch) 2006-11-21 07:20 PST, Shyam Sareen	Sachin.Padma: review-	Details Formatted Diff Diff
Check for 304 response. (1.62 KB, patch) 2006-11-21 10:53 PST, Shyam Sareen	Sachin.Padma: review-	Details Formatted Diff Diff
3.1m (1.62 KB, patch) 2006-11-21 10:54 PST, Shyam Sareen	Sachin.Padma: review-	Details Formatted Diff Diff
Setting the cache handler in RequestL(..) (2.37 KB, patch) 2006-11-21 13:03 PST, Shyam Sareen	Sachin.Padma: review+	Details Formatted Diff Diff
3.1m: Move code to cache handler. (1.82 KB, patch) 2006-11-21 13:48 PST, Shyam Sareen	Sachin.Padma: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Shyam Sareen

Comment 1 2006-11-21 07:17:20 PST

The cache is behaving correctly when it is getting filled. The problem was in the max-age cache directive.

Shyam Sareen

Comment 2 2006-11-21 07:19:14 PST

Created attachment 11593 [details] Common brach patch

Shyam Sareen

Comment 3 2006-11-21 07:20:44 PST

Created attachment 11594 [details] 3.1m branch patch

Sachin Padma

Comment 4 2006-11-21 07:51:23 PST

Comment on attachment 11593 [details] Common brach patch If netry needs validation it should not be removed from the lookup list but rather the request should be sent with validation headers

Shyam Sareen

Comment 5 2006-11-21 10:53:37 PST

Created attachment 11598 [details] Check for 304 response.

Shyam Sareen

Comment 6 2006-11-21 10:54:32 PST

Created attachment 11599 [details] 3.1m

Sachin Padma

Comment 7 2006-11-21 12:12:46 PST

Comment on attachment 11598 [details] Check for 304 response. RequestL should set the cacheHandler

Shyam Sareen

Comment 8 2006-11-21 13:03:25 PST

Created attachment 11601 [details] Setting the cache handler in RequestL(..)

Shyam Sareen

Comment 9 2006-11-21 13:48:56 PST

Created attachment 11604 [details] 3.1m: Move code to cache handler.

Sachin Padma

Comment 10 2006-11-22 14:09:07 PST

Comment on attachment 11601 [details] Setting the cache handler in RequestL(..) r=me

Sachin Padma

Comment 11 2006-11-22 14:09:54 PST

committed as r17877

Sachin Padma

Comment 12 2006-11-22 14:11:59 PST

Comment on attachment 11604 [details] 3.1m: Move code to cache handler. r=me

Sachin Padma

Comment 13 2006-11-22 14:12:30 PST

3.1m -> r17878

Krishna

Comment 14 2006-11-30 13:52:50 PST

Verified and Closed.

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