<?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>26724</bug_id>
          
          <creation_ts>2009-06-25 12:27:42 -0700</creation_ts>
          <short_desc>Move part of SetDefaultFontSize into RenderThemeChromiumSkia.h</short_desc>
          <delta_ts>2009-06-25 16:59:17 -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>WebCore Misc.</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>OS X 10.5</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>1</everconfirmed>
          <reporter name="Albert J. Wong">ajwong</reporter>
          <assigned_to name="Albert J. Wong">ajwong</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>128235</commentid>
    <comment_count>0</comment_count>
    <who name="Albert J. Wong">ajwong</who>
    <bug_when>2009-06-25 12:27:42 -0700</bug_when>
    <thetext>Currently, linux-land does not allow for setting of the default font size.

Pull up the relevant part of the code for changing font size out of RenderThemeChromiumWin into RenderThemeChromiumSkia so that it may be called from linux chromium.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>128237</commentid>
    <comment_count>1</comment_count>
      <attachid>31871</attachid>
    <who name="Albert J. Wong">ajwong</who>
    <bug_when>2009-06-25 12:37:53 -0700</bug_when>
    <thetext>Created attachment 31871
Move part of setDefaultFontSize from RenderThemeChromiumWin to RenderThemeChromiumSkia.

short patch to make the setDefaultFontSize function available on linux.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>128324</commentid>
    <comment_count>2</comment_count>
      <attachid>31871</attachid>
    <who name="Darin Fisher (:fishd, Google)">fishd</who>
    <bug_when>2009-06-25 16:57:34 -0700</bug_when>
    <thetext>Comment on attachment 31871
Move part of setDefaultFontSize from RenderThemeChromiumWin to RenderThemeChromiumSkia.

&gt; +2009-06-25  Albert J. Wong  &lt;ajwong@chromium.org&gt;
&gt; +
&gt; +        Reviewed by NOBODY (OOPS!).
&gt; +
&gt; +        Move relavent part of setDefaultFontSize from RenderThemeChromiumWin
&gt; +        up into RenderThemeChromiumSkia.

please remember to insert a bug link next time.

R=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>128325</commentid>
    <comment_count>3</comment_count>
    <who name="Darin Fisher (:fishd, Google)">fishd</who>
    <bug_when>2009-06-25 16:59:17 -0700</bug_when>
    <thetext>Landed as http://trac.webkit.org/changeset/45215</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>31871</attachid>
            <date>2009-06-25 12:37:53 -0700</date>
            <delta_ts>2009-06-25 16:57:34 -0700</delta_ts>
            <desc>Move part of setDefaultFontSize from RenderThemeChromiumWin to RenderThemeChromiumSkia.</desc>
            <filename>fontsize.patch</filename>
            <type>text/plain</type>
            <size>3867</size>
            <attacher name="Albert J. Wong">ajwong</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg
ZWRkYTM0Yy4uNTYyYzgwZSAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi
Q29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNyBAQAorMjAwOS0wNi0yNSAgQWxiZXJ0IEouIFdv
bmcgIDxhandvbmdAY2hyb21pdW0ub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAo
T09QUyEpLgorCisgICAgICAgIE1vdmUgcmVsYXZlbnQgcGFydCBvZiBzZXREZWZhdWx0Rm9udFNp
emUgZnJvbSBSZW5kZXJUaGVtZUNocm9taXVtV2luCisgICAgICAgIHVwIGludG8gUmVuZGVyVGhl
bWVDaHJvbWl1bVNraWEuCisKKyAgICAgICAgKiByZW5kZXJpbmcvUmVuZGVyVGhlbWVDaHJvbWl1
bVNraWEuY3BwOgorICAgICAgICAoV2ViQ29yZTo6UmVuZGVyVGhlbWVDaHJvbWl1bVNraWE6OnNl
dERlZmF1bHRGb250U2l6ZSk6CisgICAgICAgICogcmVuZGVyaW5nL1JlbmRlclRoZW1lQ2hyb21p
dW1Ta2lhLmg6CisgICAgICAgICogcmVuZGVyaW5nL1JlbmRlclRoZW1lQ2hyb21pdW1XaW4uY3Bw
OgorICAgICAgICAoV2ViQ29yZTo6UmVuZGVyVGhlbWVDaHJvbWl1bVdpbjo6c2V0RGVmYXVsdEZv
bnRTaXplKToKKyAgICAgICAgKiByZW5kZXJpbmcvUmVuZGVyVGhlbWVDaHJvbWl1bVdpbi5oOgor
CiAyMDA5LTA2LTIyICBBZGFtIEJhcnRoICA8YWJhcnRoQHdlYmtpdC5vcmc+CiAKICAgICAgICAg
UmV2aWV3ZWQgYnkgRGltaXRyaSBHbGF6a292LgpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9yZW5kZXJp
bmcvUmVuZGVyVGhlbWVDaHJvbWl1bVNraWEuY3BwIGIvV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVy
VGhlbWVDaHJvbWl1bVNraWEuY3BwCmluZGV4IDdkNWI0OTMuLmI3NTZhYjcgMTAwNjQ0Ci0tLSBh
L1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRlclRoZW1lQ2hyb21pdW1Ta2lhLmNwcAorKysgYi9XZWJD
b3JlL3JlbmRlcmluZy9SZW5kZXJUaGVtZUNocm9taXVtU2tpYS5jcHAKQEAgLTU4MSw2ICs1ODEs
MTIgQEAgaW50IFJlbmRlclRoZW1lQ2hyb21pdW1Ta2lhOjpidXR0b25JbnRlcm5hbFBhZGRpbmdC
b3R0b20oKSBjb25zdAogICAgIHJldHVybiAxOwogfQogCisvLyBzdGF0aWMKK3ZvaWQgUmVuZGVy
VGhlbWVDaHJvbWl1bVNraWE6OnNldERlZmF1bHRGb250U2l6ZShpbnQgZm9udFNpemUpCit7Cisg
ICAgZGVmYXVsdEZvbnRTaXplID0gc3RhdGljX2Nhc3Q8ZmxvYXQ+KGZvbnRTaXplKTsKK30KKwog
ZG91YmxlIFJlbmRlclRoZW1lQ2hyb21pdW1Ta2lhOjpjYXJldEJsaW5rSW50ZXJ2YWxJbnRlcm5h
bCgpIGNvbnN0CiB7CiAgICAgcmV0dXJuIFJlbmRlclRoZW1lOjpjYXJldEJsaW5rSW50ZXJ2YWwo
KTsKZGlmZiAtLWdpdCBhL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRlclRoZW1lQ2hyb21pdW1Ta2lh
LmggYi9XZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJUaGVtZUNocm9taXVtU2tpYS5oCmluZGV4IDQy
MWJkMjYuLmQ5MWMwZDcgMTAwNjQ0Ci0tLSBhL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRlclRoZW1l
Q2hyb21pdW1Ta2lhLmgKKysrIGIvV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyVGhlbWVDaHJvbWl1
bVNraWEuaApAQCAtMTE2LDYgKzExNiwxMyBAQCBuYW1lc3BhY2UgV2ViQ29yZSB7CiAgICAgICAg
IHZpcnR1YWwgaW50IGJ1dHRvbkludGVybmFsUGFkZGluZ1RvcCgpIGNvbnN0OwogICAgICAgICB2
aXJ0dWFsIGludCBidXR0b25JbnRlcm5hbFBhZGRpbmdCb3R0b20oKSBjb25zdDsKIAorICAgICAg
ICAvLyBQcm92aWRlIGEgd2F5IHRvIHBhc3MgdGhlIGRlZmF1bHQgZm9udCBzaXplIGZyb20gdGhl
IFNldHRpbmdzIG9iamVjdAorICAgICAgICAvLyB0byB0aGUgcmVuZGVyIHRoZW1lLiAgRklYTUU6
IGh0dHA6Ly9iLzExMjkxODYgQSBjbGVhbmVyIHdheSB3b3VsZCBiZQorICAgICAgICAvLyB0byBy
ZW1vdmUgdGhlIGRlZmF1bHQgZm9udCBzaXplIGZyb20gdGhpcyBvYmplY3QgYW5kIGhhdmUgY2Fs
bGVycworICAgICAgICAvLyB0aGF0IG5lZWQgdGhlIHZhbHVlIHRvIGdldCBpdCBkaXJlY3RseSBm
cm9tIHRoZSBhcHByb3ByaWF0ZSBTZXR0aW5ncworICAgICAgICAvLyBvYmplY3QuCisgICAgICAg
IHN0YXRpYyB2b2lkIHNldERlZmF1bHRGb250U2l6ZShpbnQpOworCiAgICAgcHJvdGVjdGVkOgog
ICAgICAgICBzdGF0aWMgY29uc3QgU3RyaW5nJiBkZWZhdWx0R1VJRm9udCgpOwogCmRpZmYgLS1n
aXQgYS9XZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJUaGVtZUNocm9taXVtV2luLmNwcCBiL1dlYkNv
cmUvcmVuZGVyaW5nL1JlbmRlclRoZW1lQ2hyb21pdW1XaW4uY3BwCmluZGV4IGZmYzEzZTEuLjRl
ZDhkODggMTAwNjQ0Ci0tLSBhL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRlclRoZW1lQ2hyb21pdW1X
aW4uY3BwCisrKyBiL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRlclRoZW1lQ2hyb21pdW1XaW4uY3Bw
CkBAIC00MjUsNyArNDI1LDcgQEAgYm9vbCBSZW5kZXJUaGVtZUNocm9taXVtV2luOjpwYWludE1l
bnVMaXN0KFJlbmRlck9iamVjdCogbywgY29uc3QgUmVuZGVyT2JqZWN0OjoKIC8vIHN0YXRpYwog
dm9pZCBSZW5kZXJUaGVtZUNocm9taXVtV2luOjpzZXREZWZhdWx0Rm9udFNpemUoaW50IGZvbnRT
aXplKQogewotICAgIGRlZmF1bHRGb250U2l6ZSA9IHN0YXRpY19jYXN0PGZsb2F0Pihmb250U2l6
ZSk7CisgICAgUmVuZGVyVGhlbWVDaHJvbWl1bVNraWE6OnNldERlZmF1bHRGb250U2l6ZShmb250
U2l6ZSk7CiAKICAgICAvLyBSZXNldCBjYWNoZWQgZm9udHMuCiAgICAgc21hbGxTeXN0ZW1Gb250
ID0gbWVudUZvbnQgPSBsYWJlbEZvbnQgPSBGb250RGVzY3JpcHRpb24oKTsKZGlmZiAtLWdpdCBh
L1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRlclRoZW1lQ2hyb21pdW1XaW4uaCBiL1dlYkNvcmUvcmVu
ZGVyaW5nL1JlbmRlclRoZW1lQ2hyb21pdW1XaW4uaAppbmRleCBhZjUxZTc5Li41ZTk4YzliIDEw
MDY0NAotLS0gYS9XZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJUaGVtZUNocm9taXVtV2luLmgKKysr
IGIvV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyVGhlbWVDaHJvbWl1bVdpbi5oCkBAIC04MSwxMSAr
ODEsOCBAQCBuYW1lc3BhY2UgV2ViQ29yZSB7CiAgICAgICAgIC8vIGVudGlyZSBtZW51bGlzdC4K
ICAgICAgICAgdmlydHVhbCBib29sIHBhaW50TWVudUxpc3QoUmVuZGVyT2JqZWN0KiwgY29uc3Qg
UmVuZGVyT2JqZWN0OjpQYWludEluZm8mLCBjb25zdCBJbnRSZWN0Jik7CiAKLSAgICAgICAgLy8g
UHJvdmlkZSBhIHdheSB0byBwYXNzIHRoZSBkZWZhdWx0IGZvbnQgc2l6ZSBmcm9tIHRoZSBTZXR0
aW5ncyBvYmplY3QKLSAgICAgICAgLy8gdG8gdGhlIHJlbmRlciB0aGVtZS4gIEZJWE1FOiBodHRw
Oi8vYi8xMTI5MTg2IEEgY2xlYW5lciB3YXkgd291bGQgYmUKLSAgICAgICAgLy8gdG8gcmVtb3Zl
IHRoZSBkZWZhdWx0IGZvbnQgc2l6ZSBmcm9tIHRoaXMgb2JqZWN0IGFuZCBoYXZlIGNhbGxlcnMK
LSAgICAgICAgLy8gdGhhdCBuZWVkIHRoZSB2YWx1ZSB0byBnZXQgaXQgZGlyZWN0bHkgZnJvbSB0
aGUgYXBwcm9wcmlhdGUgU2V0dGluZ3MKLSAgICAgICAgLy8gb2JqZWN0LgorICAgICAgICAvLyBP
dmVycmlkZSBSZW5kZXJUaGVtZUNocm9taXVtU2tpYSdzIHNldERlZmF1bHRGb250U2l6ZSBtZXRo
b2QgdG8gYWxzbyByZXNldCB0aGUgbG9jYWwgZm9udCBwcm9wZXJ0eSBjYWNoZXMuCisgICAgICAg
IC8vIFNlZSBjb21tZW50IGluIFJlbmRlclRoZW1lQ2hyb21pdW1Ta2lhOjpzZXREZWZhdWx0Rm9u
dFNpemUoKSByZWdhcmRpbmcgdWdsaW5lc3Mgb2YgdGhpcyBoYWNrLgogICAgICAgICBzdGF0aWMg
dm9pZCBzZXREZWZhdWx0Rm9udFNpemUoaW50KTsKIAogICAgIHByb3RlY3RlZDoK
</data>
<flag name="review"
          id="16433"
          type_id="1"
          status="+"
          setter="fishd"
    />
          </attachment>
      

    </bug>

</bugzilla>