• Scott Violet's avatar
    Revert "Fetch and apply Multiroom info in CAOS" · 3f6d2857
    Scott Violet authored
    This reverts commit df6e77ca.
    
    Reason for revert: Reverting as suspect of failing case_media_unittests . https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Cast%20Audio%20Linux/16824 :
    
    [ RUN      ] CastAudioManagerTest.MakeAudioOutputStreamProxy
    ../../chromecast/media/audio/cast_audio_manager_unittest.cc:106: Failure
    Actual function call count doesn't match EXPECT_CALL(*backend, Initialize())...
             Expected: to be called once
               Actual: never called - unsatisfied and active
    Stack trace:
    #0 0x00000054cffc testing::internal::UnitTestImpl::CurrentOsStackTraceExceptTop()
    #1 0x00000054c9d9 testing::internal::AssertHelper::operator=()
    #2 0x00000053ddd3 testing::internal::GoogleTestFailureReporter::ReportFailure()
    #3 0x0000005405b6 testing::internal::UntypedFunctionMockerBase::VerifyAndClearExpectationsLocked()
    #4 0x0000003e7142 chromecast::media::MockCmaBackend::~MockCmaBackend()
    #5 0x0000003a7340 testing::StrictMock<>::~StrictMock()
    #6 0x0000003a44b5 chromecast::media::(anonymous namespace)::CastAudioManagerTest_MakeAudioOutputStreamProxy_Test::TestBody()
    ../../chromecast/media/audio/cast_audio_manager_unittest.cc:105: Failure
    Actual function call count doesn't match EXPECT_CALL(*backend, CreateAudioDecoder())...
             Expected: to be called once
               Actual: never called - unsatisfied and active
    
    Original change's description:
    > Fetch and apply Multiroom info in CAOS
    > 
    > CastAudioOutputStream can now use internal backends with multizone. In
    > order for multizone to work, we first need to retrieve the multiroom
    > info and pass it to the backend.
    > 
    > BUG=b:111694183
    > TEST=Build, cast_media_unittests
    > 
    > Change-Id: I128a6b0cc1a389d01102007244fdb5594dd4c637
    > Reviewed-on: https://chromium-review.googlesource.com/1145892
    > Reviewed-by: Kenneth MacKay <kmackay@chromium.org>
    > Reviewed-by: Luke Halliwell <halliwell@chromium.org>
    > Reviewed-by: Scott Violet <sky@chromium.org>
    > Commit-Queue: Aidan Wolter <awolter@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#578670}
    
    TBR=sky@chromium.org,gab@chromium.org,halliwell@chromium.org,jameswest@chromium.org,kmackay@chromium.org,awolter@chromium.org
    
    Change-Id: I519349d1cd87709c2a1bd39299420b5a7df7bcee
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: b:111694183
    Reviewed-on: https://chromium-review.googlesource.com/1153309Reviewed-by: default avatarScott Violet <sky@chromium.org>
    Commit-Queue: Scott Violet <sky@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#578685}
    3f6d2857
cast_audio_output_stream.cc 12.6 KB