BaseFileFind/FindInFiles: Clean up enabled / valid state handling

IFindFilter:  Separate enabledChanged and validChanged
SearchEngine: Add enabledChanged
GitGrep:      Enable widget based on enabled state
FindInFiles:  Update validity on search engine enabledChanged instead of
              hardcoding it to when the search path changed

Change-Id: I0c684423b871a3a4f1f164574f42e23b64cf9da0
Reviewed-by: David Schulz <[email protected]>

1 files changed, 3 insertions, 7 deletions

diff --git a/src/plugins/git/gitgrep.cpp b/src/plugins/git/gitgrep.cpp
index 07e51399afe..cf961487d6b 100644
--- a/src/plugins/git/gitgrep.cpp
+++ b/src/plugins/git/gitgrep.cpp
@@ -198,7 +198,7 @@ private:
 
 } // namespace
 
-static bool validateDirectory(const QString &path)
+static bool isGitDirectory(const QString &path)
 {
     static IVersionControl *gitVc = VcsManager::versionControl(VcsBase::Constants::VCS_ID_GIT);
     QTC_ASSERT(gitVc, return false);
@@ -221,8 +221,9 @@ GitGrep::GitGrep()
     QTC_ASSERT(findInFiles, return);
     connect(findInFiles, &TextEditor::FindInFiles::pathChanged,
             m_widget, [this](const QString &path) {
-        m_widget->setEnabled(validateDirectory(path));
+        setEnabled(isGitDirectory(path));
     });
+    connect(this, &SearchEngine::enabledChanged, m_widget, &QWidget::setEnabled);
     findInFiles->addSearchEngine(this);
 }
 
@@ -249,11 +250,6 @@ QWidget *GitGrep::widget() const
     return m_widget;
 }
 
-bool GitGrep::isEnabled() const
-{
-    return m_widget->isEnabled();
-}
-
 QVariant GitGrep::parameters() const
 {
     GitGrepParameters params;