Bug 196517 - [ Mac ] REGRESSION (r243033) Assertion failed: m_logger Layout Test imported/w3c/web-platform-tests/mst-content-hint/MediaStreamTrack-contentHint.html is a flaky crash
Summary: [ Mac ] REGRESSION (r243033) Assertion failed: m_logger Layout Test imported/...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Eric Carlson
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-04-02 16:57 PDT by Shawn Roberts
Modified: 2019-10-14 10:39 PDT (History)
6 users (show)

See Also:


Attachments
Patch (1.67 KB, patch)
2019-10-14 09:52 PDT, Eric Carlson
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Shawn Roberts 2019-04-02 16:57:41 PDT
The following layout test is flaky on Mac Release WK2, Mac Debug WK1 & WK2

imported/w3c/web-platform-tests/mst-content-hint/MediaStreamTrack-contentHint.html

Probable cause:

Test started showing up recently on dashboard as a flaky crash. Through local testing found crashes began with https://trac.webkit.org/changeset/243033/webkit . Testing prior to this revision yield all passes.

Crashes the most in WK1 Debug, but I can repro locally in every build listed on the dashboard.

Reproduced with:

run-webkit-tests imported/w3c/web-platform-tests/mst-content-hint/MediaStreamTrack-contentHint.html --iterations 500 -f --debug -1 --exit-after-n-failures=2

Flakiness Dashboard:

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=imported%2Fw3c%2Fweb-platform-tests%2Fmst-content-hint%2FMediaStreamTrack-contentHint.html

Crash Log:

https://build.webkit.org/results/Apple%20Mojave%20Debug%20WK1%20(Tests)/r243759%20(2880)/imported/w3c/web-platform-tests/mst-content-hint/MediaStreamTrack-contentHint-crash-log.txt

ASSERTION FAILED: m_logger
/Volumes/Data/slave/mojave-debug/build/Source/WebCore/platform/mediastream/MediaStreamTrackPrivate.h(121) : virtual const WTF::Logger &WebCore::MediaStreamTrackPrivate::logger() const
1   0x104d9fdb9 WTFCrash
2   0x1139d655b WTFCrashWithInfo(int, char const*, char const*, int)
3   0x116de6b07 WebCore::MediaStreamTrackPrivate::logger() const
4   0x116de6259 WebCore::MediaStreamTrackPrivate::updateReadyState()
5   0x116de6a0e WebCore::MediaStreamTrackPrivate::audioSamplesAvailable(WTF::MediaTime const&, WebCore::PlatformAudioData const&, WebCore::AudioStreamDescription const&, unsigned long)
Comment 1 Radar WebKit Bug Importer 2019-04-02 16:58:29 PDT
<rdar://problem/49540704>
Comment 2 Shawn Roberts 2019-04-02 17:05:26 PDT
Skipping test in https://trac.webkit.org/changeset/243779/webkit while waiting for a fix.
Comment 3 Eric Carlson 2019-10-14 09:52:42 PDT
Created attachment 380888 [details]
Patch
Comment 4 Eric Carlson 2019-10-14 09:54:11 PDT
This crash was fixed by the changes for https://bugs.webkit.org/show_bug.cgi?id=198844/rdar://51729128
Comment 5 WebKit Commit Bot 2019-10-14 10:39:25 PDT
Comment on attachment 380888 [details]
Patch

Clearing flags on attachment: 380888

Committed r251083: <https://trac.webkit.org/changeset/251083>
Comment 6 WebKit Commit Bot 2019-10-14 10:39:27 PDT
All reviewed patches have been landed.  Closing bug.