<?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>226644</bug_id>
          
          <creation_ts>2021-06-04 08:41:58 -0700</creation_ts>
          <short_desc>Avoid wasteful large allocation in SpeechRecognitionRemoteRealtimeMediaSource::setStorage()</short_desc>
          <delta_ts>2021-06-04 10:15:21 -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>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=226253</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Chris Dumez">cdumez</reporter>
          <assigned_to name="Chris Dumez">cdumez</assigned_to>
          <cc>eric.carlson</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>glenn</cc>
    
    <cc>jer.noble</cc>
    
    <cc>peng.liu6</cc>
    
    <cc>philipj</cc>
    
    <cc>rbuis</cc>
    
    <cc>sergio</cc>
    
    <cc>sihui_liu</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>youennf</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1766780</commentid>
    <comment_count>0</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2021-06-04 08:41:58 -0700</bug_when>
    <thetext>Avoid wasteful large allocation in SpeechRecognitionRemoteRealtimeMediaSource::setStorage().
setStorage() sets m_buffer&apos;s size to the same size as the RingBuffer (2*sampleRate), only to resize it to the size of the rendering quantum (512) later in SpeechRecognitionRemoteRealtimeMediaSource::remoteAudioSamplesAvailable().</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1766782</commentid>
    <comment_count>1</comment_count>
      <attachid>430582</attachid>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2021-06-04 08:43:48 -0700</bug_when>
    <thetext>Created attachment 430582
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1766819</commentid>
    <comment_count>2</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2021-06-04 10:14:31 -0700</bug_when>
    <thetext>Committed r278471 (238489@main): &lt;https://commits.webkit.org/238489@main&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 430582.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1766820</commentid>
    <comment_count>3</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2021-06-04 10:15:21 -0700</bug_when>
    <thetext>&lt;rdar://problem/78874517&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>430582</attachid>
            <date>2021-06-04 08:43:48 -0700</date>
            <delta_ts>2021-06-04 10:14:32 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-226644-20210604084347.patch</filename>
            <type>text/plain</type>
            <size>2025</size>
            <attacher name="Chris Dumez">cdumez</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjc4NDQxCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCmluZGV4IDY5OGRjOWFlOGRiZDFjZDdm
NzFhMmUzZTZhNGUwYTRkODViODY1MzIuLjMyYmI0NjE1YjlhZjdjOWFmMWU5Y2Y5NzEwMTc5MmVh
YzU1ZDYzZmUgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJj
ZS9XZWJLaXQvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTggQEAKKzIwMjEtMDYtMDQgIENocmlzIER1
bWV6ICA8Y2R1bWV6QGFwcGxlLmNvbT4KKworICAgICAgICBBdm9pZCB3YXN0ZWZ1bCBsYXJnZSBh
bGxvY2F0aW9uIGluIFNwZWVjaFJlY29nbml0aW9uUmVtb3RlUmVhbHRpbWVNZWRpYVNvdXJjZTo6
c2V0U3RvcmFnZSgpCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn
aT9pZD0yMjY2NDQKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAg
ICAgICBBdm9pZCB3YXN0ZWZ1bCBsYXJnZSBhbGxvY2F0aW9uIGluIFNwZWVjaFJlY29nbml0aW9u
UmVtb3RlUmVhbHRpbWVNZWRpYVNvdXJjZTo6c2V0U3RvcmFnZSgpLgorICAgICAgICBzZXRTdG9y
YWdlKCkgd2FzIHNldHRpbmcgbV9idWZmZXIncyBzaXplIHRvIHRoZSBzYW1lIHNpemUgYXMgdGhl
IFJpbmdCdWZmZXIgKDIqc2FtcGxlUmF0ZSksCisgICAgICAgIG9ubHkgdG8gcmVzaXplIGl0IHRv
IHRoZSBzaXplIG9mIHRoZSByZW5kZXJpbmcgcXVhbnR1bSAoNTEyKSBsYXRlciBpbgorICAgICAg
ICBTcGVlY2hSZWNvZ25pdGlvblJlbW90ZVJlYWx0aW1lTWVkaWFTb3VyY2U6OnJlbW90ZUF1ZGlv
U2FtcGxlc0F2YWlsYWJsZSgpLgorCisgICAgICAgICogVUlQcm9jZXNzL1NwZWVjaFJlY29nbml0
aW9uUmVtb3RlUmVhbHRpbWVNZWRpYVNvdXJjZS5jcHA6CisgICAgICAgIChXZWJLaXQ6OlNwZWVj
aFJlY29nbml0aW9uUmVtb3RlUmVhbHRpbWVNZWRpYVNvdXJjZTo6c2V0U3RvcmFnZSk6CisKIDIw
MjEtMDYtMDMgIENocmlzIER1bWV6ICA8Y2R1bWV6QGFwcGxlLmNvbT4KIAogICAgICAgICBVc2Ug
VmVjdG9yPHVpbnQ4X3Q+IGluc3RlYWQgb2YgVmVjdG9yPGNoYXI+IHRvIHN0b3JlIGJ5dGVzIGlu
IFNoYXJlZEJ1ZmZlcgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdC9VSVByb2Nlc3MvU3BlZWNo
UmVjb2duaXRpb25SZW1vdGVSZWFsdGltZU1lZGlhU291cmNlLmNwcCBiL1NvdXJjZS9XZWJLaXQv
VUlQcm9jZXNzL1NwZWVjaFJlY29nbml0aW9uUmVtb3RlUmVhbHRpbWVNZWRpYVNvdXJjZS5jcHAK
aW5kZXggMDRjOTg0ZjY1ZmRlZTVhODdjOGUyN2Q5NjBiMTZhYmVmYTEzYWFjZS4uZTQyZGJmMTI4
MWQzNmZlZjJlMzAzZTcwN2Q1YjViYWYxNjZlZTE1MiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktp
dC9VSVByb2Nlc3MvU3BlZWNoUmVjb2duaXRpb25SZW1vdGVSZWFsdGltZU1lZGlhU291cmNlLmNw
cAorKysgYi9Tb3VyY2UvV2ViS2l0L1VJUHJvY2Vzcy9TcGVlY2hSZWNvZ25pdGlvblJlbW90ZVJl
YWx0aW1lTWVkaWFTb3VyY2UuY3BwCkBAIC04MCw3ICs4MCw3IEBAIHZvaWQgU3BlZWNoUmVjb2du
aXRpb25SZW1vdGVSZWFsdGltZU1lZGlhU291cmNlOjpzZXRTdG9yYWdlKGNvbnN0IFNoYXJlZE1l
bW9yeTo6CiAgICAgbV9kZXNjcmlwdGlvbiA9IGRlc2NyaXB0aW9uOwogCiAgICAgbV9yaW5nQnVm
ZmVyID0gV2ViQ29yZTo6Q0FSaW5nQnVmZmVyOjphZG9wdFN0b3JhZ2UobWFrZVVuaXF1ZVJlZjxS
ZWFkT25seVNoYXJlZFJpbmdCdWZmZXJTdG9yYWdlPihoYW5kbGUpLCBkZXNjcmlwdGlvbiwgbnVt
YmVyT2ZGcmFtZXMpLm1vdmVUb1VuaXF1ZVB0cigpOwotICAgIG1fYnVmZmVyID0gbWFrZVVuaXF1
ZTxXZWJDb3JlOjpXZWJBdWRpb0J1ZmZlckxpc3Q+KGRlc2NyaXB0aW9uLCBudW1iZXJPZkZyYW1l
cyk7CisgICAgbV9idWZmZXIgPSBtYWtlVW5pcXVlPFdlYkNvcmU6OldlYkF1ZGlvQnVmZmVyTGlz
dD4oZGVzY3JpcHRpb24pOwogfQogCiAjZW5kaWYK
</data>

          </attachment>
      

    </bug>

</bugzilla>