<?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>31590</bug_id>
          
          <creation_ts>2009-11-17 08:19:38 -0800</creation_ts>
          <short_desc>[Qt] unify resources used in WebKit/qt/tests/</short_desc>
          <delta_ts>2010-01-25 04:46:41 -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>WebKit Qt</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>WONTFIX</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Qt</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>32216</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Antonio Gomes">tonikitoo</reporter>
          <assigned_to name="Antonio Gomes">tonikitoo</assigned_to>
          <cc>benjamin</cc>
    
    <cc>hausmann</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>164239</commentid>
    <comment_count>0</comment_count>
    <who name="Antonio Gomes">tonikitoo</who>
    <bug_when>2009-11-17 08:19:38 -0800</bug_when>
    <thetext>Currently many autotests load their resources from external local files. see:

$ cd WebKit/qt/tests/

$ ls qwebelement/
image.png style2.css  style.css ...

$ ls qwebframe/
image.png  resources  style.css  test1.html  test2.html ...

$ ls qwebhistory/data/
page1.html  page2.html  page3.html  page4.html  page5.html  page6.html

$ ls qwebpage/frametest/
frame_a.html  iframe2.html  iframe3.html  iframe.html  index.html

$ ls qwebview/data/
frame_a.html  index.html

$ ls resources/
test.swf

It would be good to unify all of them into a single WebKit/qt/tests/resources/ directory and make all tests to use it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>182347</commentid>
    <comment_count>1</comment_count>
    <who name="Antonio Gomes">tonikitoo</who>
    <bug_when>2010-01-19 11:36:15 -0800</bug_when>
    <thetext>i can look at that.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>182658</commentid>
    <comment_count>2</comment_count>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2010-01-20 00:51:26 -0800</bug_when>
    <thetext>Does it really make sense to group them?
If there is not any resource that is a copy from another test, I would keep the resource close the where they are used.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>183733</commentid>
    <comment_count>3</comment_count>
    <who name="Antonio Gomes">tonikitoo</who>
    <bug_when>2010-01-22 11:53:05 -0800</bug_when>
    <thetext>(In reply to comment #2)
&gt; Does it really make sense to group them?
&gt; If there is not any resource that is a copy from another test, I would keep the
&gt; resource close the where they are used.

in fact, idea would be get the resources currently used more organized.

i do not know if just unifying them all is the plan here, but as you said, avoiding duplication, among others ...

@simon, benjamin: do you thing there is room for improvements here ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>184070</commentid>
    <comment_count>4</comment_count>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2010-01-24 07:26:50 -0800</bug_when>
    <thetext>(In reply to comment #3)
&gt; i do not know if just unifying them all is the plan here, but as you said,
&gt; avoiding duplication, among others ...
&gt; 
&gt; @simon, benjamin: do you thing there is room for improvements here ?

I think that make sense for the image and the swf, but I would not change the .html.

Otherwise, when someone updates the html to extend a test, that could break another test.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>184244</commentid>
    <comment_count>5</comment_count>
    <who name="Antonio Gomes">tonikitoo</who>
    <bug_when>2010-01-25 04:46:41 -0800</bug_when>
    <thetext>&gt; I think that make sense for the image and the swf, but I would not change the
&gt; .html.
&gt; 
&gt; Otherwise, when someone updates the html to extend a test, that could break
&gt; another test.

right. so there is no much gain from doing that, i believe.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>