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

Unified Diff: pkg/analyzer/test/src/context/cache_test.dart

Issue 1154563002: Fix for a memory leak in CacheFlushManager. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 5 years, 7 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
« no previous file with comments | « pkg/analyzer/lib/src/context/cache.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/analyzer/test/src/context/cache_test.dart
diff --git a/pkg/analyzer/test/src/context/cache_test.dart b/pkg/analyzer/test/src/context/cache_test.dart
index 94ae434c211328a01ce3da9489f6ca994692cbba..4e8909fb33415d15919f137882be5202e7afc196 100644
--- a/pkg/analyzer/test/src/context/cache_test.dart
+++ b/pkg/analyzer/test/src/context/cache_test.dart
@@ -689,6 +689,24 @@ class CacheFlushManagerTest {
expect(manager.recentlyUsed, orderedEquals([result1, result2]));
}
+ test_resultAccessed_unlimitedCachingPolicy() {
+ manager = new CacheFlushManager(new SimpleResultCachingPolicy(-1, -1),
+ (AnalysisTarget target) => false);
+ ResultDescriptor descriptor1 = new ResultDescriptor('result1', null);
+ ResultDescriptor descriptor2 = new ResultDescriptor('result2', null);
+ AnalysisTarget target = new TestSource();
+ TargetedResult result1 = new TargetedResult(target, descriptor1);
+ TargetedResult result2 = new TargetedResult(target, descriptor2);
+ manager.resultStored(result1, null);
+ manager.resultStored(result2, null);
+ expect(manager.currentSize, 0);
+ expect(manager.recentlyUsed, isEmpty);
+ // access result2
+ manager.resultAccessed(result2);
+ expect(manager.currentSize, 0);
+ expect(manager.recentlyUsed, isEmpty);
+ }
+
test_resultStored() {
ResultDescriptor descriptor1 = new ResultDescriptor('result1', null);
ResultDescriptor descriptor2 = new ResultDescriptor('result2', null);
@@ -721,6 +739,20 @@ class CacheFlushManagerTest {
}
}
+ test_resultStored_unlimitedCachingPolicy() {
+ manager = new CacheFlushManager(new SimpleResultCachingPolicy(-1, -1),
+ (AnalysisTarget target) => false);
+ ResultDescriptor descriptor1 = new ResultDescriptor('result1', null);
+ ResultDescriptor descriptor2 = new ResultDescriptor('result2', null);
+ AnalysisTarget target = new TestSource();
+ TargetedResult result1 = new TargetedResult(target, descriptor1);
+ TargetedResult result2 = new TargetedResult(target, descriptor2);
+ manager.resultStored(result1, null);
+ manager.resultStored(result2, null);
+ expect(manager.currentSize, 0);
+ expect(manager.recentlyUsed, isEmpty);
+ }
+
test_targetRemoved() {
ResultDescriptor descriptor1 = new ResultDescriptor('result1', null);
ResultDescriptor descriptor2 = new ResultDescriptor('result2', null);
« no previous file with comments | « pkg/analyzer/lib/src/context/cache.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698