Bug 77212 – [Qt] Spurious resize events from frameviews using fixed layout

• Component: Layout and Rendering
• Status: RESOLVED FIXED

[Allan Sandfeld Jensen]

When using a frame with fixed layout size, like Qt WebKit2 in touch mode, resize events are send every time the content resizes.

The problems seems to be that the widget has the size of the document, and resize events are send when widget size changes.

The solution should be simply to use the layout size instead of the widget size for determining if resize events should be send.

[Allan Sandfeld Jensen]

Created attachment 124316 [details]
Patch

[Kenneth Rohde Christiansen]

Comment on attachment 124316 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=124316&action=review

> ChangeLog:9
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +        * ManualTests/resize-events.html: Added.
> +

A comment why a layout test wasnt possible would have been good

> ManualTests/resize-events.html:33
> +            // 0 or 1 resize events are acceptable, more is not

Why not write this in the HTML, now it is a manual test? You could even make it write SUCCESS/FAIL

[Allan Sandfeld Jensen]

Created attachment 124507 [details]
Patch

[Kenneth Rohde Christiansen]

Comment on attachment 124507 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=124507&action=review

> ManualTests/resize-events.html:17
> +        Test who many resize events are emitted during page load and dynamic content generation.

how*

[Allan Sandfeld Jensen]

Created attachment 124513 [details]
Patch

[WebKit Review Bot]

Comment on attachment 124513 [details]
Patch

Clearing flags on attachment: 124513

Committed r106232: <http://trac.webkit.org/changeset/106232>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.