Evict CachedResources if they are cancelled due to last client being removed.

Source/core/loader/cache/CachedImageTest.cpp

 /*
  * Copyright (c) 2013, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 13 matching lines @@
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 #include "core/loader/cache/CachedImage.h"
 
+#include "core/loader/DocumentLoader.h"
+#include "core/loader/EmptyClients.h"
 #include "core/loader/cache/CachedImageClient.h"
 #include "core/loader/cache/CachedResourceHandle.h"
+#include "core/loader/cache/CachedResourceLoader.h"
+#include "core/loader/cache/MemoryCache.h"
+#include "core/page/Frame.h"
+#include "core/page/FrameView.h"
+#include "core/page/Page.h"
 #include "core/platform/SharedBuffer.h"
 #include "core/platform/graphics/Image.h"
+#include "public/platform/Platform.h"
+#include "public/platform/WebThread.h"
 #include <gtest/gtest.h>
 
 using namespace WebCore;
 
 namespace {
 
 class MockCachedImageClient : public WebCore::CachedImageClient {
 public:
     MockCachedImageClient()
         : m_imageChangedCount(0)
@@ skipping 14 matching lines @@
     }
 
     int imageChangedCount() const { return m_imageChangedCount; }
     bool notifyFinishedCalled() const { return m_notifyFinishedCalled; }
 
 private:
     int m_imageChangedCount;
     bool m_notifyFinishedCalled;
 };
 
+class QuitTask : public WebKit::WebThread::Task {
+public:
+    virtual void run()
+    {
+        WebKit::Platform::current()->currentThread()->exitRunLoop();
+    }
+};
+
+void runPendingTasks()
+{
+    WebKit::Platform::current()->currentThread()->postTask(new QuitTask);
+    WebKit::Platform::current()->currentThread()->enterRunLoop();
+}
+
 TEST(CachedImageTest, MultipartImage)
 {
     CachedResourceHandle<CachedImage> cachedImage = new CachedImage(ResourceRequest());
     cachedImage->setLoading(true);
 
     MockCachedImageClient client;
     cachedImage->addClient(&client);
 
     // Send the multipart response. No image or data buffer is created.
     cachedImage->responseReceived(ResourceResponse(KURL(), "multipart/x-mixed-replace", 0, String(), String()));
@@ skipping 24 matching lines @@
     ASSERT_FALSE(cachedImage->resourceBuffer());
     ASSERT_FALSE(cachedImage->errorOccurred());
     ASSERT_TRUE(cachedImage->hasImage());
     ASSERT_FALSE(cachedImage->image()->isNull());
     ASSERT_EQ(cachedImage->image()->width(), 1);
     ASSERT_EQ(cachedImage->image()->height(), 1);
     ASSERT_EQ(client.imageChangedCount(), 2);
     ASSERT_TRUE(client.notifyFinishedCalled());
 }
 
+TEST(CachedImageTest, CancelOnDetach)
+{
+    // Create enough of a mocked world to get a functioning ResourceLoader.
+    KURL testURL(ParsedURLString, "http://www.test.com/cancelTest.html");
+    Page::PageClients pageClients;
+    fillWithEmptyClients(pageClients);
+    EmptyFrameLoaderClient frameLoaderClient;
+    Page page(pageClients);
+    RefPtr<Frame> frame = Frame::create(&page, 0, &frameLoaderClient);
+    frame->setView(FrameView::create(frame.get()));
+    frame->init();
+    RefPtr<DocumentLoader> documentLoader = DocumentLoader::create(ResourceRequest(testURL), SubstituteData());
+    documentLoader->setFrame(frame.get());
+
+    // Emulate starting a real load.
+    CachedResourceHandle<CachedImage> cachedImage = new CachedImage(ResourceRequest(testURL));
+    cachedImage->load(documentLoader->cachedResourceLoader(), ResourceLoaderOptions());
+    memoryCache()->add(cachedImage.get());
+
+    MockCachedImageClient client;
+    cachedImage->addClient(&client);
+    EXPECT_EQ(CachedResource::Pending, cachedImage->status());
+
+    // The load should still be alive, but a timer should be started to cancel the load inside removeClient().
+    cachedImage->removeClient(&client);
+    EXPECT_EQ(CachedResource::Pending, cachedImage->status());
+    EXPECT_NE(reinterpret_cast<CachedResource*>(0), memoryCache()->resourceForURL(testURL));
+
+    // Trigger the cancel timer, ensure the load was cancelled and the resource was evicted from the cache.
+    runPendingTasks();
+    EXPECT_EQ(CachedResource::LoadError, cachedImage->status());
+    EXPECT_EQ(reinterpret_cast<CachedResource*>(0), memoryCache()->resourceForURL(testURL));
+}
+
 } // namespace