From 9496fcbc229d491cf3aab59e96c68da2787f06d8 Mon Sep 17 00:00:00 2001
From: Simjees Abraham <simjees.abraham@nokia.com>
Date: Mon, 16 Apr 2012 15:14:12 +0200
Subject: [PATCH] Debugger: Code cleanup for Inspect Tool

Code cleanup done for Inspect Tool. Member variables removed which were
not required.

Change-Id: I74ab43c4e8bf39d00a5428bb6219653b7887c46d
Reviewed-by: Aurindam Jana <aurindam.jana@nokia.com>
---
 .../qmltooling/qmldbg_qtquick2/inspecttool.cpp     | 37 ++++++++++------------
 .../qmltooling/qmldbg_qtquick2/inspecttool.h       |  2 --
 2 files changed, 16 insertions(+), 23 deletions(-)

diff --git a/src/plugins/qmltooling/qmldbg_qtquick2/inspecttool.cpp b/src/plugins/qmltooling/qmldbg_qtquick2/inspecttool.cpp
index a09d0ce..796549f 100644
--- a/src/plugins/qmltooling/qmldbg_qtquick2/inspecttool.cpp
+++ b/src/plugins/qmltooling/qmldbg_qtquick2/inspecttool.cpp
@@ -68,7 +68,6 @@ InspectTool::InspectTool(QQuickViewInspector *inspector, QQuickView *view) :
     m_tapEvent(false),
     m_rootItem(view->rootItem()),
     m_originalPosition(view->rootItem()->pos()),
-    m_currentScale(1.0f),
     m_smoothScaleFactor(Constants::ZoomSnapDelta),
     m_minScale(0.125f),
     m_maxScale(48.0f),
@@ -171,8 +170,8 @@ void InspectTool::wheelEvent(QWheelEvent *event)
     Qt::KeyboardModifier smoothZoomModifier = Qt::ControlModifier;
     if (event->modifiers() & smoothZoomModifier) {
         int numDegrees = event->delta() / 8;
-        qreal newScale = m_currentScale + m_smoothScaleFactor * (numDegrees / 15.0f);
-        scaleView(newScale / m_currentScale, m_mousePosition, m_mousePosition);
+        qreal newScale = m_rootItem->scale() + m_smoothScaleFactor * (numDegrees / 15.0f);
+        scaleView(newScale / m_rootItem->scale(), m_mousePosition, m_mousePosition);
     } else if (!event->modifiers()) {
         if (event->delta() > 0) {
             zoomIn();
@@ -201,7 +200,7 @@ void InspectTool::keyReleaseEvent(QKeyEvent *event)
     case Qt::Key_8:
     case Qt::Key_9: {
         qreal newScale = ((event->key() - Qt::Key_0) * 1.0f);
-        scaleView(newScale / m_currentScale, m_mousePosition, m_mousePosition);
+        scaleView(newScale / m_rootItem->scale(), m_mousePosition, m_mousePosition);
         break;
     }
     default:
@@ -277,38 +276,34 @@ void InspectTool::touchEvent(QTouchEvent *event)
 void InspectTool::scaleView(const qreal &factor, const QPointF &newcenter, const QPointF &oldcenter)
 {
     m_pressAndHoldTimer.stop();
-    if (((m_currentScale * factor) > m_maxScale)
-            || ((m_currentScale * factor) < m_minScale)) {
+    if (((m_rootItem->scale() * factor) > m_maxScale)
+            || ((m_rootItem->scale() * factor) < m_minScale)) {
         return;
     }
     //New position = new center + scalefactor * (oldposition - oldcenter)
-    m_adjustedOrigin = newcenter + (factor * (m_adjustedOrigin - oldcenter));
-    m_currentScale *= factor;
-
-    m_rootItem->setScale(m_currentScale);
-    m_rootItem->setPos(m_adjustedOrigin);
+    QPointF newPosition = newcenter + (factor * (m_rootItem->pos() - oldcenter));
+    m_rootItem->setScale(m_rootItem->scale() * factor);
+    m_rootItem->setPos(newPosition);
 }
 
 void InspectTool::zoomIn()
 {
     qreal newScale = nextZoomScale(ZoomIn);
-    scaleView(newScale / m_currentScale, m_mousePosition, m_mousePosition);
+    scaleView(newScale / m_rootItem->scale(), m_mousePosition, m_mousePosition);
 }
 
 void InspectTool::zoomOut()
 {
     qreal newScale = nextZoomScale(ZoomOut);
-    scaleView(newScale / m_currentScale, m_mousePosition, m_mousePosition);
+    scaleView(newScale / m_rootItem->scale(), m_mousePosition, m_mousePosition);
 }
 
 void InspectTool::zoomTo100()
 {
     m_didPressAndHold = true;
-    m_currentScale = 1.0;
-    m_adjustedOrigin = QPointF(0, 0);
 
-    m_rootItem->setPos(m_adjustedOrigin);
-    m_rootItem->setScale(m_currentScale);
+    m_rootItem->setPos(QPointF(0, 0));
+    m_rootItem->setScale(1.0);
 }
 
 qreal InspectTool::nextZoomScale(ZoomDirection direction)
@@ -336,13 +331,13 @@ qreal InspectTool::nextZoomScale(ZoomDirection direction)
 
     if (direction == ZoomIn) {
         for (int i = 0; i < zoomScales.length(); ++i) {
-            if (zoomScales[i] > m_currentScale)
+            if (zoomScales[i] > m_rootItem->scale())
                 return zoomScales[i];
         }
         return zoomScales.last();
     } else {
         for (int i = zoomScales.length() - 1; i >= 0; --i) {
-            if (zoomScales[i] < m_currentScale)
+            if (zoomScales[i] < m_rootItem->scale())
                 return zoomScales[i];
         }
         return zoomScales.first();
@@ -359,9 +354,9 @@ void InspectTool::initializeDrag(const QPointF &pos)
 
 void InspectTool::dragItemToPosition()
 {
-    m_adjustedOrigin += m_mousePosition - m_dragStartPosition;
+    QPointF newPosition = m_rootItem->pos() + m_mousePosition - m_dragStartPosition;
     m_dragStartPosition = m_mousePosition;
-    m_rootItem->setPos(m_adjustedOrigin);
+    m_rootItem->setPos(newPosition);
 }
 
 void InspectTool::moveItem(bool valid)
diff --git a/src/plugins/qmltooling/qmldbg_qtquick2/inspecttool.h b/src/plugins/qmltooling/qmldbg_qtquick2/inspecttool.h
index 25d0c63..f452802 100644
--- a/src/plugins/qmltooling/qmldbg_qtquick2/inspecttool.h
+++ b/src/plugins/qmltooling/qmldbg_qtquick2/inspecttool.h
@@ -108,11 +108,9 @@ private:
     bool m_didPressAndHold;
     bool m_tapEvent;
     QPointer<QQuickItem> m_rootItem;
-    QPointF m_adjustedOrigin;
     QPointF m_dragStartPosition;
     QPointF m_mousePosition;
     QPointF m_originalPosition;
-    qreal m_currentScale;
     qreal m_smoothScaleFactor;
     qreal m_minScale;
     qreal m_maxScale;
-- 
2.7.4