Bug 180845 – WebRTC Stats should not be console logged from a background thread

• Component: WebRTC
• Status: RESOLVED FIXED

[youenn fablet]

ASSERTION FAILED: isMainThread()
/Volumes/Data/youenn/dev/Apple/OpenSource/Source/WebCore/bindings/js/DOMWrapperWorld.cpp(72) : WebCore::DOMWrapperWorld &WebCore::mainThreadNormalWorld()
1   0x11d7fcf3d WTFCrash
2   0x110c9917a WebCore::mainThreadNormalWorld()
3   0x110d28796 WebCore::mainWorldExecState(WebCore::Frame*)
4   0x1111995a5 WebCore::Document::didLogMessage(WTFLogChannel const&, WTFLogLevel, WTF::Vector<WTF::JSONLogValue, 0ul, WTF::CrashOnOverflow, 16ul, WTF::FastMalloc>&&)
5   0x111199a7d non-virtual thunk to WebCore::Document::didLogMessage(WTFLogChannel const&, WTFLogLevel, WTF::Vector<WTF::JSONLogValue, 0ul, WTF::CrashOnOverflow, 16ul, WTF::FastMalloc>&&)
6   0x10f6706b8 void WTF::Logger::log<WTF::Logger::LogSiteIdentifier, char [19], webrtc::RTCStats>(WTFLogChannel&, WTFLogLevel, WTF::Logger::LogSiteIdentifier const&, char const (&) [19], webrtc::RTCStats const&)
7   0x10f651271 void WTF::Logger::logAlways<WTF::Logger::LogSiteIdentifier, char [19], webrtc::RTCStats>(WTFLogChannel&, WTF::Logger::LogSiteIdentifier const&, char const (&) [19], webrtc::RTCStats const&) const
8   0x10f65103f WebCore::LibWebRTCMediaEndpoint::OnStatsDelivered(rtc::scoped_refptr<webrtc::RTCStatsReport const> const&)
9   0x10f65130c non-virtual thunk to WebCore::LibWebRTCMediaEndpoint::OnStatsDelivered(rtc::scoped_refptr<webrtc::RTCStatsReport const> const&)
10  0x10bb62be3 webrtc::RTCStatsCollector::DeliverCachedReport()
11  0x10bb7167c webrtc::RTCStatsCollector::AddPartialResults_s(rtc::scoped_refptr<webrtc::RTCStatsReport>)
12  0x10bb668df webrtc::RTCStatsCollector::AddPartialResults(rtc::scoped_refptr<webrtc::RTCStatsReport> const&)
13  0x10bb66c9d webrtc::RTCStatsCollector::ProducePartialResultsOnNetworkThread(long long)
14  0x10bb81533 void rtc::MethodFunctor<webrtc::RTCStatsCollector, void (webrtc::RTCStatsCollector::*)(long long), void, long long>::CallMethod<0>(rtc::sequence<0>) const
15  0x10bb81495 rtc::MethodFunctor<webrtc::RTCStatsCollector, void (webrtc::RTCStatsCollector::*)(long long), void, long long>::operator()() const
16  0x10bb81339 rtc::FireAndForgetAsyncClosure<rtc::MethodFunctor<webrtc::RTCStatsCollector, void (webrtc::RTCStatsCollector::*)(long long), void, long long> >::Execute()
17  0x10b8b2cfe rtc::AsyncInvoker::OnMessage(rtc::Message*)

[youenn fablet]

Created attachment 329424 [details]
Patch

[EWS Watchlist]

Comment on attachment 329424 [details]
Patch

Attachment 329424 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/5667469

New failing tests:
webgl/1.0.2/conformance/context/context-release-upon-reload.html

[EWS Watchlist]

Created attachment 329437 [details]
Archive of layout-test-results from ews117 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews117  Port: mac-elcapitan  Platform: Mac OS X 10.11.6

[youenn fablet]

Comment on attachment 329424 [details]
Patch

failing test is unrelated

[WebKit Commit Bot]

Comment on attachment 329424 [details]
Patch

Clearing flags on attachment: 329424

Committed r225973: <https://trac.webkit.org/changeset/225973>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/36073411>