[Tizen] Revert "Use touch consumed return to set whether we process a gesture or not
[platform/core/uifw/dali-core.git] / dali / internal / event / events / touch-event-processor.cpp
old mode 100644 (file)
new mode 100755 (executable)
index 90f75ca..146387c
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2018 Samsung Electronics Co., Ltd.
+ * Copyright (c) 2020 Samsung Electronics Co., Ltd.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
 #endif
 
 // INTERNAL INCLUDES
-#include <dali/public-api/events/touch-data.h>
+#include <dali/public-api/events/touch-event.h>
 #include <dali/public-api/math/vector2.h>
 #include <dali/public-api/signals/callback.h>
 #include <dali/integration-api/debug.h>
 #include <dali/integration-api/events/touch-event-integ.h>
 #include <dali/internal/event/actors/actor-impl.h>
 #include <dali/internal/event/actors/layer-impl.h>
-#include <dali/internal/event/common/stage-impl.h>
+#include <dali/internal/event/common/scene-impl.h>
 #include <dali/internal/event/events/hit-test-algorithm-impl.h>
 #include <dali/internal/event/events/multi-point-event-util.h>
-#include <dali/internal/event/events/touch-data-impl.h>
+#include <dali/internal/event/events/touch-event-impl.h>
 #include <dali/internal/event/render-tasks/render-task-impl.h>
 
 namespace Dali
@@ -60,11 +60,41 @@ const char * TOUCH_POINT_STATE[ 6 ] =
 
 #endif // defined(DEBUG_ENABLED)
 
+Dali::Actor EmitInterceptTouchSignals( Dali::Actor actor, const Dali::TouchEvent& touchEvent )
+{
+  Dali::Actor interceptedActor;
+
+  if( actor )
+  {
+     Dali::Actor parent = actor.GetParent();
+     if( parent )
+     {
+       // Recursively deliver events to the actor and its parents for intercept touch event.
+       interceptedActor = EmitInterceptTouchSignals( parent, touchEvent );
+     }
+
+     if( !interceptedActor )
+     {
+       bool intercepted = false;
+       Actor& actorImpl( GetImplementation(actor) );
+       if( actorImpl.GetInterceptTouchRequired() )
+       {
+          intercepted = actorImpl.EmitInterceptTouchEventSignal( touchEvent );
+          if( intercepted )
+          {
+            interceptedActor = Dali::Actor( &actorImpl );
+          }
+       }
+     }
+  }
+
+  return interceptedActor;
+}
 
 /**
  *  Recursively deliver events to the actor and its parents, until the event is consumed or the stage is reached.
  */
-Dali::Actor EmitTouchSignals( Dali::Actor actor, const TouchEvent& event, const Dali::TouchData& touchData )
+Dali::Actor EmitTouchSignals( Dali::Actor actor, const Dali::TouchEvent& touchEvent )
 {
   Dali::Actor consumedActor;
 
@@ -79,7 +109,7 @@ Dali::Actor EmitTouchSignals( Dali::Actor actor, const TouchEvent& event, const
     // Only emit the signal if the actor's touch signal has connections (or derived actor implementation requires touch).
     if ( actorImpl.GetTouchRequired() )
     {
-      consumed = actorImpl.EmitTouchEventSignal( event, touchData );
+      consumed = actorImpl.EmitTouchEventSignal( touchEvent );
     }
 
     if ( consumed )
@@ -96,7 +126,7 @@ Dali::Actor EmitTouchSignals( Dali::Actor actor, const TouchEvent& event, const
            (parent == oldParent) )
       {
         // One of the actor's parents may consumed the event and they should be set as the consumed actor.
-        consumedActor = EmitTouchSignals( parent, event, touchData );
+        consumedActor = EmitTouchSignals( parent, touchEvent );
       }
     }
   }
@@ -106,29 +136,27 @@ Dali::Actor EmitTouchSignals( Dali::Actor actor, const TouchEvent& event, const
 
 Dali::Actor AllocAndEmitTouchSignals( unsigned long time,  Dali::Actor actor, const Integration::Point& point )
 {
-  TouchEvent touchEvent( time );
-  TouchDataPtr touchData( new TouchData( time ) );
-  Dali::TouchData touchDataHandle( touchData.Get() );
+  TouchEventPtr touchEvent( new TouchEvent( time ) );
+  Dali::TouchEvent touchEventHandle( touchEvent.Get() );
 
-  touchEvent.points.push_back( point.GetTouchPoint() );
-  touchData->AddPoint( point );
+  touchEvent->AddPoint( point );
 
-  return EmitTouchSignals( actor, touchEvent, touchDataHandle );
+  return EmitTouchSignals( actor, touchEventHandle );
 }
 
 
 /**
  * Changes the state of the primary point to leave and emits the touch signals
  */
-Dali::Actor EmitTouchSignals( Actor* actor, RenderTask& renderTask, const TouchEvent& originalEvent, const TouchDataPtr& originalTouchData, PointState::Type state )
+Dali::Actor EmitTouchSignals( Actor* actor, RenderTask& renderTask, const TouchEventPtr& originalTouchEvent, PointState::Type state )
 {
   Dali::Actor consumingActor;
 
   if( actor )
   {
-    TouchDataPtr touchData = TouchData::Clone( *originalTouchData.Get() );
+    TouchEventPtr touchEventImpl = TouchEvent::Clone( *originalTouchEvent.Get() );
 
-    Integration::Point& primaryPoint = touchData->GetPoint( 0 );
+    Integration::Point& primaryPoint = touchEventImpl->GetPoint( 0 );
 
     const Vector2& screenPosition = primaryPoint.GetScreenPosition();
     Vector2 localPosition;
@@ -138,21 +166,61 @@ Dali::Actor EmitTouchSignals( Actor* actor, RenderTask& renderTask, const TouchE
     primaryPoint.SetHitActor( Dali::Actor( actor ) );
     primaryPoint.SetState( state );
 
-    TouchEvent touchEvent( originalEvent );
-    touchEvent.points[0] = primaryPoint.GetTouchPoint();
-
-    consumingActor = EmitTouchSignals( Dali::Actor(actor), touchEvent, Dali::TouchData( touchData.Get() ) );
+    consumingActor = EmitTouchSignals( Dali::Actor(actor), Dali::TouchEvent( touchEventImpl.Get() ) );
   }
 
   return consumingActor;
 }
 
+/**
+ * @brief Parses the primary touch point by performing a hit-test if necessary
+ *
+ * @param[out] hitTestResults The hit test results are put into this variable
+ * @param[in/out] capturingTouchActorObserver The observer for the capturing touch actor member
+ * @param[in] lastRenderTask The last render task member
+ * @param[in] currentPoint The current point information
+ * @param[in] scene The scene that this touch is related to
+ */
+void ParsePrimaryTouchPoint(
+    HitTestAlgorithm::Results& hitTestResults,
+    ActorObserver& capturingTouchActorObserver,
+    const RenderTaskPtr& lastRenderTask,
+    const Integration::Point& currentPoint,
+    const Internal::Scene& scene )
+{
+  Actor* capturingTouchActor = capturingTouchActorObserver.GetActor();
+
+  // We only set the capturing touch actor when the first touch-started actor captures all touch so if it's set, just use it
+  if( capturingTouchActor && lastRenderTask )
+  {
+    hitTestResults.actor = Dali::Actor( capturingTouchActor );
+    hitTestResults.renderTask = lastRenderTask;
+    const Vector2& screenPosition = currentPoint.GetScreenPosition();
+    capturingTouchActor->ScreenToLocal( *lastRenderTask, hitTestResults.actorCoordinates.x, hitTestResults.actorCoordinates.y, screenPosition.x, screenPosition.y );
+  }
+  else
+  {
+    HitTestAlgorithm::HitTest( scene.GetSize(), scene.GetRenderTaskList(), scene.GetLayerList(), currentPoint.GetScreenPosition(), hitTestResults );
+
+    if( currentPoint.GetState() == PointState::STARTED && hitTestResults.actor )
+    {
+      // If we've just started touch, then check whether the actor has requested to capture all touch events
+      Actor* hitActor = &GetImplementation( hitTestResults.actor );
+      if( hitActor->CapturesAllTouchAfterStart() )
+      {
+        capturingTouchActorObserver.SetActor( hitActor );
+      }
+    }
+  }
+}
+
 } // unnamed namespace
 
-TouchEventProcessor::TouchEventProcessor( Stage& stage )
-: mStage( stage ),
+TouchEventProcessor::TouchEventProcessor( Scene& scene )
+: mScene( scene ),
   mLastPrimaryHitActor( MakeCallback( this, &TouchEventProcessor::OnObservedActorDisconnected ) ),
   mLastConsumedActor(),
+  mCapturingTouchActor(),
   mTouchDownConsumedActor(),
   mLastRenderTask()
 {
@@ -164,14 +232,11 @@ TouchEventProcessor::~TouchEventProcessor()
   DALI_LOG_TRACE_METHOD( gLogFilter );
 }
 
-void TouchEventProcessor::ProcessTouchEvent( const Integration::TouchEvent& event )
+bool TouchEventProcessor::ProcessTouchEvent( const Integration::TouchEvent& event )
 {
   DALI_LOG_TRACE_METHOD( gLogFilter );
-
   DALI_ASSERT_ALWAYS( !event.points.empty() && "Empty TouchEvent sent from Integration\n" );
 
-  Stage& stage = mStage;
-
   PRINT_HIERARCHY(gLogFilter);
 
   // 1) Check if it is an interrupted event - we should inform our last primary hit actor about this
@@ -215,79 +280,94 @@ void TouchEventProcessor::ProcessTouchEvent( const Integration::TouchEvent& even
       AllocAndEmitTouchSignals( event.time, touchDownConsumedActorHandle, currentPoint );
     }
 
-    mLastPrimaryHitActor.SetActor( NULL );
-    mLastConsumedActor.SetActor( NULL );
-    mTouchDownConsumedActor.SetActor( NULL );
+    mLastPrimaryHitActor.SetActor( nullptr );
+    mLastConsumedActor.SetActor( nullptr );
+    mCapturingTouchActor.SetActor( nullptr );
+    mTouchDownConsumedActor.SetActor( nullptr );
     mLastRenderTask.Reset();
 
     currentPoint.SetHitActor( Dali::Actor() );
 
-    TouchEvent touchEvent( event.time );
-    TouchDataPtr touchData( new TouchData( event.time ) );
-    Dali::TouchData touchDataHandle( touchData.Get() );
+    TouchEventPtr touchEventImpl( new TouchEvent( event.time ) );
+    Dali::TouchEvent touchEventHandle( touchEventImpl.Get() );
 
-    touchEvent.points.push_back( currentPoint.GetTouchPoint() );
-    touchData->AddPoint( currentPoint );
+    touchEventImpl->AddPoint( currentPoint );
 
-    mStage.EmitTouchedSignal( touchEvent, touchDataHandle );
-
-    return; // No need for hit testing
+    mScene.EmitTouchedSignal( touchEventHandle );
+    return false; // No need for hit testing & already an interrupted event so just return false
   }
 
   // 2) Hit Testing.
-  TouchEvent touchEvent( event.time );
-  TouchDataPtr touchData( new TouchData( event.time ) );
-  Dali::TouchData touchDataHandle( touchData.Get() );
+  TouchEventPtr touchEventImpl( new TouchEvent( event.time ) );
+  Dali::TouchEvent touchEventHandle( touchEventImpl.Get() );
 
   DALI_LOG_INFO( gLogFilter, Debug::Concise, "\n" );
   DALI_LOG_INFO( gLogFilter, Debug::General, "Point(s): %d\n", event.GetPointCount() );
 
   RenderTaskPtr currentRenderTask;
+  bool firstPointParsed = false;
 
-  for ( Integration::PointContainerConstIterator iter = event.points.begin(), beginIter = event.points.begin(), endIter = event.points.end(); iter != endIter; ++iter )
+  for ( auto&& currentPoint : event.points )
   {
     HitTestAlgorithm::Results hitTestResults;
-    HitTestAlgorithm::HitTest( stage, iter->GetScreenPosition(), hitTestResults );
+    if( !firstPointParsed )
+    {
+      firstPointParsed = true;
+      ParsePrimaryTouchPoint( hitTestResults, mCapturingTouchActor, mLastRenderTask, currentPoint, mScene );
 
-    Integration::Point newPoint( *iter );
+      // Only set the currentRenderTask for the primary hit actor.
+      currentRenderTask = hitTestResults.renderTask;
+    }
+    else
+    {
+      HitTestAlgorithm::HitTest( mScene.GetSize(), mScene.GetRenderTaskList(), mScene.GetLayerList(), currentPoint.GetScreenPosition(), hitTestResults );
+    }
+
+    Integration::Point newPoint( currentPoint );
     newPoint.SetHitActor( hitTestResults.actor );
     newPoint.SetLocalPosition( hitTestResults.actorCoordinates );
 
-    touchEvent.points.push_back( newPoint.GetTouchPoint() );
-    touchData->AddPoint( newPoint );
+    touchEventImpl->AddPoint( newPoint );
 
     DALI_LOG_INFO( gLogFilter, Debug::General, "  State(%s), Screen(%.0f, %.0f), HitActor(%p, %s), Local(%.2f, %.2f)\n",
-                   TOUCH_POINT_STATE[iter->GetState()], iter->GetScreenPosition().x, iter->GetScreenPosition().y,
+                   TOUCH_POINT_STATE[currentPoint.GetState()], currentPoint.GetScreenPosition().x, currentPoint.GetScreenPosition().y,
                    ( hitTestResults.actor ? reinterpret_cast< void* >( &hitTestResults.actor.GetBaseObject() ) : NULL ),
-                   ( hitTestResults.actor ? hitTestResults.actor.GetName().c_str() : "" ),
+                   ( hitTestResults.actor ? hitTestResults.actor.GetProperty< std::string >( Dali::Actor::Property::NAME ).c_str() : "" ),
                    hitTestResults.actorCoordinates.x, hitTestResults.actorCoordinates.y );
 
-    // Only set the currentRenderTask for the primary hit actor.
-    if ( iter == beginIter && hitTestResults.renderTask )
-    {
-      currentRenderTask = hitTestResults.renderTask;
-    }
   }
 
   // 3) Recursively deliver events to the actor and its parents, until the event is consumed or the stage is reached.
 
+  bool consumed = false;
+
   // Emit the touch signal
   Dali::Actor consumedActor;
   if ( currentRenderTask )
   {
-    consumedActor = EmitTouchSignals( touchData->GetPoint( 0 ).GetHitActor(), touchEvent, touchDataHandle );
+    // Emit the intercept touch signal
+    Dali::Actor interceptedActor = EmitInterceptTouchSignals( touchEventImpl->GetPoint( 0 ).GetHitActor(), touchEventHandle );
+    if( interceptedActor )
+    {
+      consumedActor = EmitTouchSignals( interceptedActor, touchEventHandle );
+    }
+    else
+    {
+      consumedActor = EmitTouchSignals( touchEventImpl->GetPoint( 0 ).GetHitActor(), touchEventHandle );
+    }
+    consumed = consumedActor ? true : false;
   }
 
-  Integration::Point& primaryPoint = touchData->GetPoint( 0 );
+  Integration::Point& primaryPoint = touchEventImpl->GetPoint( 0 );
   Dali::Actor primaryHitActor = primaryPoint.GetHitActor();
   PointState::Type primaryPointState = primaryPoint.GetState();
 
-  DALI_LOG_INFO( gLogFilter, Debug::Concise, "PrimaryHitActor:     (%p) %s\n", primaryHitActor ? reinterpret_cast< void* >( &primaryHitActor.GetBaseObject() ) : NULL, primaryHitActor ? primaryHitActor.GetName().c_str() : "" );
-  DALI_LOG_INFO( gLogFilter, Debug::Concise, "ConsumedActor:       (%p) %s\n", consumedActor ? reinterpret_cast< void* >( &consumedActor.GetBaseObject() ) : NULL, consumedActor ? consumedActor.GetName().c_str() : "" );
+  DALI_LOG_INFO( gLogFilter, Debug::Concise, "PrimaryHitActor:     (%p) %s\n", primaryHitActor ? reinterpret_cast< void* >( &primaryHitActor.GetBaseObject() ) : NULL, primaryHitActor ? primaryHitActor.GetProperty< std::string >( Dali::Actor::Property::NAME ).c_str() : "" );
+  DALI_LOG_INFO( gLogFilter, Debug::Concise, "ConsumedActor:       (%p) %s\n", consumedActor ? reinterpret_cast< void* >( &consumedActor.GetBaseObject() ) : NULL, consumedActor ? consumedActor.GetProperty< std::string >( Dali::Actor::Property::NAME ).c_str() : "" );
 
   if ( ( primaryPointState == PointState::DOWN ) &&
-       ( touchEvent.GetPointCount() == 1 ) &&
-       ( consumedActor && consumedActor.OnStage() ) )
+       ( touchEventImpl->GetPointCount() == 1 ) &&
+       ( consumedActor && consumedActor.GetProperty< bool >( Dali::Actor::Property::CONNECTED_TO_SCENE ) ) )
   {
     mTouchDownConsumedActor.SetActor( &GetImplementation( consumedActor ) );
   }
@@ -299,7 +379,7 @@ void TouchEventProcessor::ProcessTouchEvent( const Integration::TouchEvent& even
   Actor* lastConsumedActor( mLastConsumedActor.GetActor() );
   if( ( primaryPointState == PointState::MOTION ) || ( primaryPointState == PointState::UP ) || ( primaryPointState == PointState::STATIONARY ) )
   {
-    if ( mLastRenderTask )
+    if( mLastRenderTask )
     {
       Dali::Actor leaveEventConsumer;
       RenderTask& lastRenderTaskImpl = *mLastRenderTask.Get();
@@ -313,7 +393,7 @@ void TouchEventProcessor::ProcessTouchEvent( const Integration::TouchEvent& even
           if ( lastPrimaryHitActor->GetLeaveRequired() )
           {
             DALI_LOG_INFO( gLogFilter, Debug::Concise, "LeaveActor(Hit):     (%p) %s\n", reinterpret_cast< void* >( lastPrimaryHitActor ), lastPrimaryHitActor->GetName().c_str() );
-            leaveEventConsumer = EmitTouchSignals( mLastPrimaryHitActor.GetActor(), lastRenderTaskImpl, touchEvent, touchData, PointState::LEAVE );
+            leaveEventConsumer = EmitTouchSignals( mLastPrimaryHitActor.GetActor(), lastRenderTaskImpl, touchEventImpl, PointState::LEAVE );
           }
         }
         else
@@ -321,10 +401,12 @@ void TouchEventProcessor::ProcessTouchEvent( const Integration::TouchEvent& even
           // At this point mLastPrimaryHitActor was touchable and sensitive in the previous touch event process but is not in the current one.
           // An interrupted event is send to allow some actors to go back to their original state (i.e. Button controls)
           DALI_LOG_INFO( gLogFilter, Debug::Concise, "InterruptedActor(Hit):     (%p) %s\n", reinterpret_cast< void* >( lastPrimaryHitActor ), lastPrimaryHitActor->GetName().c_str() );
-          leaveEventConsumer = EmitTouchSignals( mLastPrimaryHitActor.GetActor(), lastRenderTaskImpl, touchEvent, touchData, PointState::INTERRUPTED );
+          leaveEventConsumer = EmitTouchSignals( mLastPrimaryHitActor.GetActor(), lastRenderTaskImpl, touchEventImpl, PointState::INTERRUPTED );
         }
       }
 
+      consumed |= leaveEventConsumer ? true : false;
+
       // Check if the motion event has been consumed by another actor's listener.  In this case, the previously
       // consumed actor's listeners may need to be informed (through a leave event).
       // Further checks here to ensure we do not signal the same actor twice for the same event.
@@ -339,7 +421,7 @@ void TouchEventProcessor::ProcessTouchEvent( const Integration::TouchEvent& even
           if( lastConsumedActor->GetLeaveRequired() )
           {
             DALI_LOG_INFO( gLogFilter, Debug::Concise, "LeaveActor(Consume): (%p) %s\n", reinterpret_cast< void* >( lastConsumedActor ), lastConsumedActor->GetName().c_str() );
-            EmitTouchSignals( lastConsumedActor, lastRenderTaskImpl, touchEvent, touchData, PointState::LEAVE );
+            EmitTouchSignals( lastConsumedActor, lastRenderTaskImpl, touchEventImpl, PointState::LEAVE );
           }
         }
         else
@@ -347,7 +429,7 @@ void TouchEventProcessor::ProcessTouchEvent( const Integration::TouchEvent& even
           // At this point mLastConsumedActor was touchable and sensitive in the previous touch event process but is not in the current one.
           // An interrupted event is send to allow some actors to go back to their original state (i.e. Button controls)
           DALI_LOG_INFO( gLogFilter, Debug::Concise, "InterruptedActor(Consume):     (%p) %s\n", reinterpret_cast< void* >( lastConsumedActor ), lastConsumedActor->GetName().c_str() );
-          EmitTouchSignals( mLastConsumedActor.GetActor(), lastRenderTaskImpl, touchEvent, touchData, PointState::INTERRUPTED );
+          EmitTouchSignals( mLastConsumedActor.GetActor(), lastRenderTaskImpl, touchEventImpl, PointState::INTERRUPTED );
         }
       }
     }
@@ -358,33 +440,35 @@ void TouchEventProcessor::ProcessTouchEvent( const Integration::TouchEvent& even
 
   if ( primaryPointState == PointState::UP )
   {
-    mLastPrimaryHitActor.SetActor( NULL );
-    mLastConsumedActor.SetActor( NULL );
+    mLastPrimaryHitActor.SetActor( nullptr );
+    mLastConsumedActor.SetActor( nullptr );
+    mCapturingTouchActor.SetActor( nullptr );
     mLastRenderTask.Reset();
   }
   else
   {
-    // The primaryHitActor may have been removed from the stage so ensure it is still on the stage before setting members.
-    if ( primaryHitActor && primaryHitActor.OnStage() )
+    // The primaryHitActor may have been removed from the scene so ensure it is still on the scene before setting members.
+    if ( primaryHitActor && GetImplementation( primaryHitActor ).OnScene() )
     {
       mLastPrimaryHitActor.SetActor( &GetImplementation( primaryHitActor ) );
 
-      // Only observe the consumed actor if we have a primaryHitActor (check if it is still on stage).
-      if ( consumedActor && consumedActor.OnStage() )
+      // Only observe the consumed actor if we have a primaryHitActor (check if it is still on the scene).
+      if ( consumedActor && GetImplementation( consumedActor ).OnScene() )
       {
         mLastConsumedActor.SetActor( &GetImplementation( consumedActor ) );
       }
       else
       {
-        mLastConsumedActor.SetActor( NULL );
+        mLastConsumedActor.SetActor( nullptr );
       }
 
       mLastRenderTask = currentRenderTask;
     }
     else
     {
-      mLastPrimaryHitActor.SetActor( NULL );
-      mLastConsumedActor.SetActor( NULL );
+      mLastPrimaryHitActor.SetActor( nullptr );
+      mLastConsumedActor.SetActor( nullptr );
+      mCapturingTouchActor.SetActor( nullptr );
       mLastRenderTask.Reset();
     }
   }
@@ -392,7 +476,7 @@ void TouchEventProcessor::ProcessTouchEvent( const Integration::TouchEvent& even
   // 6) Emit an interrupted event to the touch-down actor if it hasn't consumed the up and
   //    emit the stage touched event if required.
 
-  if ( touchEvent.GetPointCount() == 1 ) // Only want the first touch and the last release
+  if ( touchEventImpl->GetPointCount() == 1 ) // Only want the first touch and the last release
   {
     switch ( primaryPointState )
     {
@@ -406,20 +490,21 @@ void TouchEventProcessor::ProcessTouchEvent( const Integration::TouchEvent& even
         {
           Dali::Actor touchDownConsumedActorHandle( touchDownConsumedActor );
 
-          Integration::Point currentPoint = touchData->GetPoint( 0 );
+          Integration::Point currentPoint = touchEventImpl->GetPoint( 0 );
           currentPoint.SetHitActor( touchDownConsumedActorHandle );
           currentPoint.SetState( PointState::INTERRUPTED );
 
           AllocAndEmitTouchSignals( event.time, touchDownConsumedActorHandle, currentPoint );
         }
 
-        mTouchDownConsumedActor.SetActor( NULL );
+        mTouchDownConsumedActor.SetActor( nullptr );
+
+        DALI_FALLTHROUGH;
       }
-      // No break, Fallthrough
 
       case PointState::DOWN:
       {
-        mStage.EmitTouchedSignal( touchEvent, touchDataHandle );
+        mScene.EmitTouchedSignal( touchEventHandle );
         break;
       }
 
@@ -433,35 +518,34 @@ void TouchEventProcessor::ProcessTouchEvent( const Integration::TouchEvent& even
       }
     }
   }
+
+  return consumed;
 }
 
 void TouchEventProcessor::OnObservedActorDisconnected( Actor* actor )
 {
   if ( actor == mLastPrimaryHitActor.GetActor() )
   {
-    Dali::Actor handle( actor );
+    Dali::Actor actorHandle( actor );
 
     Integration::Point point;
     point.SetState( PointState::INTERRUPTED );
-    point.SetHitActor( handle );
-
-    TouchDataPtr touchData( new TouchData );
-    touchData->AddPoint( point );
-    Dali::TouchData touchDataHandle( touchData.Get() );
+    point.SetHitActor( actorHandle );
 
-    TouchEvent touchEvent( 0 );
-    touchEvent.points.push_back( point.GetTouchPoint() );
+    TouchEventPtr touchEventImpl( new TouchEvent );
+    touchEventImpl->AddPoint( point );
+    Dali::TouchEvent touchEventHandle( touchEventImpl.Get() );
 
-    Dali::Actor eventConsumer = EmitTouchSignals( handle, touchEvent, touchDataHandle );
+    Dali::Actor eventConsumer = EmitTouchSignals( actorHandle, touchEventHandle );
 
     if ( mLastConsumedActor.GetActor() != eventConsumer )
     {
-      EmitTouchSignals( Dali::Actor( mLastConsumedActor.GetActor() ), touchEvent, touchDataHandle );
+      EmitTouchSignals( Dali::Actor( mLastConsumedActor.GetActor() ), touchEventHandle );
     }
 
     // Do not set mLastPrimaryHitActor to NULL we may be iterating through its observers
 
-    mLastConsumedActor.SetActor( NULL );
+    mLastConsumedActor.SetActor( nullptr );
     mLastRenderTask.Reset();
   }
 }