OfflinePageUtilsTest#testDeleteSharedOfflineFiles failing on two builders |
||||
Issue descriptionincludes: - Lollipop Phone Tester, starting with https://build.chromium.org/p/chromium.android/builders/Lollipop%20Phone%20Tester/builds/12350 - Android Tests (dbg), starting with https://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg%29/builds/42944 Fails with java.lang.AssertionError: Should be able to copy file to shareable location. at org.junit.Assert.fail(Assert.java:88) at org.junit.Assert.assertTrue(Assert.java:41) at org.chromium.chrome.browser.offlinepages.OfflinePageUtilsTest.testDeleteSharedOfflineFiles(OfflinePageUtilsTest.java:309) ... Spec reverting https://chromium-review.googlesource.com/c/513984/. Will reland if not responsible.
,
Jun 1 2017
Issue 728520 has been merged into this issue.
,
Jun 1 2017
Failure appears to have been flaky, but the revert appears to have resolved the issue in the short term. Dropping priority & reassigning.
,
Jun 1 2017
,
Jun 20 2017
This was because a race condition introduced in the patch https://chromium-review.googlesource.com/c/513984/, the patch has been reverted and will not be reland (since the approach changed and now the patch is https://chromium-review.googlesource.com/c/540995/. The new CL shouldn't make tests flaky. Hence closing this issue as obsolete(won't fix). |
||||
►
Sign in to add a comment |
||||
Comment 1 by jbudorick@chromium.org
, Jun 1 2017