New issue
Advanced search Search tips

Issue 785936 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

PatchTest.CheckCourgetteOperation flaky on Mac10.9 Tests (dbg)

Project Member Reported by guidou@chromium.org, Nov 16 2017

Issue description

First failure:
https://build.chromium.org/p/chromium.mac/builders/Mac10.9%20Tests%20%28dbg%29/builds/47254

Sample logs:
[ RUN      ] PatchTest.CheckCourgetteOperation
[52841:55555:1116/023118.355390:WARNING:mac_util.mm(226)] Failed to set backup exclusion for file '/private/var/folders/p4/py6nlvcx4fs5s3fg51_pzqmc0000gm/T/.org.chromium.Chromium.rc1GF3/dVIuCaw/Default/History': Error Domain=NSOSStatusErrorDomain Code=-50 "The operation couldn?t be completed. (OSStatus error -50.)" (paramErr: error in user parameter list) (-50)
[52841:5891:1116/023118.373887:WARNING:notification_platform_bridge_mac.mm(526)] AlertNotificationService: XPC connection invalidated.
[52852:771:1116/023118.534997:WARNING:vt_video_decode_accelerator_mac.cc(178)] Failed to create VTDecompressionSession: Error Domain=NSOSStatusErrorDomain Code=-8973 "The operation couldn?t be completed. (OSStatus error -8973.)" (codecOpenErr) (-8973)
[52852:771:1116/023118.536547:WARNING:vt_video_decode_accelerator_mac.cc(200)] Failed to create hardware VideoToolbox session
[52852:771:1116/023118.608477:ERROR:vt_video_encode_accelerator_mac.cc(517)]  VTCompressionSessionCreate failed: -12908
[52841:55555:1116/023118.777004:WARNING:mac_util.mm(226)] Failed to set backup exclusion for file '/private/var/folders/p4/py6nlvcx4fs5s3fg51_pzqmc0000gm/T/.org.chromium.Chromium.rc1GF3/dVIuCaw/Default/Favicons': Error Domain=NSOSStatusErrorDomain Code=-50 "The operation couldn?t be completed. (OSStatus error -50.)" (paramErr: error in user parameter list) (-50)
[52841:771:1116/023119.414459:INFO:chrome_cryptauth_service.cc(222)] Profile is not authenticated yet; waiting before starting CryptAuth managers.
[52841:36611:1116/023119.445700:FATAL:scoped_refptr.h(182)] Check failed: ptr_.
0   libbase.dylib                       0x0000000124cd4afe base::debug::StackTrace::StackTrace(unsigned long) + 174
1   libbase.dylib                       0x0000000124cd4bbd base::debug::StackTrace::StackTrace(unsigned long) + 29
2   libbase.dylib                       0x0000000124cd2e6c base::debug::StackTrace::StackTrace() + 28
3   libbase.dylib                       0x0000000124d6c28f logging::LogMessage::~LogMessage() + 479
4   libbase.dylib                       0x0000000124d69c05 logging::LogMessage::~LogMessage() + 21
5   browser_tests                       0x0000000107076830 scoped_refptr<base::internal::BindStateBase>::operator->() const + 240
6   browser_tests                       0x0000000107076715 base::internal::CallbackBase::polymorphic_invoke() const + 21
7   browser_tests                       0x00000001097fdc21 base::OnceCallback<void (int)>::Run(int) && + 65
8   browser_tests                       0x00000001097fda06 patch::(anonymous namespace)::PatchDone(scoped_refptr<patch::(anonymous namespace)::PatchParams>, int) + 54
9   browser_tests                       0x00000001097fff7a void base::internal::FunctorTraits<void (*)(scoped_refptr<patch::(anonymous namespace)::PatchParams>, int), void>::Invoke<scoped_refptr<patch::(anonymous namespace)::PatchParams> const&, int const&>(void (*)(scoped_refptr<patch::(anonymous namespace)::PatchParams>, int), scoped_refptr<patch::(anonymous namespace)::PatchParams> const&&&, int const&&&) + 90
10  browser_tests                       0x00000001097ffee0 void base::internal::InvokeHelper<false, void>::MakeItSo<void (* const&)(scoped_refptr<patch::(anonymous namespace)::PatchParams>, int), scoped_refptr<patch::(anonymous namespace)::PatchParams> const&, int const&>(void (* const&&&)(scoped_refptr<patch::(anonymous namespace)::PatchParams>, int), scoped_refptr<patch::(anonymous namespace)::PatchParams> const&&&, int const&&&) + 64
11  browser_tests                       0x00000001097ffe8d void base::internal::Invoker<base::internal::BindState<void (*)(scoped_refptr<patch::(anonymous namespace)::PatchParams>, int), scoped_refptr<patch::(anonymous namespace)::PatchParams>, int>, void ()>::RunImpl<void (* const&)(scoped_refptr<patch::(anonymous namespace)::PatchParams>, int), std::__1::tuple<scoped_refptr<patch::(anonymous namespace)::PatchParams>, int> const&, 0ul, 1ul>(void (* const&&&)(scoped_refptr<patch::(anonymous namespace)::PatchParams>, int), std::__1::tuple<scoped_refptr<patch::(anonymous namespace)::PatchParams>, int> const&&&, std::__1::integer_sequence<unsigned long, 0ul, 1ul>) + 125
12  browser_tests                       0x00000001097ffd9c base::internal::Invoker<base::internal::BindState<void (*)(scoped_refptr<patch::(anonymous namespace)::PatchParams>, int), scoped_refptr<patch::(anonymous namespace)::PatchParams>, int>, void ()>::Run(base::internal::BindStateBase*) + 44
13  libbindings.dylib                   0x000000012d3ad76f base::OnceCallback<void ()>::Run() && + 95
14  libbindings.dylib                   0x000000012d3c44da mojo::InterfaceEndpointClient::NotifyError(base::Optional<mojo::DisconnectReason> const&) + 442
15  libbindings.dylib                   0x000000012d3e4bb6 mojo::internal::MultiplexRouter::ProcessNotifyErrorTask(mojo::internal::MultiplexRouter::Task*, mojo::internal::MultiplexRouter::ClientCallBehavior, base::SequencedTaskRunner*) + 806
16  libbindings.dylib                   0x000000012d3e0106 mojo::internal::MultiplexRouter::ProcessTasks(mojo::internal::MultiplexRouter::ClientCallBehavior, base::SequencedTaskRunner*) + 1334
17  libbindings.dylib                   0x000000012d3de066 mojo::internal::MultiplexRouter::OnPipeConnectionError() + 2118
18  libbindings.dylib                   0x000000012d3e94fd void base::internal::FunctorTraits<void (mojo::internal::MultiplexRouter::*)(), void>::Invoke<mojo::internal::MultiplexRouter*>(void (mojo::internal::MultiplexRouter::*)(), mojo::internal::MultiplexRouter*&&) + 125
19  libbindings.dylib                   0x000000012d3e9444 void base::internal::InvokeHelper<false, void>::MakeItSo<void (mojo::internal::MultiplexRouter::* const&)(), mojo::internal::MultiplexRouter*>(void (mojo::internal::MultiplexRouter::* const&&&)(), mojo::internal::MultiplexRouter*&&) + 68
20  libbindings.dylib                   0x000000012d3e93d3 void base::internal::Invoker<base::internal::BindState<void (mojo::internal::MultiplexRouter::*)(), base::internal::UnretainedWrapper<mojo::internal::MultiplexRouter> >, void ()>::RunImpl<void (mojo::internal::MultiplexRouter::* const&)(), std::__1::tuple<base::internal::UnretainedWrapper<mojo::internal::MultiplexRouter> > const&, 0ul>(void (mojo::internal::MultiplexRouter::* const&&&)(), std::__1::tuple<base::internal::UnretainedWrapper<mojo::internal::MultiplexRouter> > const&&&, std::__1::integer_sequence<unsigned long, 0ul>) + 99
21  libbindings.dylib                   0x000000012d3e930c base::internal::Invoker<base::internal::BindState<void (mojo::internal::MultiplexRouter::*)(), base::internal::UnretainedWrapper<mojo::internal::MultiplexRouter> >, void ()>::Run(base::internal::BindStateBase*) + 44
22  libbindings.dylib                   0x000000012d3ad76f base::OnceCallback<void ()>::Run() && + 95
23  libbindings.dylib                   0x000000012d3aadf8 mojo::Connector::HandleError(bool, bool) + 392
24  libbindings.dylib                   0x000000012d3ac95e mojo::Connector::OnHandleReadyInternal(unsigned int) + 254
25  libbindings.dylib                   0x000000012d3ac84b mojo::Connector::OnWatcherHandleReady(unsigned int) + 27
26  libbindings.dylib                   0x000000012d3b093f void base::internal::FunctorTraits<void (mojo::Connector::*)(unsigned int), void>::Invoke<mojo::Connector*, unsigned int>(void (mojo::Connector::*)(unsigned int), mojo::Connector*&&, unsigned int&&) + 143
27  libbindings.dylib                   0x000000012d3b086f void base::internal::InvokeHelper<false, void>::MakeItSo<void (mojo::Connector::* const&)(unsigned int), mojo::Connector*, unsigned int>(void (mojo::Connector::* const&&&)(unsigned int), mojo::Connector*&&, unsigned int&&) + 95
28  libbindings.dylib                   0x000000012d3b07e3 void base::internal::Invoker<base::internal::BindState<void (mojo::Connector::*)(unsigned int), base::internal::UnretainedWrapper<mojo::Connector> >, void (unsigned int)>::RunImpl<void (mojo::Connector::* const&)(unsigned int), std::__1::tuple<base::internal::UnretainedWrapper<mojo::Connector> > const&, 0ul>(void (mojo::Connector::* const&&&)(unsigned int), std::__1::tuple<base::internal::UnretainedWrapper<mojo::Connector> > const&&&, std::__1::integer_sequence<unsigned long, 0ul>, unsigned int&&) + 115
29  libbindings.dylib                   0x000000012d3b06f1 base::internal::Invoker<base::internal::BindState<void (mojo::Connector::*)(unsigned int), base::internal::UnretainedWrapper<mojo::Connector> >, void (unsigned int)>::Run(base::internal::BindStateBase*, unsigned int) + 81
30  libbindings.dylib                   0x000000012d3a3a19 base::RepeatingCallback<void (unsigned int)>::Run(unsigned int) const & + 89
31  libbindings.dylib                   0x000000012d3af18f mojo::SimpleWatcher::DiscardReadyState(base::RepeatingCallback<void (unsigned int)> const&, unsigned int, mojo::HandleSignalsState const&) + 31
32  libbindings.dylib                   0x000000012d3af484 void base::internal::FunctorTraits<void (*)(base::RepeatingCallback<void (unsigned int)> const&, unsigned int, mojo::HandleSignalsState const&), void>::Invoke<base::RepeatingCallback<void (unsigned int)> const&, unsigned int, mojo::HandleSignalsState const&>(void (*)(base::RepeatingCallback<void (unsigned int)> const&, unsigned int, mojo::HandleSignalsState const&), base::RepeatingCallback<void (unsigned int)> const&&&, unsigned int&&, mojo::HandleSignalsState const&&&) + 68
33  libbindings.dylib                   0x000000012d3af430 void base::internal::InvokeHelper<false, void>::MakeItSo<void (* const&)(base::RepeatingCallback<void (unsigned int)> const&, unsigned int, mojo::HandleSignalsState const&), base::RepeatingCallback<void (unsigned int)> const&, unsigned int, mojo::HandleSignalsState const&>(void (* const&&&)(base::RepeatingCallback<void (unsigned int)> const&, unsigned int, mojo::HandleSignalsState const&), base::RepeatingCallback<void (unsigned int)> const&&&, unsigned int&&, mojo::HandleSignalsState const&&&) + 80
34  libbindings.dylib                   0x000000012d3af3d0 void base::internal::Invoker<base::internal::BindState<void (*)(base::RepeatingCallback<void (unsigned int)> const&, unsigned int, mojo::HandleSignalsState const&), base::RepeatingCallback<void (unsigned int)> >, void (unsigned int, mojo::HandleSignalsState const&)>::RunImpl<void (* const&)(base::RepeatingCallback<void (unsigned int)> const&, unsigned int, mojo::HandleSignalsState const&), std::__1::tuple<base::RepeatingCallback<void (unsigned int)> > const&, 0ul>(void (* const&&&)(base::RepeatingCallback<void (unsigned int)> const&, unsigned int, mojo::HandleSignalsState const&), std::__1::tuple<base::RepeatingCallback<void (unsigned int)> > const&&&, std::__1::integer_sequence<unsigned long, 0ul>, unsigned int&&, mojo::HandleSignalsState const&) + 112
35  libbindings.dylib                   0x000000012d3af2b7 base::internal::Invoker<base::internal::BindState<void (*)(base::RepeatingCallback<void (unsigned int)> const&, unsigned int, mojo::HandleSignalsState const&), base::RepeatingCallback<void (unsigned int)> >, void (unsigned int, mojo::HandleSignalsState const&)>::Run(base::internal::BindStateBase*, unsigned int, mojo::HandleSignalsState const&) + 103
36  libmojo_public_system_cpp.dylib     0x000000012d56098f base::RepeatingCallback<void (unsigned int, mojo::HandleSignalsState const&)>::Run(unsigned int, mojo::HandleSignalsState const&) const + 111
37  libmojo_public_system_cpp.dylib     0x000000012d5606eb mojo::SimpleWatcher::OnHandleReady(int, unsigned int, mojo::HandleSignalsState const&) + 459
38  libmojo_public_system_cpp.dylib     0x000000012d561393 void base::internal::FunctorTraits<void (mojo::SimpleWatcher::*)(int, unsigned int, mojo::HandleSignalsState const&), void>::Invoke<base::WeakPtr<mojo::SimpleWatcher> const&, int const&, unsigned int const&, mojo::HandleSignalsState const&>(void (mojo::SimpleWatcher::*)(int, unsigned int, mojo::HandleSignalsState const&), base::WeakPtr<mojo::SimpleWatcher> const&&&, int const&&&, unsigned int const&&&, mojo::HandleSignalsState const&&&) + 179
39  libmojo_public_system_cpp.dylib     0x000000012d561215 void base::internal::InvokeHelper<true, void>::MakeItSo<void (mojo::SimpleWatcher::* const&)(int, unsigned int, mojo::HandleSignalsState const&), base::WeakPtr<mojo::SimpleWatcher> const&, int const&, unsigned int const&, mojo::HandleSignalsState const&>(void (mojo::SimpleWatcher::* const&&&)(int, unsigned int, mojo::HandleSignalsState const&), base::WeakPtr<mojo::SimpleWatcher> const&&&, int const&&&, unsigned int const&&&, mojo::HandleSignalsState const&&&) + 149
40  libmojo_public_system_cpp.dylib     0x000000012d561172 void base::internal::Invoker<base::internal::BindState<void (mojo::SimpleWatcher::*)(int, unsigned int, mojo::HandleSignalsState const&), base::WeakPtr<mojo::SimpleWatcher>, int, unsigned int, mojo::HandleSignalsState>, void ()>::RunImpl<void (mojo::SimpleWatcher::* const&)(int, unsigned int, mojo::HandleSignalsState const&), std::__1::tuple<base::WeakPtr<mojo::SimpleWatcher>, int, unsigned int, mojo::HandleSignalsState> const&, 0ul, 1ul, 2ul, 3ul>(void (mojo::SimpleWatcher::* const&&&)(int, unsigned int, mojo::HandleSignalsState const&), std::__1::tuple<base::WeakPtr<mojo::SimpleWatcher>, int, unsigned int, mojo::HandleSignalsState> const&&&, std::__1::integer_sequence<unsigned long, 0ul, 1ul, 2ul, 3ul>) + 242
41  libmojo_public_system_cpp.dylib     0x000000012d560fbc base::internal::Invoker<base::internal::BindState<void (mojo::SimpleWatcher::*)(int, unsigned int, mojo::HandleSignalsState const&), base::WeakPtr<mojo::SimpleWatcher>, int, unsigned int, mojo::HandleSignalsState>, void ()>::Run(base::internal::BindStateBase*) + 44
42  libbase.dylib                       0x0000000124c733cf base::OnceCallback<void ()>::Run() && + 95
43  libbase.dylib                       0x0000000124cd7134 base::debug::TaskAnnotator::RunTask(char const*, base::PendingTask*) + 884
44  libbase.dylib                       0x0000000124f96811 base::internal::TaskTracker::RunOrSkipTask(std::__1::unique_ptr<base::internal::Task, std::__1::default_delete<base::internal::Task> >, base::internal::Sequence*, bool) + 3857
45  libbase.dylib                       0x0000000124f9b2d6 base::internal::TaskTrackerPosix::RunOrSkipTask(std::__1::unique_ptr<base::internal::Task, std::__1::default_delete<base::internal::Task> >, base::internal::Sequence*, bool) + 710
46  libbase.dylib                       0x0000000124f93ae9 base::internal::TaskTracker::RunNextTask(scoped_refptr<base::internal::Sequence>, base::internal::CanScheduleSequenceObserver*) + 1145
47  libbase.dylib                       0x0000000124f789d9 base::internal::SchedulerWorker::Thread::ThreadMain() + 921
48  libbase.dylib                       0x0000000124fa7b58 base::(anonymous namespace)::ThreadFunc(void*) + 632
49  libsystem_pthread.dylib             0x00007fff9687c899 _pthread_body + 138
50  libsystem_pthread.dylib             0x00007fff9687c72a _pthread_struct_init + 0
51  libsystem_pthread.dylib             0x00007fff96880fc9 thread_start + 13
 

Comment 1 by guidou@chromium.org, Nov 16 2017

PatchTest.CheckBsdiffOperation is flaky too.
Disabling both on crrev.com/c/774840
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 16 2017

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

commit 3d6c1ff7646ffc8c5da8d781f54a03dd6cc60ec6
Author: Guido Urdaneta <guidou@chromium.org>
Date: Thu Nov 16 16:04:00 2017

Disable flaky PatchTest.CheckCourgetteOperation on Mac

TBR=laforge@chromium.org

Bug:  785936 
Change-Id: I4c4b1cf5a589ea837e460fde508a7e2e83dc21cd
Reviewed-on: https://chromium-review.googlesource.com/774840
Commit-Queue: Guido Urdaneta <guidou@chromium.org>
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517096}
[modify] https://crrev.com/3d6c1ff7646ffc8c5da8d781f54a03dd6cc60ec6/chrome/browser/component_updater/component_patcher_operation_browsertest.cc

Comment 3 by guidou@chromium.org, Nov 16 2017

Labels: -Sheriff-Chromium

Comment 4 by laforge@google.com, Nov 16 2017

Cc: waff...@chromium.org sorin@chromium.org

Comment 5 by sorin@chromium.org, Nov 16 2017

Status: Started (was: Untriaged)
Owner: waff...@chromium.org
Project Member

Comment 7 by bugdroid1@chromium.org, Nov 16 2017

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

commit 9fb91d70045741cdcbfad8df694abb5845000fbb
Author: Joshua Pawlicki <waffles@google.com>
Date: Thu Nov 16 21:37:14 2017

Fix flaky behavior (and potential crash) in the patch service.

Bug:  785936 
Change-Id: I08bf3e8a66291df8fffa256bad765c6e0743d29b
Reviewed-on: https://chromium-review.googlesource.com/775555
Reviewed-by: Sorin Jianu <sorin@chromium.org>
Commit-Queue: Joshua Pawlicki <waffles@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517198}
[modify] https://crrev.com/9fb91d70045741cdcbfad8df694abb5845000fbb/chrome/browser/component_updater/component_patcher_operation_browsertest.cc
[modify] https://crrev.com/9fb91d70045741cdcbfad8df694abb5845000fbb/components/patch_service/public/cpp/patch.cc

Status: Fixed (was: Started)

Sign in to add a comment