[Webkit-unassigned] [Bug 95557] New: [Qt] ASSERT(m_useFixedLayout) in WebPage when using QRawWebView without setting fixed layout

[bugzilla-daemon at webkit.org]

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

           Summary: [Qt] ASSERT(m_useFixedLayout) in WebPage when using
                    QRawWebView without setting fixed layout
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebKit Qt
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: kbalazs at webkit.org
                CC: kenneth at webkit.org, abecsi at webkit.org,
                    jocelyn.turcotte at nokia.com, noam.rosenthal at nokia.com


This has been revealed in https://bugs.webkit.org/show_bug.cgi?id=90394.
At first I was thinking that this is simply a matter of a missing call to WebPageProxy::setUseFixedLayout.
But if I made this call, than I could not use qrawwebview for layout tests because it always sets the viewport size to 980x800. I was trying setFixedLayoutSize but it does not help. In WebPage::sendViewportAttributesChanged the given size argument is adjusted in accordance to the viewport tag specification (WebCore::computeViewportAttributes). It is strange to me that calling this function with default desktop arguments (devicewidth/height==viewportSize.width()/height(), args=default, minimumWidth=viewportSize.width(), scale=1, viewportSize=800x600) adjust the size to 980x800 but I don't know the specification.
In order to fix this I see two option:
 1. introduce a setting in WebPage/~Proxy that enables/disables handling the viewport attributes (qrawwebview is missing the UI side of it anyway)
 2. let WTR with qrawwebview run in non fixed layout mode and don't call webpageproxy->setViewportSize.
 3. find out how can we set a custom viewport size in fixed layout mode

3 seems to be the best but I had no success with that, so for now I would like to go with 2.

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