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

Issue 646965 link

Starred by 1 user
Status: Assigned
Owner:
alemate@chromium.org
Cc:
█ dgn@chromium.org
achuith@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

"ExistingUserControllerTest.ExistingUserLogin" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Sep 14 2016 
"ExistingUserControllerTest.ExistingUserLogin" is flaky.

This issue was created automatically by the chromium-try-flakes app. Please find the right owner to fix the respective test/step and assign this issue to them. If the step/test is infrastructure-related, please add Infra-Troopers label and change issue status to Untriaged. When done, please remove the issue from Sheriff Bug Queue by removing the Sheriff-Chromium label.

We have detected 3 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyNwsSBUZsYWtlIixFeGlzdGluZ1VzZXJDb250cm9sbGVyVGVzdC5FeGlzdGluZ1VzZXJMb2dpbgw.

Flaky tests should be disabled within 30 minutes unless culprit CL is found and reverted. Please see more details here: https://sites.google.com/a/chromium.org/dev/developers/tree-sheriffs/sheriffing-bug-queues#triaging-auto-filed-flakiness-bugs

Comment 1 by jbroman@chromium.org, Sep 14 2016

Cc: alemate@chromium.org
Labels: OS-Chrome
Owner: achuith@chromium.org
Status: Assigned (was: Untriaged)
Failure:

[5437:5437:0914/112950:FATAL:keep_alive_registry.cc(83)] Check failed: !g_browser_process->IsShuttingDown().
#0 0x0000030aec4e base::debug::StackTrace::StackTrace()
#1 0x0000030c481a logging::LogMessage::~LogMessage()
#2 0x0000033a0401 KeepAliveRegistry::Register()
#3 0x0000034dedd8 ProfileNotification::ProfileNotification()
#4 0x0000034d9d00 MessageCenterNotificationManager::Add()
#5 0x0000071f657d SigninErrorNotifier::OnErrorChanged()
#6 0x000004a47ab1 SigninErrorController::AuthStatusChanged()
#7 0x000003468c84 MutableProfileOAuth2TokenServiceDelegate::UpdateAuthError()
#8 0x00000346947a MutableProfileOAuth2TokenServiceDelegate::OnWebDataServiceRequestDone()
#9 0x000004a66d13 WebDataRequestManager::RequestCompletedOnThread()
#10 0x000001abf47f _ZN4base8internal7InvokerINS0_9BindStateIMN5media20AudioInputControllerEFvSt10unique_ptrINS3_8AudioBusESt14default_deleteIS6_EEEJ13scoped_refptrIS4_ENS0_13PassedWrapperIS9_EEEEEFvvEE3RunEPNS0_13BindStateBaseE
#11 0x000003139064 base::debug::TaskAnnotator::RunTask()
#12 0x0000030cbf85 base::MessageLoop::RunTask()
#13 0x0000030cc2a8 base::MessageLoop::DeferOrRunPendingTask()
#14 0x0000030cc5eb base::MessageLoop::DoWork()
#15 0x0000030ce0ea base::(anonymous namespace)::WorkSourceDispatch()
#16 0x7f26dc51cd13 g_main_context_dispatch
#17 0x7f26dc51d060 <unknown>
#18 0x7f26dc51d124 g_main_context_iteration
#19 0x0000030cdf06 base::MessagePumpGlib::Run()
#20 0x0000030cbacd base::MessageLoop::RunHandler()
#21 0x0000030eb760 base::RunLoop::Run()
#22 0x0000036d7dfc content::RunThisRunLoop()
#23 0x0000036d864c content::MessageLoopRunner::Run()
#24 0x0000036d8ac7 content::WindowedNotificationObserver::Wait()
#25 0x0000012d7d03 chromeos::ExistingUserControllerTest_ExistingUserLogin_Test::RunTestOnMainThread()

No obvious recent culprit, so assigning to chrome/browser/chromeos/login/OWNERS.

Comment 2 by jbroman@chromium.org, Sep 14 2016 

Disabling CL in the CQ: https://codereview.chromium.org/2341833003/

Comment 3 by jbroman@chromium.org, Sep 14 2016 

Note that this check also fails in ExistingUserControllerPublicSessionTest.LoginStopsAutoLogin fairly often, but it doesn't seem to happen frequently enough to trigger chromium-try-flakes yet.

Comment 4 by achuith@chromium.org, Sep 14 2016

Cc: -alemate@chromium.org achuith@chromium.org
Owner: alemate@chromium.org
Sorry Alex, you'll have to handle this.
Project Member

Comment 5 by bugdroid1@chromium.org, Sep 14 2016 

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

commit efd2bb86ff473f9ae895f9b845f196a2c5f21706
Author: jbroman <jbroman@chromium.org>
Date: Wed Sep 14 20:38:02 2016

Disable ExistingUserControllerTest.ExistingUserLogin, which is flaky.

TBR=achuith@chromium.org
BUG=646965

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

[modify] https://crrev.com/efd2bb86ff473f9ae895f9b845f196a2c5f21706/chrome/browser/chromeos/login/existing_user_controller_browsertest.cc

Comment 6 by grt@chromium.org, Sep 15 2016

Labels: -Sheriff-Chromium
Removing Sheriff-Chromium since this is assigned.

Comment 7 by ericwilligers@chromium.org, Dec 8 2016

Cc: dgn@chromium.org
►

Sign in to add a comment