<?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>276717</bug_id>
          
          <creation_ts>2024-07-17 07:53:23 -0700</creation_ts>
          <short_desc>A crash with multiple video tags that contains webm source</short_desc>
          <delta_ts>2024-07-30 01:51:19 -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>Safari 17</version>
          <rep_platform>iPhone / iPad</rep_platform>
          <op_sys>iOS 17</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=277327</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Major</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Dmitry Vakhnenko">dima</reporter>
          <assigned_to name="Jean-Yves Avenard [:jya]">jean-yves.avenard</assigned_to>
          <cc>ap</cc>
    
    <cc>dima</cc>
    
    <cc>jean-yves.avenard</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2046722</commentid>
    <comment_count>0</comment_count>
      <attachid>471904</attachid>
    <who name="Dmitry Vakhnenko">dima</who>
    <bug_when>2024-07-17 07:53:23 -0700</bug_when>
    <thetext>Created attachment 471904
Crush demo

For some reason a tab with multiple &lt;video&gt; tags crashes if it has a &lt;source&gt; in webm format.

We consistently catch this error on iPhone 12 only. On other devices we have not been able to catch this bug.

I have prepared a minimal example html file on which we catch this behavior.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2046724</commentid>
    <comment_count>1</comment_count>
    <who name="Dmitry Vakhnenko">dima</who>
    <bug_when>2024-07-17 07:55:57 -0700</bug_when>
    <thetext>Hmm, on 13 iphone it crashes too

iOS 17.5.1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2046868</commentid>
    <comment_count>2</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2024-07-17 17:43:07 -0700</bug_when>
    <thetext>Thank you for the report! Perhaps it is just running out of memory? When opening on a Mac, I see the videos resize to become pretty huge.

Could you please see if there is a crash log stored? I don&apos;t have the configuration to quickly try it myself.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2048026</commentid>
    <comment_count>3</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2024-07-24 07:54:42 -0700</bug_when>
    <thetext>&lt;rdar://problem/132405520&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2048532</commentid>
    <comment_count>4</comment_count>
    <who name="Jean-Yves Avenard [:jya]">jean-yves.avenard</who>
    <bug_when>2024-07-25 23:42:07 -0700</bug_when>
    <thetext>In this page we have videos with posters and have preload=&quot;metadata&quot;.

The webm player doesn&apos;t respect the preload=&quot;metadata&quot; and so will download all the videos immediately.
They aren&apos;t particularly big here, 11.27MB, 74MB, 23MB, 5.6MB, 5.7MB and 11.8MB 

I haven&apos;t been able to reproduce on an iPhone running 21F90 (17.5.1) it takes forever to load but it doesn&apos;t crash.

Dmitry, can you make it crash with the test file attached on an iPhone 12 or 13? is is the attached example just to show what your page typically looks like and there could be much more videos present ?

A workaround for now would be to change 

```
					&lt;video
						preload=&quot;metadata&quot;
						class=&quot;videoPlayer&quot;
						poster=&quot;https://optim.tildacdn.net/tild3162-3030-4635-b934-393766343537/-/format/webp/mike.png&quot;
					&gt;
						&lt;source
							src=&quot;https://practicum-content.s3.us-west-1.amazonaws.com/usa-main/guidance/Tutors/2022_11_03_Videos%20of%20guidance%20team_Mike%20Lee.webm&quot;
							type=&quot;video/webm&quot;
						/&gt;
						&lt;source
							src=&quot;https://practicum-content.s3.us-west-1.amazonaws.com/us-videos/2022_11_03_Videos%20of%20guidance%20team_Mike%20Lee.mp4&quot;
							type=&quot;video/mp4&quot;
						/&gt;
					&lt;/video&gt;
```

into:
					&lt;video
						preload=&quot;metadata&quot;
						class=&quot;videoPlayer&quot;
						poster=&quot;https://optim.tildacdn.net/tild3162-3030-4635-b934-393766343537/-/format/webp/mike.png&quot;
					&gt;
						&lt;source
							src=&quot;https://practicum-content.s3.us-west-1.amazonaws.com/us-videos/2022_11_03_Videos%20of%20guidance%20team_Mike%20Lee.mp4&quot;
							type=&quot;video/mp4&quot;
						/&gt;
						&lt;source
							src=&quot;https://practicum-content.s3.us-west-1.amazonaws.com/usa-main/guidance/Tutors/2022_11_03_Videos%20of%20guidance%20team_Mike%20Lee.webm&quot;
							type=&quot;video/webm&quot;
						/&gt;
					&lt;/video&gt;
```

place the mp4 file first in your list instead of the webm. It will reduce much the networking usage.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2049119</commentid>
    <comment_count>5</comment_count>
    <who name="Jean-Yves Avenard [:jya]">jean-yves.avenard</who>
    <bug_when>2024-07-29 08:56:08 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/31396</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2049414</commentid>
    <comment_count>6</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2024-07-30 01:51:17 -0700</bug_when>
    <thetext>Committed 281554@main (75467caf79bb): &lt;https://commits.webkit.org/281554@main&gt;

Reviewed commits have been landed. Closing PR #31396 and removing active labels.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>471904</attachid>
            <date>2024-07-17 07:53:23 -0700</date>
            <delta_ts>2024-07-17 07:53:23 -0700</delta_ts>
            <desc>Crush demo</desc>
            <filename>index.html</filename>
            <type>text/html</type>
            <size>5262</size>
            <attacher name="Dmitry Vakhnenko">dima</attacher>
            
              <data encoding="base64">PCFET0NUWVBFIGh0bWw+CjxodG1sIGxhbmc9ImVuIj4KCTxoZWFkPgoJCTxtZXRhIGNoYXJzZXQ9
IlVURi04IiAvPgoJCTxtZXRhIG5hbWU9InZpZXdwb3J0IiBjb250ZW50PSJ3aWR0aD1kZXZpY2Ut
d2lkdGgsIGluaXRpYWwtc2NhbGU9MS4wIiAvPgoJCTx0aXRsZT5Eb2N1bWVudDwvdGl0bGU+Cgk8
L2hlYWQ+Cgk8Ym9keT4KCQk8c2VjdGlvbiBjbGFzcz0idmlkZW8iPgoJCQk8ZGl2IGNsYXNzPSJ2
aWRlb19fdGl0bGUiPgoJCQkJQmFja2VkIGJ5IGV4cGVyaWVuY2U6IG91ciB0dXRvcnMgaGF2ZQoJ
CQkJPHNwYW4gY2xhc3M9InZpZGVvX19zcGFuIj41KyZuYnNwO3llYXJzPC9zcGFuPiBpbiB0aGVp
ciBmaWVsZCBhbmQKCQkJCW1lbnRvcnNoaXAgdHJhaW5pbmcgZnJvbSBUcmlwbGVUZW4KCQkJPC9k
aXY+CgkJCTxkaXYgY2xhc3M9InZpZGVvX19jb250YWluZXIiPgoJCQkJPGRpdiBjbGFzcz0idmlk
ZW9fX2l0ZW0iPgoJCQkJCTxidXR0b24gY2xhc3M9InZpZGVvX19idXR0b24iPjwvYnV0dG9uPgoJ
CQkJCTx2aWRlbwoJCQkJCQlwcmVsb2FkPSJtZXRhZGF0YSIKCQkJCQkJY2xhc3M9InZpZGVvUGxh
eWVyIgoJCQkJCQlwb3N0ZXI9Imh0dHBzOi8vb3B0aW0udGlsZGFjZG4ubmV0L3RpbGQzNzMwLTY2
NjMtNDE2NS1iNTM2LTY1NjMzNzY1Mzk2NS8tL2Zvcm1hdC93ZWJwL2FuZHJldy5wbmciCgkJCQkJ
PgoJCQkJCQk8c291cmNlCgkJCQkJCQlzcmM9Imh0dHBzOi8vcHJhY3RpY3VtLWNvbnRlbnQuczMu
dXMtd2VzdC0xLmFtYXpvbmF3cy5jb20vdXNhLW1haW4vZ3VpZGFuY2UvVHV0b3JzL1RyaXBsZVRl
bl9UdXRvcnNfQW5kcmV3LndlYm0iCgkJCQkJCQl0eXBlPSJ2aWRlby93ZWJtIgoJCQkJCQkvPgoJ
CQkJCQk8c291cmNlCgkJCQkJCQlzcmM9Imh0dHBzOi8vcHJhY3RpY3VtLWNvbnRlbnQuczMudXMt
d2VzdC0xLmFtYXpvbmF3cy5jb20vdXNhLW1haW4vZ3VpZGFuY2UvVHJpcGxlVGVuX1R1dG9yc19B
bmRyZXcubXA0IgoJCQkJCQkJdHlwZT0idmlkZW8vbXA0IgoJCQkJCQkvPgoJCQkJCTwvdmlkZW8+
CgkJCQkJPHAgY2xhc3M9InZpZGVvX19uYW1lIj5BbmRyZXc8L3A+CgkJCQkJPHAgY2xhc3M9InZp
ZGVvX19leHBlcmllbmNlIj43IHllYXJzIGV4cGVyaWVuY2U8L3A+CgkJCQkJPHAgY2xhc3M9InZp
ZGVvX19qb2IiPlNvZnR3YXJlIEVuZ2luZWVyIGF0IE1pY3Jvc29mdDwvcD4KCQkJCTwvZGl2PgoJ
CQkJPGRpdiBjbGFzcz0idmlkZW9fX2l0ZW0iPgoJCQkJCTxidXR0b24gY2xhc3M9InZpZGVvX19i
dXR0b24iPjwvYnV0dG9uPgoJCQkJCTx2aWRlbwoJCQkJCQlwcmVsb2FkPSJtZXRhZGF0YSIKCQkJ
CQkJY2xhc3M9InZpZGVvUGxheWVyIgoJCQkJCQlwb3N0ZXI9Imh0dHBzOi8vb3B0aW0udGlsZGFj
ZG4ubmV0L3RpbGQzMDYyLTMyNjItNDEzMS1iNjM4LTM3MzU2MjM4MzYzMC8tL2Zvcm1hdC93ZWJw
L21heC5wbmciCgkJCQkJPgoJCQkJCQk8c291cmNlCgkJCQkJCQlzcmM9Imh0dHBzOi8vcHJhY3Rp
Y3VtLWNvbnRlbnQuczMudXMtd2VzdC0xLmFtYXpvbmF3cy5jb20vdXNhLW1haW4vZ3VpZGFuY2Uv
VHV0b3JzL1RyaXBsZVRlbl9UdXRvcnNfTWF4LndlYm0iCgkJCQkJCQl0eXBlPSJ2aWRlby93ZWJt
IgoJCQkJCQkvPgoJCQkJCQk8c291cmNlCgkJCQkJCQlzcmM9Imh0dHBzOi8vcHJhY3RpY3VtLWNv
bnRlbnQuczMudXMtd2VzdC0xLmFtYXpvbmF3cy5jb20vdXNhLW1haW4vZ3VpZGFuY2UvVHJpcGxl
VGVuX1R1dG9yc19NYXgubXA0IgoJCQkJCQkJdHlwZT0idmlkZW8vbXA0IgoJCQkJCQkvPgoJCQkJ
CTwvdmlkZW8+CgkJCQkJPHAgY2xhc3M9InZpZGVvX19uYW1lIj5NYXg8L3A+CgkJCQkJPHAgY2xh
c3M9InZpZGVvX19leHBlcmllbmNlIj43IHllYXJzIGV4cGVyaWVuY2U8L3A+CgkJCQkJPHAgY2xh
c3M9InZpZGVvX19qb2IiPgoJCQkJCQlTZW5pb3IgU29mdHdhcmUgRW5naW5lZXIgYXQgUmVwdWJs
aWMgU2VydmljZXMKCQkJCQk8L3A+CgkJCQk8L2Rpdj4KCQkJCTxkaXYgY2xhc3M9InZpZGVvX19p
dGVtIj4KCQkJCQk8YnV0dG9uIGNsYXNzPSJ2aWRlb19fYnV0dG9uIj48L2J1dHRvbj4KCQkJCQk8
dmlkZW8KCQkJCQkJcHJlbG9hZD0ibWV0YWRhdGEiCgkJCQkJCWNsYXNzPSJ2aWRlb1BsYXllciIK
CQkJCQkJcG9zdGVyPSJodHRwczovL29wdGltLnRpbGRhY2RuLm5ldC90aWxkMzk2NS0zNTY2LTQx
MzAtYTYzNS02NTY2MzY2MzM2MzMvLS9mb3JtYXQvd2VicC9qb3JnZS5wbmciCgkJCQkJPgoJCQkJ
CQk8c291cmNlCgkJCQkJCQlzcmM9Imh0dHBzOi8vcHJhY3RpY3VtLWNvbnRlbnQuczMudXMtd2Vz
dC0xLmFtYXpvbmF3cy5jb20vdXNhLW1haW4vZ3VpZGFuY2UvVHV0b3JzL1RyaXBsZVRlbl9UdXRv
cnNfSm9yZ2Uud2VibSIKCQkJCQkJCXR5cGU9InZpZGVvL3dlYm0iCgkJCQkJCS8+CgkJCQkJCTxz
b3VyY2UKCQkJCQkJCXNyYz0iaHR0cHM6Ly9wcmFjdGljdW0tY29udGVudC5zMy51cy13ZXN0LTEu
YW1hem9uYXdzLmNvbS91c2EtbWFpbi9ndWlkYW5jZS9UcmlwbGVUZW5fVHV0b3JzX0dlb3JnZS5t
cDQiCgkJCQkJCQl0eXBlPSJ2aWRlby9tcDQiCgkJCQkJCS8+CgkJCQkJPC92aWRlbz4KCQkJCQk8
cCBjbGFzcz0idmlkZW9fX25hbWUiPkpvcmdlPC9wPgoJCQkJCTxwIGNsYXNzPSJ2aWRlb19fZXhw
ZXJpZW5jZSI+OCB5ZWFycyBleHBlcmllbmNlPC9wPgoJCQkJCTxwIGNsYXNzPSJ2aWRlb19fam9i
Ij5Tb2Z0d2FyZSBFbmdpbmVlciBhdCBCdWxsaG9ybjwvcD4KCQkJCTwvZGl2PgoJCQkJPGRpdiBj
bGFzcz0idmlkZW9fX2l0ZW0iPgoJCQkJCTxidXR0b24gY2xhc3M9InZpZGVvX19idXR0b24iPjwv
YnV0dG9uPgoJCQkJCTx2aWRlbwoJCQkJCQlwcmVsb2FkPSJtZXRhZGF0YSIKCQkJCQkJY2xhc3M9
InZpZGVvUGxheWVyIgoJCQkJCQlwb3N0ZXI9Imh0dHBzOi8vb3B0aW0udGlsZGFjZG4ubmV0L3Rp
bGQzMTYyLTMwMzAtNDYzNS1iOTM0LTM5Mzc2NjM0MzUzNy8tL2Zvcm1hdC93ZWJwL21pa2UucG5n
IgoJCQkJCT4KCQkJCQkJPHNvdXJjZQoJCQkJCQkJc3JjPSJodHRwczovL3ByYWN0aWN1bS1jb250
ZW50LnMzLnVzLXdlc3QtMS5hbWF6b25hd3MuY29tL3VzYS1tYWluL2d1aWRhbmNlL1R1dG9ycy8y
MDIyXzExXzAzX1ZpZGVvcyUyMG9mJTIwZ3VpZGFuY2UlMjB0ZWFtX01pa2UlMjBMZWUud2VibSIK
CQkJCQkJCXR5cGU9InZpZGVvL3dlYm0iCgkJCQkJCS8+CgkJCQkJCTxzb3VyY2UKCQkJCQkJCXNy
Yz0iaHR0cHM6Ly9wcmFjdGljdW0tY29udGVudC5zMy51cy13ZXN0LTEuYW1hem9uYXdzLmNvbS91
cy12aWRlb3MvMjAyMl8xMV8wM19WaWRlb3MlMjBvZiUyMGd1aWRhbmNlJTIwdGVhbV9NaWtlJTIw
TGVlLm1wNCIKCQkJCQkJCXR5cGU9InZpZGVvL21wNCIKCQkJCQkJLz4KCQkJCQk8L3ZpZGVvPgoJ
CQkJCTxwIGNsYXNzPSJ2aWRlb19fbmFtZSI+TWlrZTwvcD4KCQkJCQk8cCBjbGFzcz0idmlkZW9f
X2V4cGVyaWVuY2UiPjMgeWVhcnMgZXhwZXJpZW5jZTwvcD4KCQkJCQk8cCBjbGFzcz0idmlkZW9f
X2pvYiI+CgkJCQkJCURpcmVjdG9yIG9mIEFkdmFuY2VkIEFuYWx5dGljcyBhdCBPTEcgYW5kIFNj
aWVuY2UgQ29uc3VsdGFudCBhdAoJCQkJCQlFbGl0ZSBBbmFseXRpY2FsIFNvbHV0aW9ucwoJCQkJ
CTwvcD4KCQkJCTwvZGl2PgoJCQkJPGRpdiBjbGFzcz0idmlkZW9fX2l0ZW0iPgoJCQkJCTxidXR0
b24gY2xhc3M9InZpZGVvX19idXR0b24iPjwvYnV0dG9uPgoJCQkJCTx2aWRlbwoJCQkJCQlwcmVs
b2FkPSJtZXRhZGF0YSIKCQkJCQkJY2xhc3M9InZpZGVvUGxheWVyIgoJCQkJCQlwb3N0ZXI9Imh0
dHBzOi8vb3B0aW0udGlsZGFjZG4ubmV0L3RpbGQzMDY0LTMxMzYtNDM2MS1hNTY1LTYyNjU2NTYx
MzMzMS8tL2Zvcm1hdC93ZWJwL2FudGhvbnkucG5nIgoJCQkJCT4KCQkJCQkJPHNvdXJjZQoJCQkJ
CQkJc3JjPSJodHRwczovL3ByYWN0aWN1bS1jb250ZW50LnMzLnVzLXdlc3QtMS5hbWF6b25hd3Mu
Y29tL3VzYS1tYWluL2d1aWRhbmNlL1R1dG9ycy8yMDIyXzExXzAzX1ZpZGVvcyUyMG9mJTIwZ3Vp
ZGFuY2UlMjB0ZWFtX0FudGhvbnklMjBNb3JnYW5fMDIud2VibSIKCQkJCQkJCXR5cGU9InZpZGVv
L3dlYm0iCgkJCQkJCS8+CgkJCQkJCTxzb3VyY2UKCQkJCQkJCXNyYz0iaHR0cHM6Ly9wcmFjdGlj
dW0tY29udGVudC5zMy51cy13ZXN0LTEuYW1hem9uYXdzLmNvbS91cy12aWRlb3MvMjAyMl8xMV8w
M19WaWRlb3MlMjBvZiUyMGd1aWRhbmNlJTIwdGVhbV9BbnRob255JTIwTW9yZ2FuXzAyLm1wNCIK
CQkJCQkJCXR5cGU9InZpZGVvL21wNCIKCQkJCQkJLz4KCQkJCQk8L3ZpZGVvPgoJCQkJCTxwIGNs
YXNzPSJ2aWRlb19fbmFtZSI+QW50aG9ueTwvcD4KCQkJCQk8cCBjbGFzcz0idmlkZW9fX2V4cGVy
aWVuY2UiPjMgeWVhcnMgZXhwZXJpZW5jZTwvcD4KCQkJCQk8cCBjbGFzcz0idmlkZW9fX2pvYiI+
CgkJCQkJCVNlbmlvciBNYWNoaW5lIExlYXJuaW5nIEVuZ2luZWVyIGFuZCBUZWNobmljYWwgQ29u
c3VsdGFudAoJCQkJCTwvcD4KCQkJCTwvZGl2PgoJCQkJPGRpdiBjbGFzcz0idmlkZW9fX2l0ZW0i
PgoJCQkJCTxidXR0b24gY2xhc3M9InZpZGVvX19idXR0b24iPjwvYnV0dG9uPgoJCQkJCTx2aWRl
bwoJCQkJCQlwcmVsb2FkPSJtZXRhZGF0YSIKCQkJCQkJY2xhc3M9InZpZGVvUGxheWVyIgoJCQkJ
CQlwb3N0ZXI9Imh0dHBzOi8vb3B0aW0udGlsZGFjZG4ubmV0L3RpbGQzNzY2LTM0MzktNDQ2Mi1i
NzM2LTYyNjEzNjMxMzQ2NC8tL2Zvcm1hdC93ZWJwL3NlcmdlLnBuZyIKCQkJCQk+CgkJCQkJCTxz
b3VyY2UKCQkJCQkJCXNyYz0iaHR0cHM6Ly9wcmFjdGljdW0tY29udGVudC5zMy51cy13ZXN0LTEu
YW1hem9uYXdzLmNvbS91c2EtbWFpbi9ndWlkYW5jZS9UdXRvcnMvVHJpcGxlVGVuX1R1dG9yc19T
ZXJnZS53ZWJtIgoJCQkJCQkJdHlwZT0idmlkZW8vd2VibSIKCQkJCQkJLz4KCQkJCQkJPHNvdXJj
ZQoJCQkJCQkJc3JjPSJodHRwczovL3ByYWN0aWN1bS1jb250ZW50LnMzLnVzLXdlc3QtMS5hbWF6
b25hd3MuY29tL3VzYS1tYWluL2d1aWRhbmNlL1RyaXBsZVRlbl9UdXRvcnNfU2VyZ2UubXA0IgoJ
CQkJCQkJdHlwZT0idmlkZW8vbXA0IgoJCQkJCQkvPgoJCQkJCTwvdmlkZW8+CgkJCQkJPHAgY2xh
c3M9InZpZGVvX19uYW1lIj5TZXJnZTwvcD4KCQkJCQk8cCBjbGFzcz0idmlkZW9fX2V4cGVyaWVu
Y2UiPjEyIHllYXJzIGV4cGVyaWVuY2U8L3A+CgkJCQkJPHAgY2xhc3M9InZpZGVvX19qb2IiPkRl
diBUZWFtIExlYWQgYXQgV29sdDwvcD4KCQkJCTwvZGl2PgoJCQk8L2Rpdj4KCQk8L3NlY3Rpb24+
Cgk8L2JvZHk+CjwvaHRtbD4K
</data>

          </attachment>
      

    </bug>

</bugzilla>