<?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>36553</bug_id>
          
          <creation_ts>2010-03-24 13:50:21 -0700</creation_ts>
          <short_desc>[Qt] Missing QWebSettings for Frame Flattening</short_desc>
          <delta_ts>2010-03-26 07:05:40 -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>New Bugs</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>CLOSED</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>
          
          <blocked>35784</blocked>
    
    <blocked>36558</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Jesus Sanchez-Palencia">jesus</reporter>
          <assigned_to name="Jesus Sanchez-Palencia">jesus</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>eric</cc>
    
    <cc>hausmann</cc>
    
    <cc>kenneth</cc>
    
    <cc>ossy</cc>
    
    <cc>tonikitoo</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>203560</commentid>
    <comment_count>0</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2010-03-24 13:50:21 -0700</bug_when>
    <thetext>[Qt] Missing QWebSettings for Frame Flattening</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>203564</commentid>
    <comment_count>1</comment_count>
      <attachid>51539</attachid>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2010-03-24 13:54:58 -0700</bug_when>
    <thetext>Created attachment 51539
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>203622</commentid>
    <comment_count>2</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2010-03-24 15:03:16 -0700</bug_when>
    <thetext>This same setting will also set iFrame Flattening when available.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>203639</commentid>
    <comment_count>3</comment_count>
    <who name="Kenneth Rohde Christiansen">kenneth</who>
    <bug_when>2010-03-24 15:12:19 -0700</bug_when>
    <thetext>Looks good, but it would be nice if one from the doc team could review the documentation.

Simon, could you get one to do that?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>203644</commentid>
    <comment_count>4</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2010-03-24 15:15:50 -0700</bug_when>
    <thetext>(In reply to comment #3)
&gt; Looks good, but it would be nice if one from the doc team could review the
&gt; documentation.
&gt; 
&gt; Simon, could you get one to do that?

Yes, this would be nice. Also I noticed that documentation for QWebSettings:WebGL is missing. Should I open a bug report for that or the doc team is working on this?!

thanks for the review!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>203690</commentid>
    <comment_count>5</comment_count>
      <attachid>51539</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-03-24 17:02:45 -0700</bug_when>
    <thetext>Comment on attachment 51539
Patch

Clearing flags on attachment: 51539

Committed r56473: &lt;http://trac.webkit.org/changeset/56473&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>203691</commentid>
    <comment_count>6</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-03-24 17:02:49 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>203873</commentid>
    <comment_count>7</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2010-03-25 00:42:53 -0700</bug_when>
    <thetext>Guys, you broke ( but most of all fix :) ) fast/frames/flattening/frameset-flattening-grid.html with this patch
http://build.webkit.org/results/Qt%20Linux%20Release/r56474%20%289010%29/results.html

Could you update the platform specific expected file if the new result is correct?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204075</commentid>
    <comment_count>8</comment_count>
    <who name="Antonio Gomes">tonikitoo</who>
    <bug_when>2010-03-25 08:25:01 -0700</bug_when>
    <thetext>(In reply to comment #7)
&gt; Guys, you broke ( but most of all fix :) )
&gt; fast/frames/flattening/frameset-flattening-grid.html with this patch
&gt; http://build.webkit.org/results/Qt%20Linux%20Release/r56474%20%289010%29/results.html
&gt; 
&gt; Could you update the platform specific expected file if the new result is
&gt; correct?

diego found the problem, got a patch, kenneth reviewed and i landed.

r56547</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204182</commentid>
    <comment_count>9</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-03-25 11:59:56 -0700</bug_when>
    <thetext>(In reply to comment #8)
&gt; (In reply to comment #7)
&gt; &gt; Guys, you broke ( but most of all fix :) )
&gt; &gt; fast/frames/flattening/frameset-flattening-grid.html with this patch
&gt; &gt; http://build.webkit.org/results/Qt%20Linux%20Release/r56474%20%289010%29/results.html
&gt; &gt; 
&gt; &gt; Could you update the platform specific expected file if the new result is
&gt; &gt; correct?
&gt; 
&gt; diego found the problem, got a patch, kenneth reviewed and i landed.
&gt; 
&gt; r56547

Leopard is still broken.

The patch you committed did not reference this bug. :(</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204194</commentid>
    <comment_count>10</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2010-03-25 12:43:47 -0700</bug_when>
    <thetext>(In reply to comment #9)
&gt; Leopard is still broken.
&gt; 
&gt; The patch you committed did not reference this bug. :(
Needless newline removed by http://trac.webkit.org/changeset/56561
and Leopard is green again.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204520</commentid>
    <comment_count>11</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2010-03-26 05:41:00 -0700</bug_when>
    <thetext>cherry-pick-for-backport: &lt;r56547&gt;
cherry-pick-for-backport: &lt;r56561&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204522</commentid>
    <comment_count>12</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2010-03-26 05:42:13 -0700</bug_when>
    <thetext>cherry-pick-for-backport: &lt;r56561&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204537</commentid>
    <comment_count>13</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2010-03-26 06:29:56 -0700</bug_when>
    <thetext>cherry-pick-for-backport: &lt;r56557&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204545</commentid>
    <comment_count>14</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2010-03-26 07:04:59 -0700</bug_when>
    <thetext>Revision r56473 cherry-picked into qtwebkit-2.0 with commit 289b7bf993b1cb6c896142c868c674925182a197</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204546</commentid>
    <comment_count>15</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2010-03-26 07:05:09 -0700</bug_when>
    <thetext>Revision r56547 cherry-picked into qtwebkit-2.0 with commit c4fe1ca4a566b93a2f35a0d112fc1b2da12ba2dc</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204547</commentid>
    <comment_count>16</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2010-03-26 07:05:18 -0700</bug_when>
    <thetext>Revision r56557 cherry-picked into qtwebkit-2.0 with commit 9b559418e4e199551fb30218390d03e596f80f9b</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204548</commentid>
    <comment_count>17</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2010-03-26 07:05:29 -0700</bug_when>
    <thetext>Revision r56561 cherry-picked into qtwebkit-2.0 with commit 1989edbb25a211a0638ff8369999e02d254c8630</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>51539</attachid>
            <date>2010-03-24 13:54:58 -0700</date>
            <delta_ts>2010-03-24 17:02:44 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-36553-20100324175657.patch</filename>
            <type>text/plain</type>
            <size>3317</size>
            <attacher name="Jesus Sanchez-Palencia">jesus</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdC9xdC9BcGkvcXdlYnNldHRpbmdzLmNwcCBiL1dlYktpdC9xdC9B
cGkvcXdlYnNldHRpbmdzLmNwcAppbmRleCBjYTdkYWYwLi5kZWY0MjJkIDEwMDY0NAotLS0gYS9X
ZWJLaXQvcXQvQXBpL3F3ZWJzZXR0aW5ncy5jcHAKKysrIGIvV2ViS2l0L3F0L0FwaS9xd2Vic2V0
dGluZ3MuY3BwCkBAIC0xOTUsNiArMTk1LDEwIEBAIHZvaWQgUVdlYlNldHRpbmdzUHJpdmF0ZTo6
YXBwbHkoKQogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBnbG9iYWwtPmF0
dHJpYnV0ZXMudmFsdWUoUVdlYlNldHRpbmdzOjpEZXZlbG9wZXJFeHRyYXNFbmFibGVkKSk7CiAg
ICAgICAgIHNldHRpbmdzLT5zZXREZXZlbG9wZXJFeHRyYXNFbmFibGVkKHZhbHVlKTsKIAorICAg
ICAgICB2YWx1ZSA9IGF0dHJpYnV0ZXMudmFsdWUoUVdlYlNldHRpbmdzOjpGcmFtZUZsYXR0ZW5p
bmdFbmFibGVkLAorICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBnbG9iYWwt
PmF0dHJpYnV0ZXMudmFsdWUoUVdlYlNldHRpbmdzOjpGcmFtZUZsYXR0ZW5pbmdFbmFibGVkKSk7
CisgICAgICAgIHNldHRpbmdzLT5zZXRGcmFtZVNldEZsYXR0ZW5pbmdFbmFibGVkKHZhbHVlKTsK
KwogICAgICAgICBRVXJsIGxvY2F0aW9uID0gIXVzZXJTdHlsZVNoZWV0TG9jYXRpb24uaXNFbXB0
eSgpID8gdXNlclN0eWxlU2hlZXRMb2NhdGlvbiA6IGdsb2JhbC0+dXNlclN0eWxlU2hlZXRMb2Nh
dGlvbjsKICAgICAgICAgc2V0dGluZ3MtPnNldFVzZXJTdHlsZVNoZWV0TG9jYXRpb24oV2ViQ29y
ZTo6S1VSTChsb2NhdGlvbikpOwogCkBAIC00MDEsNiArNDA1LDEyIEBAIFFXZWJTZXR0aW5ncyog
UVdlYlNldHRpbmdzOjpnbG9iYWxTZXR0aW5ncygpCiAgICAgICAgIFFHcmFwaGljc1dlYlZpZXcs
IGFjY2VsZXJhdGVzIGFuaW1hdGlvbnMgb2Ygd2ViIGNvbnRlbnQuIENTUyBhbmltYXRpb25zIG9m
IHRoZSB0cmFuc2Zvcm0gYW5kCiAgICAgICAgIG9wYWNpdHkgcHJvcGVydGllcyB3aWxsIGJlIHJl
bmRlcmVkIGJ5IGNvbXBvc2luZyB0aGUgY2FjaGVkIGNvbnRlbnQgb2YgdGhlIGFuaW1hdGVkIGVs
ZW1lbnRzLgogICAgICAgICBUaGlzIGZlYXR1cmUgaXMgZW5hYmxlZCBieSBkZWZhdWx0CisgICAg
XHZhbHVlIEZyYW1lRmxhdHRlbmluZ0VuYWJsZWQgV2l0aCB0aGlzIHNldHRpbmcgZWFjaCBzdWJm
cmFtZSBpcyBleHBhbmRlZCB0byBpdHMgY29udGVudHMuCisgICAgICAgIE9uIHRvdWNoIGRldmlj
ZXMsIGl0IGlzIGRlc2lyZWQgdG8gbm90IGhhdmUgYW55IHNjcm9sbGFibGUgc3ViIHBhcnRzIG9m
IHRoZSBwYWdlCisgICAgICAgIGFzIGl0IHJlc3VsdHMgaW4gYSBjb25mdXNpbmcgdXNlciBleHBl
cmllbmNlLCB3aXRoIHNjcm9sbGluZyBzb21ldGltZXMgc2Nyb2xsaW5nIHN1YiBwYXJ0cworICAg
ICAgICBhbmQgYXQgb3RoZXIgdGltZXMgc2Nyb2xsaW5nIHRoZSBwYWdlIGl0c2VsZi4gRm9yIHRo
aXMgcmVhc29uIGlmcmFtZXMgYW5kIGZyYW1lc2V0cyBhcmUKKyAgICAgICAgYmFyZWx5IHVzYWJs
ZSBvbiB0b3VjaCBkZXZpY2VzLiBUaGlzIHdpbGwgZmxhdHRlbiBhbGwgdGhlIGZyYW1lcyB0byBi
ZWNvbWUgb25lIHNjcm9sbGFibGUgcGFnZS4KKyAgICAgICAgRGlzYWJsZWQgYnkgZGVmYXVsdC4K
ICovCiAKIC8qIQpAQCAtNDM1LDYgKzQ0NSw3IEBAIFFXZWJTZXR0aW5nczo6UVdlYlNldHRpbmdz
KCkKICAgICBkLT5hdHRyaWJ1dGVzLmluc2VydChRV2ViU2V0dGluZ3M6OkxvY2FsQ29udGVudENh
bkFjY2Vzc0ZpbGVVcmxzLCB0cnVlKTsKICAgICBkLT5hdHRyaWJ1dGVzLmluc2VydChRV2ViU2V0
dGluZ3M6OkFjY2VsZXJhdGVkQ29tcG9zaXRpbmdFbmFibGVkLCB0cnVlKTsKICAgICBkLT5hdHRy
aWJ1dGVzLmluc2VydChRV2ViU2V0dGluZ3M6OldlYkdMRW5hYmxlZCwgZmFsc2UpOworICAgIGQt
PmF0dHJpYnV0ZXMuaW5zZXJ0KFFXZWJTZXR0aW5nczo6RnJhbWVGbGF0dGVuaW5nRW5hYmxlZCwg
ZmFsc2UpOwogICAgIGQtPm9mZmxpbmVTdG9yYWdlRGVmYXVsdFF1b3RhID0gNSAqIDEwMjQgKiAx
MDI0OwogICAgIGQtPmRlZmF1bHRUZXh0RW5jb2RpbmcgPSBRTGF0aW4xU3RyaW5nKCJpc28tODg1
OS0xIik7CiB9CmRpZmYgLS1naXQgYS9XZWJLaXQvcXQvQXBpL3F3ZWJzZXR0aW5ncy5oIGIvV2Vi
S2l0L3F0L0FwaS9xd2Vic2V0dGluZ3MuaAppbmRleCBkYzJiY2UwLi5jYWJhMGQ1IDEwMDY0NAot
LS0gYS9XZWJLaXQvcXQvQXBpL3F3ZWJzZXR0aW5ncy5oCisrKyBiL1dlYktpdC9xdC9BcGkvcXdl
YnNldHRpbmdzLmgKQEAgLTczLDcgKzczLDggQEAgcHVibGljOgogICAgICAgICBXZWJHTEVuYWJs
ZWQsCiAgICAgICAgIFNwYXRpYWxOYXZpZ2F0aW9uRW5hYmxlZCwKICAgICAgICAgTG9jYWxDb250
ZW50Q2FuQWNjZXNzRmlsZVVybHMsCi0gICAgICAgIFRpbGVkQmFja2luZ1N0b3JlRW5hYmxlZAor
ICAgICAgICBUaWxlZEJhY2tpbmdTdG9yZUVuYWJsZWQsCisgICAgICAgIEZyYW1lRmxhdHRlbmlu
Z0VuYWJsZWQKICAgICB9OwogICAgIGVudW0gV2ViR3JhcGhpYyB7CiAgICAgICAgIE1pc3NpbmdJ
bWFnZUdyYXBoaWMsCmRpZmYgLS1naXQgYS9XZWJLaXQvcXQvQ2hhbmdlTG9nIGIvV2ViS2l0L3F0
L0NoYW5nZUxvZwppbmRleCA2MGI2NWE3Li5mZjlmNWU5IDEwMDY0NAotLS0gYS9XZWJLaXQvcXQv
Q2hhbmdlTG9nCisrKyBiL1dlYktpdC9xdC9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNyBAQAorMjAx
MC0wMy0yNCAgSmVzdXMgU2FuY2hlei1QYWxlbmNpYSAgPGplc3VzLnBhbGVuY2lhQG9wZW5ib3Nz
YS5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg
QWRkIHRoZSBGcmFtZUZsYXR0ZW5pbmdFbmFibGVkIFdlYkF0dHJpYnV0ZSB0byBRV2ViU2V0dGlu
Z3MuCisKKyAgICAgICAgW1F0XSBNaXNzaW5nIFFXZWJTZXR0aW5ncyBmb3IgRnJhbWUgRmxhdHRl
bmluZworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzY1
NTMKKworICAgICAgICAqIEFwaS9xd2Vic2V0dGluZ3MuY3BwOgorICAgICAgICAoUVdlYlNldHRp
bmdzUHJpdmF0ZTo6YXBwbHkpOgorICAgICAgICAoUVdlYlNldHRpbmdzOjpRV2ViU2V0dGluZ3Mp
OgorICAgICAgICAqIEFwaS9xd2Vic2V0dGluZ3MuaDoKKwogMjAxMC0wMy0yMyAgQW5kZXJzIEJh
a2tlbiA8YW5kZXJzLmJha2tlbkBub2tpYS5jb20+LCBKZXN1cyBTYW5jaGV6LVBhbGVuY2lhIDxq
ZXN1cy5wYWxlbmNpYUBvcGVuYm9zc2Eub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IE5PQk9E
WSAoT09QUyEpLgo=
</data>

          </attachment>
      

    </bug>

</bugzilla>