<?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>75889</bug_id>
          
          <creation_ts>2012-01-09 14:15:08 -0800</creation_ts>
          <short_desc>[Qt][WK2] Remove workaround introduced into bug 75470</short_desc>
          <delta_ts>2012-01-12 10:05: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>WebKit Qt</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</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>Trivial</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Hugo Parente Lima">hugo.lima</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>hausmann</cc>
    
    <cc>kbalazs</cc>
    
    <cc>vestbo</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>532524</commentid>
    <comment_count>0</comment_count>
    <who name="Hugo Parente Lima">hugo.lima</who>
    <bug_when>2012-01-09 14:15:08 -0800</bug_when>
    <thetext>Remove QML workaround introduced into bug 75470, clip property is working again.
Tested on Qt5 rev. f917ea79ef75917ce9f7d166fd5f8e2b772b9058</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>532531</commentid>
    <comment_count>1</comment_count>
      <attachid>121725</attachid>
    <who name="Hugo Parente Lima">hugo.lima</who>
    <bug_when>2012-01-09 14:24:58 -0800</bug_when>
    <thetext>Created attachment 121725
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>533969</commentid>
    <comment_count>2</comment_count>
      <attachid>121725</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-01-11 09:49:58 -0800</bug_when>
    <thetext>Comment on attachment 121725
Patch

Clearing flags on attachment: 121725

Committed r104718: &lt;http://trac.webkit.org/changeset/104718&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>533971</commentid>
    <comment_count>3</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-01-11 09:50:12 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>534776</commentid>
    <comment_count>4</comment_count>
    <who name="Balazs Kelemen">kbalazs</who>
    <bug_when>2012-01-12 06:12:18 -0800</bug_when>
    <thetext>Well, I think this was a bit too early to remove that since most of us use the qt5 version that we have on the bot and that one would still need the workaround.
Normally Ossy updates it weekly, however now it&apos;s a bit older because qt5 trunk was buggy. Not a big problem of course, but next time please take this into consideration. I think the most straightforward is to use the bot&apos;s qt5 version for development. Anyway, no need to revert this, tomorrow Ossy will update and we can live with that for a day.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>534785</commentid>
    <comment_count>5</comment_count>
    <who name="Tor Arne Vestbø">vestbo</who>
    <bug_when>2012-01-12 06:32:45 -0800</bug_when>
    <thetext>Sorry about that :/</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>534931</commentid>
    <comment_count>6</comment_count>
    <who name="Hugo Parente Lima">hugo.lima</who>
    <bug_when>2012-01-12 10:05:24 -0800</bug_when>
    <thetext>Yep, this is why I did put the hash of Qt5 I used to test it. Anyway my fault for upload the patch too early.

P.S.: I did need to update my Qt5 to avoid some segfaults and slowness on QtQuick that were already fixed, this is why I&apos;m using a Qt different from the used on build bots.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>121725</attachid>
            <date>2012-01-09 14:24:58 -0800</date>
            <delta_ts>2012-01-11 09:49:58 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-75889-20120109192404.patch</filename>
            <type>text/plain</type>
            <size>1131</size>
            <attacher name="Hugo Parente Lima">hugo.lima</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTA0NDY5CmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg
Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggMDM4MTg3N2E0ZmE4ODYxNmM4YmU1NTYxNzJhNWRiMTlj
YjdlNDJjZS4uYmUzZmQ1NzZiYjhmYWZkNGYyOTEwNTkwYmNjMzgzNThhNGNhNDk0NyAxMDA2NDQK
LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEy
IEBACisyMDEyLTAxLTA5ICBIdWdvIFBhcmVudGUgTGltYSAgPGh1Z28ubGltYUBvcGVuYm9zc2Eu
b3JnPgorCisgICAgICAgIFtRdF1bV0syXSBSZW1vdmUgd29ya2Fyb3VuZCBpbnRyb2R1Y2VkIGlu
dG8gYnVnIDc1NDcwCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn
aT9pZD03NTg4OQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAg
ICAgICogTWluaUJyb3dzZXIvcXQvcW1sL0Jyb3dzZXJXaW5kb3cucW1sOgorCiAyMDEyLTAxLTA2
ICBBZGFtIFJvYmVuICA8YXJvYmVuQGFwcGxlLmNvbT4KIAogICAgICAgICBQcmludCBhbiBlcnJv
ciBtZXNzYWdlIHdoZW4gTlJXVCBjYW4ndCBydW4gQXBhY2hlCmRpZmYgLS1naXQgYS9Ub29scy9N
aW5pQnJvd3Nlci9xdC9xbWwvQnJvd3NlcldpbmRvdy5xbWwgYi9Ub29scy9NaW5pQnJvd3Nlci9x
dC9xbWwvQnJvd3NlcldpbmRvdy5xbWwKaW5kZXggZDI2NGI1NTE2MDA5MzYzNGFmZmFjZjRjZGVi
MDFjZjQyYzQ5MTkyYS4uYjMzNzMyZDNmMTU5OTk2MGRlN2FlZGE0YTc2N2IzM2RjM2I0ZWNjZSAx
MDA2NDQKLS0tIGEvVG9vbHMvTWluaUJyb3dzZXIvcXQvcW1sL0Jyb3dzZXJXaW5kb3cucW1sCisr
KyBiL1Rvb2xzL01pbmlCcm93c2VyL3F0L3FtbC9Ccm93c2VyV2luZG93LnFtbApAQCAtMjQ1LDYg
KzI0NSw3IEBAIFJlY3RhbmdsZSB7CiAgICAgICAgICAgICB9CiAgICAgICAgICAgICBUZXh0SW5w
dXQgewogICAgICAgICAgICAgICAgIGlkOiBhZGRyZXNzTGluZQorICAgICAgICAgICAgICAgIGNs
aXA6IHRydWUKICAgICAgICAgICAgICAgICBzZWxlY3RCeU1vdXNlOiB0cnVlCiAgICAgICAgICAg
ICAgICAgZm9udCB7CiAgICAgICAgICAgICAgICAgICAgIHBvaW50U2l6ZTogMTEK
</data>

          </attachment>
      

    </bug>

</bugzilla>