New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 615877 link

Starred by 1 user
Status: Fixed
Owner:
█ wnwen@chromium.org
OOO until Feb 4th
Closed: May 2016
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

tabmodel tests very flaky

Project Member Reported by agrieve@chromium.org, May 30 2016 
Looks like mainly these two:
org.chromium.chrome.browser.tabmodel.RestoreMigrateTest#testMigrationLeavesOtherFilesAlone
org.chromium.chrome.browser.tabmodel.RestoreMigrateTest#testMigrateData

Fails the majority of the time on:
https://build.chromium.org/p/chromium.android/builders/Lollipop%20Phone%20Tester?numbuilds=200
and:
https://build.chromium.org/p/chromium.android/builders/Lollipop%20Low-end%20Tester
Project Member

Comment 1 by bugdroid1@chromium.org, May 31 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/eed074ef476f639f864dfb0511eb56b7693abc6a

commit eed074ef476f639f864dfb0511eb56b7693abc6a
Author: wnwen <wnwen@chromium.org>
Date: Tue May 31 18:31:01 2016

Fix downstream tab state migration test.

Since the SharedPreferences is kept in memory, it is not cleared and
reloaded even when the underlying file is deleted. Using an
AdvancedMockContext should ensure that we use a fresh in-memory
SharedPreference every time, this may speed up the test too.

BUG= 615877 

Review-Url: https://codereview.chromium.org/2027503002
Cr-Commit-Position: refs/heads/master@{#396872}

[modify] https://crrev.com/eed074ef476f639f864dfb0511eb56b7693abc6a/chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/RestoreMigrateTest.java

Comment 2 by wnwen@chromium.org, May 31 2016

Status: Fixed (was: Assigned)
►

Sign in to add a comment