<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>161582</bug_id>
          
          <creation_ts>2016-09-04 19:21:39 -0700</creation_ts>
          <short_desc>WebRTC: Several media stream tests crashing</short_desc>
          <delta_ts>2016-09-05 06:56:44 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Media</component>
          <version>Other</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Michael Catanzaro">mcatanzaro</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>alex</cc>
    
    <cc>pnormand</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1226324</commentid>
    <comment_count>0</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2016-09-04 19:21:39 -0700</bug_when>
    <thetext>We have several media stream tests crashing with this assertion in OpenWebRTC:

owr_video_renderer.c:314:owr_video_renderer_get_element: assertion failed: (flip)

Each test has been broken since it was added (though some of the tests originally timed out instead of crashed). The affected tests are:

fast/mediastream/apply-constraints-video.html
fast/mediastream/MediaStream-video-element-displays-buffer.html
fast/mediastream/MediaStream-video-element-video-tracks-disabled-then-enabled.html
fast/mediastream/MediaStream-video-element-video-tracks-disabled.html

Example backtrace:

Thread 1 (Thread 0x2b0031e90f80 (LWP 14747)):
#0  0x00002b002aa89067 in __GI_raise (sig=sig@entry=6) at ../nptl/sysdeps/unix/sysv/linux/raise.c:56
#1  0x00002b002aa8a448 in __GI_abort () at abort.c:89
#2  0x00002b0027d84105 in g_assertion_message () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/glib-2.44.1/glib/gtestutils.c:2356
#3  0x00002b0027d8418a in g_assertion_message_expr () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/glib-2.44.1/glib/gtestutils.c:2371
#4  0x00002b00264ef131 in owr_video_renderer_get_element () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/openwebrtc/local/owr_video_renderer.c:314
#5  0x00002b00264ef1fd in owr_video_renderer_constructed () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/openwebrtc/local/owr_video_renderer.c:360
#6  0x00002b0027cd5534 in g_object_new_internal () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/glib-2.44.1/gobject/gobject.c:1814
#7  0x00002b0027cd70f5 in g_object_new_valist () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/glib-2.44.1/gobject/gobject.c:2033
#8  0x00002b0027cd7404 in g_object_new () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/glib-2.44.1/gobject/gobject.c:1617
#9  0x00002b00220428cf in _ZN7WebCore30MediaPlayerPrivateGStreamerOwr15createVideoSinkEv () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#10 0x00002b00220421b5 in _ZN7WebCore30MediaPlayerPrivateGStreamerOwr4loadERNS_18MediaStreamPrivateE () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#11 0x00002b0021b2e4d2 in _ZN7WebCore11MediaPlayer23loadWithNextMediaEngineEPKNS_18MediaPlayerFactoryE () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#12 0x00002b0021b2ef9b in _ZN7WebCore11MediaPlayer4loadEPNS_18MediaStreamPrivateE () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#13 0x00002b00217674b8 in _ZN7WebCore16HTMLMediaElement12loadResourceERKNS_3URLERNS_11ContentTypeERKN3WTF6StringE () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#14 0x00002b0021767e11 in _ZN7WebCore16HTMLMediaElement19selectMediaResourceEv () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#15 0x00002b002176974a in _ZN7WebCore16HTMLMediaElement23pendingActionTimerFiredEv () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#16 0x00002b0021abab5f in _ZN7WebCore12ThreadTimers24sharedTimerFiredInternalEv () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#17 0x00002b002412ec1a in _ZZN3WTF7RunLoop9TimerBaseC4ERS0_ENUlPvE_4_FUNES3_ () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libjavascriptcoregtk-4.0.so.18
#18 0x00002b0027d5eecd in g_main_dispatch () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/glib-2.44.1/glib/gmain.c:3122
#19 g_main_context_dispatch () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/glib-2.44.1/glib/gmain.c:3737
#20 0x00002b0027d5f268 in g_main_context_iterate () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/glib-2.44.1/glib/gmain.c:3808
#21 0x00002b0027d5f582 in g_main_loop_run () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/glib-2.44.1/glib/gmain.c:4002
#22 0x00002b002412efc0 in _ZN3WTF7RunLoop3runEv () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libjavascriptcoregtk-4.0.so.18
#23 0x00002b002112b7c2 in WebProcessMainUnix () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#24 0x00002b002aa75b45 in __libc_start_main (main=0x400bf0 &lt;main&gt;, argc=2, argv=0x7fff6f6f5bd8, init=&lt;optimized out&gt;, fini=&lt;optimized out&gt;, rtld_fini=&lt;optimized out&gt;, stack_end=0x7fff6f6f5bc8) at libc-start.c:287
#25 0x0000000000400c45 in _start ()

Updating expectations accordingly.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1226386</commentid>
    <comment_count>1</comment_count>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2016-09-05 01:02:59 -0700</bug_when>
    <thetext>Oops this is my fault, sorry.

With this patch in OpenWebRTC things should improve a bit:

https://github.com/EricssonResearch/openwebrtc/pull/629

I&apos;ll try to have it merged upstream and will bump our OpenWebRTC version in JHBuild.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1226412</commentid>
    <comment_count>2</comment_count>
      <attachid>287947</attachid>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2016-09-05 06:05:28 -0700</bug_when>
    <thetext>Created attachment 287947
patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1226417</commentid>
    <comment_count>3</comment_count>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2016-09-05 06:56:44 -0700</bug_when>
    <thetext>Committed r205451: &lt;http://trac.webkit.org/changeset/205451&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>287947</attachid>
            <date>2016-09-05 06:05:28 -0700</date>
            <delta_ts>2016-09-05 06:08:46 -0700</delta_ts>
            <desc>patch</desc>
            <filename>patch.patch</filename>
            <type>text/plain</type>
            <size>3274</size>
            <attacher name="Philippe Normand">pnormand</attacher>
            
              <data encoding="base64">RnJvbSA1NTc2NTYyMjE2ODk5M2U3ODZhZGNhZGY3NDI2Y2JjOTBiYTFlNzVkIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBQaGlsaXBwZSBOb3JtYW5kIDxwbm9ybWFuZEBpZ2FsaWEuY29t
PgpEYXRlOiBNb24sIDUgU2VwIDIwMTYgMTU6MDA6MjggKzAyMDAKU3ViamVjdDogW1BBVENIXSBw
YXRjaAoKLS0tCiBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cgICAgICAgICAgICAgICAgICAgICB8ICA5
ICsrKysrKysrKwogTGF5b3V0VGVzdHMvcGxhdGZvcm0vZ3RrL1Rlc3RFeHBlY3RhdGlvbnMgfCAg
NSAtLS0tLQogVG9vbHMvQ2hhbmdlTG9nICAgICAgICAgICAgICAgICAgICAgICAgICAgfCAxMCAr
KysrKysrKysrCiBUb29scy9ndGsvamhidWlsZC5tb2R1bGVzICAgICAgICAgICAgICAgICB8ICAy
ICstCiA0IGZpbGVzIGNoYW5nZWQsIDIwIGluc2VydGlvbnMoKyksIDYgZGVsZXRpb25zKC0pCgpk
aWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9n
CmluZGV4IGIxMTA0NWQuLmMyZTRlZDkgMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3RzL0NoYW5nZUxv
ZworKysgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxMiBAQAorMjAxNi0wOS0w
NSAgUGhpbGlwcGUgTm9ybWFuZCAgPHBub3JtYW5kQGlnYWxpYS5jb20+CisKKyAgICAgICAgV2Vi
UlRDOiBTZXZlcmFsIG1lZGlhIHN0cmVhbSB0ZXN0cyBjcmFzaGluZworICAgICAgICBodHRwczov
L2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTYxNTgyCisKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ndGsvVGVzdEV4cGVj
dGF0aW9uczogVW5za2lwIG5vdy1wYXNzaW5nIHRlc3RzLgorCiAyMDE2LTA5LTA1ICBZb3Vlbm4g
RmFibGV0ICA8eW91ZW5uQGFwcGxlLmNvbT4KIAogICAgICAgICBDYWNoZWRSZXNvdXJjZUxvYWRl
ciBpcyBub3QgdGFraW5nIGludG8gYWNjb3VudCBmZXRjaCBvcHRpb25zIHRvIHVzZSBvciBub3Qg
Y2FjaGVkIHJlc291cmNlcwpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vZ3RrL1Rl
c3RFeHBlY3RhdGlvbnMgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9ndGsvVGVzdEV4cGVjdGF0aW9u
cwppbmRleCBmZGQ1NTBkLi5lZDNiYmZhIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9wbGF0Zm9y
bS9ndGsvVGVzdEV4cGVjdGF0aW9ucworKysgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9ndGsvVGVz
dEV4cGVjdGF0aW9ucwpAQCAtODIzLDExICs4MjMsNiBAQCB3ZWJraXQub3JnL2IvMTE3NzU2IG1l
ZGlhL3RyYWNrL21lZGlhLWVsZW1lbnQtZW5xdWV1ZS1ldmVudC1jcmFzaC5odG1sIFsgVGltZW91
dAogCiB3ZWJraXQub3JnL2IvMTUzOTM3IGh0dHAvdGVzdHMvbWlzYy9kZXRhY2gtZHVyaW5nLW5v
dGlmeURvbmUuaHRtbCBbIENyYXNoIFBhc3MgXQogCi13ZWJraXQub3JnL2IvMTYxNTgyIGZhc3Qv
bWVkaWFzdHJlYW0vYXBwbHktY29uc3RyYWludHMtdmlkZW8uaHRtbCBbIENyYXNoIF0KLXdlYmtp
dC5vcmcvYi8xNjE1ODIgZmFzdC9tZWRpYXN0cmVhbS9NZWRpYVN0cmVhbS12aWRlby1lbGVtZW50
LWRpc3BsYXlzLWJ1ZmZlci5odG1sIFsgQ3Jhc2ggXQotd2Via2l0Lm9yZy9iLzE2MTU4MiBmYXN0
L21lZGlhc3RyZWFtL01lZGlhU3RyZWFtLXZpZGVvLWVsZW1lbnQtdmlkZW8tdHJhY2tzLWRpc2Fi
bGVkLXRoZW4tZW5hYmxlZC5odG1sIFsgQ3Jhc2ggXQotd2Via2l0Lm9yZy9iLzE2MTU4MiBmYXN0
L21lZGlhc3RyZWFtL01lZGlhU3RyZWFtLXZpZGVvLWVsZW1lbnQtdmlkZW8tdHJhY2tzLWRpc2Fi
bGVkLmh0bWwgWyBDcmFzaCBdCi0KICMvLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8v
Ly8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vCiAj
IEVuZCBvZiBDcmFzaGluZyB0ZXN0cwogIy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8v
Ly8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8K
ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCAyMDAy
OTQxLi44NTY2ZWY4IDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMTMgQEAKKzIwMTYtMDktMDUgIFBoaWxpcHBlIE5vcm1hbmQgIDxw
bm9ybWFuZEBpZ2FsaWEuY29tPgorCisgICAgICAgIFdlYlJUQzogU2V2ZXJhbCBtZWRpYSBzdHJl
YW0gdGVzdHMgY3Jhc2hpbmcKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTE2MTU4MgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor
CisgICAgICAgICogZ3RrL2poYnVpbGQubW9kdWxlczogQnVtcCB0byBsYXRlc3QgT3BlbldlYlJU
QyBtYXN0ZXIgZml4aW5nIHRoZSBjcmFzaCBvbiBtaXNzaW5nCisgICAgICAgIGdsdmlkZW9mbGlw
IEdTdHJlYW1lciBlbGVtZW50LgorCiAyMDE2LTA5LTA1ICBZb3Vlbm4gRmFibGV0ICA8eW91ZW5u
QGFwcGxlLmNvbT4KIAogICAgICAgICBXM0MgdGVzdCBpbXBvcnRlciBzaG91bGQgZ2VuZXJhdGUg
dGhlIGxpc3Qgb2YgcmVzb3VyY2UgZmlsZXMKZGlmZiAtLWdpdCBhL1Rvb2xzL2d0ay9qaGJ1aWxk
Lm1vZHVsZXMgYi9Ub29scy9ndGsvamhidWlsZC5tb2R1bGVzCmluZGV4IDdkZjg0ZWMuLjdiNTFh
ZjEgMTAwNjQ0Ci0tLSBhL1Rvb2xzL2d0ay9qaGJ1aWxkLm1vZHVsZXMKKysrIGIvVG9vbHMvZ3Rr
L2poYnVpbGQubW9kdWxlcwpAQCAtNTE0LDcgKzUxNCw3IEBACiAgICAgICA8ZGVwIHBhY2thZ2U9
ImdzdC1wbHVnaW5zLW9wZW53ZWJydGMiLz4KICAgICAgIDxkZXAgcGFja2FnZT0ibGlibmljZSIv
PgogICAgICA8L2RlcGVuZGVuY2llcz4KLSAgICA8YnJhbmNoIHJlcG89ImdpdGh1Yi5jb20iIG1v
ZHVsZT0iRXJpY3Nzb25SZXNlYXJjaC9vcGVud2VicnRjLmdpdCIgY2hlY2tvdXRkaXI9Im9wZW53
ZWJydGMiIHRhZz0iOWIxOGM5NGExYzVlNWNhZDE2ZjI0MTk1NDYxM2MyNjdjYjAzMmM4NSIvPgor
ICAgIDxicmFuY2ggcmVwbz0iZ2l0aHViLmNvbSIgbW9kdWxlPSJFcmljc3NvblJlc2VhcmNoL29w
ZW53ZWJydGMuZ2l0IiBjaGVja291dGRpcj0ib3BlbndlYnJ0YyIgdGFnPSJjOTk3YmZmMTRhOTM4
OTU4MmJjMTA3ZTJhZWE3Y2U3N2ZiNzAwZWQ5Ii8+CiAgIDwvYXV0b3Rvb2xzPgogCiAgIDxhdXRv
dG9vbHMgaWQ9Imxsdm0iCi0tIAoyLjkuMw==
</data>
<flag name="review"
          id="311350"
          type_id="1"
          status="+"
          setter="alex"
    />
          </attachment>
      

    </bug>

</bugzilla>