[webkit-reviews] review granted: [Bug 86317] Seamless iframe should not announce a new browsing context : [Attachment 179692] patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Dec 16 23:45:34 PST 2012


Eric Seidel <eric at webkit.org> has granted chris fleizach
<cfleizach at apple.com>'s request for review:
Bug 86317: Seamless iframe should not announce a new browsing context
https://bugs.webkit.org/show_bug.cgi?id=86317

Attachment 179692: patch
https://bugs.webkit.org/attachment.cgi?id=179692&action=review

------- Additional Comments from Eric Seidel <eric at webkit.org>
OK.  I thought there was a FIXME in the code somewhere?  Or an existing bug...
maybe we just discussed this over IRC.	<iframe seamless> is in a 95%-shippable
state.	We are discussing disabling it for Chrome, as it's currently w/o an
owner and needs a few bug fixes before it's robust enough for adoption.  This
is still a good fix to have regardless, of course. :)

Looks like you got a cr-linux failure, btw.


More information about the webkit-reviews mailing list