Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(235)

Unified Diff: third_party/WebKit/Source/core/css/CSSStyleSheetResourceTest.cpp

Issue 2411243004: [WeakMemoryCache] Remove LRU lists, prune order control and live/dead distinction (Closed)
Patch Set: Rebase, rename MemoryCache::setCapacities() Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/css/CSSStyleSheetResourceTest.cpp
diff --git a/third_party/WebKit/Source/core/css/CSSStyleSheetResourceTest.cpp b/third_party/WebKit/Source/core/css/CSSStyleSheetResourceTest.cpp
index 830ebd89d61d7f0f6b5ac1a8f89618ce239d5f92..f7fa614f371efe5f19b07e4ad5849e9ffb7b25d5 100644
--- a/third_party/WebKit/Source/core/css/CSSStyleSheetResourceTest.cpp
+++ b/third_party/WebKit/Source/core/css/CSSStyleSheetResourceTest.cpp
@@ -105,22 +105,6 @@ TEST_F(CSSStyleSheetResourceTest, PruneCanCauseEviction) {
contents->checkLoaded();
cssResource->saveParsedStyleSheet(contents);
-
- memoryCache()->update(cssResource, cssResource->size(), cssResource->size(),
- false);
- memoryCache()->update(imageResource, imageResource->size(),
- imageResource->size(), false);
- if (!memoryCache()->isInSameLRUListForTest(cssResource, imageResource)) {
- // We assume that the LRU list is determined by |size / accessCount|.
- for (size_t i = 0; i < cssResource->size() + 1; ++i)
- memoryCache()->update(cssResource, cssResource->size(),
- cssResource->size(), true);
- for (size_t i = 0; i < imageResource->size() + 1; ++i)
- memoryCache()->update(imageResource, imageResource->size(),
- imageResource->size(), true);
- }
- ASSERT_TRUE(
- memoryCache()->isInSameLRUListForTest(cssResource, imageResource));
}
ThreadState::current()->collectAllGarbage();
// This operation should not lead to crash!

Powered by Google App Engine
This is Rietveld 408576698