From 5ac85ccabbaa02623152f140b9233c6cefcf3b77 Mon Sep 17 00:00:00 2001 From: Patrick Scott Date: Thu, 11 Mar 2010 09:00:08 -0500 Subject: [PATCH] Do not send the not-modified response when validating the cache. Bug: 2450517 Change-Id: Ibdff4c4b573bb63ac5de94ec7277241bb0242203 --- core/java/android/webkit/LoadListener.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/core/java/android/webkit/LoadListener.java b/core/java/android/webkit/LoadListener.java index 7da99cad9a663..4bdab0a59bd9e 100644 --- a/core/java/android/webkit/LoadListener.java +++ b/core/java/android/webkit/LoadListener.java @@ -1014,8 +1014,12 @@ class LoadListener extends Handler implements EventHandler { // do not call webcore if it is redirect. According to the code in // InspectorController::willSendRequest(), the response is only updated - // when it is not redirect. - if ((mStatusCode >= 301 && mStatusCode <= 303) || mStatusCode == 307) { + // when it is not redirect. If we received a not-modified response from + // the server and mCacheLoader is not null, do not send the response to + // webkit. This is just a validation response for loading from the + // cache. + if ((mStatusCode >= 301 && mStatusCode <= 303) || mStatusCode == 307 || + (mStatusCode == 304 && mCacheLoader != null)) { return; }