<?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>36348</bug_id>
          
          <creation_ts>2010-03-19 00:16:59 -0700</creation_ts>
          <short_desc>[Qt] Disable TILED_BACKING_STORE if Qt version is earlier than Qt4.6</short_desc>
          <delta_ts>2010-03-19 09:42:00 -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>WebCore Misc.</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</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>Qt</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Laszlo Gombos">laszlo.gombos</reporter>
          <assigned_to name="QtWebKit Unassigned">webkit-qt-unassigned</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>hausmann</cc>
    
    <cc>koivisto</cc>
    
    <cc>tonikitoo</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>201749</commentid>
    <comment_count>0</comment_count>
    <who name="Laszlo Gombos">laszlo.gombos</who>
    <bug_when>2010-03-19 00:16:59 -0700</bug_when>
    <thetext>It seems that the QtWebKit tiled backing store needs the QGraphicsItem::scale function (seeQGraphicsWebViewPrivate::updateTiledBackingStoreScale).

I propose to turn off TILED_BACKING_STORE if Qt version is earlier than Qt4.6 (instead of making it work on Qt 4.5 or earlier).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201752</commentid>
    <comment_count>1</comment_count>
      <attachid>51130</attachid>
    <who name="Laszlo Gombos">laszlo.gombos</who>
    <bug_when>2010-03-19 00:21:11 -0700</bug_when>
    <thetext>Created attachment 51130
proposed patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201763</commentid>
    <comment_count>2</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2010-03-19 00:35:41 -0700</bug_when>
    <thetext>QGraphicsItem::scale is present in Qt 4.5, but it&apos;s not a Q_PROPERTY in QGraphicsObject.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201867</commentid>
    <comment_count>3</comment_count>
    <who name="Antonio Gomes">tonikitoo</who>
    <bug_when>2010-03-19 08:49:56 -0700</bug_when>
    <thetext>i confess I do not know what is the current minimum version of Qt that QtWebKit has to support. 

one mentioned it is current release (4.6) and the next (4.7). If so, should we care much about 4.5?

just out of curiosity.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201897</commentid>
    <comment_count>4</comment_count>
      <attachid>51130</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-03-19 09:41:55 -0700</bug_when>
    <thetext>Comment on attachment 51130
proposed patch

Clearing flags on attachment: 51130

Committed r56243: &lt;http://trac.webkit.org/changeset/56243&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>201898</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-03-19 09:42:00 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>51130</attachid>
            <date>2010-03-19 00:21:11 -0700</date>
            <delta_ts>2010-03-19 09:41:55 -0700</delta_ts>
            <desc>proposed patch</desc>
            <filename>36348.patch</filename>
            <type>text/plain</type>
            <size>1333</size>
            <attacher name="Laszlo Gombos">laszlo.gombos</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA1NjIyMSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTcgQEAKKzIwMTAtMDMtMTkgIExhc3psbyBHb21ib3MgIDxsYXN6bG8uMS5nb21i
b3NAbm9raWEuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisg
ICAgICAgIFtRdF0gRGlzYWJsZSBUSUxFRF9CQUNLSU5HX1NUT1JFIGlmIFF0IHZlcnNpb24gaXMg
ZWFybGllciB0aGFuIFF0NC42CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3df
YnVnLmNnaT9pZD0zNjM0OAorCisgICAgICAgIEJhY2tpbmcgc3RvcmUgaW1wbGVtZW50YXRpb24g
Zm9yIFF0V2ViS2l0IHJlcXVpcmVzIGF0IGxlYXN0IFF0CisgICAgICAgIHZlcnNpb24gNC42Lgor
CisgICAgICAgIE5vIG5ldyB0ZXN0cyBhcyB0aGVyZSBpcyBubyBuZXcgZnVuY3Rpb25hbGl0eS4K
KworICAgICAgICAqIFdlYkNvcmUucHJpOgorCiAyMDEwLTAzLTE4ICBDaHJpcyBFdmFucyAgPGNl
dmFuc0BjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgTmlrb2xhcyBaaW1tZXJt
YW5uLgpJbmRleDogV2ViQ29yZS9XZWJDb3JlLnByaQo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL1dl
YkNvcmUucHJpCShyZXZpc2lvbiA1NjIxMikKKysrIFdlYkNvcmUvV2ViQ29yZS5wcmkJKHdvcmtp
bmcgY29weSkKQEAgLTgwLDcgKzgwLDkgQEAgZ3JlYXRlclRoYW4oUVRfTUlOT1JfVkVSU0lPTiwg
NSkgewogIWNvbnRhaW5zKERFRklORVMsIEVOQUJMRV9EQVRBTElTVD0uKTogREVGSU5FUyArPSBF
TkFCTEVfREFUQUxJU1Q9MQogCiAjIFRpbGVkIEJhY2tpbmcgU3RvcmUgc3VwcG9ydAotIWNvbnRh
aW5zKERFRklORVMsIEVOQUJMRV9USUxFRF9CQUNLSU5HX1NUT1JFPS4pOiBERUZJTkVTICs9IEVO
QUJMRV9USUxFRF9CQUNLSU5HX1NUT1JFPTEKK2dyZWF0ZXJUaGFuKFFUX01JTk9SX1ZFUlNJT04s
IDUpIHsKKyAgICAhY29udGFpbnMoREVGSU5FUywgRU5BQkxFX1RJTEVEX0JBQ0tJTkdfU1RPUkU9
Lik6IERFRklORVMgKz0gRU5BQkxFX1RJTEVEX0JBQ0tJTkdfU1RPUkU9MQorfQogCiAjIE5lc2Nh
cGUgcGx1Z2lucyBzdXBwb3J0IChOUEFQSSkKICFjb250YWlucyhERUZJTkVTLCBFTkFCTEVfTkVU
U0NBUEVfUExVR0lOX0FQST0uKSB7Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>