Commit ebc1b2d2 by vyao Committed by Oliver Woodman

Automated g4 rollback of changelist 191834511.

*** Reason for rollback ***

Rolling back to re-disable the exception as the crash is still appearing on latest version

*** Original change description ***

Automated g4 rollback of changelist 191128111.

*** Reason for rollback ***

This was a temp fix to be cherrypicked into YouTube Andorid Main App 13.12  to resolve P0 b/77315136.

*** Original change description ***

Temp fix for b/77315136: Don't lock or check cache directories in SimpleCache

To be immediately rolled back after submission

Submitting on behalf of cblay.

***

***

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=192789263
parent 0b086888
...@@ -93,10 +93,6 @@ public final class SimpleCache implements Cache { ...@@ -93,10 +93,6 @@ public final class SimpleCache implements Cache {
* @param index The CachedContentIndex to be used. * @param index The CachedContentIndex to be used.
*/ */
/*package*/ SimpleCache(File cacheDir, CacheEvictor evictor, CachedContentIndex index) { /*package*/ SimpleCache(File cacheDir, CacheEvictor evictor, CachedContentIndex index) {
if (!lockFolder(cacheDir)) {
throw new IllegalStateException("Another SimpleCache instance uses the folder: " + cacheDir);
}
this.cacheDir = cacheDir; this.cacheDir = cacheDir;
this.evictor = evictor; this.evictor = evictor;
this.index = index; this.index = index;
......
...@@ -296,17 +296,17 @@ public class SimpleCacheTest { ...@@ -296,17 +296,17 @@ public class SimpleCacheTest {
} }
} }
@Test // @Test
public void testMultipleSimpleCacheWithSameCacheDirThrowsException() throws Exception { // public void testMultipleSimpleCacheWithSameCacheDirThrowsException() throws Exception {
new SimpleCache(cacheDir, new NoOpCacheEvictor()); // new SimpleCache(cacheDir, new NoOpCacheEvictor());
//
try { // try {
new SimpleCache(cacheDir, new NoOpCacheEvictor()); // new SimpleCache(cacheDir, new NoOpCacheEvictor());
assertWithMessage("Exception was expected").fail(); // assertWithMessage("Exception was expected").fail();
} catch (IllegalStateException e) { // } catch (IllegalStateException e) {
// Expected. Do nothing. // // Expected. Do nothing.
} // }
} // }
@Test @Test
public void testMultipleSimpleCacheWithSameCacheDirDoesNotThrowsExceptionAfterRelease() public void testMultipleSimpleCacheWithSameCacheDirDoesNotThrowsExceptionAfterRelease()
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment