V4 debugger: Drop redundantRefs option

No client is using this anymore and we can eliminate a lot of code this
way.

Change-Id: I454581928fe88e59cd28738c4f7fddd060999181
Reviewed-by: Simon Hausmann <simon.hausmann@qt.io>

1 files changed, 39 insertions, 65 deletions

diff --git a/tests/auto/qml/debugger/qqmldebugjs/tst_qqmldebugjs.cpp b/tests/auto/qml/debugger/qqmldebugjs/tst_qqmldebugjs.cpp
index e3996e0c18..afd1afaabc 100644
--- a/tests/auto/qml/debugger/qqmldebugjs/tst_qqmldebugjs.cpp
+++ b/tests/auto/qml/debugger/qqmldebugjs/tst_qqmldebugjs.cpp
@@ -260,7 +260,7 @@ public:
                          this, &QJSDebugClient::onStateChanged);
     }
 
-    void connect(bool redundantRefs = false, bool namesAsObjects = false);
+    void connect(bool namesAsObjects = false);
     void interrupt();
 
     void continueDebugging(StepAction stepAction);
@@ -306,10 +306,9 @@ public:
 
 };
 
-void QJSDebugClient::connect(bool redundantRefs, bool namesAsObjects)
+void QJSDebugClient::connect(bool namesAsObjects)
 {
     QJSValue jsonVal = parser.call(QJSValueList() << QLatin1String("{}"));
-    jsonVal.setProperty("redundantRefs", QJSValue(redundantRefs));
     jsonVal.setProperty("namesAsObjects", QJSValue(namesAsObjects));
     sendMessage(packMessage(CONNECT,
                             stringify.call(QJSValueList() << jsonVal).toString().toUtf8()));
@@ -798,6 +797,7 @@ void tst_QQmlDebugJS::connect()
     QFETCH(bool, blockMode);
     QFETCH(bool, restrictMode);
     QFETCH(bool, qmlscene);
+
     QCOMPARE(init(qmlscene, QString(TEST_QMLFILE), blockMode, restrictMode), ConnectSuccess);
     m_client->connect();
     QVERIFY(waitForClientSignal(SIGNAL(connected())));
@@ -806,12 +806,11 @@ void tst_QQmlDebugJS::connect()
 void tst_QQmlDebugJS::interrupt()
 {
     //void connect()
-
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
+
     QCOMPARE(init(qmlscene), ConnectSuccess);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
 
     m_client->interrupt();
     QVERIFY(waitForClientSignal(SIGNAL(interruptRequested())));
@@ -820,12 +819,11 @@ void tst_QQmlDebugJS::interrupt()
 void tst_QQmlDebugJS::getVersion()
 {
     //void version()
-
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
+
     QCOMPARE(init(qmlscene), ConnectSuccess);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QVERIFY(waitForClientSignal(SIGNAL(connected())));
 
     m_client->version();
@@ -836,11 +834,10 @@ void tst_QQmlDebugJS::getVersionWhenAttaching()
 {
     //void version()
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
 
     QCOMPARE(init(qmlscene, QLatin1String(TIMER_QMLFILE), false), ConnectSuccess);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
 
     m_client->version();
     QVERIFY(waitForClientSignal(SIGNAL(result())));
@@ -849,12 +846,11 @@ void tst_QQmlDebugJS::getVersionWhenAttaching()
 void tst_QQmlDebugJS::disconnect()
 {
     //void disconnect()
-
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
+
     QCOMPARE(init(qmlscene), ConnectSuccess);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
 
     m_client->disconnect();
     QVERIFY(waitForClientSignal(SIGNAL(result())));
@@ -864,14 +860,13 @@ void tst_QQmlDebugJS::setBreakpointInScriptOnCompleted()
 {
     //void setBreakpoint(QString type, QString target, int line = -1, int column = -1, bool enabled = false, QString condition = QString(), int ignoreCount = -1)
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
 
     int sourceLine = 34;
     QCOMPARE(init(qmlscene, ONCOMPLETED_QMLFILE), ConnectSuccess);
 
     m_client->setBreakpoint(QLatin1String(ONCOMPLETED_QMLFILE), sourceLine, -1, true);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QVERIFY(waitForClientSignal(SIGNAL(stopped())));
 
     QString jsonString(m_client->response);
@@ -887,14 +882,13 @@ void tst_QQmlDebugJS::setBreakpointInScriptOnComponentCreated()
 {
     //void setBreakpoint(QString type, QString target, int line = -1, int column = -1, bool enabled = false, QString condition = QString(), int ignoreCount = -1)
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
 
     int sourceLine = 34;
     QCOMPARE(init(qmlscene, CREATECOMPONENT_QMLFILE), ConnectSuccess);
 
     m_client->setBreakpoint(QLatin1String(ONCOMPLETED_QMLFILE), sourceLine, -1, true);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QVERIFY(waitForClientSignal(SIGNAL(stopped())));
 
     QString jsonString(m_client->response);
@@ -909,12 +903,12 @@ void tst_QQmlDebugJS::setBreakpointInScriptOnComponentCreated()
 void tst_QQmlDebugJS::setBreakpointInScriptOnTimerCallback()
 {
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
+
     int sourceLine = 35;
     QCOMPARE(init(qmlscene, TIMER_QMLFILE), ConnectSuccess);
 
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     //We can set the breakpoint after connect() here because the timer is repeating and if we miss
     //its first iteration we can still catch the second one.
     m_client->setBreakpoint(QLatin1String(TIMER_QMLFILE), sourceLine, -1, true);
@@ -933,14 +927,13 @@ void tst_QQmlDebugJS::setBreakpointInScriptInDifferentFile()
 {
     //void setBreakpoint(QString type, QString target, int line = -1, int column = -1, bool enabled = false, QString condition = QString(), int ignoreCount = -1)
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
 
     int sourceLine = 31;
     QCOMPARE(init(qmlscene, LOADJSFILE_QMLFILE), ConnectSuccess);
 
     m_client->setBreakpoint(QLatin1String(TEST_JSFILE), sourceLine, -1, true);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QVERIFY(waitForClientSignal(SIGNAL(stopped())));
 
     QString jsonString(m_client->response);
@@ -956,7 +949,6 @@ void tst_QQmlDebugJS::setBreakpointInScriptOnComment()
 {
     //void setBreakpoint(QString type, QString target, int line = -1, int column = -1, bool enabled = false, QString condition = QString(), int ignoreCount = -1)
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
 
     int sourceLine = 34;
@@ -964,7 +956,7 @@ void tst_QQmlDebugJS::setBreakpointInScriptOnComment()
     QCOMPARE(init(qmlscene, BREAKPOINTRELOCATION_QMLFILE), ConnectSuccess);
 
     m_client->setBreakpoint(QLatin1String(BREAKPOINTRELOCATION_QMLFILE), sourceLine, -1, true);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QEXPECT_FAIL("", "Relocation of breakpoints is disabled right now", Abort);
     QVERIFY(waitForClientSignal(SIGNAL(stopped()), 1));
 
@@ -981,7 +973,6 @@ void tst_QQmlDebugJS::setBreakpointInScriptOnEmptyLine()
 {
     //void setBreakpoint(QString type, QString target, int line = -1, int column = -1, bool enabled = false, QString condition = QString(), int ignoreCount = -1)
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
 
     int sourceLine = 35;
@@ -989,7 +980,7 @@ void tst_QQmlDebugJS::setBreakpointInScriptOnEmptyLine()
     QCOMPARE(init(qmlscene, BREAKPOINTRELOCATION_QMLFILE), ConnectSuccess);
 
     m_client->setBreakpoint(QLatin1String(BREAKPOINTRELOCATION_QMLFILE), sourceLine, -1, true);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QEXPECT_FAIL("", "Relocation of breakpoints is disabled right now", Abort);
     QVERIFY(waitForClientSignal(SIGNAL(stopped()), 1));
 
@@ -1006,14 +997,13 @@ void tst_QQmlDebugJS::setBreakpointInScriptOnOptimizedBinding()
 {
     //void setBreakpoint(QString type, QString target, int line = -1, int column = -1, bool enabled = false, QString condition = QString(), int ignoreCount = -1)
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
 
     int sourceLine = 39;
     QCOMPARE(init(qmlscene, BREAKPOINTRELOCATION_QMLFILE), ConnectSuccess);
 
     m_client->setBreakpoint(QLatin1String(BREAKPOINTRELOCATION_QMLFILE), sourceLine, -1, true);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QVERIFY(waitForClientSignal(SIGNAL(stopped())));
 
     QString jsonString(m_client->response);
@@ -1028,13 +1018,13 @@ void tst_QQmlDebugJS::setBreakpointInScriptOnOptimizedBinding()
 void tst_QQmlDebugJS::setBreakpointInScriptWithCondition()
 {
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
+
     int out = 10;
     int sourceLine = 37;
     QCOMPARE(init(qmlscene, CONDITION_QMLFILE), ConnectSuccess);
 
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     //The breakpoint is in a timer loop so we can set it after connect().
     m_client->setBreakpoint(QLatin1String(CONDITION_QMLFILE), sourceLine, 1, true, QLatin1String("a > 10"));
     QVERIFY(waitForClientSignal(SIGNAL(stopped())));
@@ -1067,14 +1057,14 @@ void tst_QQmlDebugJS::setBreakpointInScriptWithCondition()
 void tst_QQmlDebugJS::setBreakpointInScriptThatQuits()
 {
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
+
     QCOMPARE(init(qmlscene, QUIT_QMLFILE), ConnectSuccess);
 
     int sourceLine = 36;
 
     m_client->setBreakpoint(QLatin1String(QUIT_QMLFILE), sourceLine, -1, true);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QVERIFY(waitForClientSignal(SIGNAL(stopped())));
 
     QString jsonString(m_client->response);
@@ -1111,14 +1101,13 @@ void tst_QQmlDebugJS::clearBreakpoint()
 {
     //void clearBreakpoint(int breakpoint);
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
 
     int sourceLine1 = 37;
     int sourceLine2 = 38;
     QCOMPARE(init(qmlscene, CHANGEBREAKPOINT_QMLFILE), ConnectSuccess);
 
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     //The breakpoints are in a timer loop so we can set them after connect().
     //Furthermore the breakpoints should be hit in the right order because setting of breakpoints
     //can only occur in the QML event loop. (see QCOMPARE for sourceLine2 below)
@@ -1161,12 +1150,11 @@ void tst_QQmlDebugJS::setExceptionBreak()
 {
     //void setExceptionBreak(QString type, bool enabled = false);
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
 
     QCOMPARE(init(qmlscene, EXCEPTION_QMLFILE), ConnectSuccess);
     m_client->setExceptionBreak(QJSDebugClient::All,true);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QVERIFY(waitForClientSignal(SIGNAL(stopped())));
 }
 
@@ -1174,14 +1162,13 @@ void tst_QQmlDebugJS::stepNext()
 {
     //void continueDebugging(StepAction stepAction, int stepCount = 1);
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
 
     int sourceLine = 37;
     QCOMPARE(init(qmlscene, STEPACTION_QMLFILE), ConnectSuccess);
 
     m_client->setBreakpoint(QLatin1String(STEPACTION_QMLFILE), sourceLine, -1, true);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QVERIFY(waitForClientSignal(SIGNAL(stopped())));
 
     m_client->continueDebugging(QJSDebugClient::Next);
@@ -1208,7 +1195,6 @@ void tst_QQmlDebugJS::stepIn()
 {
     //void continueDebugging(StepAction stepAction, int stepCount = 1);
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
 
     int sourceLine = 41;
@@ -1216,7 +1202,7 @@ void tst_QQmlDebugJS::stepIn()
     QCOMPARE(init(qmlscene, STEPACTION_QMLFILE), ConnectSuccess);
 
     m_client->setBreakpoint(QLatin1String(STEPACTION_QMLFILE), sourceLine, 1, true);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QVERIFY(waitForClientSignal(SIGNAL(stopped())));
     QCOMPARE(responseBody(m_client).value("sourceLine").toInt(), sourceLine);
 
@@ -1232,7 +1218,6 @@ void tst_QQmlDebugJS::stepOut()
 {
     //void continueDebugging(StepAction stepAction, int stepCount = 1);
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
 
     int sourceLine = 37;
@@ -1240,7 +1225,7 @@ void tst_QQmlDebugJS::stepOut()
     QCOMPARE(init(qmlscene, STEPACTION_QMLFILE), ConnectSuccess);
 
     m_client->setBreakpoint(QLatin1String(STEPACTION_QMLFILE), sourceLine, -1, true);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QVERIFY(waitForClientSignal(SIGNAL(stopped())));
     QCOMPARE(responseBody(m_client).value("sourceLine").toInt(), sourceLine);
 
@@ -1256,7 +1241,6 @@ void tst_QQmlDebugJS::continueDebugging()
 {
     //void continueDebugging(StepAction stepAction, int stepCount = 1);
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
 
     int sourceLine1 = 41;
@@ -1265,7 +1249,7 @@ void tst_QQmlDebugJS::continueDebugging()
 
     m_client->setBreakpoint(QLatin1String(STEPACTION_QMLFILE), sourceLine1, -1, true);
     m_client->setBreakpoint(QLatin1String(STEPACTION_QMLFILE), sourceLine2, -1, true);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QVERIFY(waitForClientSignal(SIGNAL(stopped())));
 
     m_client->continueDebugging(QJSDebugClient::Continue);
@@ -1284,14 +1268,13 @@ void tst_QQmlDebugJS::backtrace()
 {
     //void backtrace(int fromFrame = -1, int toFrame = -1, bool bottom = false);
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
 
     int sourceLine = 34;
     QCOMPARE(init(qmlscene, ONCOMPLETED_QMLFILE), ConnectSuccess);
 
     m_client->setBreakpoint(QLatin1String(ONCOMPLETED_QMLFILE), sourceLine, -1, true);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QVERIFY(waitForClientSignal(SIGNAL(stopped())));
 
     m_client->backtrace();
@@ -1302,14 +1285,13 @@ void tst_QQmlDebugJS::getFrameDetails()
 {
     //void frame(int number = -1);
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
 
     int sourceLine = 34;
     QCOMPARE(init(qmlscene, ONCOMPLETED_QMLFILE), ConnectSuccess);
 
     m_client->setBreakpoint(QLatin1String(ONCOMPLETED_QMLFILE), sourceLine, -1, true);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QVERIFY(waitForClientSignal(SIGNAL(stopped())));
 
     m_client->frame();
@@ -1320,14 +1302,13 @@ void tst_QQmlDebugJS::getScopeDetails()
 {
     //void scope(int number = -1, int frameNumber = -1);
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
 
     int sourceLine = 34;
     QCOMPARE(init(qmlscene, ONCOMPLETED_QMLFILE), ConnectSuccess);
 
     m_client->setBreakpoint(QLatin1String(ONCOMPLETED_QMLFILE), sourceLine, -1, true);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QVERIFY(waitForClientSignal(SIGNAL(stopped())));
 
     m_client->scope();
@@ -1355,15 +1336,14 @@ void tst_QQmlDebugJS::evaluateInGlobalScope()
 void tst_QQmlDebugJS::evaluateInLocalScope()
 {
     //void evaluate(QString expr, bool global = false, bool disableBreak = false, int frame = -1, const QVariantMap &addContext = QVariantMap());
-
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
+
     int sourceLine = 34;
     QCOMPARE(init(qmlscene, ONCOMPLETED_QMLFILE), ConnectSuccess);
 
     m_client->setBreakpoint(QLatin1String(ONCOMPLETED_QMLFILE), sourceLine, -1, true);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QVERIFY(waitForClientSignal(SIGNAL(stopped())));
 
     m_client->frame();
@@ -1440,14 +1420,13 @@ void tst_QQmlDebugJS::evaluateInContext()
 void tst_QQmlDebugJS::getScripts()
 {
     //void scripts(int types = -1, QList<int> ids = QList<int>(), bool includeSource = false, QVariant filter = QVariant());
-
     QFETCH(bool, qmlscene);
-    QFETCH(bool, redundantRefs);
     QFETCH(bool, namesAsObjects);
+
     QCOMPARE(init(qmlscene), ConnectSuccess);
 
     m_client->setBreakpoint(QString(TEST_QMLFILE), 35, -1, true);
-    m_client->connect(redundantRefs, namesAsObjects);
+    m_client->connect(namesAsObjects);
     QVERIFY(waitForClientSignal(SIGNAL(stopped())));
 
     m_client->scripts();
@@ -1535,16 +1514,11 @@ QList<QQmlDebugClient *> tst_QQmlDebugJS::createClients()
 void tst_QQmlDebugJS::targetData()
 {
     QTest::addColumn<bool>("qmlscene");
-    QTest::addColumn<bool>("redundantRefs");
     QTest::addColumn<bool>("namesAsObjects");
-    QTest::newRow("custom / redundant / objects")   << false << true  << true;
-    QTest::newRow("qmlscene / redundant / objects") << true  << true  << true;
-    QTest::newRow("custom / redundant / strings")   << false << true  << false;
-    QTest::newRow("qmlscene / redundant / strings") << true  << true  << false;
-    QTest::newRow("custom / sparse / objects")      << false << false << true;
-    QTest::newRow("qmlscene / sparse / objects")    << true  << false << true;
-    QTest::newRow("custom / sparse / strings")      << false << false << false;
-    QTest::newRow("qmlscene / sparse / strings")    << true  << false << false;
+    QTest::newRow("custom / objects")   << false << true;
+    QTest::newRow("qmlscene / objects") << true  << true;
+    QTest::newRow("custom / strings")   << false << false;
+    QTest::newRow("qmlscene / strings") << true  << false;
 }
 
 bool tst_QQmlDebugJS::waitForClientSignal(const char *signal, int timeout)