Fixing overlay position and memory leaks in slider.
authorkishorekumar Dachinamoorthy <kishore.d@samsung.com>
Thu, 4 Jul 2013 06:12:01 +0000 (11:42 +0530)
committerkishorekumar Dachinamoorthy <kishore.d@samsung.com>
Thu, 4 Jul 2013 06:53:49 +0000 (12:23 +0530)
Change-Id: I7259255c894b0b6df7c8dce1e62b3aac71e3e8c5

src/ui/controls/FUiCtrl_SliderOverlay.cpp
src/ui/controls/FUiCtrl_SliderPresenter.cpp

index edb61f3..4e3ade5 100644 (file)
@@ -198,12 +198,6 @@ _SliderOverlay::DrawSliderOverlay(int positionX, int handleWidth, int value)
                                                                        __sliderRect.y + overlayY - (overlayHeight + overlayBottomMargin),
                                                                        overlayWidth, overlayHeight);
 
-       // Check SliderOverlay Position
-       if (__sliderOverlayRect.x < __sliderRect.x)
-       {
-               __sliderOverlayRect.x = __sliderRect.x;
-       }
-
        SetBounds(__sliderOverlayRect);
 
        // Get canvas & Clear BG
index aceb42e..8b89d16 100644 (file)
@@ -1528,6 +1528,9 @@ _SliderPresenter::OnTouchReleased(const _Control& source, const _TouchInfo& touc
                point.x = pCanvas->GetBoundsF().x + pCanvas->GetBoundsF().width;
        }
 
+       delete pCanvas;
+       pCanvas = null;
+
        if (__isSliderPressed == true)
        {
                PLAY_FEEDBACK(_RESOURCE_FEEDBACK_PATTERN_TAP, __pSlider);
@@ -1619,6 +1622,9 @@ _SliderPresenter::OnTouchMoved(const _Control& source, const _TouchInfo& touchin
                __pSlideTimer->Start(__timerValue);
        }
 
+       delete pCanvas;
+       pCanvas = null;
+
        return true;
 }