<?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>193579</bug_id>
          
          <creation_ts>2019-01-18 09:49:09 -0800</creation_ts>
          <short_desc>Duplicate tests between canvas/philip/tests and imported/w3c/canvas</short_desc>
          <delta_ts>2019-01-21 01:07:02 -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>Tools / Tests</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=193042</see_also>
    
    <see_also>https://bugs.webkit.org/show_bug.cgi?id=179769</see_also>
          <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="David Kilzer (:ddkilzer)">ddkilzer</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>cdumez</cc>
    
    <cc>dino</cc>
    
    <cc>jlewis3</cc>
    
    <cc>lforschler</cc>
    
    <cc>Ms2ger</cc>
    
    <cc>rniwa</cc>
    
    <cc>ryanhaddad</cc>
    
    <cc>simon.fraser</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1495889</commentid>
    <comment_count>0</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2019-01-18 09:49:09 -0800</bug_when>
    <thetext>There appear to be a large number of duplicate tests between canvas/philip/tests and imported/w3c/canvas in the LayoutTests directory.

Removing the duplicate tests could marginally speed up layout test runs.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1495890</commentid>
    <comment_count>1</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2019-01-18 09:49:46 -0800</bug_when>
    <thetext>The canvas/philip/tests were imported in &lt;https://trac.webkit.org/r59954&gt; on May 14, 2010 long before the W3C test suite existed.

It seems like these tests have been upstreamed to the W3C test suite, but the older copies have not all been removed.

In fact, there appear to be 741 tests that are duplicates by name:

$ ls `find LayoutTests/canvas/philip/tests -name \*.html | sed -e &apos;s#canvas/philip/tests#imported/w3c/canvas#&apos;` 2&gt; /dev/null | wc -l
     741

And there are 34 tests that only exist in canvas/philip/tests:

$ ls `find LayoutTests/canvas/philip/tests -name \*.html | sed -e &apos;s#canvas/philip/tests#imported/w3c/canvas#&apos;` &gt; /dev/null
ls: LayoutTests/imported/w3c/canvas/2d.clearRect+fillRect.alpha0.5.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.clearRect+fillRect.alpha0.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.clearRect+fillRect.basic.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.composite.globalComposite.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.composite.operation.foobar.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.composite.operation.nullsuffix.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.drawImage.incomplete.emptysrc.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.drawImage.incomplete.nosrc.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.drawImage.incomplete.removedsrc.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.drawImage.negativeSourceHeight.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.drawImage.negativeSourceHeight2.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.drawImage.negativeSourceHeightAndWidth.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.drawImage.outsidesource.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.line.cap.invalid.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.line.join.invalid.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.missingargs.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.pattern.image.broken.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.pattern.image.incomplete.emptysrc.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.pattern.image.incomplete.removedsrc.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.pattern.repeat.nullsuffix.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.strokeRect.zero.4.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.text-custom-font-load-crash.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.text.align.invalid.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.text.baseline.invalid.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.text.draw.baseline.bottom.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.text.draw.baseline.hanging.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.text.draw.baseline.ideographic.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.text.draw.baseline.middle.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.text.draw.baseline.top.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.text.draw.space.collapse.end.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.text.draw.space.collapse.other.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.text.draw.space.collapse.space.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.text.draw.space.collapse.start.html: No such file or directory
ls: LayoutTests/imported/w3c/canvas/2d.text.measure.width.space.html: No such file or directory</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1495891</commentid>
    <comment_count>2</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2019-01-18 09:50:57 -0800</bug_when>
    <thetext>I noticed this while working on finding leaks in WebKit2 tests, but I don&apos;t have time to investigate this right now (other than blindly removing the duplicate canvas/philip/tests--which I did not want to do), so filed this bug to track the issue.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1495942</commentid>
    <comment_count>3</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2019-01-18 11:22:32 -0800</bug_when>
    <thetext>See bug 193042</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>