DescriptionRevert of Fix DexLoaderTest#testLoadAfterDeleteCachedDexes() flakiness (patchset #3 id:60001 of https://codereview.chromium.org/2033953002/ )
Reason for revert:
looks like this broke the Android Tests (dbg) bot
https://uberchromegw.corp.google.com/i/chromium.linux/builders/Android%20Tests%20(dbg)
also see crbug.com/617935
Original issue's description:
> Fix DexLoaderTest#testLoadAfterDeleteCachedDexes() flakiness
>
> There seems to be data cached by the OS which is not deleted by
> DexLoader#deleteCachedDexes() which is causing the test to be flaky.
> This CL changes DexLoaderTest#testLoadAfterDeleteCachedDexes() to test
> DexLoader#deleteCachedDexes() in a scenario which is closer to how the function
> is used in production.
>
> The test used to test that calling DexLoader#deleteCachedDexes() would make
> DexLoader#load() load a dex file from scratch.
>
> The CL changes the test to test that:
> - DexLoader#deleteCachedDexes() deletes previously extracted dexes from the local
> data directory
> - It is possible to use reuse a data directory to load several dexes
>
> BUG=616923
>
> Committed: https://crrev.com/c59b7de12ab8878ddfe4df4fe5cd91b0d526b8c2
> Cr-Commit-Position: refs/heads/master@{#398198}
TBR=hanxi@chromium.org,pkotwicz@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=616923
Committed: https://crrev.com/1514931b7dd9bcb5706e2dfa32e3d4e64f60d887
Cr-Commit-Position: refs/heads/master@{#398281}
Patch Set 1 #
Messages
Total messages: 7 (3 generated)
|