[PlacesItemModelTest] Try to use an event loop instead of an hardcoded timeout

[dolphin.git] / src / search / dolphinsearchbox.cpp

diff --git a/src/search/dolphinsearchbox.cpp b/src/search/dolphinsearchbox.cpp
index 6cae37e197dd2e1fdcee56c722e1f6c786006bd8..61f5c2db4f8e341ed22fe0ef079a27a554d10c48 100644 (file)
--- a/src/search/dolphinsearchbox.cpp
+++ b/src/search/dolphinsearchbox.cpp
@@ -21,31 +21,29 @@
 
 #include "dolphin_searchsettings.h"
 #include "dolphinfacetswidget.h"
 
 #include "dolphin_searchsettings.h"
 #include "dolphinfacetswidget.h"

+#include "panels/places/placesitemmodel.h"

 
 

-#include <panels/places/placesitemmodel.h>
-
-#include <QIcon>
-#include <QLineEdit>

 #include <KLocalizedString>
 #include <KLocalizedString>

-#include <KSeparator>

 #include <KNS3/KMoreToolsMenuFactory>
 #include <KNS3/KMoreToolsMenuFactory>

+#include <KSeparator>
+#include <config-baloo.h>
+#ifdef HAVE_BALOO
+#include <Baloo/Query>
+#include <Baloo/IndexerConfig>
+#endif

 
 #include <QButtonGroup>
 #include <QDir>
 
 #include <QButtonGroup>
 #include <QDir>

+#include <QFontDatabase>

 #include <QHBoxLayout>
 #include <QHBoxLayout>

+#include <QIcon>

 #include <QLabel>
 #include <QLabel>

+#include <QLineEdit>

 #include <QScrollArea>
 #include <QTimer>
 #include <QToolButton>
 #include <QUrlQuery>
 
 #include <QScrollArea>
 #include <QTimer>
 #include <QToolButton>
 #include <QUrlQuery>
 

-#include <config-baloo.h>
-#ifdef HAVE_BALOO
-    #include <Baloo/Query>
-    #include <Baloo/IndexerConfig>
-#endif
-#include <QFontDatabase>
-

 DolphinSearchBox::DolphinSearchBox(QWidget* parent) :
     QWidget(parent),
     m_startedSearching(false),
 DolphinSearchBox::DolphinSearchBox(QWidget* parent) :
     QWidget(parent),
     m_startedSearching(false),


@@ -233,10 +231,14 @@ bool DolphinSearchBox::eventFilter(QObject* obj, QEvent* event)
         // #379135: we get the FocusIn event when we close a tab but we don't want to emit
         // the activated() signal before the removeTab() call in DolphinTabWidget::closeTab() returns.
         // To avoid this issue, we delay the activation of the search box.
         // #379135: we get the FocusIn event when we close a tab but we don't want to emit
         // the activated() signal before the removeTab() call in DolphinTabWidget::closeTab() returns.
         // To avoid this issue, we delay the activation of the search box.

-        QTimer::singleShot(0, this, [this] {
-            setActive(true);
-            setFocus();
-        });
+        // We also don't want to schedule the activation process if we are already active,
+        // otherwise we can enter in a loop of FocusIn/FocusOut events with the searchbox of another tab.
+        if (!isActive()) {
+            QTimer::singleShot(0, this, [this] {
+                setActive(true);
+                setFocus();
+            });
+        }

         break;
 
     default:
         break;
 
     default: