Bug 19348 – REGRESSION (r34193): Setting the size of a frame with javascript document.body.row no longer works

• Component: Layout and Rendering
• Status: RESOLVED FIXED

[Peer Bremer]

calling a javascript function:

function resizeFrame(contractsetting){
	document.body.rows=contractsetting;
}
or using document.body.rows='100,*';
No longer alters teh existing frame set height.
See the example at:
http://www.smilecms.com/webkit/expand_frame.html

Works with Safari 3.1.1 (4525.18) but not with the latest webkit builds

[Alexey Proskuryakov]

Confirmed as a regression with r34291. Resizing the window to trigger relayout fixes the height.

[Alexey Proskuryakov]

<rdar://problem/5978447>

Caused by moving the base class recalcStyle() to the beginning of HTMLFrameSetElement::recalcStyle() in <http://trac.webkit.org/changeset/34193#file15>.

*** Bug 20140 has been marked as a duplicate of this bug. ***

[Peer Bremer]

There is also a redraw problem when a page is reloaded inside a frame set and a div uses an image as a background. Part of the background stays white and does not show the image. Solved when the window is resized.

Created attachment 22753 [details]
Call the base class implementation last

[Darin Adler]

Comment on attachment 22753 [details]
Call the base class implementation last

r=me

Fixed in <http://trac.webkit.org/changeset/35690>.