Bug 113077 - REGRESSION(r143102): iframe with percentage height within table with anonymous cell fails
Summary: REGRESSION(r143102): iframe with percentage height within table with anonymou...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Major
Assignee: zalan
URL:
Keywords: InRadar
Depends on: 14762
Blocks:
  Show dependency treegraph
 
Reported: 2013-03-22 09:58 PDT by zalan
Modified: 2013-03-27 16:58 PDT (History)
10 users (show)

See Also:


Attachments
iframe testcase (1.62 KB, text/html)
2013-03-22 09:59 PDT, zalan
no flags Details
iframe testcase (620 bytes, text/html)
2013-03-22 10:05 PDT, zalan
no flags Details
Patch (6.10 KB, patch)
2013-03-22 11:51 PDT, zalan
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description zalan 2013-03-22 09:58:33 PDT
iframe in strict mode should take 100% height instead of auto.
see test case
Comment 1 zalan 2013-03-22 09:59:28 PDT
Created attachment 194578 [details]
iframe testcase
Comment 2 zalan 2013-03-22 10:05:33 PDT
Created attachment 194579 [details]
iframe testcase
Comment 3 zalan 2013-03-22 10:07:41 PDT
<rdar://problem/13396183>
Comment 4 zalan 2013-03-22 11:51:18 PDT
Created attachment 194608 [details]
Patch
Comment 5 Robert Hogan 2013-03-27 00:54:03 PDT
Looks good to me. Thanks for spotting the mistake and fixing it.
Comment 6 Antti Koivisto 2013-03-27 05:23:28 PDT
Comment on attachment 194608 [details]
Patch

r=me
Comment 7 WebKit Review Bot 2013-03-27 16:58:07 PDT
Comment on attachment 194608 [details]
Patch

Clearing flags on attachment: 194608

Committed r147021: <http://trac.webkit.org/changeset/147021>
Comment 8 WebKit Review Bot 2013-03-27 16:58:11 PDT
All reviewed patches have been landed.  Closing bug.