<?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>87646</bug_id>
          
          <creation_ts>2012-05-28 03:15:51 -0700</creation_ts>
          <short_desc>Fix ENABLE_IFRAME_SEAMLESS to actually fully disable &lt;iframe seamless&gt;</short_desc>
          <delta_ts>2012-09-18 12:01:58 -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>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>45950</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Eric Seidel (no email)">eric</reporter>
          <assigned_to name="Eric Seidel (no email)">eric</assigned_to>
          <cc>abarth</cc>
    
    <cc>aestes</cc>
    
    <cc>bdakin</cc>
    
    <cc>jberlin</cc>
    
    <cc>lforschler</cc>
    
    <cc>sam</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>635081</commentid>
    <comment_count>0</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2012-05-28 03:15:51 -0700</bug_when>
    <thetext>Fix ENABLE_IFRAME_SEAMLESS to actually fully disable &lt;iframe seamless&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>635084</commentid>
    <comment_count>1</comment_count>
      <attachid>144320</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2012-05-28 03:18:46 -0700</bug_when>
    <thetext>Created attachment 144320
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>635098</commentid>
    <comment_count>2</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2012-05-28 03:51:21 -0700</bug_when>
    <thetext>Apple may want to pull this into their release branches, since I believe they&apos;ve disabled seamless there.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>636380</commentid>
    <comment_count>3</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2012-05-29 13:23:19 -0700</bug_when>
    <thetext>&lt;rdar://problem/11550608&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>636752</commentid>
    <comment_count>4</comment_count>
      <attachid>144320</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-05-29 19:28:33 -0700</bug_when>
    <thetext>Comment on attachment 144320
Patch

Clearing flags on attachment: 144320

Committed r118883: &lt;http://trac.webkit.org/changeset/118883&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>636753</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-05-29 19:28:38 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>637590</commentid>
    <comment_count>6</comment_count>
    <who name="Sam Weinig">sam</who>
    <bug_when>2012-05-30 13:13:23 -0700</bug_when>
    <thetext>What is the symptom if this bug is not incorporated?  Is there a user visible effect?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>637614</commentid>
    <comment_count>7</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2012-05-30 13:27:29 -0700</bug_when>
    <thetext>&gt; What is the symptom if this bug is not incorporated?  Is there a user visible effect?

The symptom is that an iframe with the seamless attribute would exhibit some of the seamless behaviors (e.g., style inheritance), but not all of them (e.g., redirected navigation).  The effect should only be visible in content that attempts to use the seamless attribute.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>637706</commentid>
    <comment_count>8</comment_count>
    <who name="Sam Weinig">sam</who>
    <bug_when>2012-05-30 14:41:12 -0700</bug_when>
    <thetext>(In reply to comment #7)
&gt; &gt; What is the symptom if this bug is not incorporated?  Is there a user visible effect?
&gt; 
&gt; The symptom is that an iframe with the seamless attribute would exhibit some of the seamless behaviors (e.g., style inheritance), but not all of them (e.g., redirected navigation).  The effect should only be visible in content that attempts to use the seamless attribute.

Thanks Adam.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>144320</attachid>
            <date>2012-05-28 03:18:46 -0700</date>
            <delta_ts>2012-05-29 19:28:33 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-87646-20120528031845.patch</filename>
            <type>text/plain</type>
            <size>2715</size>
            <attacher name="Eric Seidel (no email)">eric</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTE4NjYwCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggMTYzNDMwMjc4MmVmN2Zk
ZGMxY2MwYTRjNGE2YTFkYzZhN2RhMDM0Ny4uNmFkYTFkYTA2MmRhNDhjNjVhZmIyZDM5YTY3MjEz
NWZiNjliMGQzMCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDI2IEBACisyMDEyLTA1LTI4ICBFcmlj
IFNlaWRlbCAgPGVyaWNAd2Via2l0Lm9yZz4KKworICAgICAgICBGaXggRU5BQkxFX0lGUkFNRV9T
RUFNTEVTUyB0byBhY3R1YWxseSBmdWxseSBkaXNhYmxlIDxpZnJhbWUgc2VhbWxlc3M+CisgICAg
ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04NzY0NgorCisgICAg
ICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEluIHRoZSBwcm9jZXNz
IG9mIG1vdmluZyB0aGUgc2VhbWxlc3MgZmVhdHVyZSBvdXQgb2YgZ2l0aHViIGFuZCBpbnRvIGJ1
Z3Mud2Via2l0Lm9yZworICAgICAgICBtdWx0aXBsZSB2ZXJzaW9ucyBvZiB0aGUgc2hvdWxkRGlz
cGxheVNlYW1sZXNzbHkgZnVuY3Rpb24gZ290IHdyaXR0ZW4KKyAgICAgICAgKGFuZCBtb3ZlZCBm
cm9tIEhUTUxJRnJhbWVFbGVtZW50IHRvIERvY3VtZW50KSwgYnV0IG9ubHkgb25lIG9mIHRoZW0g
d2FzIHdyYXBwZWQKKyAgICAgICAgaW4gRU5BQkxFX0lGUkFNRV9TRUFNTEVTUy4gIEhUTUxJRnJh
bWVFbGVtZW50IHdhcyBjaGVja2luZyBtYXlEaXNwbGF5U2VhbWxlc3NseQorICAgICAgICBkaXJl
Y3RseSAoYXMgd2FzIG15IG9yaWdpbmFsIGRlc2lnbiksIHdoaWNoIGdvdCBhcm91bmQgdGhlIEVO
QUJMRV9JRlJBTUVfU0VBTUxFU1MgY2hlY2suCisgICAgICAgIEkndmUgZml4ZWQgdGhpcyBvdmVy
c2lnaHQsIGFuZCB0aGUgZmVhdHVyZSBpcyBub3cgb2ZmIHdoZW4gd2UgdGVsbCBpdCB0byBiZSBv
ZmYuCisKKyAgICAgICAgVGhpcyBpcyBjb3ZlcmVkIGJ5IG1hbnkgZXhpc3RpbmcgdGVzdHMuICBJ
J3ZlIHZlcmlmaWVkIGxvY2FsbHkgdGhhdAorICAgICAgICBhbGwgdGVzdHMgZmFpbCB3aGVuIEVO
QUJMRV9JRlJBTUVfU0VBTUxFU1MgaXMgZGlzYWJsZWQgaW5zdGVhZCBvZgorICAgICAgICBvbmx5
IHNvbWUgb2YgdGhlbS4KKworICAgICAgICAqIGRvbS9TZWN1cml0eUNvbnRleHQuaDoKKyAgICAg
ICAgKFNlY3VyaXR5Q29udGV4dCk6CisgICAgICAgICogaHRtbC9IVE1MSUZyYW1lRWxlbWVudC5j
cHA6CisgICAgICAgIChXZWJDb3JlOjpIVE1MSUZyYW1lRWxlbWVudDo6c2hvdWxkRGlzcGxheVNl
YW1sZXNzbHkpOgorCiAyMDEyLTA1LTI4ICBTaGVyaWZmIEJvdCAgPHdlYmtpdC5yZXZpZXcuYm90
QGdtYWlsLmNvbT4KIAogICAgICAgICBVbnJldmlld2VkLCByb2xsaW5nIG91dCByMTE4NjUwLgpk
aWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvZG9tL1NlY3VyaXR5Q29udGV4dC5oIGIvU291cmNl
L1dlYkNvcmUvZG9tL1NlY3VyaXR5Q29udGV4dC5oCmluZGV4IGZlYmQxMjI2MTFhY2I3ZTA4NzJj
YzA3MDY2ZTJjMGNmODMzMzdkZjMuLjdiNDc0NmQxMzdlOGEzNzMzYmFjNTEyZDMwZjU4M2UzZjE1
YWUyNGYgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL2RvbS9TZWN1cml0eUNvbnRleHQuaAor
KysgYi9Tb3VyY2UvV2ViQ29yZS9kb20vU2VjdXJpdHlDb250ZXh0LmgKQEAgLTY3LDggKzY3LDYg
QEAgcHVibGljOgogCiAgICAgc3RhdGljIFNhbmRib3hGbGFncyBwYXJzZVNhbmRib3hQb2xpY3ko
Y29uc3QgU3RyaW5nJiBwb2xpY3kpOwogCi0gICAgYm9vbCBtYXlEaXNwbGF5U2VhbWxlc3NXaXRo
UGFyZW50KCkgY29uc3QgeyByZXR1cm4gbV9tYXlEaXNwbGF5U2VhbWxlc3NXaXRoUGFyZW50OyB9
Ci0KIHByb3RlY3RlZDoKICAgICBTZWN1cml0eUNvbnRleHQoKTsKICAgICB+U2VjdXJpdHlDb250
ZXh0KCk7CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9odG1sL0hUTUxJRnJhbWVFbGVtZW50
LmNwcCBiL1NvdXJjZS9XZWJDb3JlL2h0bWwvSFRNTElGcmFtZUVsZW1lbnQuY3BwCmluZGV4IGRj
NGQ0MzU4NjVjYjgzMGE5NGY1NmNmMzZkM2MyYzMwOTA5MmMxYmUuLjBlYmUwOTQ2OWM2YzA0MTE1
YmY5OGFiNTA1NmRlYjJjZmEyYzhlYWIgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL2h0bWwv
SFRNTElGcmFtZUVsZW1lbnQuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL2h0bWwvSFRNTElGcmFt
ZUVsZW1lbnQuY3BwCkBAIC0xMjIsNyArMTIyLDcgQEAgdm9pZCBIVE1MSUZyYW1lRWxlbWVudDo6
cmVtb3ZlZEZyb20oQ29udGFpbmVyTm9kZSogaW5zZXJ0aW9uUG9pbnQpCiAKIGJvb2wgSFRNTElG
cmFtZUVsZW1lbnQ6OnNob3VsZERpc3BsYXlTZWFtbGVzc2x5KCkgY29uc3QKIHsKLSAgICByZXR1
cm4gY29udGVudERvY3VtZW50KCkgJiYgY29udGVudERvY3VtZW50KCktPm1heURpc3BsYXlTZWFt
bGVzc1dpdGhQYXJlbnQoKSAmJiBoYXNBdHRyaWJ1dGUoc2VhbWxlc3NBdHRyKTsKKyAgICByZXR1
cm4gY29udGVudERvY3VtZW50KCkgJiYgY29udGVudERvY3VtZW50KCktPnNob3VsZERpc3BsYXlT
ZWFtbGVzc2x5V2l0aFBhcmVudCgpOwogfQogCiB2b2lkIEhUTUxJRnJhbWVFbGVtZW50OjpkaWRS
ZWNhbGNTdHlsZShTdHlsZUNoYW5nZSBzdHlsZUNoYW5nZSkK
</data>

          </attachment>
      

    </bug>

</bugzilla>