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

Issue 603823 link

Starred by 1 user
Status: Assigned
Owner:
█ gyzhou@chromium.org
Last visit > 30 days ago
Cc:
braveyao@chromium.org
guidou@chromium.org
niklase@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

NativeDesktopMediaListTest.AddAuraWindow is flaky on Linux

Project Member Reported by perkj@chromium.org, Apr 15 2016 
From: https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29%2832%29/builds/27697/steps/unit_tests%20on%20Ubuntu-12.04/logs/NativeDesktopMediaListTest.AddAuraWindow

[ RUN      ] NativeDesktopMediaListTest.AddAuraWindow
Xlib:  extension "RANDR" missing on display ":9".

GMOCK WARNING:
Uninteresting mock function call - returning directly.
    Function call: OnSourceThumbnailChanged(0x1506b7a0, 3)
NOTE: You can safely ignore the above warning unless this call should not happen.  Do not suppress it by blindly adding an EXPECT_CALL() if you don't mean to enforce the call.  See http://code.google.com/p/googlemock/wiki/CookBook#Knowing_When_to_Expect for details.

GMOCK WARNING:
Uninteresting mock function call - returning directly.
    Function call: OnSourceRemoved(0x1506b7a0, 2)
NOTE: You can safely ignore the above warning unless this call should not happen.  Do not suppress it by blindly adding an EXPECT_CALL() if you don't mean to enforce the call.  See http://code.google.com/p/googlemock/wiki/CookBook#Knowing_When_to_Expect for details.

GMOCK WARNING:
Uninteresting mock function call - returning directly.
    Function call: OnSourceRemoved(0x1506b7a0, 2)
NOTE: You can safely ignore the above warning unless this call should not happen.  Do not suppress it by blindly adding an EXPECT_CALL() if you don't mean to enforce the call.  See http://code.google.com/p/googlemock/wiki/CookBook#Knowing_When_to_Expect for details.
unknown file: Failure

Unexpected mock function call - returning directly.
    Function call: OnSourceAdded(0x1506b7a0, 2)
Google Mock tried the following 1 expectation, but it didn't match:

../../chrome/browser/media/native_desktop_media_list_unittest.cc:396: EXPECT_CALL(observer_, OnSourceAdded(model_.get(), index))...
  Expected arg #1: is equal to 3
           Actual: 2
         Expected: to be called once
           Actual: called once - saturated and active
../../chrome/browser/media/native_desktop_media_list_unittest.cc:396: Failure
Mock function called more times than expected - returning directly.
    Function call: OnSourceAdded(0x1506b7a0, 3)
         Expected: to be called once
           Actual: called twice - over-saturated and active
[  FAILED  ] NativeDesktopMediaListTest.AddAuraWindow (294 ms)

Comment 1 by perkj@chromium.org, Apr 15 2016

Labels: -Sheriff-Chromium

Comment 2 by gyzhou@chromium.org, Apr 18 2016 

This issue should be similar to https://bugs.chromium.org/p/chromium/issues/detail?id=602494

Comment 4 by gyzhou@chromium.org, Apr 30 2016

Status: Fixed (was: Untriaged)

Comment 5 by sabbaku...@yandex-team.ru, Jul 5 

The test also fails in Windows:

[ RUN      ] NativeDesktopMediaListTest.AddAuraWindow

GMOCK WARNING:
Uninteresting mock function call - returning directly.
    Function call: OnSourceThumbnailChanged(0EA32D30, 2)
NOTE: You can safely ignore the above warning unless this call should not happen.  Do not suppress it by blindly adding an EXPECT_CALL() if you don't mean to enforce the call.  See https://github.com/google/googletest/blob/master/googlemock/docs/CookBook.md#knowing-when-to-expect for details.

GMOCK WARNING:
Uninteresting mock function call - returning directly.
    Function call: OnSourceRemoved(0EA32D30, 1)
NOTE: You can safely ignore the above warning unless this call should not happen.  Do not suppress it by blindly adding an EXPECT_CALL() if you don't mean to enforce the call.  See https://github.com/google/googletest/blob/master/googlemock/docs/CookBook.md#knowing-when-to-expect for details.

GMOCK WARNING:
Uninteresting mock function call - returning directly.
    Function call: OnSourceRemoved(0EA32D30, 1)
NOTE: You can safely ignore the above warning unless this call should not happen.  Do not suppress it by blindly adding an EXPECT_CALL() if you don't mean to enforce the call.  See https://github.com/google/googletest/blob/master/googlemock/docs/CookBook.md#knowing-when-to-expect for details.
unknown file: error:
Unexpected mock function call - returning directly.
    Function call: OnSourceAdded(0EA32D30, 1)
Google Mock tried the following 1 expectation, but it didn't match:

../../chrome/browser/media/webrtc/native_desktop_media_list_unittest.cc(399): EXPECT_CALL(observer_, OnSourceAdded(model_.get(), index))...
  Expected arg #1: is equal to 2
           Actual: 1
         Expected: to be called once
           Actual: called once - saturated and active
Stack trace:
Backtrace:
        StackTraceGetter::CurrentStackTrace [0x02D2C55A+90]
        testing::internal::UnitTestImpl::CurrentOsStackTraceExceptTop [0x02D42F6A+74]
        testing::internal::AssertHelper::operator= [0x02D42A7B+75]
        testing::internal::GoogleTestFailureReporter::ReportFailure [0x02D1237E+142]
        testing::internal::Expect [0x00210B7B+91]
        testing::internal::UntypedFunctionMockerBase::UntypedInvokeWith [0x02D18FC0+2112]
        testing::internal::FunctionMockerBase<void __cdecl(DesktopMediaList *,int)>::InvokeWith [0x00F0D70E+46]
        testing::internal::FunctionMocker<void __cdecl(DesktopMediaList *,int)>::Invoke [0x00F0D6C6+102]
        NativeDesktopMediaListTest::`scalar deleting destructor' [0x028B628E+158]
        DesktopMediaListBase::UpdateSourcesList [0x08F2F09B+1611]
        NativeDesktopMediaList::RefreshForAuraWindows [0x0729AE69+393]
        base::internal::FunctorTraits<void (__thiscall NativeDesktopMediaList::*)(std::vector<DesktopMediaListBase::SourceDescription,std::allocator<DesktopMediaListBase::SourceDescription> >),void>::Invoke<void (__thiscall NativeDesktopMediaList::*)(std::vector< [0x072A091B+107]
        base::internal::InvokeHelper<1,void>::MakeItSo<void (__thiscall NativeDesktopMediaList::*)(std::vector<DesktopMediaListBase::SourceDescription,std::allocator<DesktopMediaListBase::SourceDescription> >),base::WeakPtr<NativeDesktopMediaList>,std::vector<Des [0x072A07B2+130]
        base::internal::Invoker<base::internal::BindState<void (__thiscall NativeDesktopMediaList::*)(std::vector<DesktopMediaListBase::SourceDescription,std::allocator<DesktopMediaListBase::SourceDescription> >),base::WeakPtr<NativeDesktopMediaList>,std::vector< [0x072A06DF+111]
        base::internal::Invoker<base::internal::BindState<void (__thiscall NativeDesktopMediaList::*)(std::vector<DesktopMediaListBase::SourceDescription,std::allocator<DesktopMediaListBase::SourceDescription> >),base::WeakPtr<NativeDesktopMediaList>,std::vector< [0x072A0544+84]
        base::OnceCallback<void __cdecl(void)>::Run [0x6347CC20+80]
        base::debug::TaskAnnotator::RunTask [0x634DE1E3+1075]
        base::internal::IncomingTaskQueue::RunTask [0x63568738+232]
        base::MessageLoop::RunTask [0x6357269B+875]
        base::MessageLoop::DeferOrRunPendingTask [0x63572E59+73]
        base::MessageLoop::DoWork [0x635730DA+442]
        base::MessagePumpForUI::DoRunLoop [0x6357E93E+62]
        base::MessagePumpWin::Run [0x6357DE89+185]
        base::MessageLoop::Run [0x63571FA6+486]
        base::RunLoop::Run [0x6363DA88+488]
        base::RunLoop::RunUntilIdle [0x6363E277+199]
        views::ViewsTestBase::RunPendingMessages [0x05113F48+56]
        views::ViewsTestBase::TearDown [0x05113EDD+29]
        NativeDesktopMediaListTest::TearDown [0x028B60C1+129]

../../chrome/browser/media/webrtc/native_desktop_media_list_unittest.cc(399): error: Mock function called more times than expected - returning directly.
    Function call: OnSourceAdded(0EA32D30, 2)
         Expected: to be called once
           Actual: called twice - over-saturated and active
Stack trace:
Backtrace:
        StackTraceGetter::CurrentStackTrace [0x02D2C55A+90]
        testing::internal::UnitTestImpl::CurrentOsStackTraceExceptTop [0x02D42F6A+74]
        testing::internal::AssertHelper::operator= [0x02D42A7B+75]
        testing::internal::GoogleTestFailureReporter::ReportFailure [0x02D1237E+142]
        testing::internal::Expect [0x00210B7B+91]
        testing::internal::UntypedFunctionMockerBase::UntypedInvokeWith [0x02D1903D+2237]
        testing::internal::FunctionMockerBase<void __cdecl(DesktopMediaList *,int)>::InvokeWith [0x00F0D70E+46]
        testing::internal::FunctionMocker<void __cdecl(DesktopMediaList *,int)>::Invoke [0x00F0D6C6+102]
        NativeDesktopMediaListTest::`scalar deleting destructor' [0x028B628E+158]
        DesktopMediaListBase::UpdateSourcesList [0x08F2F09B+1611]
        NativeDesktopMediaList::RefreshForAuraWindows [0x0729AE69+393]
        base::internal::FunctorTraits<void (__thiscall NativeDesktopMediaList::*)(std::vector<DesktopMediaListBase::SourceDescription,std::allocator<DesktopMediaListBase::SourceDescription> >),void>::Invoke<void (__thiscall NativeDesktopMediaList::*)(std::vector< [0x072A091B+107]
        base::internal::InvokeHelper<1,void>::MakeItSo<void (__thiscall NativeDesktopMediaList::*)(std::vector<DesktopMediaListBase::SourceDescription,std::allocator<DesktopMediaListBase::SourceDescription> >),base::WeakPtr<NativeDesktopMediaList>,std::vector<Des [0x072A07B2+130]
        base::internal::Invoker<base::internal::BindState<void (__thiscall NativeDesktopMediaList::*)(std::vector<DesktopMediaListBase::SourceDescription,std::allocator<DesktopMediaListBase::SourceDescription> >),base::WeakPtr<NativeDesktopMediaList>,std::vector< [0x072A06DF+111]
        base::internal::Invoker<base::internal::BindState<void (__thiscall NativeDesktopMediaList::*)(std::vector<DesktopMediaListBase::SourceDescription,std::allocator<DesktopMediaListBase::SourceDescription> >),base::WeakPtr<NativeDesktopMediaList>,std::vector< [0x072A0544+84]
        base::OnceCallback<void __cdecl(void)>::Run [0x6347CC20+80]
        base::debug::TaskAnnotator::RunTask [0x634DE1E3+1075]
        base::internal::IncomingTaskQueue::RunTask [0x63568738+232]
        base::MessageLoop::RunTask [0x6357269B+875]
        base::MessageLoop::DeferOrRunPendingTask [0x63572E59+73]
        base::MessageLoop::DoWork [0x635730DA+442]
        base::MessagePumpForUI::DoRunLoop [0x6357E93E+62]
        base::MessagePumpWin::Run [0x6357DE89+185]
        base::MessageLoop::Run [0x63571FA6+486]
        base::RunLoop::Run [0x6363DA88+488]
        base::RunLoop::RunUntilIdle [0x6363E277+199]
        views::ViewsTestBase::RunPendingMessages [0x05113F48+56]
        views::ViewsTestBase::TearDown [0x05113EDD+29]
        NativeDesktopMediaListTest::TearDown [0x028B60C1+129]

[  FAILED  ] NativeDesktopMediaListTest.AddAuraWindow (642 ms)

Comment 6 by sergeyu@chromium.org, Jul 9

Components: Blink>GetUserMedia>Desktop
Status: Untriaged (was: Fixed)
Reopening - the bug was marked as Fixed incorrectly - the tests were disabled, but the root cause was not investigated or fixed.
Project Member

Comment 7 by bugdroid1@chromium.org, Jul 11 

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

commit f522512015974f151b5fd6c2c37254deefbdf253
Author: Sergey Abbakumov <sabbakumov@yandex-team.ru>
Date: Wed Jul 11 17:50:12 2018

Completely disable aura window capture unit tests

The tests are flaky on all platforms.
Adding gtest_repeat=100 reveals the issue.

Bug: 603823
Change-Id: Ic24db4872df66233634bbc19b2803980b520c43a
Reviewed-on: https://chromium-review.googlesource.com/1127029
Commit-Queue: Sergey Ulanov <sergeyu@chromium.org>
Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#574235}
[modify] https://crrev.com/f522512015974f151b5fd6c2c37254deefbdf253/chrome/browser/media/webrtc/native_desktop_media_list_unittest.cc

Comment 8 by niklase@chromium.org, Jul 12

Cc: braveyao@chromium.org
Status: Available (was: Untriaged)

Comment 9 by benhenry@chromium.org, Aug 2

Status: Assigned (was: Available)
►

Sign in to add a comment