<?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>32273</bug_id>
          
          <creation_ts>2009-12-08 05:08:06 -0800</creation_ts>
          <short_desc>add ANDROID to STORE_FONT_CUSTOM_PLATFORM_DATA</short_desc>
          <delta_ts>2010-07-12 08:06:51 -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>Platform</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Android</rep_platform>
          <op_sys>Android</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>0</everconfirmed>
          <reporter name="Mike Reed">mikerreed</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>android-webkit-unforking</cc>
    
    <cc>commit-queue</cc>
    
    <cc>jorlow</cc>
    
    <cc>steveblock</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>169648</commentid>
    <comment_count>0</comment_count>
    <who name="Mike Reed">mikerreed</who>
    <bug_when>2009-12-08 05:08:06 -0800</bug_when>
    <thetext>ANDROID can support STORE_FONT_CUSTOM_PLATFORM_DATA, so I am adding its platform to the list at the head of CachedFont.cpp</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>170761</commentid>
    <comment_count>1</comment_count>
      <attachid>44689</attachid>
    <who name="Mike Reed">mikerreed</who>
    <bug_when>2009-12-11 07:54:03 -0800</bug_when>
    <thetext>Created attachment 44689
this fixes the issue, by adding PLATFORM(ANDROID) to the check in CachedFont.cpp</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>170762</commentid>
    <comment_count>2</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2009-12-11 07:56:02 -0800</bug_when>
    <thetext>style-queue ran check-webkit-style on attachment 44689 without any errors.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>170767</commentid>
    <comment_count>3</comment_count>
      <attachid>44689</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2009-12-11 08:12:58 -0800</bug_when>
    <thetext>Comment on attachment 44689
this fixes the issue, by adding PLATFORM(ANDROID) to the check in CachedFont.cpp

Change is fine. Can&apos;t land without a change log, though, so review-.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>170789</commentid>
    <comment_count>4</comment_count>
      <attachid>44692</attachid>
    <who name="Mike Reed">mikerreed</who>
    <bug_when>2009-12-11 09:37:54 -0800</bug_when>
    <thetext>Created attachment 44692
add PLATFORM(ANDROID)

same as previous patch, but this one includes the ChangeLog</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>237359</commentid>
    <comment_count>5</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2010-06-12 15:51:39 -0700</bug_when>
    <thetext>This patch has been overlooked, because it isn&apos;t marked for review. Please see &lt;http://webkit.org/coding/contributing.html&gt; for more information.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>248114</commentid>
    <comment_count>6</comment_count>
      <attachid>60860</attachid>
    <who name="Steve Block">steveblock</who>
    <bug_when>2010-07-08 02:54:19 -0700</bug_when>
    <thetext>Created attachment 60860
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>248116</commentid>
    <comment_count>7</comment_count>
    <who name="Steve Block">steveblock</who>
    <bug_when>2010-07-08 02:55:21 -0700</bug_when>
    <thetext>Taking over this patch from Mike.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>249835</commentid>
    <comment_count>8</comment_count>
      <attachid>60860</attachid>
    <who name="Jeremy Orlow">jorlow</who>
    <bug_when>2010-07-12 07:47:25 -0700</bug_when>
    <thetext>Comment on attachment 60860
Patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>249844</commentid>
    <comment_count>9</comment_count>
      <attachid>60860</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-07-12 08:06:46 -0700</bug_when>
    <thetext>Comment on attachment 60860
Patch

Clearing flags on attachment: 60860

Committed r63088: &lt;http://trac.webkit.org/changeset/63088&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>249845</commentid>
    <comment_count>10</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-07-12 08:06:51 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>44689</attachid>
            <date>2009-12-11 07:54:03 -0800</date>
            <delta_ts>2010-07-08 02:54:08 -0700</delta_ts>
            <desc>this fixes the issue, by adding PLATFORM(ANDROID) to the check in CachedFont.cpp</desc>
            <filename>androidfontpatch.txt</filename>
            <type>text/plain</type>
            <size>651</size>
            <attacher name="Mike Reed">mikerreed</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvbG9hZGVyL0NhY2hlZEZvbnQuY3BwCj09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNv
cmUvbG9hZGVyL0NhY2hlZEZvbnQuY3BwCShyZXZpc2lvbiA1MTc4OSkKKysrIFdlYkNvcmUvbG9h
ZGVyL0NhY2hlZEZvbnQuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0yNyw3ICsyNyw3IEBACiAjaW5j
bHVkZSAiY29uZmlnLmgiCiAjaW5jbHVkZSAiQ2FjaGVkRm9udC5oIgogCi0jaWYgUExBVEZPUk0o
Q0cpIHx8IFBMQVRGT1JNKFFUKSB8fCBQTEFURk9STShHVEspIHx8IChQTEFURk9STShDSFJPTUlV
TSkgJiYgKFBMQVRGT1JNKFdJTl9PUykgfHwgUExBVEZPUk0oTElOVVgpKSkgfHwgUExBVEZPUk0o
SEFJS1UpIHx8IFBMQVRGT1JNKFdJTkNFKQorI2lmIFBMQVRGT1JNKEFORFJPSUQpIHx8IFBMQVRG
T1JNKENHKSB8fCBQTEFURk9STShRVCkgfHwgUExBVEZPUk0oR1RLKSB8fCAoUExBVEZPUk0oQ0hS
T01JVU0pICYmIChQTEFURk9STShXSU5fT1MpIHx8IFBMQVRGT1JNKExJTlVYKSkpIHx8IFBMQVRG
T1JNKEhBSUtVKSB8fCBQTEFURk9STShXSU5DRSkKICNkZWZpbmUgU1RPUkVfRk9OVF9DVVNUT01f
UExBVEZPUk1fREFUQQogI2VuZGlmCiAK
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>44692</attachid>
            <date>2009-12-11 09:37:54 -0800</date>
            <delta_ts>2010-07-08 02:54:13 -0700</delta_ts>
            <desc>add PLATFORM(ANDROID)</desc>
            <filename>cachedfont_patch.txt</filename>
            <type>text/plain</type>
            <size>1300</size>
            <attacher name="Mike Reed">mikerreed</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA1MTk5OCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTYgQEAKKzIwMDktMTItMTEgIE1pa2UgUmVlZCAgPG1pa2VycmVlZEBnbWFpbC5j
b20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgYWRk
IFBMQVRGT1JNKEFORFJPSUQpIHRvIHRoZSBsaXN0IGlmIHBsYXRmb3JtcyB0aGF0IHN1cHBvcnQK
KyAgICAgICAgU1RPUkVfRk9OVF9DVVNUT01fUExBVEZPUk1fREFUQQorICAgICAgICAKKyAgICAg
ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTMyMjczCisKKyAgICAg
ICAgTm8gbmV3IHRlc3RzLiAoT09QUyEpIFRlc3RpbmcgaXMgZG9uZSBpbnNpZGUgdGhlIGFuZHJv
aWQgZW52aXJvbm1lbnQKKworICAgICAgICAqIGxvYWRlci9DYWNoZWRGb250LmNwcDoKKwogMjAw
OS0xMi0xMSAgYW50b24gbXVoaW4gIDxhbnRvbm1AY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFJl
dmlld2VkIGJ5IEFkYW0gQmFydGguCkluZGV4OiBXZWJDb3JlL2xvYWRlci9DYWNoZWRGb250LmNw
cAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL2xvYWRlci9DYWNoZWRGb250LmNwcAkocmV2aXNpb24g
NTE3ODkpCisrKyBXZWJDb3JlL2xvYWRlci9DYWNoZWRGb250LmNwcAkod29ya2luZyBjb3B5KQpA
QCAtMjcsNyArMjcsNyBAQAogI2luY2x1ZGUgImNvbmZpZy5oIgogI2luY2x1ZGUgIkNhY2hlZEZv
bnQuaCIKIAotI2lmIFBMQVRGT1JNKENHKSB8fCBQTEFURk9STShRVCkgfHwgUExBVEZPUk0oR1RL
KSB8fCAoUExBVEZPUk0oQ0hST01JVU0pICYmIChQTEFURk9STShXSU5fT1MpIHx8IFBMQVRGT1JN
KExJTlVYKSkpIHx8IFBMQVRGT1JNKEhBSUtVKSB8fCBQTEFURk9STShXSU5DRSkKKyNpZiBQTEFU
Rk9STShBTkRST0lEKSB8fCBQTEFURk9STShDRykgfHwgUExBVEZPUk0oUVQpIHx8IFBMQVRGT1JN
KEdUSykgfHwgKFBMQVRGT1JNKENIUk9NSVVNKSAmJiAoUExBVEZPUk0oV0lOX09TKSB8fCBQTEFU
Rk9STShMSU5VWCkpKSB8fCBQTEFURk9STShIQUlLVSkgfHwgUExBVEZPUk0oV0lOQ0UpCiAjZGVm
aW5lIFNUT1JFX0ZPTlRfQ1VTVE9NX1BMQVRGT1JNX0RBVEEKICNlbmRpZgogCg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>60860</attachid>
            <date>2010-07-08 02:54:19 -0700</date>
            <delta_ts>2010-07-12 08:06:46 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-32273-20100708105417.patch</filename>
            <type>text/plain</type>
            <size>1239</size>
            <attacher name="Steve Block">steveblock</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA2Mjc3NCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTQgQEAKKzIwMTAtMDctMDggIFN0ZXZlIEJsb2NrICA8c3RldmVibG9ja0Bnb29n
bGUuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAg
IGFkZCBBTkRST0lEIHRvIFNUT1JFX0ZPTlRfQ1VTVE9NX1BMQVRGT1JNX0RBVEEKKyAgICAgICAg
aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTMyMjczCisKKyAgICAgICAg
VGVzdGVkIGJ5IGV4aXN0aW5nIHRlc3RzLCBqdXN0IGFkZGluZyBBTkRST0lEIHRvIHRoZSBsaXN0
IG9mIHBsYXRmb3JtcyB0aGF0IHVzZSB0aGlzIGZlYXR1cmUuCisKKyAgICAgICAgKiBsb2FkZXIv
Q2FjaGVkRm9udC5jcHA6CisKIDIwMTAtMDctMDcgIEFsZXhhbmRlciBQYXZsb3YgIDxhcGF2bG92
QGNocm9taXVtLm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBEYXJpbiBGaXNoZXIuCkluZGV4
OiBXZWJDb3JlL2xvYWRlci9DYWNoZWRGb250LmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL2xv
YWRlci9DYWNoZWRGb250LmNwcAkocmV2aXNpb24gNjI3NzQpCisrKyBXZWJDb3JlL2xvYWRlci9D
YWNoZWRGb250LmNwcAkod29ya2luZyBjb3B5KQpAQCAtMjcsNyArMjcsNyBAQAogI2luY2x1ZGUg
ImNvbmZpZy5oIgogI2luY2x1ZGUgIkNhY2hlZEZvbnQuaCIKIAotI2lmIFBMQVRGT1JNKENHKSB8
fCBQTEFURk9STShRVCkgfHwgUExBVEZPUk0oR1RLKSB8fCAoUExBVEZPUk0oQ0hST01JVU0pICYm
IChPUyhXSU5ET1dTKSB8fCBPUyhMSU5VWCkpKSB8fCBQTEFURk9STShIQUlLVSkgfHwgT1MoV0lO
Q0UpCisjaWYgUExBVEZPUk0oQ0cpIHx8IFBMQVRGT1JNKFFUKSB8fCBQTEFURk9STShHVEspIHx8
IChQTEFURk9STShDSFJPTUlVTSkgJiYgKE9TKFdJTkRPV1MpIHx8IE9TKExJTlVYKSkpIHx8IFBM
QVRGT1JNKEhBSUtVKSB8fCBPUyhXSU5DRSkgfHwgUExBVEZPUk0oQU5EUk9JRCkKICNkZWZpbmUg
U1RPUkVfRk9OVF9DVVNUT01fUExBVEZPUk1fREFUQQogI2VuZGlmCiAK
</data>

          </attachment>
      

    </bug>

</bugzilla>