Remove QWidgetBackingStore::fullUpdatePending

This was no longer used and always false.

Change-Id: I981055e6cc736cfe3432914c173c8b3edc31b46e
Reviewed-by: Laszlo Agocs <laszlo.agocs@theqtcompany.com>
This commit is contained in:
Allan Sandfeld Jensen 2016-06-08 11:01:06 +02:00 committed by Allan Sandfeld Jensen
parent 923d869b0f
commit bdf270202b
2 changed files with 4 additions and 21 deletions

View File

@ -285,7 +285,7 @@ bool QWidgetBackingStore::bltRect(const QRect &rect, int dx, int dy, QWidget *wi
{ {
const QPoint pos(tlwOffset + widget->mapTo(tlw, rect.topLeft())); const QPoint pos(tlwOffset + widget->mapTo(tlw, rect.topLeft()));
const QRect tlwRect(QRect(pos, rect.size())); const QRect tlwRect(QRect(pos, rect.size()));
if (fullUpdatePending || dirty.intersects(tlwRect)) if (dirty.intersects(tlwRect))
return false; // We don't want to scroll junk. return false; // We don't want to scroll junk.
return store->scroll(tlwRect, dx, dy); return store->scroll(tlwRect, dx, dy);
} }
@ -354,7 +354,7 @@ QRegion QWidgetBackingStore::dirtyRegion(QWidget *widget) const
const bool widgetDirty = widget && widget != tlw; const bool widgetDirty = widget && widget != tlw;
const QRect tlwRect(topLevelRect()); const QRect tlwRect(topLevelRect());
const QRect surfaceGeometry(tlwRect.topLeft(), store->size()); const QRect surfaceGeometry(tlwRect.topLeft(), store->size());
if (fullUpdatePending || (surfaceGeometry != tlwRect && surfaceGeometry.size() != tlwRect.size())) { if (surfaceGeometry != tlwRect && surfaceGeometry.size() != tlwRect.size()) {
if (widgetDirty) { if (widgetDirty) {
const QRect dirtyTlwRect = QRect(QPoint(), tlwRect.size()); const QRect dirtyTlwRect = QRect(QPoint(), tlwRect.size());
const QPoint offset(widget->mapTo(tlw, QPoint())); const QPoint offset(widget->mapTo(tlw, QPoint()));
@ -532,13 +532,6 @@ void QWidgetBackingStore::markDirty(const QRegion &rgn, QWidget *widget,
return; return;
} }
//### FIXME fullUpdatePending seems to be always false????
if (fullUpdatePending) {
if (updateTime == UpdateNow)
sendUpdateRequest(tlw, updateTime);
return;
}
const QPoint offset = widget->mapTo(tlw, QPoint()); const QPoint offset = widget->mapTo(tlw, QPoint());
if (QWidgetPrivate::get(widget)->renderToTexture) { if (QWidgetPrivate::get(widget)->renderToTexture) {
@ -631,12 +624,6 @@ void QWidgetBackingStore::markDirty(const QRect &rect, QWidget *widget,
return; return;
} }
if (fullUpdatePending) {
if (updateTime == UpdateNow)
sendUpdateRequest(tlw, updateTime);
return;
}
if (QWidgetPrivate::get(widget)->renderToTexture) { if (QWidgetPrivate::get(widget)->renderToTexture) {
if (!widget->d_func()->inDirtyList) if (!widget->d_func()->inDirtyList)
addDirtyRenderToTextureWidget(widget); addDirtyRenderToTextureWidget(widget);
@ -773,7 +760,6 @@ void QWidgetBackingStore::updateLists(QWidget *cur)
QWidgetBackingStore::QWidgetBackingStore(QWidget *topLevel) QWidgetBackingStore::QWidgetBackingStore(QWidget *topLevel)
: tlw(topLevel), : tlw(topLevel),
dirtyOnScreenWidgets(0), dirtyOnScreenWidgets(0),
fullUpdatePending(0),
updateRequestSent(0), updateRequestSent(0),
textureListWatcher(0), textureListWatcher(0),
perfFrames(0) perfFrames(0)
@ -1142,7 +1128,6 @@ void QWidgetBackingStore::sync()
for (int i = 0; i < dirtyWidgets.size(); ++i) for (int i = 0; i < dirtyWidgets.size(); ++i)
resetWidget(dirtyWidgets.at(i)); resetWidget(dirtyWidgets.at(i));
dirtyWidgets.clear(); dirtyWidgets.clear();
fullUpdatePending = false;
} }
return; return;
} }
@ -1159,7 +1144,7 @@ void QWidgetBackingStore::doSync()
const bool inTopLevelResize = tlw->d_func()->maybeTopData()->inTopLevelResize; const bool inTopLevelResize = tlw->d_func()->maybeTopData()->inTopLevelResize;
const QRect tlwRect(topLevelRect()); const QRect tlwRect(topLevelRect());
const QRect surfaceGeometry(tlwRect.topLeft(), store->size()); const QRect surfaceGeometry(tlwRect.topLeft(), store->size());
if ((fullUpdatePending || inTopLevelResize || surfaceGeometry.size() != tlwRect.size()) && !updatesDisabled) { if ((inTopLevelResize || surfaceGeometry.size() != tlwRect.size()) && !updatesDisabled) {
if (hasStaticContents() && !store->size().isEmpty() ) { if (hasStaticContents() && !store->size().isEmpty() ) {
// Repaint existing dirty area and newly visible area. // Repaint existing dirty area and newly visible area.
const QRect clipRect(0, 0, surfaceGeometry.width(), surfaceGeometry.height()); const QRect clipRect(0, 0, surfaceGeometry.width(), surfaceGeometry.height());
@ -1245,7 +1230,6 @@ void QWidgetBackingStore::doSync()
tlwExtra->widgetTextures.clear(); tlwExtra->widgetTextures.clear();
findAllTextureWidgetsRecursively(tlw, tlw); findAllTextureWidgetsRecursively(tlw, tlw);
qt_window_private(tlw->windowHandle())->compositing = false; // will get updated in qt_flush() qt_window_private(tlw->windowHandle())->compositing = false; // will get updated in qt_flush()
fullUpdatePending = false;
#endif #endif
if (toClean.isEmpty()) { if (toClean.isEmpty()) {

View File

@ -116,7 +116,7 @@ public:
inline bool isDirty() const inline bool isDirty() const
{ {
return !(dirtyWidgets.isEmpty() && dirty.isEmpty() && !fullUpdatePending); return !(dirtyWidgets.isEmpty() && dirty.isEmpty());
} }
// ### Qt 4.6: Merge into a template function (after MSVC isn't supported anymore). // ### Qt 4.6: Merge into a template function (after MSVC isn't supported anymore).
@ -135,7 +135,6 @@ private:
QVector<QWidget *> *dirtyOnScreenWidgets; QVector<QWidget *> *dirtyOnScreenWidgets;
QList<QWidget *> staticWidgets; QList<QWidget *> staticWidgets;
QBackingStore *store; QBackingStore *store;
uint fullUpdatePending : 1;
uint updateRequestSent : 1; uint updateRequestSent : 1;
QPoint tlwOffset; QPoint tlwOffset;