Changeset 37753 in webkit

Timestamp:

Oct 20, 2008 6:22:51 PM (16 years ago)

Author:

Antti Koivisto

Message:

2008-10-20 Antti Koivisto <Antti Koivisto>

Reviewed by Adele Peterson.

Fix ​https://bugs.webkit.org/show_bug.cgi?id=21763
REGRESSION (r36108-r36113): Can't load picture content on wired.com galleries
<rdar://problem/6285025

If we get 304 response we need to explicitly check if that completes the frame load.

No test case, simulating the condition is pretty difficult.

• loader/loader.cpp: (WebCore::Loader::Host::didReceiveResponse):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• loader/loader.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2008-10-20  Antti Koivisto  <antti@apple.com>
+
+        Reviewed by Adele Peterson.
+
+        Fix https://bugs.webkit.org/show_bug.cgi?id=21763
+        REGRESSION (r36108-r36113): Can't load picture content on wired.com galleries
+        <rdar://problem/6285025
+        
+        If we get 304 response we need to explicitly check if that completes the frame load.
+        
+        No test case, simulating the condition is pretty difficult.
+
+        * loader/loader.cpp:
+        (WebCore::Loader::Host::didReceiveResponse):
+
 2008-10-20  Anders Carlsson  <andersca@apple.com>
 

trunk/WebCore/loader/loader.cpp

@@ -384 +384 @@
             // Existing resource is ok, just use it updating the expiration time.
             cache()->revalidationSucceeded(resource, response);
+            
+            if (request->docLoader()->frame())
+                request->docLoader()->frame()->loader()->checkCompleted();
+
             delete request;