<?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>42168</bug_id>
          
          <creation_ts>2010-07-13 08:40:41 -0700</creation_ts>
          <short_desc>REGRESSION (r63192): Pixel test failure in contain-and-cover.html</short_desc>
          <delta_ts>2010-07-13 09:38:48 -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>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>0</everconfirmed>
          <reporter name="Pavel Podivilov">podivilov</reporter>
          <assigned_to name="Andreas Kling">kling</assigned_to>
          <cc>jorlow</cc>
    
    <cc>kling</cc>
    
    <cc>mnaganov</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>250529</commentid>
    <comment_count>0</comment_count>
    <who name="Pavel Podivilov">podivilov</who>
    <bug_when>2010-07-13 08:40:41 -0700</bug_when>
    <thetext>contain-and-cover.html start failing after http://trac.webkit.org/changeset/63192, http://build.chromium.org/buildbot/waterfall.fyi/builders/Webkit%20Linux%20(webkit.org)/builds/32026
Could you please have a look?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>250531</commentid>
    <comment_count>1</comment_count>
    <who name="Jeremy Orlow">jorlow</who>
    <bug_when>2010-07-13 08:47:55 -0700</bug_when>
    <thetext>Why am I cc&apos;ed on this?  Also you really need to provide WAY more information to non-Chromium contributors.  There&apos;s basically no context here.  And no link to where he can see the actual failing test/results.  (Hopefully we&apos;ll be able to get pixel tests working on build.webkit.org soon!)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>250534</commentid>
    <comment_count>2</comment_count>
      <attachid>61383</attachid>
    <who name="Pavel Podivilov">podivilov</who>
    <bug_when>2010-07-13 08:51:02 -0700</bug_when>
    <thetext>Created attachment 61383
add contain-and-cover.html to test_expectations</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>250535</commentid>
    <comment_count>3</comment_count>
    <who name="Jeremy Orlow">jorlow</who>
    <bug_when>2010-07-13 08:53:18 -0700</bug_when>
    <thetext>Is this a real regression?  If so (and you can&apos;t get ahold of Andreas) then you should revert it and not paper over the regression.  If it&apos;s not a real regression then you should probably go ahead and check in a new baseline.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>250540</commentid>
    <comment_count>4</comment_count>
    <who name="Pavel Podivilov">podivilov</who>
    <bug_when>2010-07-13 09:03:43 -0700</bug_when>
    <thetext>(In reply to comment #3)
&gt; Is this a real regression?  If so (and you can&apos;t get ahold of Andreas) then you should revert it and not paper over the regression.  If it&apos;s not a real regression then you should probably go ahead and check in a new baseline.

I&apos;ve cc&apos;ed you just because you were in https://bugs.webkit.org/show_bug.cgi?id=38108 which seemed very similar to this problem. 

I&apos;m not sure if it&apos;s a regression or not, that&apos;s why I&apos;ve created this bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>250542</commentid>
    <comment_count>5</comment_count>
    <who name="Andreas Kling">kling</who>
    <bug_when>2010-07-13 09:09:53 -0700</bug_when>
    <thetext>Just a quick note to let you know I&apos;m looking at this right now.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>250545</commentid>
    <comment_count>6</comment_count>
    <who name="Pavel Podivilov">podivilov</who>
    <bug_when>2010-07-13 09:15:08 -0700</bug_when>
    <thetext>(In reply to comment #5)
&gt; Just a quick note to let you know I&apos;m looking at this right now.

Andreas, it&apos;s my bad, this is actually the same problem as https://bugs.webkit.org/show_bug.cgi?id=38108, and absolutely not connected to r63192. For some reason image diff was first detected by buildbots just after your change.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>250547</commentid>
    <comment_count>7</comment_count>
    <who name="Pavel Podivilov">podivilov</who>
    <bug_when>2010-07-13 09:19:25 -0700</bug_when>
    <thetext>

*** This bug has been marked as a duplicate of bug 38108 ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>250551</commentid>
    <comment_count>8</comment_count>
    <who name="Andreas Kling">kling</who>
    <bug_when>2010-07-13 09:22:36 -0700</bug_when>
    <thetext>Not so fast, this actually did regress with my change (r63192). Didn&apos;t notice it since we don&apos;t have pixel tests for Qt. ;(
Working on a fix right now.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>250559</commentid>
    <comment_count>9</comment_count>
    <who name="Pavel Podivilov">podivilov</who>
    <bug_when>2010-07-13 09:29:41 -0700</bug_when>
    <thetext>If so, could you please revert it?

*** This bug has been marked as a duplicate of bug 38108 ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>250564</commentid>
    <comment_count>10</comment_count>
    <who name="Pavel Podivilov">podivilov</who>
    <bug_when>2010-07-13 09:32:50 -0700</bug_when>
    <thetext>Duplicate -&gt; Unconfirmed</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>250570</commentid>
    <comment_count>11</comment_count>
    <who name="Andreas Kling">kling</who>
    <bug_when>2010-07-13 09:38:48 -0700</bug_when>
    <thetext>Reverted with &lt;http://trac.webkit.org/changeset/63213&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>61383</attachid>
            <date>2010-07-13 08:51:02 -0700</date>
            <delta_ts>2010-07-13 09:15:42 -0700</delta_ts>
            <desc>add contain-and-cover.html to test_expectations</desc>
            <filename>tmp.patch</filename>
            <type>text/plain</type>
            <size>1080</size>
            <attacher name="Pavel Podivilov">podivilov</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv
ZwppbmRleCAxMGI2MmYxLi5kNmYxMTI3IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VM
b2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTIgQEAKKzIwMTAtMDct
MTMgIFBhdmVsIFBvZGl2aWxvdiAgPHBvZGl2aWxvdkBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgW0Nocm9taXVtXSBBZGQgY29u
dGFpbi1hbmQtY292ZXIuaHRtbCB0byB0ZXN0X2V4cGVjdGF0aW9ucy50eHQKKyAgICAgICAgaHR0
cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTQyMTY4CisKKyAgICAgICAgKiBw
bGF0Zm9ybS9jaHJvbWl1bS90ZXN0X2V4cGVjdGF0aW9ucy50eHQ6CisKIDIwMTAtMDctMTMgIEFu
ZHJlYXMgS2xpbmcgIDxhbmRyZWFzLmtsaW5nQG5va2lhLmNvbT4KIAogICAgICAgICBSdWJiZXIt
c3RhbXBlZCBieSBTaW1vbiBIYXVzbWFubi4KZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL3BsYXRm
b3JtL2Nocm9taXVtL3Rlc3RfZXhwZWN0YXRpb25zLnR4dCBiL0xheW91dFRlc3RzL3BsYXRmb3Jt
L2Nocm9taXVtL3Rlc3RfZXhwZWN0YXRpb25zLnR4dAppbmRleCA4ODc2YTFmLi43ZDdmZGU0IDEw
MDY0NAotLS0gYS9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9jaHJvbWl1bS90ZXN0X2V4cGVjdGF0aW9u
cy50eHQKKysrIGIvTGF5b3V0VGVzdHMvcGxhdGZvcm0vY2hyb21pdW0vdGVzdF9leHBlY3RhdGlv
bnMudHh0CkBAIC0zMDEzLDMgKzMwMTMsNSBAQCBCVUdBV09ORyBNQUMgOiBzdmcvY3VzdG9tL2Ns
aXAtcGF0aC1jaGlsZC1jaGFuZ2VzLnN2ZyA9IFRFWFQKIAogQlVHNDg4NTkgV0lOIDogZmFzdC9p
bmxpbmUvY29udGludWF0aW9uLW91dGxpbmVzLXdpdGgtbGF5ZXJzLTIuaHRtbCA9IElNQUdFK1RF
WFQKIAorLy8gU3RhcnRlZCBmYWlsaW5nIGF0IDYzMTkyCitCVUdXSzQyMTY4IFdJTiBMSU5VWCA6
IGZhc3QvYmFja2dyb3VuZHMvc2l6ZS9jb250YWluLWFuZC1jb3Zlci5odG1sID0gSU1BR0UK
</data>

          </attachment>
      

    </bug>

</bugzilla>