JavaScriptFilter: Add engine reset

Change-Id: I80c0e1c9270c9a67aa2494eeab1ccd942bf1885e
Reviewed-by: André Hartmann <aha_1980@gmx.de>
Reviewed-by: Eike Ziller <eike.ziller@qt.io>

src/plugins/coreplugin/locator/javascriptfilter.cpp

@@ -32,6 +32,11 @@
 namespace Core {
 namespace Internal {
 
+enum JavaScriptAction
+{
+    ResetEngine = QVariant::UserType + 1
+};
+
 JavaScriptFilter::JavaScriptFilter()
 {
     setId("JavaScriptFilter");
@@ -48,7 +53,8 @@ void JavaScriptFilter::prepareSearch(const QString &entry)
 {
     Q_UNUSED(entry);
 
-    setupEngine();
+    if (!m_engine)
+        setupEngine();
 }
 
 QList<LocatorFilterEntry> JavaScriptFilter::matchesFor(
@@ -56,13 +62,17 @@ QList<LocatorFilterEntry> JavaScriptFilter::matchesFor(
 {
     Q_UNUSED(future);
 
-    const QString result = m_engine->evaluate(entry).toString();
-    const QString expression = entry + " = " + result;
-
     QList<LocatorFilterEntry> entries;
-    entries.append({this, expression, QVariant()});
+    if (entry.trimmed().isEmpty()) {
-    entries.append({this, tr("Copy to clipboard: %1").arg(result), result});
+        entries.append({this, tr("Reset Engine"), QVariant(ResetEngine, nullptr)});
-    entries.append({this, tr("Copy to clipboard: %1").arg(expression), expression});
+    } else {
+        const QString result = m_engine->evaluate(entry).toString();
+        const QString expression = entry + " = " + result;
+
+        entries.append({this, expression, QVariant()});
+        entries.append({this, tr("Copy to clipboard: %1").arg(result), result});
+        entries.append({this, tr("Copy to clipboard: %1").arg(expression), expression});
+    }
 
     return entries;
 }
@@ -77,6 +87,11 @@ void JavaScriptFilter::accept(Core::LocatorFilterEntry selection, QString *newTe
     if (selection.internalData.isNull())
         return;
 
+    if (selection.internalData.userType() == ResetEngine) {
+        m_engine.reset();
+        return;
+    }
+
     QClipboard *clipboard = QGuiApplication::clipboard();
     clipboard->setText(selection.internalData.toString());
 }
@@ -89,10 +104,7 @@ void JavaScriptFilter::refresh(QFutureInterface<void> &future)
 
 void JavaScriptFilter::setupEngine()
 {
-    if (m_engine)
+    m_engine.reset(new QJSEngine);
-        return;
-
-    m_engine = new QJSEngine(this);
     m_engine->evaluate(
                 "function abs(x) { return Math.abs(x); }\n"
                 "function acos(x) { return Math.acos(x); }\n"

src/plugins/coreplugin/locator/javascriptfilter.h

@@ -27,6 +27,8 @@
 
 #include <coreplugin/locator/ilocatorfilter.h>
 
+#include <memory>
+
 QT_BEGIN_NAMESPACE
 class QJSEngine;
 QT_END_NAMESPACE
@@ -51,7 +53,7 @@ public:
 private:
     void setupEngine();
 
-    QJSEngine *m_engine = nullptr;
+    mutable std::unique_ptr<QJSEngine> m_engine;
 };
 
 } // namespace Internal