99289 2012-10-14 20:44:56 -0700 Iframe seamless not applying styles with dynamic iframe content 2013-01-21 11:04:52 -0800 1 1 1 Unclassified WebKit Frames 528+ (Nightly build) Mac OS X 10.7 RESOLVED FIXED P2 Normal --- 45950 1 williamson.michaelr mkwst eric mkwst ojan webkit.review.bot oldest_to_newest 741703 0 williamson.michaelr 2012-10-14 20:44:56 -0700 There were other seamless issues, but I don't believe this is related, so apologies if this turns out to be a duplicate. First noticed on Chrome, reproduced on Webkit Nightly from October 12th. Steps to reproduce the problem: 1. I've attached 2 files: test.html and iframe.html. 2. Download them to the same folder and open test.html locally in your browser 3. You'll notice that the HELLO WORLD text isn't red. 4. Edit text.html to uncomment the iframe tag that has the src="iframe.html" attribute set. Reload test.html, and now you'll see that the text is red. What is the expected behavior? What went wrong? Parent styles in a seamless iframe are being applied correctly only when the iframe content is loaded using a src="" attribute. When the content is loaded dynamically with JavaScript, the parent styles aren't applied, even though it's still a seamless frame. My understanding of the spec is that parent styles should be applied to the seamless frame regardless of where the content comes from, but I might be misunderstanding something. 741705 1 eric 2012-10-14 20:47:26 -0700 Thanks for the bug! 742479 2 168789 williamson.michaelr 2012-10-15 14:50:01 -0700 Created attachment 168789 The testcase that I seem to have forgotten. 812068 3 mkwst 2013-01-21 05:52:29 -0800 I believe this was fixed in bug 103539. I'll add a test to ensure this doesn't regress, but it seems to be working now on trunk. 812070 4 183765 mkwst 2013-01-21 05:57:18 -0800 Created attachment 183765 Patch 812265 5 183765 eric 2013-01-21 10:47:35 -0800 Comment on attachment 183765 Patch ok 812275 6 183765 webkit.review.bot 2013-01-21 11:04:47 -0800 Comment on attachment 183765 Patch Clearing flags on attachment: 183765 Committed r140346: <http://trac.webkit.org/changeset/140346> 812276 7 webkit.review.bot 2013-01-21 11:04:52 -0800 All reviewed patches have been landed. Closing bug. 168789 2012-10-15 14:50:01 -0700 2012-10-15 14:50:01 -0700 The testcase that I seem to have forgotten. seamless_bug.zip application/zip 843 williamson.michaelr UEsDBAoAAAAAAHGtTkEAAAAAAAAAAAAAAAAJABwAc2VhbWxlc3MvVVQJAANlhntQboZ7UHV4CwAB BPUBAAAEFAAAAFBLAwQUAAAACACiVUxBB9Z5OkQAAACAAAAAFAAcAHNlYW1sZXNzL2lmcmFtZS5o dG1sVVQJAAMPSXhQ84N7UHV4CwABBPUBAAAEFAAAAONSAAObjJLcHDsuBSiwScpPqURwgQIpmWUK yTmJxcW2SkWpKUrIcgoKHq4+Pv4K4f5BPi7IevSBmpDM1Ec21EYfYiMAUEsDBBQAAAAIACasTkFZ dXMpEwEAACkCAAASABwAc2VhbWxlc3MvdGVzdC5odG1sVVQJAAP4g3tQ+4N7UHV4CwABBPUBAAAE FAAAAHVSsW6DMBTc+YpXFsgApGtiGKpEaiSkSF2yZKH2a7BkMLLdpKjqv9fENq2Kuh2nu3v3nolI azpRRQCkxYZNwEJtRoEOA+QKGXz6DwAqhVQbsOTWc1+RB/xNNR3+0rbIL63ZwON6PXws5Ixf/9PO 0nudYu5DitCSvEo2+roPWUb8bI1NJ1Br0IqWsSPzacW4yrKw0ULMWdBmTNLYxxaOcnPDOKKp4oNx kmujoBsPLq0E633vsDf5Bc1e4ASfxgNLk5/sZLWdnY7dSWqtISWnsjfWd+I9k7c8JDrTbMjlgH26 +sveFDeYJmQ6LBWN1uU5tu90jit43tf1EU7Hl3pnl7GCKlnYqZAaXao9uV/TXvz+g3wDUEsBAh4D CgAAAAAAca1OQQAAAAAAAAAAAAAAAAkAGAAAAAAAAAAQAO1BAAAAAHNlYW1sZXNzL1VUBQADZYZ7 UHV4CwABBPUBAAAEFAAAAFBLAQIeAxQAAAAIAKJVTEEH1nk6RAAAAIAAAAAUABgAAAAAAAEAAACk gUMAAABzZWFtbGVzcy9pZnJhbWUuaHRtbFVUBQADD0l4UHV4CwABBPUBAAAEFAAAAFBLAQIeAxQA AAAIACasTkFZdXMpEwEAACkCAAASABgAAAAAAAEAAACkgdUAAABzZWFtbGVzcy90ZXN0Lmh0bWxV VAUAA/iDe1B1eAsAAQT1AQAABBQAAABQSwUGAAAAAAMAAwABAQAANAIAAAAA 183765 2013-01-21 05:57:18 -0800 2013-01-21 11:04:47 -0800 Patch bug-99289-20130121145413.patch text/plain 3005 mkwst U3VidmVyc2lvbiBSZXZpc2lvbjogMTQwMzIxCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggYzczNTY2YjZiYjIzNGI5ZjE5OGFl NTYzZmQxYTNhYWM2M2IyMjBlMS4uMWI4ZmZmNTU2Njg1OTk1ZTNjODhiMzFkMjM2MzhhODI4ZWY1 ODU2NiAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDEzLTAxLTIxICBNaWtlIFdlc3QgIDxta3dz dEBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgQWRkIGEgdGVzdCB0byBlbnN1cmUgdGhhdCBjb250 ZW50IHdyaXR0ZW4gaW50byBhIHNlYW1sZXNzIElGcmFtZSBpbmhlcml0cyBzdHlsZS4KKyAgICAg ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTk5Mjg5CisKKyAgICAg ICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVGhpcyBwYXRjaCBhZGRz IGEgc2luZ2xlIHRlc3QgdG8gZW5zdXJlIHRoYXQgY29udGVudCB3cml0dGVuIGludG8gYQorICAg ICAgICBzZWFtbGVzcyBJRnJhbWUgdmlhICdjb250ZW50RG9jdW1lbnQud3JpdGUoKScgcHJvcGVy bHkgaW5oZXJpdHMgc3R5bGUuCisgICAgICAgIFRoZSByb290IG9mIHRoZSBpc3N1ZSB3YXMgZml4 ZWQgaW4gd2tidWcuY29tLzEwMzUzOSwgc28gbm8gY29kZSBjaGFuZ2VzCisgICAgICAgIGFyZSBu ZWNlc2FyeSwgYnV0IHRoYXQgcGF0Y2ggZGlkbid0IHRlc3QgdGhpcyBwYXJ0aWN1bGFyIGVkZ2Ug Y2FzZS4KKworICAgICAgICAqIGZhc3QvZnJhbWVzL3NlYW1sZXNzL3NlYW1sZXNzLWRvY3VtZW50 LXdyaXRlLWV4cGVjdGVkLnR4dDogQWRkZWQuCisgICAgICAgICogZmFzdC9mcmFtZXMvc2VhbWxl c3Mvc2VhbWxlc3MtZG9jdW1lbnQtd3JpdGUuaHRtbDogQWRkZWQuCisKIDIwMTMtMDEtMjEgIEFs ZXhhbmRlciBQYXZsb3YgIDxhcGF2bG92QGNocm9taXVtLm9yZz4KIAogICAgICAgICBXZWIgSW5z cGVjdG9yOiBEbyBub3QgZGlzcGF0Y2ggbW91c2Vtb3ZlIHdoZW4gZW11bGF0aW5nIHRvdWNoIGV2 ZW50IGFuZCBubyB0b3VjaCBpcyBhY3RpdmUKZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL2Zhc3Qv ZnJhbWVzL3NlYW1sZXNzL3NlYW1sZXNzLWRvY3VtZW50LXdyaXRlLWV4cGVjdGVkLnR4dCBiL0xh eW91dFRlc3RzL2Zhc3QvZnJhbWVzL3NlYW1sZXNzL3NlYW1sZXNzLWRvY3VtZW50LXdyaXRlLWV4 cGVjdGVkLnR4dApuZXcgZmlsZSBtb2RlIDEwMDY0NAppbmRleCAwMDAwMDAwMDAwMDAwMDAwMDAw MDAwMDAwMDAwMDAwMDAwMDAwMDAwLi45ZDU3MTRlYTgwOTFhMzZiYWRmMDczNzY0MGNiZGQ3NjRl NTc2ZmUxCi0tLSAvZGV2L251bGwKKysrIGIvTGF5b3V0VGVzdHMvZmFzdC9mcmFtZXMvc2VhbWxl c3Mvc2VhbWxlc3MtZG9jdW1lbnQtd3JpdGUtZXhwZWN0ZWQudHh0CkBAIC0wLDAgKzEsMyBAQAor VGhpcyB0ZXN0IGVuc3VyZXMgdGhhdCBjb250ZW50IHdyaXR0ZW4gaW50byBhIHNlYW1sZXNzIGlm cmFtZSBjb3JyZWN0bHkgaW5oZXJpdHMgc3R5bGUuCitQQVNTIHdpbmRvdy5nZXRDb21wdXRlZFN0 eWxlKGZyYW1lZEVsZW1lbnQpLmdldFByb3BlcnR5Q1NTVmFsdWUoJ2NvbG9yJykuY3NzVGV4dCBp cyAicmdiKDI1NSwgMCwgMCkiCisKZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL2Zhc3QvZnJhbWVz L3NlYW1sZXNzL3NlYW1sZXNzLWRvY3VtZW50LXdyaXRlLmh0bWwgYi9MYXlvdXRUZXN0cy9mYXN0 L2ZyYW1lcy9zZWFtbGVzcy9zZWFtbGVzcy1kb2N1bWVudC13cml0ZS5odG1sCm5ldyBmaWxlIG1v ZGUgMTAwNjQ0CmluZGV4IDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAu LmU2ZTYzZThlMmUwMDdkNTZlYWU1MjBlMDdhMDI1YWU4YmVhNTY2NzUKLS0tIC9kZXYvbnVsbAor KysgYi9MYXlvdXRUZXN0cy9mYXN0L2ZyYW1lcy9zZWFtbGVzcy9zZWFtbGVzcy1kb2N1bWVudC13 cml0ZS5odG1sCkBAIC0wLDAgKzEsMzAgQEAKKzwhRE9DVFlQRSBodG1sPgorPGh0bWw+Cis8aGVh ZD4KKyAgICA8c3R5bGU+CisgICAgICAucmVkIHsgY29sb3I6IHJlZDsgfQorICAgIDwvc3R5bGU+ CisgICAgPHNjcmlwdCBzcmM9Ii4uLy4uL2pzL3Jlc291cmNlcy9qcy10ZXN0LXByZS5qcyI+PC9z Y3JpcHQ+CisgICAgPHNjcmlwdD4KKyAgICAgICAgZnVuY3Rpb24gd3JpdGVUb0ZyYW1lKGh0bWwp IHsKKyAgICAgICAgICAgIHZhciBmcmFtZSA9IGRvY3VtZW50LnF1ZXJ5U2VsZWN0b3IoJ2lmcmFt ZScpOworICAgICAgICAgICAgdmFyIGlmcmFtZURvYyA9IGZyYW1lLmNvbnRlbnRXaW5kb3cuZG9j dW1lbnQ7CisgICAgICAgICAgICBpZnJhbWVEb2Mub3BlbigpOworICAgICAgICAgICAgaWZyYW1l RG9jLndyaXRlKGh0bWwpOworICAgICAgICAgICAgaWZyYW1lRG9jLmNsb3NlKCk7CisgICAgICAg IH0KKworICAgICAgICB3aW5kb3cub25sb2FkID0gZnVuY3Rpb24gKCkgeworICAgICAgICAgICAg ZGVidWcoIlRoaXMgdGVzdCBlbnN1cmVzIHRoYXQgY29udGVudCB3cml0dGVuIGludG8gYSBzZWFt bGVzcyBpZnJhbWUgY29ycmVjdGx5IGluaGVyaXRzIHN0eWxlLiIpOworCisgICAgICAgICAgICB3 cml0ZVRvRnJhbWUoJzxkaXYgY2xhc3M9XCJyZWRcIj5UZXh0LjwvZGl2PicpOworICAgICAgICAg ICAgd2luZG93LmZyYW1lZEVsZW1lbnQgPSBkb2N1bWVudC5xdWVyeVNlbGVjdG9yKCdpZnJhbWUn KS5jb250ZW50RG9jdW1lbnQucXVlcnlTZWxlY3RvcignZGl2Jyk7CisKKyAgICAgICAgICAgIHNo b3VsZEJlRXF1YWxUb1N0cmluZygid2luZG93LmdldENvbXB1dGVkU3R5bGUoZnJhbWVkRWxlbWVu dCkuZ2V0UHJvcGVydHlDU1NWYWx1ZSgnY29sb3InKS5jc3NUZXh0IiwgInJnYigyNTUsIDAsIDAp Iik7CisgICAgICAgIH07CisgICAgPC9zY3JpcHQ+Cis8L2hlYWQ+Cis8Ym9keT4KKyAgICA8aWZy YW1lIHNlYW1sZXNzPjwvaWZyYW1lPgorPC9ib2R5PgorPC9odG1sPgo=