Summary: | [Qt][WK2] Add missing ViewportUpdateDeferrer guarding to setPageItemRectVisible() calls. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | zalan <zalan> | ||||||
Component: | WebKit Qt | Assignee: | zalan <zalan> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | cmarcelo, dinu.jacob, kenneth, menard, webkit.review.bot, zoltan | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
zalan
2012-05-31 00:50:59 PDT
Created attachment 145011 [details]
Patch
Comment on attachment 145011 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=145011&action=review > Source/WebKit2/UIProcess/qt/QtViewportInteractionEngine.cpp:345 > void QtViewportInteractionEngine::scaleAnimationValueChanged(QVariant value) > { > + ViewportUpdateDeferrer guard(this); > setPageItemRectVisible(value.toRectF()); > } This one is not needed... it should be guarded already (In reply to comment #2) > (From update of attachment 145011 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=145011&action=review > > > Source/WebKit2/UIProcess/qt/QtViewportInteractionEngine.cpp:345 > > void QtViewportInteractionEngine::scaleAnimationValueChanged(QVariant value) > > { > > + ViewportUpdateDeferrer guard(this); > > setPageItemRectVisible(value.toRectF()); > > } > > This one is not needed... it should be guarded already Good catch, but unfortunately it is not guarded. Created a new bug report on that bug#87954 Created attachment 145052 [details]
Patch
Comment on attachment 145052 [details] Patch Clearing flags on attachment: 145052 Committed r119095: <http://trac.webkit.org/changeset/119095> All reviewed patches have been landed. Closing bug. |