Changeset 141462 in webkit

Timestamp:

Jan 31, 2013 12:38:19 PM (11 years ago)

Author:

mjs@apple.com

Message:

REGRESSION (r138962): Fails to show "confirm form resubmission", hangs browser
​https://bugs.webkit.org/show_bug.cgi?id=108214

Reviewed by Adam Barth.

Attempt a workaround suggested by Nate Chapin by putting some code
in #ifdef !PLATFORM(CHROMIUM).

• loader/MainResourceLoader.cpp:

(WebCore::MainResourceLoader::notifyFinished):

Location:

trunk/Source/WebCore

Files:

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

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-01-31  Maciej Stachowiak  <mjs@apple.com>
+
+        REGRESSION (r138962): Fails to show "confirm form resubmission", hangs browser
+        https://bugs.webkit.org/show_bug.cgi?id=108214
+
+        Reviewed by Adam Barth.
+        
+        Attempt a workaround suggested by Nate Chapin by putting some code
+        in #ifdef !PLATFORM(CHROMIUM).
+
+        * loader/MainResourceLoader.cpp:
+        (WebCore::MainResourceLoader::notifyFinished):
+
 2013-01-31  Ojan Vafai  <ojan@chromium.org>
 

trunk/Source/WebCore/loader/MainResourceLoader.cpp

@@ -577 +577 @@
     }
 
+    // FIXME: we should fix the design to eliminate the need for a platform ifdef here
+#if !PLATFORM(CHROMIUM)
     if (m_documentLoader->request().cachePolicy() == ReturnCacheDataDontLoad && !m_resource->wasCanceled()) {
         frameLoader()->retryAfterFailedCacheOnlyMainResourceLoad();
         return;
     }
+#endif
 
     const ResourceError& error = m_resource->resourceError();