<?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>53363</bug_id>
          
          <creation_ts>2011-01-28 21:11:46 -0800</creation_ts>
          <short_desc>Use clampToInteger() functions in a few places</short_desc>
          <delta_ts>2011-01-29 11:44:24 -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>CSS</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="Simon Fraser (smfr)">simon.fraser</reporter>
          <assigned_to name="Simon Fraser (smfr)">simon.fraser</assigned_to>
          <cc>simon.fraser</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>341982</commentid>
    <comment_count>0</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2011-01-28 21:11:46 -0800</bug_when>
    <thetext>Now that we have clamping functions in MathExtras, use them!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>341983</commentid>
    <comment_count>1</comment_count>
      <attachid>80543</attachid>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2011-01-28 21:13:24 -0800</bug_when>
    <thetext>Created attachment 80543
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>342120</commentid>
    <comment_count>2</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2011-01-29 11:44:24 -0800</bug_when>
    <thetext>http://trac.webkit.org/changeset/77064</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>80543</attachid>
            <date>2011-01-28 21:13:24 -0800</date>
            <delta_ts>2011-01-29 11:38:46 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-53363-20110128211323.patch</filename>
            <type>text/plain</type>
            <size>4344</size>
            <attacher name="Simon Fraser (smfr)">simon.fraser</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCAzYWQ4ZmFiZDIyZjYyOTc3YjM4MWUyYjcwZGJlZWVlMzY0Zjg0OTI3Li45
ODE1ZjViZDk0NTYxMmRlNjBhYTI5NDU1YTlmN2Y2NDExN2VhYTlhIDEwMDY0NAotLS0gYS9Tb3Vy
Y2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0x
LDUgKzEsMTggQEAKIDIwMTEtMDEtMjggIFNpbW9uIEZyYXNlciAgPHNpbW9uLmZyYXNlckBhcHBs
ZS5jb20+CiAKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg
VXNlIGNsYW1wVG9JbnRlZ2VyKCkgZnVuY3Rpb25zIGluIGEgZmV3IHBsYWNlcworICAgICAgICBo
dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTMzNjMKKyAgICAgICAgCisg
ICAgICAgICogY3NzL0NTU1N0eWxlU2VsZWN0b3IuY3BwOgorICAgICAgICAoV2ViQ29yZTo6Q1NT
U3R5bGVTZWxlY3Rvcjo6YXBwbHlQcm9wZXJ0eSk6IFVzZSBjbGFtcFRvSW50ZWdlcigpIGZvciB6
LWluZGV4LgorICAgICAgICAoV2ViQ29yZTo6Q1NTU3R5bGVTZWxlY3Rvcjo6Y3JlYXRlVHJhbnNm
b3JtT3BlcmF0aW9ucyk6IFVzZSBjbGFtcFRvUG9zaXRpdmVJbnRlZ2VyKCkuCisgICAgICAgICog
cGxhdGZvcm0vZ3JhcGhpY3MvdHJhbnNmb3Jtcy9QZXJzcGVjdGl2ZVRyYW5zZm9ybU9wZXJhdGlv
bi5jcHA6IERpdHRvLgorICAgICAgICAoV2ViQ29yZTo6UGVyc3BlY3RpdmVUcmFuc2Zvcm1PcGVy
YXRpb246OmJsZW5kKTogRGl0dG8uCisKKzIwMTEtMDEtMjggIFNpbW9uIEZyYXNlciAgPHNpbW9u
LmZyYXNlckBhcHBsZS5jb20+CisKICAgICAgICAgUmV2aWV3ZWQgYnkgR2F2aW4gQmFycmFjbG91
Z2guCiAKICAgICAgICAgQWRkIHZhcmlvdXMgY2xhbXBUb0ludCgpIG1ldGhvZHMgdG8gTWF0aEV4
dHJhcy5oCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9jc3MvQ1NTU3R5bGVTZWxlY3Rvci5j
cHAgYi9Tb3VyY2UvV2ViQ29yZS9jc3MvQ1NTU3R5bGVTZWxlY3Rvci5jcHAKaW5kZXggNjUzNjE5
M2Y4ZmIwN2QwYzZhNTllMWRmYjAyNzg0ZmZhYzY2ZjAwZS4uMGM0MDdhNjM5NzVlMWRhOGVmMWZk
MTg3ODc2N2Q2YmExZjEyN2EzMCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvY3NzL0NTU1N0
eWxlU2VsZWN0b3IuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL2Nzcy9DU1NTdHlsZVNlbGVjdG9y
LmNwcApAQCAtNDI5Niw5ICs0Mjk2LDcgQEAgdm9pZCBDU1NTdHlsZVNlbGVjdG9yOjphcHBseVBy
b3BlcnR5KGludCBpZCwgQ1NTVmFsdWUgKnZhbHVlKQogICAgICAgICB9CiAgICAgICAgIAogICAg
ICAgICAvLyBGSVhNRTogU2hvdWxkIGNsYW1wIGFsbCBzb3J0cyBvZiBvdGhlciBpbnRlZ2VyIHBy
b3BlcnRpZXMgdG9vLgotICAgICAgICBjb25zdCBkb3VibGUgbWluSW50QXNEb3VibGUgPSBJTlRf
TUlOOwotICAgICAgICBjb25zdCBkb3VibGUgbWF4SW50QXNEb3VibGUgPSBJTlRfTUFYOwotICAg
ICAgICBtX3N0eWxlLT5zZXRaSW5kZXgoc3RhdGljX2Nhc3Q8aW50PihtYXgobWluSW50QXNEb3Vi
bGUsIG1pbihwcmltaXRpdmVWYWx1ZS0+Z2V0RG91YmxlVmFsdWUoKSwgbWF4SW50QXNEb3VibGUp
KSkpOworICAgICAgICBtX3N0eWxlLT5zZXRaSW5kZXgoY2xhbXBUb0ludGVnZXIocHJpbWl0aXZl
VmFsdWUtPmdldERvdWJsZVZhbHVlKCkpKTsKICAgICAgICAgcmV0dXJuOwogICAgIH0KICAgICBj
YXNlIENTU1Byb3BlcnR5V2lkb3dzOgpAQCAtNzA2MSw3ICs3MDU5LDcgQEAgYm9vbCBDU1NTdHls
ZVNlbGVjdG9yOjpjcmVhdGVUcmFuc2Zvcm1PcGVyYXRpb25zKENTU1ZhbHVlKiBpblZhbHVlLCBS
ZW5kZXJTdHlsZSoKICAgICAgICAgICAgICAgICAgICAgLy8gVGhpcyBpcyBhIHF1aXJrIHRoYXQg
c2hvdWxkIGdvIGF3YXkgd2hlbiAzZCB0cmFuc2Zvcm1zIGFyZSBmaW5hbGl6ZWQuCiAgICAgICAg
ICAgICAgICAgICAgIGRvdWJsZSB2YWwgPSBmaXJzdFZhbHVlLT5nZXREb3VibGVWYWx1ZSgpOwog
ICAgICAgICAgICAgICAgICAgICBvayA9IHZhbCA+PSAwOwotICAgICAgICAgICAgICAgICAgICB2
YWwgPSBtaW48ZG91YmxlPih2YWwsIG51bWVyaWNfbGltaXRzPGludD46Om1heCgpKTsKKyAgICAg
ICAgICAgICAgICAgICAgdmFsID0gY2xhbXBUb1Bvc2l0aXZlSW50ZWdlcih2YWwpOwogICAgICAg
ICAgICAgICAgICAgICBwID0gTGVuZ3RoKHN0YXRpY19jYXN0PGludD4odmFsKSwgRml4ZWQpOwog
ICAgICAgICAgICAgICAgIH0KICAgICAgICAgICAgICAgICAKZGlmZiAtLWdpdCBhL1NvdXJjZS9X
ZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL3RyYW5zZm9ybXMvUGVyc3BlY3RpdmVUcmFuc2Zvcm1P
cGVyYXRpb24uY3BwIGIvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvdHJhbnNmb3Jt
cy9QZXJzcGVjdGl2ZVRyYW5zZm9ybU9wZXJhdGlvbi5jcHAKaW5kZXggMjU4Y2ZiZTVjNzNjZWVj
YTY1NDhmNjJhZDI1ZjE0MzY4NWUxZDg0Ni4uMThiZmUzN2Q0OGI0ODhiZTgyYzk0NWQ2ZjYyMjBh
OTVjZmY1NzBhZSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3Mv
dHJhbnNmb3Jtcy9QZXJzcGVjdGl2ZVRyYW5zZm9ybU9wZXJhdGlvbi5jcHAKKysrIGIvU291cmNl
L1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvdHJhbnNmb3Jtcy9QZXJzcGVjdGl2ZVRyYW5zZm9y
bU9wZXJhdGlvbi5jcHAKQEAgLTI2LDE4ICsyNiwxMiBAQAogI2luY2x1ZGUgImNvbmZpZy5oIgog
I2luY2x1ZGUgIlBlcnNwZWN0aXZlVHJhbnNmb3JtT3BlcmF0aW9uLmgiCiAKLSNpbmNsdWRlIDxh
bGdvcml0aG0+Ci0jaW5jbHVkZSA8bGltaXRzPgorI2luY2x1ZGUgPHd0Zi9NYXRoRXh0cmFzLmg+
CiAKIHVzaW5nIG5hbWVzcGFjZSBzdGQ7CiAKIG5hbWVzcGFjZSBXZWJDb3JlIHsKIAotc3RhdGlj
IGludCBjbGFtcFRvUG9zdGl2ZUludGVnZXIoZG91YmxlIHZhbCkKLXsKLSAgICByZXR1cm4gc3Rh
dGljX2Nhc3Q8aW50PihtYXg8ZG91YmxlPihtaW48ZG91YmxlPih2YWwsIG51bWVyaWNfbGltaXRz
PGludD46Om1heCgpKSwgMCkpOwotfQotCiBQYXNzUmVmUHRyPFRyYW5zZm9ybU9wZXJhdGlvbj4g
UGVyc3BlY3RpdmVUcmFuc2Zvcm1PcGVyYXRpb246OmJsZW5kKGNvbnN0IFRyYW5zZm9ybU9wZXJh
dGlvbiogZnJvbSwgZG91YmxlIHByb2dyZXNzLCBib29sIGJsZW5kVG9JZGVudGl0eSkKIHsKICAg
ICBpZiAoZnJvbSAmJiAhZnJvbS0+aXNTYW1lVHlwZSgqdGhpcykpCkBAIC00Niw3ICs0MCw3IEBA
IFBhc3NSZWZQdHI8VHJhbnNmb3JtT3BlcmF0aW9uPiBQZXJzcGVjdGl2ZVRyYW5zZm9ybU9wZXJh
dGlvbjo6YmxlbmQoY29uc3QgVHJhbnNmCiAgICAgaWYgKGJsZW5kVG9JZGVudGl0eSkgewogICAg
ICAgICBkb3VibGUgcCA9IG1fcC5jYWxjRmxvYXRWYWx1ZSgxKTsKICAgICAgICAgcCA9IHAgKyAo
MS4gLSBwKSAqIHByb2dyZXNzOyAvLyBGSVhNRTogdGhpcyBzZWVtcyB3cm9uZy4gaHR0cHM6Ly9i
dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTUyNzAwCi0gICAgICAgIHJldHVybiBQZXJz
cGVjdGl2ZVRyYW5zZm9ybU9wZXJhdGlvbjo6Y3JlYXRlKExlbmd0aChjbGFtcFRvUG9zdGl2ZUlu
dGVnZXIocCksIEZpeGVkKSk7CisgICAgICAgIHJldHVybiBQZXJzcGVjdGl2ZVRyYW5zZm9ybU9w
ZXJhdGlvbjo6Y3JlYXRlKExlbmd0aChjbGFtcFRvUG9zaXRpdmVJbnRlZ2VyKHApLCBGaXhlZCkp
OwogICAgIH0KICAgICAKICAgICBjb25zdCBQZXJzcGVjdGl2ZVRyYW5zZm9ybU9wZXJhdGlvbiog
ZnJvbU9wID0gc3RhdGljX2Nhc3Q8Y29uc3QgUGVyc3BlY3RpdmVUcmFuc2Zvcm1PcGVyYXRpb24q
Pihmcm9tKTsKQEAgLTYzLDcgKzU3LDcgQEAgUGFzc1JlZlB0cjxUcmFuc2Zvcm1PcGVyYXRpb24+
IFBlcnNwZWN0aXZlVHJhbnNmb3JtT3BlcmF0aW9uOjpibGVuZChjb25zdCBUcmFuc2YKIAogICAg
IGlmIChkZWNvbXAucGVyc3BlY3RpdmVaKSB7CiAgICAgICAgIGRvdWJsZSB2YWwgPSAtMS4wIC8g
ZGVjb21wLnBlcnNwZWN0aXZlWjsKLSAgICAgICAgcmV0dXJuIFBlcnNwZWN0aXZlVHJhbnNmb3Jt
T3BlcmF0aW9uOjpjcmVhdGUoTGVuZ3RoKGNsYW1wVG9Qb3N0aXZlSW50ZWdlcih2YWwpLCBGaXhl
ZCkpOworICAgICAgICByZXR1cm4gUGVyc3BlY3RpdmVUcmFuc2Zvcm1PcGVyYXRpb246OmNyZWF0
ZShMZW5ndGgoY2xhbXBUb1Bvc2l0aXZlSW50ZWdlcih2YWwpLCBGaXhlZCkpOwogICAgIH0KICAg
ICByZXR1cm4gUGVyc3BlY3RpdmVUcmFuc2Zvcm1PcGVyYXRpb246OmNyZWF0ZShMZW5ndGgoMCwg
Rml4ZWQpKTsKIH0K
</data>
<flag name="review"
          id="72040"
          type_id="1"
          status="+"
          setter="abarth"
    />
          </attachment>
      

    </bug>

</bugzilla>