11663 – [S60]Cache not cleared when it gets filled

Bug 11663 - [S60]Cache not cleared when it gets filled

Summary: [S60]Cache not cleared when it gets filled

Status:	CLOSED FIXED

Alias:	None

Product:	WebKit
Classification:	Unclassified
Component:	WebKit Misc. (show other bugs)
Version:	420+
Hardware:	S60 Emulator S60 3rd edition

Importance:	P2 Normal
Assignee:	Nobody

URL:
Keywords:

Depends on:
Blocks:

Reported:	2006-11-21 07:02 PST by Shyam Sareen
Modified:	2006-11-30 13:52 PST (History)
CC List:	1 user (show)

See Also:

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 an attachment (proposed patch, testcase, etc.)

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

Description Shyam Sareen 2006-11-21 07:02:24 PST

TSW: AHUN-6VBVP8

Comment 1 Shyam Sareen 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.

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

Created attachment 11593 [details]
Common brach patch

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

Created attachment 11594 [details]
3.1m branch patch

Comment 4 Sachin Padma 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

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

Created attachment 11598 [details]
Check for 304 response.

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

Created attachment 11599 [details]
3.1m

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

Comment on attachment 11598 [details]
Check for 304 response.

RequestL should set the cacheHandler

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

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

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

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

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

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

r=me

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

committed as r17877

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

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

r=me

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

3.1m -> r17878

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

Verified and Closed.