<?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>66148</bug_id>
          
          <creation_ts>2011-08-12 11:01:13 -0700</creation_ts>
          <short_desc>Replace == comparison with epsilon aware roughlyEquals for LayoutUnits</short_desc>
          <delta_ts>2011-11-01 17:04:47 -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>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>INVALID</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>
          
          <blocked>63656</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Emil A Eklund">eae</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>ap</cc>
    
    <cc>hyatt</cc>
    
    <cc>leviw</cc>
    
    <cc>simon.fraser</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>450510</commentid>
    <comment_count>0</comment_count>
    <who name="Emil A Eklund">eae</who>
    <bug_when>2011-08-12 11:01:13 -0700</bug_when>
    <thetext>Once we move to float we can no longer safely use == to compare two LayoutUnits.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>450803</commentid>
    <comment_count>1</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2011-08-12 21:10:51 -0700</bug_when>
    <thetext>So, are we decided that we move to floats, not to fixed precision numbers?

Rough equality has a number of highly undesirable features - it&apos;s not transitive, it breaks with zoom etc.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>451110</commentid>
    <comment_count>2</comment_count>
    <who name="Emil A Eklund">eae</who>
    <bug_when>2011-08-15 10:22:02 -0700</bug_when>
    <thetext>No firm decision yet, we have a prototype running on trunk that we are running a set of tests against to see if it&apos;s feasible to use floats throughout and help us discover problems.

We&apos;re currently leaning towards using floats as both javascript and the graphics sub system operates on floating point but we&apos;ll revisit this before committing to go down that path.

This bug is for tracking places where we might need to change how we do comparisons if we decide to go with floats.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>494315</commentid>
    <comment_count>3</comment_count>
    <who name="Emil A Eklund">eae</who>
    <bug_when>2011-11-01 17:04:47 -0700</bug_when>
    <thetext>No longer applicable as we decided to go with fixed point.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>