[Webkit-unassigned] [Bug 82600] New: [Qt][WK2] contentViewportChanged is triggered twice after gesture end

[bugzilla-daemon at webkit.org]

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

           Summary: [Qt][WK2] contentViewportChanged is triggered twice
                    after gesture end
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Keywords: Qt, QtTriaged
          Severity: Normal
          Priority: P2
         Component: Platform
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: kbalazs at webkit.org
                CC: kenneth at webkit.org


Not a very big issue, just a small bug I have noticed.
After a gesture ended, we clear the ViewportUpdateDeferrer, and it emits contentResumeRequested and contentViewportChanged (via the interaction engine). The first is connected to QQuickWebViewFlickablePrivate::_q_resume the second is to QQuickWebViewFlickablePrivate::_q_contentViewportChanged. But _q_resume also calls directly the second which is unnecessary and there is no check later so we will schedule a layerflush in LayerTreeHostQt (and send a message unnecessarily). I have removed the call in _q_resume, put a call to _q_contentViewportChanged to onComponentComplete where _q_resume is called explicitly, and it seems like everything works fine with it.

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