<?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>233976</bug_id>
          
          <creation_ts>2021-12-07 22:19:43 -0800</creation_ts>
          <short_desc>Assert isMainThread() in hasValidAverageCharWidth</short_desc>
          <delta_ts>2022-07-19 21:58:49 -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>Layout and Rendering</component>
          <version>WebKit Local Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <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>
          
          <blocked>233488</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Cameron McCormack (:heycam)">heycam</reporter>
          <assigned_to name="Cameron McCormack (:heycam)">heycam</assigned_to>
          <cc>bfulgham</cc>
    
    <cc>mattwoodrow</cc>
    
    <cc>mmaxfield</cc>
    
    <cc>simon.fraser</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>zalan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1821385</commentid>
    <comment_count>0</comment_count>
    <who name="Cameron McCormack (:heycam)">heycam</who>
    <bug_when>2021-12-07 22:19:43 -0800</bug_when>
    <thetext>.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1821387</commentid>
    <comment_count>1</comment_count>
      <attachid>446298</attachid>
    <who name="Cameron McCormack (:heycam)">heycam</who>
    <bug_when>2021-12-07 22:21:18 -0800</bug_when>
    <thetext>Created attachment 446298
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1821989</commentid>
    <comment_count>2</comment_count>
      <attachid>446298</attachid>
    <who name="Myles C. Maxfield">mmaxfield</who>
    <bug_when>2021-12-08 21:05:08 -0800</bug_when>
    <thetext>Comment on attachment 446298
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=446298&amp;action=review

&gt; Source/WebCore/platform/graphics/FontCascade.cpp:344
&gt; +    ASSERT(isMainThread());

This is fine, but why not any of the other functions in FontCascade?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1821991</commentid>
    <comment_count>3</comment_count>
    <who name="Cameron McCormack (:heycam)">heycam</who>
    <bug_when>2021-12-08 21:09:04 -0800</bug_when>
    <thetext>(In reply to Myles C. Maxfield from comment #2)
&gt; This is fine, but why not any of the other functions in FontCascade?

Because most FontCascade functions are called from worker threads too.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1824250</commentid>
    <comment_count>4</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2021-12-14 22:20:17 -0800</bug_when>
    <thetext>&lt;rdar://problem/86506432&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1885215</commentid>
    <comment_count>5</comment_count>
      <attachid>461021</attachid>
    <who name="Matt Woodrow">mattwoodrow</who>
    <bug_when>2022-07-19 15:48:06 -0700</bug_when>
    <thetext>Created attachment 461021
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1885302</commentid>
    <comment_count>6</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2022-07-19 21:58:47 -0700</bug_when>
    <thetext>Committed 252634@main (4c3c8c892f78): &lt;https://commits.webkit.org/252634@main&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 461021.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>446298</attachid>
            <date>2021-12-07 22:21:18 -0800</date>
            <delta_ts>2021-12-08 21:05:08 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-233976-20211208172117.patch</filename>
            <type>text/plain</type>
            <size>1467</size>
            <attacher name="Cameron McCormack (:heycam)">heycam</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjg2NjM3CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNzJhMTBjYzUwZWRjZDBm
ZTJjYmJiMjEwODYzMzNmMjhhODE2NDgyOS4uZDI1NThlY2JkYWVkNTVmNWU1OWRjYmU4NjdmNjI4
ZTE4MDBlMGI5NSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDIxLTEyLTAyICBDYW1l
cm9uIE1jQ29ybWFjayAgPGhleWNhbUBhcHBsZS5jb20+CisKKyAgICAgICAgQXNzZXJ0IGlzTWFp
blRocmVhZCgpIGluIGhhc1ZhbGlkQXZlcmFnZUNoYXJXaWR0aAorICAgICAgICBodHRwczovL2J1
Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjMzOTc2CisKKyAgICAgICAgUmV2aWV3ZWQg
YnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVGhpcyBmdW5jdGlvbiBpcyBvbmx5IGNhbGxl
ZCBmcm9tIHJlbmRlciBvYmplY3RzLCBhbmQgc28gZG9lcyBub3QgbmVlZAorICAgICAgICB0byBi
ZSBtYWRlIHRocmVhZCBzYWZlIGZvciBPZmZzY3JlZW5DYW52YXMuCisKKyAgICAgICAgKiBwbGF0
Zm9ybS9ncmFwaGljcy9Gb250Q2FzY2FkZS5jcHA6CisgICAgICAgIChXZWJDb3JlOjpGb250Q2Fz
Y2FkZTo6aGFzVmFsaWRBdmVyYWdlQ2hhcldpZHRoIGNvbnN0KToKKwogMjAyMS0xMi0wNyAgQ2Ft
ZXJvbiBNY0Nvcm1hY2sgIDxoZXljYW1AYXBwbGUuY29tPgogCiAgICAgICAgIE1ha2UgdGhlIHVz
ZUJhY2tzbGFzaEFzWWVuU2lnbkZvckZhbWlseSB0YWJsZSB0aHJlYWQgc2FmZQpkaWZmIC0tZ2l0
IGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvRm9udENhc2NhZGUuY3BwIGIvU291
cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvRm9udENhc2NhZGUuY3BwCmluZGV4IGIzYzE0
ZmUxMmVjM2RjNmY3NmE0MzM3YmVkMzQ2M2Y5OTgzYWUwNDYuLjM1MjBkNWY2Y2JjMGU5YWYwODQ4
YjZhODFkYmI5ZTQzNjFlNDZiMTggMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3Jt
L2dyYXBoaWNzL0ZvbnRDYXNjYWRlLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9n
cmFwaGljcy9Gb250Q2FzY2FkZS5jcHAKQEAgLTM0MSw2ICszNDEsOCBAQCBHbHlwaERhdGEgRm9u
dENhc2NhZGU6OmdseXBoRGF0YUZvckNoYXJhY3RlcihVQ2hhcjMyIGMsIGJvb2wgbWlycm9yLCBG
b250VmFyaWFudAogLy8gYWxsIHBsYXRmb3Jtcy4KIGJvb2wgRm9udENhc2NhZGU6Omhhc1ZhbGlk
QXZlcmFnZUNoYXJXaWR0aCgpIGNvbnN0CiB7CisgICAgQVNTRVJUKGlzTWFpblRocmVhZCgpKTsK
KwogICAgIGNvbnN0IEF0b21TdHJpbmcmIGZhbWlseSA9IGZpcnN0RmFtaWx5KCk7CiAgICAgaWYg
KGZhbWlseS5pc0VtcHR5KCkpCiAgICAgICAgIHJldHVybiBmYWxzZTsK
</data>
<flag name="review"
          id="471658"
          type_id="1"
          status="+"
          setter="mmaxfield"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>461021</attachid>
            <date>2022-07-19 15:48:06 -0700</date>
            <delta_ts>2022-07-19 21:58:48 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-233976-20220720104804.patch</filename>
            <type>text/plain</type>
            <size>1170</size>
            <attacher name="Matt Woodrow">mattwoodrow</attacher>
            
              <data encoding="base64">RnJvbSBlOGZlOWEzNTFkZTQwYWZlZjc1ZDU2NWNlMmEyYTVmZmM0NjkzNDc0IE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBDYW1lcm9uIE1jQ29ybWFjayA8aGV5Y2FtQGFwcGxlLmNvbT4K
RGF0ZTogVHVlLCAyMyBOb3YgMjAyMSAxNjowNjoxNiArMTEwMApTdWJqZWN0OiBbUEFUQ0hdIEFz
c2VydCBpc01haW5UaHJlYWQoKSBpbiBoYXNWYWxpZEF2ZXJhZ2VDaGFyV2lkdGgKIGh0dHBzOi8v
YnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMzM5NzYKClJldmlld2VkIGJ5IE15bGVz
IEMuIE1heGZpZWxkLgoKVGhpcyBmdW5jdGlvbiBpcyBvbmx5IGNhbGxlZCBmcm9tIHJlbmRlciBv
YmplY3RzLCBhbmQgc28gZG9lcyBub3QgbmVlZAp0byBiZSBtYWRlIHRocmVhZCBzYWZlIGZvciBP
ZmZzY3JlZW5DYW52YXMuCgoqIHBsYXRmb3JtL2dyYXBoaWNzL0ZvbnRDYXNjYWRlLmNwcDoKKFdl
YkNvcmU6OkZvbnRDYXNjYWRlOjpoYXNWYWxpZEF2ZXJhZ2VDaGFyV2lkdGggY29uc3QpOgotLS0K
IFNvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL0ZvbnRDYXNjYWRlLmNwcCB8IDIgKysK
IDEgZmlsZSBjaGFuZ2VkLCAyIGluc2VydGlvbnMoKykKCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2Vi
Q29yZS9wbGF0Zm9ybS9ncmFwaGljcy9Gb250Q2FzY2FkZS5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9w
bGF0Zm9ybS9ncmFwaGljcy9Gb250Q2FzY2FkZS5jcHAKaW5kZXggYWM0ZWZjZjA1NDNiLi43NDI4
ODUxOGRjOWIgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL0Zv
bnRDYXNjYWRlLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9Gb250
Q2FzY2FkZS5jcHAKQEAgLTMzOSw2ICszMzksOCBAQCBHbHlwaERhdGEgRm9udENhc2NhZGU6Omds
eXBoRGF0YUZvckNoYXJhY3RlcihVQ2hhcjMyIGMsIGJvb2wgbWlycm9yLCBGb250VmFyaWFudAog
Ly8gYWxsIHBsYXRmb3Jtcy4KIGJvb2wgRm9udENhc2NhZGU6Omhhc1ZhbGlkQXZlcmFnZUNoYXJX
aWR0aCgpIGNvbnN0CiB7CisgICAgQVNTRVJUKGlzTWFpblRocmVhZCgpKTsKKwogICAgIGNvbnN0
IEF0b21TdHJpbmcmIGZhbWlseSA9IGZpcnN0RmFtaWx5KCk7CiAgICAgaWYgKGZhbWlseS5pc0Vt
cHR5KCkpCiAgICAgICAgIHJldHVybiBmYWxzZTsK
</data>

          </attachment>
      

    </bug>

</bugzilla>