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

Issue 630338 link

Starred by 1 user
Status: Fixed
Owner: ----
Closed: Jul 2016
Cc:
tedc...@chromium.org
█ dozsa@google.com
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Android Tests (dbg) fails PasswordViewingTypeTest#testUserRedirectSyncSettings

Project Member Reported by timvolod...@chromium.org, Jul 21 2016 
https://uberchromegw.corp.google.com/i/chromium.linux/builders/Android%20Tests%20%28dbg%29/builds/35371/steps/chrome_public_test_apk/logs/stdio

C  946.806s Main  Detailed Logs
C  946.806s Main  ********************************************************************************
C  946.809s Main  [FAIL] org.chromium.chrome.browser.preferences.PasswordViewingTypeTest#testUserRedirectSyncSettings:
C  946.809s Main  java.lang.AssertionError
C  946.809s Main  	at org.chromium.base.ThreadUtils.assertOnUiThread(ThreadUtils.java:192)
C  946.809s Main  	at org.chromium.chrome.browser.sync.ProfileSyncService.get(ProfileSyncService.java:104)
C  946.809s Main  	at org.chromium.chrome.browser.preferences.PasswordViewingTypeTest.testUserRedirectSyncSettings(PasswordViewingTypeTest.java:133)
C  946.809s Main  	at java.lang.reflect.Method.invokeNative(Native Method)
C  946.809s Main  	at android.test.InstrumentationTestCase.runMethod(InstrumentationTestCase.java:214)
C  946.809s Main  	at android.test.InstrumentationTestCase.runTest(InstrumentationTestCase.java:199)
C  946.809s Main  	at org.chromium.base.test.BaseTestResult.run(BaseTestResult.java:129)
C  946.809s Main  	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:191)
C  946.809s Main  	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:176)
C  946.809s Main  	at android.test.InstrumentationTestRunner.onStart(InstrumentationTestRunner.java:554)
C  946.809s Main  	at android.app.Instrumentation$InstrumentationThread.run(Instrumentation.java:1701)
C  946.809s Main  ********************************************************************************
C  946.809s Main  Summary

Comment 1 by timvolod...@chromium.org, Jul 21 2016

Cc: dozsa@google.com
this has been added in https://codereview.chromium.org/2092723002
+cc:dozsa@

going to disable it

Comment 2 by tedc...@chromium.org, Jul 21 2016 

I reverted the change already.  If a new change introduces a flaky test and is recent, it is prudent to just revert the change and have them fix the test and resubmit.

Comment 3 by timvolod...@chromium.org, Jul 21 2016

Status: Fixed (was: Untriaged)
oh ok I see, was trying a 'softer' approach since other tests actually seem to work.

closing for now
►

Sign in to add a comment