Bug 84596 – Layout Test media/encrypted-media/encrypted-media-events.html is flaky

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Mikhail Naganov]

The following layout test is flaky on all platforms

media/encrypted-media/encrypted-media-events.html

See http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=media%2Fencrypted-media%2Fencrypted-media-events.html

Probable cause:

(from the Linux bot http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Linux%2032/builds/16740/steps/webkit_tests/logs/stdio)

07:16:06.401 3304 worker/0 media/encrypted-media/encrypted-media-events.html crashed, stack trace:
07:16:06.401 3304   	base::debug::StackTrace::StackTrace() [0x82c145c]
07:16:06.401 3304   	base::(anonymous namespace)::StackDumpSignalHandler() [0x82aa2ed]
07:16:06.401 3304   	0xb782d400
07:16:06.401 3304   	base::internal::LockImpl::Lock() [0x82b23a1]
07:16:06.401 3304   	media::AesDecryptor::AddKey() [0x96e26b9]
07:16:06.402 3304   	webkit_media::WebMediaPlayerImpl::addKey() [0x91acdb7]
07:16:06.402 3304   	WebKit::WebMediaPlayerClientImpl::addKey() [0x80fbfd2]
07:16:06.402 3304   	WebCore::MediaPlayer::addKey() [0x87e279b]
07:16:06.402 3304   	WebCore::HTMLMediaElement::webkitAddKey() [0x86d0db9]
07:16:06.402 3304   	WebCore::HTMLMediaElementV8Internal::webkitAddKeyCallback() [0x8e134c4]
07:16:06.402 3304   	v8::internal::Builtin_HandleApiCall() [0x83fea2c]
07:16:06.402 3304   	0xbf925f88
07:16:06.402 3304   	0xbf925f18

[David Dorwin]

Below are more call stacks from various platforms.

http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Win%20(dbg)(2)/builds/9564/steps/webkit_tests/logs/stdio

16:42:18.252 3328 worker/1 media/encrypted-media/encrypted-media-events.html output stderr lines:
16:42:18.252 3328   Assertion failed: ptr_ != NULL, file c:\b\build\slave\webkit-win-latest-dbg\build\src\base/memory/ref_counted.h, line 249
16:42:18.252 3328   #URL:file:///E:/b/build/slave/Webkit_Win__dbg__2_/build/src/third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-events.html


http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Win/builds/13413/steps/webkit_tests/logs/stdio

14:45:03.802 608 worker/1 media/encrypted-media/encrypted-media-events.html crashed, stack trace:
14:45:03.802 608   Backtrace:
14:45:03.802 608   	RtlEnterCriticalSection [0x7C90100B+11]
14:45:03.802 608   	(No symbol) [0x0077F0CE]
14:45:03.802 608   	(No symbol) [0x006ECFF6]
14:45:03.802 608   	(No symbol) [0x00D606F7]
14:45:03.802 608   	(No symbol) [0x00DEE0B3]
14:45:03.802 608   	(No symbol) [0x009621C5]
14:45:03.802 608   	(No symbol) [0x004DCABB]
14:45:03.802 608   	(No symbol) [0x8100961E]


http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac10.5%20(dbg)(2)/builds/6332/steps/webkit_tests/logs/stdio

13:52:19.529 2576 worker/2 media/encrypted-media/encrypted-media-events.html crashed, stack trace:
13:52:19.529 2576   Assertion failed: (ptr_ != NULL), function operator->, file ../../base/memory/ref_counted.h, line 249.
13:52:19.529 2576   [3155:-1603913952:12116066056372:ERROR:process_util_posix.cc(143)] Received signal 6
13:52:19.529 2576   	0   DumpRenderTree                      0x13a62f1f start + 8501255
13:52:19.529 2576   	1   DumpRenderTree                      0x13a62ebb start + 8501155
13:52:19.529 2576   	2   DumpRenderTree                      0x13b197f7 start + 9248991
13:52:19.529 2576   	3   libSystem.B.dylib                   0x94e551cb _sigtramp + 43
13:52:19.529 2576   	4   ???                                 0xffffffff 0x0 + 4294967295
13:52:19.529 2576   	5   libSystem.B.dylib                   0x94ec9242 raise + 26
13:52:19.529 2576   	6   libSystem.B.dylib                   0x94ed5681 abort + 73
13:52:19.529 2576   	7   libSystem.B.dylib                   0x94eca3e3 __assert_rtn + 101
13:52:19.530 2576   	8   DumpRenderTree                      0x1710bc4c webkit::npapi::CarbonPluginWindowTracker::DestroyDummyWindowForDelegate(void*, OpaqueWindowPtr*) + 799052
13:52:19.530 2576   	9   DumpRenderTree                      0x17108b45 webkit::npapi::CarbonPluginWindowTracker::DestroyDummyWindowForDelegate(void*, OpaqueWindowPtr*) + 786501
13:52:19.530 2576   	10  DumpRenderTree                      0x13420b5a start + 1938498
13:52:19.530 2576   	11  DumpRenderTree                      0x154506c5 start + 35688365
13:52:19.530 2576   	12  DumpRenderTree                      0x151817d1 start + 32743609
13:52:19.530 2576   	13  DumpRenderTree                      0x1670de62 start + 55338826
13:52:19.530 2576   	14  DumpRenderTree                      0x142bfb36 start + 17269790
13:52:19.530 2576   	15  DumpRenderTree                      0x142bf5da start + 17268418
13:52:19.530 2576   	16  DumpRenderTree                      0x142b58ec start + 17228244
13:52:19.530 2576   	17  ???                                 0x2740a336 0x0 + 658547510
13:52:19.530 2576   ax: 0, bx: 94ed5641, cx: bfffb76c, dx: 94e56d52
13:52:19.530 2576   di: 172e1be0, si: 172e1be0, bp: bfffb788, sp: bfffb76c, ss: 1f, flags: 282
13:52:19.530 2576   ip: 94e56d52, cs: 7, ds: 1f, es: 1f, fs: 0, gs: 37

[David Dorwin]

This needs to be fixed in Chromium code: http://code.google.com/p/chromium/issues/detail?id=124885

[David Dorwin]

Created attachment 139490 [details]
Patch

[WebKit Review Bot]

Comment on attachment 139490 [details]
Patch

Clearing flags on attachment: 139490

Committed r115757: <http://trac.webkit.org/changeset/115757>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.