<?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>205822</bug_id>
          
          <creation_ts>2020-01-06 11:40:53 -0800</creation_ts>
          <short_desc>[iOS] Only prewarm fonts with valid font names</short_desc>
          <delta_ts>2020-01-10 14:43:47 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit Misc.</component>
          <version>WebKit Nightly 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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Per Arne Vollan">pvollan</reporter>
          <assigned_to name="Per Arne Vollan">pvollan</assigned_to>
          <cc>bfulgham</cc>
    
    <cc>commit-queue</cc>
    
    <cc>mmaxfield</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1603182</commentid>
    <comment_count>0</comment_count>
    <who name="Per Arne Vollan">pvollan</who>
    <bug_when>2020-01-06 11:40:53 -0800</bug_when>
    <thetext>The font names &quot;.SF NS Text&quot; and &quot;.SF NS Display&quot; are not valid on iOS.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1603183</commentid>
    <comment_count>1</comment_count>
      <attachid>386872</attachid>
    <who name="Per Arne Vollan">pvollan</who>
    <bug_when>2020-01-06 11:43:13 -0800</bug_when>
    <thetext>Created attachment 386872
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1603198</commentid>
    <comment_count>2</comment_count>
      <attachid>386872</attachid>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2020-01-06 12:13:01 -0800</bug_when>
    <thetext>Comment on attachment 386872
Patch

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

r=me, but I wonder if that conditional should also allow MacCatalyst.

&gt; Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:1697
&gt; +#if PLATFORM(MAC)

What about MacCatalyst?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1603219</commentid>
    <comment_count>3</comment_count>
      <attachid>386878</attachid>
    <who name="Per Arne Vollan">pvollan</who>
    <bug_when>2020-01-06 12:49:33 -0800</bug_when>
    <thetext>Created attachment 386878
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1603221</commentid>
    <comment_count>4</comment_count>
    <who name="Per Arne Vollan">pvollan</who>
    <bug_when>2020-01-06 12:50:18 -0800</bug_when>
    <thetext>(In reply to Brent Fulgham from comment #2)
&gt; Comment on attachment 386872 [details]
&gt; Patch
&gt; 
&gt; View in context:
&gt; https://bugs.webkit.org/attachment.cgi?id=386872&amp;action=review
&gt; 
&gt; r=me, but I wonder if that conditional should also allow MacCatalyst.
&gt; 
&gt; &gt; Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:1697
&gt; &gt; +#if PLATFORM(MAC)
&gt; 
&gt; What about MacCatalyst?

Fixed.

Thanks for reviewing!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1603257</commentid>
    <comment_count>5</comment_count>
      <attachid>386878</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2020-01-06 14:00:45 -0800</bug_when>
    <thetext>Comment on attachment 386878
Patch

Rejecting attachment 386878 from commit-queue.

Failed to run &quot;[&apos;/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch&apos;, &apos;--status-host=webkit-queues.webkit.org&apos;, &apos;--bot-id=webkit-cq-01&apos;, &apos;validate-changelog&apos;, &apos;--check-oops&apos;, &apos;--non-interactive&apos;, 386878, &apos;--port=mac&apos;]&quot; exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

ChangeLog entry in Source/WebCore/ChangeLog contains OOPS!.

Full output: https://webkit-queues.webkit.org/results/13300244</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1603285</commentid>
    <comment_count>6</comment_count>
      <attachid>386890</attachid>
    <who name="Per Arne Vollan">pvollan</who>
    <bug_when>2020-01-06 14:45:49 -0800</bug_when>
    <thetext>Created attachment 386890
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1603374</commentid>
    <comment_count>7</comment_count>
      <attachid>386890</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2020-01-06 16:49:39 -0800</bug_when>
    <thetext>Comment on attachment 386890
Patch

Clearing flags on attachment: 386890

Committed r254099: &lt;https://trac.webkit.org/changeset/254099&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1603422</commentid>
    <comment_count>8</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2020-01-06 20:10:39 -0800</bug_when>
    <thetext>&lt;rdar://problem/58362968&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>386872</attachid>
            <date>2020-01-06 11:43:13 -0800</date>
            <delta_ts>2020-01-06 12:13:01 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-205822-20200106114313.patch</filename>
            <type>text/plain</type>
            <size>1382</size>
            <attacher name="Per Arne Vollan">pvollan</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDI1NDA3MSkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE3IEBACisyMDIwLTAxLTA2ICBQZXIgQXJu
ZSBWb2xsYW4gIDxwdm9sbGFuQGFwcGxlLmNvbT4KKworICAgICAgICBbaU9TXSBPbmx5IHByZXdh
cm0gZm9udHMgd2l0aCB2YWxpZCBmb250IG5hbWVzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJr
aXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMDU4MjIKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JP
RFkgKE9PUFMhKS4KKworICAgICAgICBUaGUgZm9udCBuYW1lcyAiLlNGIE5TIFRleHQiIGFuZCAi
LlNGIE5TIERpc3BsYXkiIGFyZSBub3QgdmFsaWQgb24gaU9TLCBhbmQgc2hvdWxkIG5vdCBiZSBw
cmV3YXJtZWQuCisKKyAgICAgICAgTm8gbmV3IHRlc3RzLCBubyBiZWhhdmlvciBjaGFuZ2UuCisK
KyAgICAgICAgKiBwbGF0Zm9ybS9ncmFwaGljcy9jb2NvYS9Gb250Q2FjaGVDb3JlVGV4dC5jcHA6
CisgICAgICAgIChXZWJDb3JlOjpGb250Q2FjaGU6OnByZXdhcm1HbG9iYWxseSk6CisKIDIwMjAt
MDEtMDYgIFBlciBBcm5lIFZvbGxhbiAgPHB2b2xsYW5AYXBwbGUuY29tPgogCiAgICAgICAgIFJF
R1JFU1NJT04ocjI0NzYyNik6IEludHJvZHVjZWQgbWVtb3J5IHJlZ3Jlc3Npb24KSW5kZXg6IFNv
dXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2NvY29hL0ZvbnRDYWNoZUNvcmVUZXh0LmNw
cAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9jb2NvYS9G
b250Q2FjaGVDb3JlVGV4dC5jcHAJKHJldmlzaW9uIDI1NDA2NykKKysrIFNvdXJjZS9XZWJDb3Jl
L3BsYXRmb3JtL2dyYXBoaWNzL2NvY29hL0ZvbnRDYWNoZUNvcmVUZXh0LmNwcAkod29ya2luZyBj
b3B5KQpAQCAtMTY5NCw4ICsxNjk0LDEwIEBAIHZvaWQgRm9udENhY2hlOjpwcmV3YXJtR2xvYmFs
bHkoKQogICAgICAgICByZXR1cm47CiAKICAgICBWZWN0b3I8U3RyaW5nPiBmYW1pbGllcyA9IHN0
ZDo6aW5pdGlhbGl6ZXJfbGlzdDxTdHJpbmc+IHsKKyNpZiBQTEFURk9STShNQUMpCiAgICAgICAg
ICIuU0YgTlMgVGV4dCJfcywKICAgICAgICAgIi5TRiBOUyBEaXNwbGF5Il9zLAorI2VuZGlmCiAg
ICAgICAgICJBcmlhbCJfcywKICAgICAgICAgIkhlbHZldGljYSJfcywKICAgICAgICAgIkhlbHZl
dGljYSBOZXVlIl9zLAo=
</data>
<flag name="review"
          id="402627"
          type_id="1"
          status="+"
          setter="bfulgham"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>386878</attachid>
            <date>2020-01-06 12:49:33 -0800</date>
            <delta_ts>2020-01-06 14:46:42 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-205822-20200106124933.patch</filename>
            <type>text/plain</type>
            <size>1407</size>
            <attacher name="Per Arne Vollan">pvollan</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDI1NDA3MSkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE3IEBACisyMDIwLTAxLTA2ICBQZXIgQXJu
ZSBWb2xsYW4gIDxwdm9sbGFuQGFwcGxlLmNvbT4KKworICAgICAgICBbaU9TXSBPbmx5IHByZXdh
cm0gZm9udHMgd2l0aCB2YWxpZCBmb250IG5hbWVzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJr
aXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMDU4MjIKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JP
RFkgKE9PUFMhKS4KKworICAgICAgICBUaGUgZm9udCBuYW1lcyAiLlNGIE5TIFRleHQiIGFuZCAi
LlNGIE5TIERpc3BsYXkiIGFyZSBub3QgdmFsaWQgb24gaU9TLCBhbmQgc2hvdWxkIG5vdCBiZSBw
cmV3YXJtZWQuCisKKyAgICAgICAgTm8gbmV3IHRlc3RzLCBubyBiZWhhdmlvciBjaGFuZ2UuCisK
KyAgICAgICAgKiBwbGF0Zm9ybS9ncmFwaGljcy9jb2NvYS9Gb250Q2FjaGVDb3JlVGV4dC5jcHA6
CisgICAgICAgIChXZWJDb3JlOjpGb250Q2FjaGU6OnByZXdhcm1HbG9iYWxseSk6CisKIDIwMjAt
MDEtMDYgIFBlciBBcm5lIFZvbGxhbiAgPHB2b2xsYW5AYXBwbGUuY29tPgogCiAgICAgICAgIFJF
R1JFU1NJT04ocjI0NzYyNik6IEludHJvZHVjZWQgbWVtb3J5IHJlZ3Jlc3Npb24KSW5kZXg6IFNv
dXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2NvY29hL0ZvbnRDYWNoZUNvcmVUZXh0LmNw
cAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9jb2NvYS9G
b250Q2FjaGVDb3JlVGV4dC5jcHAJKHJldmlzaW9uIDI1NDA2NykKKysrIFNvdXJjZS9XZWJDb3Jl
L3BsYXRmb3JtL2dyYXBoaWNzL2NvY29hL0ZvbnRDYWNoZUNvcmVUZXh0LmNwcAkod29ya2luZyBj
b3B5KQpAQCAtMTY5NCw4ICsxNjk0LDEwIEBAIHZvaWQgRm9udENhY2hlOjpwcmV3YXJtR2xvYmFs
bHkoKQogICAgICAgICByZXR1cm47CiAKICAgICBWZWN0b3I8U3RyaW5nPiBmYW1pbGllcyA9IHN0
ZDo6aW5pdGlhbGl6ZXJfbGlzdDxTdHJpbmc+IHsKKyNpZiBQTEFURk9STShNQUMpIHx8IFBMQVRG
T1JNKE1BQ0NBVEFMWVNUKQogICAgICAgICAiLlNGIE5TIFRleHQiX3MsCiAgICAgICAgICIuU0Yg
TlMgRGlzcGxheSJfcywKKyNlbmRpZgogICAgICAgICAiQXJpYWwiX3MsCiAgICAgICAgICJIZWx2
ZXRpY2EiX3MsCiAgICAgICAgICJIZWx2ZXRpY2EgTmV1ZSJfcywK
</data>
<flag name="commit-queue"
          id="402639"
          type_id="3"
          status="-"
          setter="commit-queue"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>386890</attachid>
            <date>2020-01-06 14:45:49 -0800</date>
            <delta_ts>2020-01-06 16:49:39 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-205822-20200106144549.patch</filename>
            <type>text/plain</type>
            <size>1406</size>
            <attacher name="Per Arne Vollan">pvollan</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDI1NDA3MSkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE3IEBACisyMDIwLTAxLTA2ICBQZXIgQXJu
ZSBWb2xsYW4gIDxwdm9sbGFuQGFwcGxlLmNvbT4KKworICAgICAgICBbaU9TXSBPbmx5IHByZXdh
cm0gZm9udHMgd2l0aCB2YWxpZCBmb250IG5hbWVzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJr
aXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMDU4MjIKKworICAgICAgICBSZXZpZXdlZCBieSBCcmVu
dCBGdWxnaGFtLgorCisgICAgICAgIFRoZSBmb250IG5hbWVzICIuU0YgTlMgVGV4dCIgYW5kICIu
U0YgTlMgRGlzcGxheSIgYXJlIG5vdCB2YWxpZCBvbiBpT1MsIGFuZCBzaG91bGQgbm90IGJlIHBy
ZXdhcm1lZC4KKworICAgICAgICBObyBuZXcgdGVzdHMsIG5vIGJlaGF2aW9yIGNoYW5nZS4KKwor
ICAgICAgICAqIHBsYXRmb3JtL2dyYXBoaWNzL2NvY29hL0ZvbnRDYWNoZUNvcmVUZXh0LmNwcDoK
KyAgICAgICAgKFdlYkNvcmU6OkZvbnRDYWNoZTo6cHJld2FybUdsb2JhbGx5KToKKwogMjAyMC0w
MS0wNiAgUGVyIEFybmUgVm9sbGFuICA8cHZvbGxhbkBhcHBsZS5jb20+CiAKICAgICAgICAgUkVH
UkVTU0lPTihyMjQ3NjI2KTogSW50cm9kdWNlZCBtZW1vcnkgcmVncmVzc2lvbgpJbmRleDogU291
cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvY29jb2EvRm9udENhY2hlQ29yZVRleHQuY3Bw
Cj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2NvY29hL0Zv
bnRDYWNoZUNvcmVUZXh0LmNwcAkocmV2aXNpb24gMjU0MDY3KQorKysgU291cmNlL1dlYkNvcmUv
cGxhdGZvcm0vZ3JhcGhpY3MvY29jb2EvRm9udENhY2hlQ29yZVRleHQuY3BwCSh3b3JraW5nIGNv
cHkpCkBAIC0xNjk0LDggKzE2OTQsMTAgQEAgdm9pZCBGb250Q2FjaGU6OnByZXdhcm1HbG9iYWxs
eSgpCiAgICAgICAgIHJldHVybjsKIAogICAgIFZlY3RvcjxTdHJpbmc+IGZhbWlsaWVzID0gc3Rk
Ojppbml0aWFsaXplcl9saXN0PFN0cmluZz4geworI2lmIFBMQVRGT1JNKE1BQykgfHwgUExBVEZP
Uk0oTUFDQ0FUQUxZU1QpCiAgICAgICAgICIuU0YgTlMgVGV4dCJfcywKICAgICAgICAgIi5TRiBO
UyBEaXNwbGF5Il9zLAorI2VuZGlmCiAgICAgICAgICJBcmlhbCJfcywKICAgICAgICAgIkhlbHZl
dGljYSJfcywKICAgICAgICAgIkhlbHZldGljYSBOZXVlIl9zLAo=
</data>

          </attachment>
      

    </bug>

</bugzilla>