Issue metadata
Sign in to add a comment
|
AudioRendererSinkCacheTest.SmokeTest in content_unittests sometimes fails on Fuchsia |
||||||||||||||||||||||||
Issue description
I think this is cross-platform, just happens to fail somewhat more reliably on Fuchsia.
[00008.944] 01111.01150> <== fatal exception: process content_unittests[4130] thread initial-thread[4157]
[00008.944] 01111.01150> <== fatal page fault, PC at 0x4bfe6818eded
[00008.944] 01111.01150> CS: 0 RIP: 0x4bfe6818eded EFL: 0x10287 CR2: 0
[00008.944] 01111.01150> RAX: 0 RBX: 0x19b6208dfc80 RCX: 0x1a12957bf5e0 RDX: 0x1a129580f2c0
[00008.944] 01111.01150> RSI: 0x1a25239506a0 RDI: 0x1a12957bf5e0 RBP: 0x46d93328bf10 RSP: 0x46d93328bde0
[00008.944] 01111.01150> R8: 0x508 R9: 0x1a25238d R10: 0x1980dad56880 R11: 0x46d93328c3b8
[00008.944] 01111.01150> R12: 0xa48c1b3 R13: 0x4ed73003ad40 R14: 0x7bbcec6415f8 R15: 0xd
[00008.944] 01111.01150> errc: 0x4
[00008.944] 01111.01150> bottom of user stack:
[00008.944] 01111.01150> 0x000046d93328bde0: 238d09b0 00001a25 00000000 00000000 |...#%...........|
[00008.944] 01111.01150> 0x000046d93328bdf0: 00000000 00000000 957bf4a0 00001a12 |..........{.....|
[00008.944] 01111.01150> 0x000046d93328be00: 238d09b0 01001a25 00000000 00000000 |...#%...........|
[00008.944] 01111.01150> 0x000046d93328be10: 00000000 00000000 957bf4a0 00001a12 |..........{.....|
[00008.944] 01111.01150> 0x000046d93328be20: 239506a0 00001a25 957bf4a0 00001a12 |...#%.....{.....|
[00008.944] 01111.01150> 0x000046d93328be30: 957bf4a0 00001a12 957bf4a0 00001a12 |..{.......{.....|
[00008.944] 01111.01150> 0x000046d93328be40: 3328bfc0 000046d9 238d09b0 00001a25 |..(3.F.....#%...|
[00008.944] 01111.01150> 0x000046d93328be50: 3328bfc0 000046d9 6a08efb8 00004bfe |..(3.F.....j.K..|
[00008.944] 01111.01150> 0x000046d93328be60: 238d09b0 00001a25 3328be98 000046d9 |...#%.....(3.F..|
[00008.944] 01111.01150> 0x000046d93328be70: 23950710 00001a25 23950718 00001a25 |...#%......#%...|
[00008.944] 01111.01150> 0x000046d93328be80: 239506a0 00001a25 23950710 00001a25 |...#%......#%...|
[00008.944] 01111.01150> 0x000046d93328be90: 238d09b0 00001a25 23950718 00001a25 |...#%......#%...|
[00008.944] 01111.01150> 0x000046d93328bea0: 3328be90 000046d9 00000001 00000000 |..(3.F..........|
[00008.944] 01111.01150> 0x000046d93328beb0: 238d09b0 00001a25 23950718 00001a25 |...#%......#%...|
[00008.944] 01111.01150> 0x000046d93328bec0: 957bf4a0 00001a12 238e0620 00001a25 |..{..... ..#%...|
[00008.944] 01111.01150> 0x000046d93328bed0: 23950718 00001a25 238d09b0 00001a25 |...#%......#%...|
[00008.944] 01111.01150> arch: x86_64
[00008.945] 01111.01150> dso: id=59eb1facf07e3b6fba76fc9ecd92dffa5d3a6e84 base=0x7bbcec564000 name=libc.so
[00008.945] 01111.01150> dso: id=2dadccdbdcf830271fac95b55ac6778beed7366a base=0x6b1f3acd2000 name=<vDSO>
[00008.945] 01111.01150> dso: id=2994b0ee947a258a base=0x4bfe64a85000 name=app:content_unittests
[00008.945] 01111.01150> dso: id=b83e2f93c48fea7dd1237153594f0cea9638a5d6 base=0x32635a365000 name=liblaunchpad.so
[00008.945] 01111.01150> dso: id=51d865a9ca0fd097dbc7def848f3678b551a9bba base=0x1290bdea8000 name=libmxio.so
#01: void std::__1::__tree_remove<std::__1::__tree_node_base<void*>*>(std::__1::__tree_node_base<void*>*, std::__1::__tree_node_base<void*>*) at buildtools/third_party/libc++/trunk/include/__tree:440
#02: std::__1::__tree<std::__1::__value_type<void const*, testing::(anonymous namespace)::MockObjectState>, std::__1::__map_value_compare<void const*, std::__1::__value_type<void const*, testing::(anonymous namespace)::MockObjectState>, std::__1::less<void const*>, true>, std::__1::allocator<std::__1::__value_type<void const*, testing::(anonymous namespace)::MockObjectState> > >::erase(std::__1::__tree_const_iterator<std::__1::__value_type<void const*, testing::(anonymous namespace)::MockObjectState>, std::__1::__tree_node<std::__1::__value_type<void const*, testing::(anonymous namespace)::MockObjectState>, void*>*, long>) at buildtools/third_party/libc++/trunk/include/__tree:2364
#03: erase at buildtools/third_party/libc++/trunk/include/map:1194
(inlined by) UnregisterLocked at third_party/googletest/src/googlemock/src/gmock-spec-builders.cc:754
#04: testing::internal::FunctionMockerBase<void ()>::~FunctionMockerBase() at third_party/googletest/src/googlemock/include/gmock/gmock-spec-builders.h:1476
#05: testing::internal::FunctionMocker<void ()>::~FunctionMocker() at third_party/googletest/src/googlemock/include/gmock/gmock-generated-function-mockers.h:62
#06: media::MockAudioRendererSink::~MockAudioRendererSink() at media/base/mock_audio_renderer_sink.cc:31
#07: media::MockAudioRendererSink::~MockAudioRendererSink() at media/base/mock_audio_renderer_sink.cc:31
#08: base::RefCountedThreadSafe<media::AudioRendererSink, base::DefaultRefCountedThreadSafeTraits<media::AudioRendererSink> >::DeleteInternal(media::AudioRendererSink const*) at base/memory/ref_counted.h:381
#09: base::DefaultRefCountedThreadSafeTraits<media::AudioRendererSink>::Destruct(media::AudioRendererSink const*) at base/memory/ref_counted.h:338
#10: base::RefCountedThreadSafe<media::AudioRendererSink, base::DefaultRefCountedThreadSafeTraits<media::AudioRendererSink> >::Release() const at base/memory/ref_counted.h:374
#11: scoped_refptr<media::AudioRendererSink const>::Release(media::AudioRendererSink const*) at base/memory/ref_counted.h:641
#12: scoped_refptr<media::AudioRendererSink const>::~scoped_refptr() at base/memory/ref_counted.h:536
#13: base::internal::RetainedRefWrapper<media::AudioRendererSink const>::~RetainedRefWrapper() at base/bind_helpers.h:204
#14: std::__1::__tuple_leaf<1ul, base::internal::RetainedRefWrapper<media::AudioRendererSink const>, false>::~__tuple_leaf() at buildtools/third_party/libc++/trunk/include/tuple:170
#15: std::__1::__tuple_impl<std::__1::__tuple_indices<0ul, 1ul, 2ul>, base::WeakPtr<content::AudioRendererSinkCacheImpl>, base::internal::RetainedRefWrapper<media::AudioRendererSink const>, bool>::~__tuple_impl() at buildtools/third_party/libc++/trunk/include/tuple:369
#16: std::__1::tuple<base::WeakPtr<content::AudioRendererSinkCacheImpl>, base::internal::RetainedRefWrapper<media::AudioRendererSink const>, bool>::~tuple() at buildtools/third_party/libc++/trunk/include/tuple:474
#17: base::internal::BindState<void (content::AudioRendererSinkCacheImpl::*)(media::AudioRendererSink const*, bool), base::WeakPtr<content::AudioRendererSinkCacheImpl>, base::internal::RetainedRefWrapper<media::AudioRendererSink const>, bool>::~BindState() at base/bind_internal.h:469
#18: base::internal::BindState<void (content::AudioRendererSinkCacheImpl::*)(media::AudioRendererSink const*, bool), base::WeakPtr<content::AudioRendererSinkCacheImpl>, base::internal::RetainedRefWrapper<media::AudioRendererSink const>, bool>::Destroy(base::internal::BindStateBase const*) at base/bind_internal.h:472
#19: base::internal::BindStateBaseRefCountTraits::Destruct(base::internal::BindStateBase const*) at base/callback_internal.cc:22
#20: base::RefCountedThreadSafe<base::internal::BindStateBase, base::internal::BindStateBaseRefCountTraits>::Release() const at base/memory/ref_counted.h:374
#21: scoped_refptr<base::internal::BindStateBase>::Release(base::internal::BindStateBase*) at base/memory/ref_counted.h:641
#22: scoped_refptr<base::internal::BindStateBase>::~scoped_refptr() at base/memory/ref_counted.h:536
#23: base::internal::CallbackBase::~CallbackBase() at base/callback_internal.cc:74
#24: base::OnceCallback<void ()>::~OnceCallback() at base/callback_forward.h:11
#25: base::PendingTask::~PendingTask() at base/pending_task.cc:42
#26: __destruct_at_end at buildtools/third_party/libc++/trunk/include/vector:417
(inlined by) __destruct_at_end at buildtools/third_party/libc++/trunk/include/vector:804
(inlined by) pop_back at buildtools/third_party/libc++/trunk/include/vector:1663
(inlined by) pop at buildtools/third_party/libc++/trunk/include/queue:701
(inlined by) DeletePendingTasks at base/message_loop/message_loop.cc:472
#27: base::MessageLoop::~MessageLoop() at base/message_loop/message_loop.cc:124
#28: content::AudioRendererSinkCacheTest::~AudioRendererSinkCacheTest() at content/renderer/media/audio_renderer_sink_cache_unittest.cc:34
#29: content::AudioRendererSinkCacheTest_SmokeTest_Test::~AudioRendererSinkCacheTest_SmokeTest_Test() at content/renderer/media/audio_renderer_sink_cache_unittest.cc:355
#30: content::AudioRendererSinkCacheTest_SmokeTest_Test::~AudioRendererSinkCacheTest_SmokeTest_Test() at content/renderer/media/audio_renderer_sink_cache_unittest.cc:355
#31: testing::Test::DeleteSelf_() at third_party/googletest/src/googletest/include/gtest/gtest.h:453
#32: void testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::*)(), char const*) at third_party/googletest/src/googletest/src/gtest.cc:2399
#33: void testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::*)(), char const*) at third_party/googletest/src/googletest/src/gtest.cc:2452
#34: testing::TestInfo::Run() at third_party/googletest/src/googletest/src/gtest.cc:2661
#35: testing::TestCase::Run() at third_party/googletest/src/googletest/src/gtest.cc:2770
#36: testing::internal::UnitTestImpl::RunAllTests() at third_party/googletest/src/googletest/src/gtest.cc:4647
#37: bool testing::internal::HandleSehExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>(testing::internal::UnitTestImpl*, bool (testing::internal::UnitTestImpl::*)(), char const*) at third_party/googletest/src/googletest/src/gtest.cc:2399
#38: bool testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>(testing::internal::UnitTestImpl*, bool (testing::internal::UnitTestImpl::*)(), char const*) at third_party/googletest/src/googletest/src/gtest.cc:2452
#39: testing::UnitTest::Run() at third_party/googletest/src/googletest/src/gtest.cc:4256
#40: RUN_ALL_TESTS() at third_party/googletest/src/googletest/include/gtest/gtest.h:2237
#41: base::TestSuite::Run() at base/test/test_suite.cc:270
#42: content::UnitTestTestSuite::Run() at content/public/test/unittest_test_suite.cc:45
#43: int base::internal::FunctorTraits<int (content::UnitTestTestSuite::*)(), void>::Invoke<content::UnitTestTestSuite*>(int (content::UnitTestTestSuite::*)(), content::UnitTestTestSuite*&&) at base/bind_internal.h:194
#44: int base::internal::InvokeHelper<false, int>::MakeItSo<int (content::UnitTestTestSuite::* const&)(), content::UnitTestTestSuite*>(int (content::UnitTestTestSuite::* const&)(), content::UnitTestTestSuite*&&) at base/bind_internal.h:277
#45: int base::internal::Invoker<base::internal::BindState<int (content::UnitTestTestSuite::*)(), base::internal::UnretainedWrapper<content::UnitTestTestSuite> >, int ()>::RunImpl<int (content::UnitTestTestSuite::* const&)(), std::__1::tuple<base::internal::UnretainedWrapper<content::UnitTestTestSuite> > const&, 0ul>(int (content::UnitTestTestSuite::* const&)(), std::__1::tuple<base::internal::UnretainedWrapper<content::UnitTestTestSuite> > const&, std::__1::integer_sequence<unsigned long, 0ul>) at base/bind_internal.h:349
#46: base::internal::Invoker<base::internal::BindState<int (content::UnitTestTestSuite::*)(), base::internal::UnretainedWrapper<content::UnitTestTestSuite> >, int ()>::Run(base::internal::BindStateBase*) at base/bind_internal.h:331
#47: base::RepeatingCallback<int ()>::Run() const & at base/callback.h:92
#48: base::(anonymous namespace)::LaunchUnitTestsInternal(base::RepeatingCallback<int ()> const&, unsigned long, int, bool, base::RepeatingCallback<void ()> const&) at base/test/launcher/unit_test_launcher.cc:216
#49: base::LaunchUnitTests(int, char**, base::RepeatingCallback<int ()> const&) at base/test/launcher/unit_test_launcher.cc:475
[00008.959] 02455.02504> [1/1] AudioRendererSinkCacheTest.SmokeTest (CRASHED)
,
Aug 29 2017
Actually I'm not sure. I misread CacheSink as RendererImpl... :|
,
Aug 29 2017
Yeah, not the same it looks like. I just tried pulling the CL from the other bug to check and I can still repro (sometimes) locally.
,
Aug 29 2017
I believe this one isn't cross-platform after all. GTest is (silently! why!) using this implementation of Mutex https://cs.chromium.org/chromium/src/third_party/googletest/src/googletest/include/gtest/internal/gtest-port.h . While it looks quite fast, it doesn't appear to do too much in the way of "ex"clusion.
,
Aug 29 2017
,
Aug 31 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c7d3d02c6e92331f88bd3e070fc8d7aee365aa63 commit c7d3d02c6e92331f88bd3e070fc8d7aee365aa63 Author: Scott Graham <scottmg@chromium.org> Date: Thu Aug 31 03:22:14 2017 Roll src/third_party/googletest/src/ 42bc671f4..7f8fefabe (121 commits) https://chromium.googlesource.com/external/github.com/google/googletest.git/+log/42bc671f47b1..7f8fefabedf2 $ git log 42bc671f4..7f8fefabe --date=short --no-merges --format='%ad %ae %s' 2017-08-30 gennadiycivil Removed "Trivial" 2017-08-30 gennadiycivil removed internal link ( not allowed in OSS) 2017-08-30 scottmg Note that it is preferable for Googlers to create a CL internally first 2017-08-30 thielen fix SetUp/TearDownTestCase() in AdvancedGuide 2017-08-29 scottmg Detect Fuchsia, and set GTEST_HAS_PTHREAD on GTEST_OS_FUCHSIA 2017-08-29 thielen use plural verb as mentioned in issue #1105 2017-08-29 thielen remove Yob's comma mentioned in issue #1105 2017-08-29 thielen run combined build only 2017-08-29 thielen switch on verbose make 2017-08-29 thielen fix typo: xUnit 2017-08-14 thielen fix typo in comment and string (SetUpTestCase) 2017-08-28 alyssar Applying lint checks from upstream google3 2017-08-22 arkady.shapkin Support x64 configuration for old VS2015 projects 2016-02-23 arkady.shapkin Remove gtest VS2005 projects 2016-02-23 arkady.shapkin Support x64 configuration for old VS2010 projects 2017-08-22 roman.perepelitsa Support ref-qualified member functions in Property(). 2017-08-21 Maurice.Gilden Remove unused variable 2017-08-20 gennadiycivil Added "explicit" as per compiler suggestion 2017-08-20 gennadiycivil Proposing these changes, please review 2017-08-18 henryschreineriii Adding CMake visibility policy setting 2017-08-18 Maurice.Gilden Change tabs to spaces in test case 2017-08-18 Maurice.Gilden Switch return type to class without default constructor 2017-08-18 Maurice.Gilden Fix test if exceptions are not supported 2017-08-18 Maurice.Gilden adds test for NiceMock with unknown return value 2017-08-16 gennadiycivil Update README.md 2017-08-16 alyssar Handling invalid flag values 2017-07-27 Maurice.Gilden Add function name to exception if there's no default action 2017-08-16 matthew.woehlke Fix problem installing gtest when gmock enabled 2017-08-14 misterg Change AppVeyor Status Badge to point to new AppVeyor Project Location 2017-08-14 misterg Change AppVeyor Status Badge to point to new AppVeyor Project Location 2017-08-14 soap Add documentation for pkg-config 2017-08-14 soap Add support for pkgconfig 2017-08-10 misterg Addressing Comments 2017-08-10 misterg Addressing comments 2017-08-10 gennadiycivil Update WORKSPACE 2017-08-10 thielen say "former version" instead of "released version" 2017-08-10 edu_kueyar Minor style fixes 2017-08-10 misterg Added Copyright 2017-08-10 misterg Initial Revision, review 164634031 2017-08-10 alyssar Adding a flag option to change the default mock type 2017-08-10 thielen clarify distinction regarding Test Case 2017-08-09 matthew.woehlke Fix library install destinations 2017-08-09 misterg WIP 2017-08-09 daniel.kruegler Infinite Loop when calling a mock function that takes boost::filesystem::path as parameter #521: Add is_same type trait and prevent infinite loops for recursive containers 2017-08-09 daniel.kruegler Infinite Loop when calling a mock function that takes boost::filesystem::path as parameter #521: Add is_same type trait 2017-08-09 misterg Added googlemock tests 2017-08-09 misterg Added googlemock tests 2017-08-09 misterg WIP 2017-08-09 thielen be more specific on Test Case 2017-08-09 eduherminio Punctuation 2017-08-08 misterg WIP, win testing 2017-08-08 misterg WIP, win testing 2017-08-08 misterg WIP, windows testing 2017-08-08 misterg WIP, windows testing 2017-08-08 misterg WIP, windows testing 2017-08-08 misterg WIP 2017-08-07 misterg Samples changes upstreaming 2017-08-07 misterg Samples changes upstreaming 2017-08-05 thielen add note about different definitions of Test Case 2017-08-05 thielen fix small typo in comment 2017-01-04 shlomif Correct some typos in a comment 2017-04-11 aconverse Use wider types to prevent unsigned overflow diagnostics 2017-08-03 joel.laity Fixed cmake policy issue 2017-08-02 git Fix policy issue with old cmakes 2017-08-02 misterg WIP 2017-08-02 misterg WIP 2017-08-01 misterg WIP 2017-08-01 misterg WIP 2016-07-10 kriss Fixed typo 2017-07-30 thielen fix links to Google C++ Style Guide 2017-07-30 thielen remove doc of former versions 2017-07-30 thielen remove obsolete reference to SVN trunk and fix link to pump manual 2017-07-30 thielen remove doc of former versions 2017-07-30 thielen remove obsolete reference to SVN trunk 2017-07-26 jnino docs: fix broken link 2017-07-16 zulkarnine2076 Remove duplicate code 2017-07-15 zulkarnine2076 Fix assumption for foreground bit offset 2017-07-11 pbos Add gtest-parallel to open-source projects. 2017-07-05 vincent.palancher Fixes a typo in FAQ.md 2017-07-01 coryan More tables that did not render correctly. 2017-07-01 coryan Remove silly claim that C++ lacks lambdas. 2017-07-01 coryan Same fixes for "current" version. 2017-07-01 coryan Fix typos too s/destoyed/destroyed/ 2017-07-01 coryan Fix table formatting. 2017-06-29 zulkarnine2076 Remove unnecessary const 2017-06-29 zulkarnine2076 Add helper functions for text color calculation 2017-06-22 zulkarnine2076 Fix a problem when bg_color == fg_color 2017-06-21 chris.sharpe.99 Colouring in help text 2017-06-21 zulkarnine2076 Add background_mask instead of using magic number 2017-06-20 zulkarnine2076 Fix background color in ColoredPrintf 2017-06-19 jnino docs: fix broken link from dummies guide to cook book 2017-06-19 hjd Fix typo in gmock-actions.h 2017-06-19 hjd Fixed misspelling in assertion message. 2017-06-11 BillyDonahue Create gtest-internal.h 2017-06-08 hernandezbravojorgeluis Fixing float comparison broken link 2017-05-30 lipkab Remove unnecessary 'the'. 2017-05-06 n54 Add NetBSD support 2017-03-12 danilcha Again rewrote everything 2017-03-12 danilcha Added explicit gtest library dependency 2017-03-11 danilcha Update README.md (...) Created with: roll-dep src/third_party/googletest/src R=pwnall@chromium.org Bug: 759916 , 754861 Change-Id: I72f8b283bcd29cc67575765fb3d5cace98b6151e Reviewed-on: https://chromium-review.googlesource.com/644161 Reviewed-by: Victor Costan <pwnall@chromium.org> Commit-Queue: Scott Graham <scottmg@chromium.org> Cr-Commit-Position: refs/heads/master@{#498745} [modify] https://crrev.com/c7d3d02c6e92331f88bd3e070fc8d7aee365aa63/DEPS
,
Aug 31 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f7bb7ea4f91fcc4cd9c140f8b2c092db704e268c commit f7bb7ea4f91fcc4cd9c140f8b2c092db704e268c Author: Scott Graham <scottmg@chromium.org> Date: Thu Aug 31 17:32:46 2017 fuchsia: Update content_unittests filter for recent changes/flakes - AudioRendererSinkCacheTest.SmokeTest was fixed by making gtest threadsafe (rolled already) - BrowsingDataRemoverImplTest.MultipleTasks is yet-another sqlite one that needs locks (just moved in the filter) - CacheStorageManagerTest.GetAllOriginsUsageWithOldIndex was fixed in Fuchsia and by SDK roll elsewhere (mtime fix for parent dirs) - RendererAudioOutputStreamFactoryIntegrationTest.StreamIntegrationTest was (likely) also fixed by the gtest fix. (At least, it uses gmock, and I can't reproduce failure any more. - URLLoaderImplTest.CantSniffEmptyHtml has flaked a few times, disable it along with the rest of the URLLoaderImplTest - NavigationURLLoaderTest.RequestFailedCertError and NavigationURLLoaderTest.RequestFailedCertErrorFatal started failing after the SDK roll, disable. Bug: 760687, 759916 , 754861 , 761033 Change-Id: I53baaa13f7526172675f44cba1aee9371b6dfac2 TBR: wez@chromium.org Reviewed-on: https://chromium-review.googlesource.com/646527 Commit-Queue: Scott Graham <scottmg@chromium.org> Reviewed-by: Wez <wez@chromium.org> Reviewed-by: Scott Graham <scottmg@chromium.org> Cr-Commit-Position: refs/heads/master@{#498909} [modify] https://crrev.com/f7bb7ea4f91fcc4cd9c140f8b2c092db704e268c/testing/buildbot/filters/fuchsia.content_unittests.filter
,
Aug 31 2017
,
Sep 11
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by dalecur...@chromium.org
, Aug 29 2017Status: Duplicate (was: Started)