[Webkit-unassigned] [Bug 32807] New: FrameView::adjustSize does not size to true content height

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=32807

           Summary: FrameView::adjustSize does not size to true content
                    height
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
        OS/Version: Mac OS X 10.5
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Layout and Rendering
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: aa at chromium.org
                CC: hyatt at apple.com


Currently, FrameView::adjustSize sets the content size to the max() of the
content's height and the container height. This means that if the content does
not fill the container, the content size will still be set to the contianer's
height.

This makes it impossible to implement containers that shrinkwrap their contents
height using APIs like FrameView::contentSize() and
ChromeClient::contentsSizeChanged().

It is possible to get the content's true height using
frame->document()->documentElement()->scrollHeight(), but there is no
corresponding event for when this changes.

Instead of adding more API for when the scroll height changes, it seems to me
that it would be better to just change ::adjustSize() to set the content size
to the true height. Applications that want the old behavior will still be able
to get it, since they can get the container height.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.