From 96f0746298a1b93d0d22e3678a17e2a58d4010c7 Mon Sep 17 00:00:00 2001 From: Jason Barron Date: Sun, 20 Sep 2009 14:28:47 +0200 Subject: Add a new compressable event QEvent::UpdateSoftKeys. Now that the softkey framework triggers updates on focus changed, window activated, and action added/removed/changed the softkey bar was updating many times resulting in flicker. Solve that by introducing a new event type that is posted to the softkey framework. Since we only need to update the softkeys once per event loop iteration, the event is compressible in the event loop. Reviewed-by: Alessandro Portale --- src/corelib/kernel/qcoreevent.h | 2 + src/gui/kernel/qapplication.cpp | 3 + src/gui/kernel/qsoftkeymanager.cpp | 52 ++++---- src/gui/kernel/qsoftkeymanager_p.h | 7 +- src/gui/kernel/qwidget.cpp | 2 +- tests/auto/qsoftkeymanager/qsoftkeymanager.pro | 4 + tests/auto/qsoftkeymanager/tst_qsoftkeymanager.cpp | 138 +++++++++++++++++++++ 7 files changed, 183 insertions(+), 25 deletions(-) create mode 100644 tests/auto/qsoftkeymanager/qsoftkeymanager.pro create mode 100644 tests/auto/qsoftkeymanager/tst_qsoftkeymanager.cpp diff --git a/src/corelib/kernel/qcoreevent.h b/src/corelib/kernel/qcoreevent.h index 11fc988..bc96918 100644 --- a/src/corelib/kernel/qcoreevent.h +++ b/src/corelib/kernel/qcoreevent.h @@ -283,6 +283,8 @@ public: SymbianDeferredFocusChanged = 201, // Internal for generating asynchronous focus events on Symbian + UpdateSoftKeys = 202, // Internal for compressing soft key updates + // 512 reserved for Qt Jambi's MetaCall event // 513 reserved for Qt Jambi's DeleteOnMainThread event diff --git a/src/gui/kernel/qapplication.cpp b/src/gui/kernel/qapplication.cpp index df4a935..774ec23 100644 --- a/src/gui/kernel/qapplication.cpp +++ b/src/gui/kernel/qapplication.cpp @@ -1160,6 +1160,7 @@ bool QApplication::compressEvent(QEvent *event, QObject *receiver, QPostEventLis || event->type() == QEvent::Resize || event->type() == QEvent::Move || event->type() == QEvent::LanguageChange + || event->type() == QEvent::UpdateSoftKeys || event->type() == QEvent::InputMethod)) { for (int i = 0; i < postedEvents->size(); ++i) { const QPostEvent &cur = postedEvents->at(i); @@ -1177,6 +1178,8 @@ bool QApplication::compressEvent(QEvent *event, QObject *receiver, QPostEventLis ((QMoveEvent *)(cur.event))->p = ((QMoveEvent *)event)->p; } else if (cur.event->type() == QEvent::LanguageChange) { ; + } else if (cur.event->type() == QEvent::UpdateSoftKeys) { + ; } else if ( cur.event->type() == QEvent::InputMethod ) { *(QInputMethodEvent *)(cur.event) = *(QInputMethodEvent *)event; } else { diff --git a/src/gui/kernel/qsoftkeymanager.cpp b/src/gui/kernel/qsoftkeymanager.cpp index a7a5223..76fb378 100644 --- a/src/gui/kernel/qsoftkeymanager.cpp +++ b/src/gui/kernel/qsoftkeymanager.cpp @@ -139,32 +139,40 @@ void QSoftKeyManager::sendKeyEvent() new QKeyEvent(QEvent::KeyPress, keyToSend, Qt::NoModifier)); } -void QSoftKeyManager::updateSoftKeys(bool force) +void QSoftKeyManager::updateSoftKeys() { - QList softKeys; - QWidget *source = QApplication::focusWidget(); - do { - if (source) { - QList actions = source->actions(); - for (int i = 0; i < actions.count(); ++i) { - if (actions.at(i)->softKeyRole() != QAction::NoSoftKey) - softKeys.append(actions.at(i)); - } + QEvent *event = new QEvent(QEvent::UpdateSoftKeys); + QApplication::postEvent(QSoftKeyManager::instance(), event); +} - QWidget *parent = source->parentWidget(); - if (parent && softKeys.isEmpty()) - source = parent; - else - break; - } else { - source = QApplication::activeWindow(); - } - } while (source); +bool QSoftKeyManager::event(QEvent *e) +{ + if (e->type() == QEvent::UpdateSoftKeys) { + QList softKeys; + QWidget *source = QApplication::focusWidget(); + do { + if (source) { + QList actions = source->actions(); + for (int i = 0; i < actions.count(); ++i) { + if (actions.at(i)->softKeyRole() != QAction::NoSoftKey) + softKeys.append(actions.at(i)); + } + + QWidget *parent = source->parentWidget(); + if (parent && softKeys.isEmpty()) + source = parent; + else + break; + } else { + source = QApplication::activeWindow(); + } + } while (source); - if (force || (source != QSoftKeyManager::softKeySource)) { - QSoftKeyManager::softKeySource = source; - QSoftKeyManager::updateSoftKeys_sys(softKeys); + QSoftKeyManager::softKeySource = source; + QSoftKeyManager::updateSoftKeys_sys(softKeys); + return true; } + return false; } #ifdef Q_WS_S60 diff --git a/src/gui/kernel/qsoftkeymanager_p.h b/src/gui/kernel/qsoftkeymanager_p.h index 743b4d4..229df98 100644 --- a/src/gui/kernel/qsoftkeymanager_p.h +++ b/src/gui/kernel/qsoftkeymanager_p.h @@ -60,7 +60,7 @@ QT_BEGIN_HEADER QT_BEGIN_NAMESPACE -class QSoftKeyManager : public QObject +class Q_AUTOTEST_EXPORT QSoftKeyManager : public QObject { Q_OBJECT @@ -73,7 +73,7 @@ public: CancelSoftKey, }; - static void updateSoftKeys(bool force = false); + static void updateSoftKeys(); static QAction *createAction(StandardSoftKey standardKey, QWidget *actionWidget); static QAction *createKeyedAction(StandardSoftKey standardKey, Qt::Key key, QWidget *actionWidget); @@ -91,6 +91,9 @@ private: static QWidget *softKeySource; QHash keyedActions; +protected: + bool event(QEvent *e); + Q_DISABLE_COPY(QSoftKeyManager) private Q_SLOTS: diff --git a/src/gui/kernel/qwidget.cpp b/src/gui/kernel/qwidget.cpp index 41d0b23..00ad430 100644 --- a/src/gui/kernel/qwidget.cpp +++ b/src/gui/kernel/qwidget.cpp @@ -8257,7 +8257,7 @@ bool QWidget::event(QEvent *event) case QEvent::ActionRemoved: case QEvent::ActionChanged: #ifdef QT_SOFTKEYS_ENABLED - QSoftKeyManager::updateSoftKeys(true); + QSoftKeyManager::updateSoftKeys(); #endif actionEvent((QActionEvent*)event); break; diff --git a/tests/auto/qsoftkeymanager/qsoftkeymanager.pro b/tests/auto/qsoftkeymanager/qsoftkeymanager.pro new file mode 100644 index 0000000..61ed4dc --- /dev/null +++ b/tests/auto/qsoftkeymanager/qsoftkeymanager.pro @@ -0,0 +1,4 @@ +load(qttest_p4) +SOURCES += tst_qsoftkeymanager.cpp + +requires(symbian) diff --git a/tests/auto/qsoftkeymanager/tst_qsoftkeymanager.cpp b/tests/auto/qsoftkeymanager/tst_qsoftkeymanager.cpp new file mode 100644 index 0000000..81ef498 --- /dev/null +++ b/tests/auto/qsoftkeymanager/tst_qsoftkeymanager.cpp @@ -0,0 +1,138 @@ +/**************************************************************************** +** +** Copyright (C) 2009 Nokia Corporation and/or its subsidiary(-ies). +** All rights reserved. +** Contact: Nokia Corporation (qt-info@nokia.com) +** +** This file is part of the test suite of the Qt Toolkit. +** +** $QT_BEGIN_LICENSE:LGPL$ +** No Commercial Usage +** This file contains pre-release code and may not be distributed. +** You may use this file in accordance with the terms and conditions +** contained in the Technology Preview License Agreement accompanying +** this package. +** +** GNU Lesser General Public License Usage +** Alternatively, this file may be used under the terms of the GNU Lesser +** General Public License version 2.1 as published by the Free Software +** Foundation and appearing in the file LICENSE.LGPL included in the +** packaging of this file. Please review the following information to +** ensure the GNU Lesser General Public License version 2.1 requirements +** will be met: http://www.gnu.org/licenses/old-licenses/lgpl-2.1.html. +** +** In addition, as a special exception, Nokia gives you certain additional +** rights. These rights are described in the Nokia Qt LGPL Exception +** version 1.1, included in the file LGPL_EXCEPTION.txt in this package. +** +** If you have questions regarding the use of this file, please contact +** Nokia at qt-info@nokia.com. +** +** +** +** +** +** +** +** +** $QT_END_LICENSE$ +** +****************************************************************************/ + +#include + +#include "qevent.h" +#include "private/qsoftkeymanager_p.h" + +class tst_QSoftKeyManager : public QObject +{ +Q_OBJECT + +public: + tst_QSoftKeyManager(); + virtual ~tst_QSoftKeyManager(); + +public slots: + void initTestCase(); + void cleanupTestCase(); + void init(); + void cleanup(); +private slots: + void updateSoftKeysCompressed(); +}; + +class EventListener : public QObject +{ +public: + EventListener(QObject *listenTo) + { + resetCounts(); + if (listenTo) + listenTo->installEventFilter(this); + } + + void resetCounts() + { + numUpdateSoftKeys = 0; + } + + int numUpdateSoftKeys; + +protected: + bool eventFilter(QObject * /*object*/, QEvent *event) + { + if (event->type() == QEvent::UpdateSoftKeys) + numUpdateSoftKeys++; + return false; + } +}; + +tst_QSoftKeyManager::tst_QSoftKeyManager() : QObject() +{ +} + +tst_QSoftKeyManager::~tst_QSoftKeyManager() +{ +} + +void tst_QSoftKeyManager::initTestCase() +{ +} + +void tst_QSoftKeyManager::cleanupTestCase() +{ +} + +void tst_QSoftKeyManager::init() +{ +} + +void tst_QSoftKeyManager::cleanup() +{ +} + +/* + This tests that we only get one UpdateSoftKeys event even though + multiple events that trigger soft keys occur. +*/ +void tst_QSoftKeyManager::updateSoftKeysCompressed() +{ + QWidget w; + EventListener listener(qApp); + + QList softKeys; + for (int i = 0; i < 10; ++i) { + QAction *action = new QAction("foo", &w); + action->setSoftKeyRole(QAction::PositiveSoftKey); + softKeys << action; + } + w.addActions(softKeys); + + QApplication::processEvents(); + + QVERIFY(listener.numUpdateSoftKeys == 1); +} + + +QTEST_MAIN(tst_QSoftKeyManager) +#include "tst_qsoftkeymanager.moc" -- cgit v0.12