The DEPTH value was being retrieved incorrectly. Fix this

Change-Id: Ie0c7556bdaaa661cdfbcf5b298166a94048151fa

Reset memory pool when core terminate

Since we can create new Core without process terminate, (e.g. offscreen application)
We should ensurely clear the memory usage.

Without this clean-up process, some memory might be remained after
core terminate. For example, some OwnerPointer<Node> are exist at Message during
app terminate, it might not be release surely.

Change-Id: Iaef0b16bcd6631aa8a52e9b52bfa39374be770b6
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Fix UTC build error

Change-Id: Ie78675bd4a10301423214d163f4c38fe99599451
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Get the actor who trigger visiblity changed signal" into devel/master

Merge "InheritedVisibilityChangedCallback with window Visibility" into devel/master

Change the trace filter name : DALI_TRACE_UPDATE/RENDER_PROCESS

Now VD performance tracing system prepare the environment value name.

Let we hide the combined keyword at dali core side.

Change-Id: I17dc2cded6b89a1f593335299a092d3034d9ccdf
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Fix compile error at gcc-13 (Do not use operator< with nullptr)

Change-Id: Ic40ab0752eebbc33f13d1cd7c6f0d14ac424d8c6
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

Merge "Print more logs when (animation finished case + previous state)" into devel/master

Merge "Allow to compare IntrusivePtr comparision" into devel/master

Merge "DALi Version 2.3.35" into devel/master

Print more logs when (animation finished case + previous state)

Since we don't print any logs if animation finished naturally,
user might not know whether given rendering loops occured due to animation, or not.

To make ensure the animation state changeness, let we also print log
if animation finished naturally.

Also, print the log before change the animation state. It will be useful
when we trace the animation state changing.

Change-Id: I642057b56b86b2aca27874995ee1ff9adcfae30e
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

DALi Version 2.3.35

Change-Id: I463177231febbb34e2e1ef3d8607ebe3f11fdc6a

Merge "Make bake flag don't chage the set dirty flag for VisualRenderer" into devel/master

Get the actor who trigger visiblity changed signal

Since we cannot change the parameter of Visibility changed signal,
we need to make another API to get the actor who trigger visibility changed.

Change-Id: I990338910efc72fda941a66b0c9a802b22ff7f49
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Add GetLastPanGestureState

Change-Id: Ica2861de9274023bb6f0c7b8c79c105224badea8

Allow to compare IntrusivePtr comparision

Let we allow to compare IntrusivePtr operator<.

After this patch, we can use std::set<IntrusivePtr<T>> or map, without
custom comparisoin functor.

Change-Id: Ifce5ad1fe951837d21766daba4cda1546c263262
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Make bake flag don't chage the set dirty flag for VisualRenderer

Since VisualRendererProperty try to call AnimatablePropertyBase::OnBake directly,
it might change the overlaped bake flags.

To avoid this issue, let we call AnimableProperty<T>::OnBake, instead of
supersuper class

Change-Id: Ic57c34834b46e889f438b7b19a8a592e77a7fbcb
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

InheritedVisibilityChangedCallback with window Visibility

Change-Id: I0b99cb7052380537ff8f0924af3855445d9041cd
Signed-off-by: Seungho Baek <sbsh.baek@samsung.com>

Make Actor's name don't use ConstString

Since ConstString register into internal string pool
and never be released,
Their might be occured some memory limits if user use randomized name.

Until now, we use raw string comparision at FindChildByName.
So their is no performance issue after this patch.

TODO : How could we resolve same issue for RegisterProperty?

Change-Id: Ia4258d204e986c117f19489a075e3c5afff9fab9
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "DALi Version 2.3.34" into devel/master

DALi Version 2.3.34

Change-Id: I41cd021c91e5e4ca71e5aa9bc36b9dfe3b974b54

Merge "Cache string based shader if required" into devel/master

Merge "Trigger Program cleaning by the size of cache container" into devel/master

Make bake flag dont change set dirty flag

Since we can bake componet side, it may have problem
if component property baked, and full property is set.

In this case, property call reset to base value only 1 times.
So it will show flickering.

To fix this issue, let we make mDirtyFlag |= BAKED_FLAG instead of
mDirtyFlag = BAKED_FLAG

Change-Id: I7269ac79ab1ec7796f62eaea438ee47dd104e963
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Trigger Program cleaning by the size of cache container

Current logic try to clean program cache every 10 seconds.
That mean, we need to re-create program.
It might make some unneccessary performance slow-down actions.

To avoid it, let we try to GC only if the number of program cache size
is bigger than threshold, not always.

And also, for every GC time, we delete only 5 programs incrementally per each frame.
So make we less-block the rendering time.

It will be useful for real world app who don't use custom shader.

Change-Id: Ie4e3bfb3486984673caa79750f55d813f256c0bf
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Cache string based shader if required

Let we cache string based shader's ShaderData.
It will be useful to avoid use multiple ShaderData creation
even if we use same source code.

Change-Id: Ifeaa9c2fa47b402690dab9b650b01ce874002069
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Do not calculate transform if there is nothing changed as transform" into devel/master

Add internal UTC to use ShaderFactory.SaveShader

Since there was no UTC relative with it, platform abstraction code become broken.

To make code clean, let we implement something internal UTC about shader cache.

Change-Id: I83f10b5a7231af9f6a68c33336c92bc1f7ea1a68
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Do not calculate transform if there is nothing changed as transform

Since we can know every cases when the local-world matrix changeness,
we can "skip" if there is nothing changed during current frame.

Transform calculation is kind of heavy operation.
So if there is any timing that we can skip it, it could be
increase performance for non-transform cases (a.k.a Lottie, AniamtedImage, Particle)

Change-Id: Ie7b6fdd8099810e599bf696fca267e48c7f83066
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge changes Ia910ced0,Ifa949d9b into devel/master

* changes:
  Print log if TypeInfo asserted + Do not assert if we call SetProperty for readonly + Add keyword LIKELY / UNLIKELY
  Remove useless macro

Merge "DALi Version 2.3.33" into devel/master

DALi Version 2.3.33

Change-Id: I42061d53504977f1cd2b39eb984ecc9b9625f166

Fix secondary command buffer Begin flag

Change-Id: I52db5f89da394444bcb248ff97ed0a2b0bf2b6da

Adding Begin/End commands to graphics API

Change-Id: Iff0f91133f4a8646f17ccaebf6d7d95e84c8fc17

Merge "Cleaned up render pass creation comments" into devel/master

(TestApplication) Fix SVACE issue

Change-Id: I654a48b18ef90fb7c0a4737ac9dc273f4c3e661a

Merge changes I03cb1f37,I66ce8530,Iad1ace55,Ib9eef3cf into devel/master

* changes:
  Making test-render-surface.h/cpp common across repos
  Updating framebuffer to use renderpass
  Minor test harness updates
  Updating graphics-api for Vulkan

Cleaned up render pass creation comments

Change-Id: I92a7a4c4beb236fd2caa9207c7726079374bc480

DALi Version 2.3.32

Change-Id: If9a370530f2668add53306b58726d6cfb7c5f60a

Calculate Has with std::string_view

Since convert from std::string_view to std::string
might requred memory copy.

To avoid this overhead, let we make a API that calculate
hash as string_view, instead of full string

Change-Id: Ie5455a5001e1c1f423023708a4988d53badff74c
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Print log if TypeInfo asserted + Do not assert if we call SetProperty for readonly + Add keyword LIKELY / UNLIKELY

Change-Id: Ia910ced0e022f95a262cd508b02b31917b1e3a83
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Making test-render-surface.h/cpp common across repos

Change-Id: I03cb1f37ad9f51bc96b08bb31b6ee9d7d20cd083
Signed-off-by: David Steele <david.steele@samsung.com>

Updating framebuffer to use renderpass

Vulkan framebuffer requires a renderpass to be defined before
the framebuffer can be created. Currently in gles backend, this
order is irrelevant, and is not defined in the graphics api.

Added Graphics::RenderPass vector to the
Graphics::FramebufferCreateInfo - this vector may have more than one
compatible render pass in it. The first render pass will be used to
create the framebuffer, but BeginRenderPass may use any of the defined
render passes with that framebuffer (as long as they are compatible in
vulkan, i.e. only differ by layout or load/store ops).

Change-Id: I66ce8530a34bcb1da57461db5644771efe0af48c

Minor test harness updates

Adding test-render-surface to dali-core

Change-Id: Iad1ace557fd4bab95a5e5cae5cf8281a9f4d2c64

Updating graphics-api for Vulkan

Change-Id: Ib9eef3cf2901b4fae094891d9ac82939c5be9d49

Remove useless macro

Since we use IndexedMap for a long time and it doesn't show any problem, let we remove this macro

Change-Id: Ifa949d9b4a2b40238770d63b170d9dd462ff0f95
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

DALi Version 2.3.31

Change-Id: Ifa20f9c8ef8faea482e55bcfd24cf71f1460af05

Merge "Print log if worker thread destruction occured for UI items." into devel/master

Print log if worker thread destruction occured for UI items.

Change-Id: I15db9dd5d6763b403b4b5015c95e3491c18443dc
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Make Log::LogMessage print to stderr / stdout if log function not installed

Until now, we just ignore the messages when we use Dali::Integration::Log::LogMessage(Something like, DALI_LOG_ERROR macro) when we don't install log function.

The log function exist per each threads.
But it is difficult to debug something at worker thread jobs
what DALi cannot controled. (Like GC thread for dotnet Application)

For more general and each debugging, let we print 'something' even if
log function is not installed, like stderr.

For now, Tizen platform re-open the stderr / stdout as dlog.
and Linux-friendly + Windows support to print logs into stderr, stdout file.

Change-Id: Iaf6a49d8aa9f53108b4c83bbdb18df406e3b57fa
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Fix coverity issue : Uninitialized value at HitTestResult + PanGestureDetector SG delete

Since PanGestureDetector use SG::PanGesture, which the memory ownership is on UpdateManager,
we need to take carefully to control the memory usage.

Change-Id: I7c2929c97b57e6e14072e0598bcd1788aa43af98
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Added element stride to the UniformInfo" into devel/master

DALi Version 2.3.30

Change-Id: I4ac6554c928cb7a90cea3574308c40634476147d

Added element stride to the UniformInfo

Added element stride to the UniformInfo when uniform is an array.

Change-Id: Ib006d0ce50589b2ef5cf1a5bd1e6b3ab7fe9ec45

Merge "Add TouchPropagationType enum. PARENT events are propagated to parents in the traditional way. GEOMETRY is a new way in which events are propagated based on geometry." into devel/master

Add TouchPropagationType enum.
PARENT events are propagated to parents in the traditional way.
GEOMETRY is a new way in which events are propagated based on geometry.

Change-Id: I3a717d5433e73f22e835ed3e6cd384a337160688

Merge changes I576e6929,Ib8bae68c into devel/master

* changes:
  Mark dirty flag into SG::Renderer directly if visual property changed
  Remove static_cast when we try to get decorated visual properties

DALi Version 2.3.29

Change-Id: I130c77353c8488d8d62b02bbbff076bd016acdb5

Mark dirty flag into SG::Renderer directly if visual property changed

Since age-down the visual property every frames make performance down,
let we update VisualRenderer's property send updated flag into SG::Renderer directly.

Now it will reduce some useless call of PrepareProperties of visual renderer.

Change-Id: I576e6929fca6945379b72988b4f3633e2ad70e22
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Remove static_cast when we try to get decorated visual properties

Since we don't have any plan to add extra data for VisualRenderer now,
let we specify the class of Extension of VisualRenderer, instead of void*

Change-Id: Ib8bae68c58082c93fb9a7dbe673c589f3b523766
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "If Hittable is false, actor should not receive events except INTERRUPTED" into devel/master

Merge "For Frame Event" into devel/master

If Hittable is false, actor should not receive events except INTERRUPTED

Change-Id: I6f782f0e3021ea1d3bcc8a9ce630956bd2128fda

Call OnBake(), OnSet only if the value is really changed

Let we check the current value and inputed value, so if they are same, ignore.

It will be useful when we don't want to mark dirty for animatable property doesn't changed.

For performance issue, let we just check only for bool / int / float.

TODO : Chould we check them also at Vector2,3,4?

Change-Id: I70b36eab97585a09c2c021da4f37a1d983a771af
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "If USER_INTERACTION_ENABLED is false, actor should not receive events." into devel/master

Merge "A new gesture recognition method. 1. Add HandleEvent(actor, touch) api at GestureDetector    This is an API that recognizes gestures by passing the actor for which the gesture should be recognized and the touch event to GestureDetector.    You can use it in the touch event callback. This should be used with SetGeometryHittestEnabled(true). 2. To support multi-gesture, we hit-test each multi-touch" into devel/master

Merge "Reverts the previously implemented FeedTouch. The new Logic FeedTouch is coming soon." into devel/master

For Frame Event

When processing multiple multi-touches, they are grouped and processed. Add handledMultiTouch condition for grouping.

Change-Id: I9870485f03c0ef16f7ba7231f47d5bd3e8bc1f84

Merge "Make Transform Component dirty as 2 frames, not for true/false" into devel/master

DALi Version 2.3.28

Change-Id: I6208c856efbdcce2f56dc53d8745062c3c012f94

If USER_INTERACTION_ENABLED is false, actor should not receive events.

Change-Id: I21ca9b8587dfecf7412d6dff736f0bbcbc9b8bed

Remove old pan gesture history when guesture started.

Change-Id: I2cec5ea23b801c5dc6e7ac2e0fd0da0c780c42c2
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

A new gesture recognition method.
1. Add HandleEvent(actor, touch) api at GestureDetector
   This is an API that recognizes gestures by passing the actor for which the gesture should be recognized and the touch event to GestureDetector.
   You can use it in the touch event callback. This should be used with SetGeometryHittestEnabled(true).
2. To support multi-gesture, we hit-test each multi-touch

Change-Id: I6ea2d42fe54d1ebd7ce8a1844f91434765f9b478

Fix partial update issue after VisualRenderer patch applied

Change-Id: I0df716d6a4607ada01119c67f77f127a70566ab2
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Make Transform Component dirty as 2 frames, not for true/false

Since ResetToBaseValue for TransfromProperty doesn't use basic
AnimatableProperty logic, the dirty flag is not match with common sence.

To match it as frame-by-frame, let we make component dirty as 2 bit,
and age-down every frame.

And after we allow to use dirty flag feature for transform,
we can determine whether we need to re-calculate matrix or not.

If that flag be used, we can 'skip' heavy matrix multiply operation.

Change-Id: Icf51e0be37962bee5147c08bb0cd3302bf203f89
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Reverts the previously implemented FeedTouch. The new Logic FeedTouch is coming soon.

Revert "1. Fixes a bug where events do not occur when longPressGestures overlap. This is when using gestures with FeedTouch."

This reverts commit b0516efeefaff68139bee29a7317eccd3729adbb.

Revert "actorCoordinates were missing from TapGesture when FeedTouch was used"

This reverts commit 9e32a10a253c53d66ec41fe915d6d378bf2d8aa4.

Revert "Touch and Hover event propagrated by geometry way.(5)"

This reverts commit c1ead670ee0d10718449be889b6bdacf98d93a60.

Revert "Add FeedTouch api to GestureDetector."

This reverts commit 54e52fd9d4556e19b2e97082b057ad886952e55e.

Change-Id: Iffe838047df670006ad23f7c5d46eb7f938cd16c

Merge "Fix for GCC14" into devel/master

Call ResetToBaseValue only 1 times per PropertyBase

If multiple animation try to change single property, the dirty flag
of PropertyBase would not be works well.

To ensure the dirty flag, let we collect resetter required properties
into single container, and call ResetToBaseValue only one times.

It will ensure that we call the resetter only 1 times per 1 frame.

Change-Id: I1a3792993f2ef681399f72f6f1718731408f2128
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

Fix for GCC14

Change-Id: I3a0528f8c6752e2f042a5cf6bfa11b66d875b22a
Signed-off-by: Seungho Baek <sbsh.baek@samsung.com>

Fix Animation with EndAction::DISCARD dont reset properties.

If Animation::EndAction::DISCARD finisehd normally cases,
The mDirtyFlag was not matched with real world

For example

ResetToBaseValue[0](mDirtyFlag become 1)
Animate and finished (mValue[0] changed.)
(update)
ResetToBaseValue[1](mDirtyFlag become 0)
(update)
(ResetToBaseValue did not called. So mValue[0] is last frame value)
(update)
(ResetToBaseValue did not called. So mValue[1] is BaseValue)

Now, mValue become flickering.

To avoid this problem, let we call ResetToBaseValue at least 2 frames
if finished animation's EndAction is DISCARD.
(Note that we don't consider Stop() call cases, since Stop() will not Animate
Animator, so mValue[0] is BaseValue)

And also, There was some issue that visual renderer property changeness not updated to
the dirty rect infomation.
(Since SG::Renderer don't be mark as updated)

To fix this issue, let we ensure to check the visual renderer property dirty.

And also, for apply Animation::EndAction::DISCARD case,
Let we make visual renderer coefficient use double buffered flags,
and age down every frames.

TODO : UpdateManager need to ResetBaseValue at least 2 frames
if Finished animation is EndAction::DISCARD.

This will need update manager side fix. So just keep this bug and fix as another patch.

Change-Id: Ibf654f723e1f986843cda620bc741b1121ee95d7
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Make InternalState for animation. So ensure Stop() Play() make the playstate is Playing

Since there is no way to determine that Animation::HasFinished() function called
due to we call Stop() API, or Animation finished normally.

To seperate the state, let we make new internal state
: STOPPING, PLAYING_DURING_STOPPING, PAUSED_DURING_STOPPING

STOPPING will be changed as STOPPED after Animation::HasFinished() function called.

And also, make CLEARED state so we can easly control Clear() function call.

Change-Id: I2acccfaaa034dd1edc956e4a23b0726b5d1b65e7
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

DALi Version 2.3.27

Change-Id: Icdbaecd822027f41547003ee98b716227d07306d

Ensure cleared animation not emit finished signal during finished signal emit

Since we collect all finished animations and emit after,
It is possible that finisehd signal emitted what we should not
(e.g. Animation::Clear() called during another animation's finished signal)

To avoid it, let we ensure to check whether given animation is ignored or not.

Change-Id: I681c2c8e0f1624e0846409eb862862e5392ab7ec
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Ignore AnimationFinishedCallback when we call Animation::Clear() hard + Resolve bug when we call Play multiple times" into devel/master

DALi Version 2.3.26

Change-Id: I8aa7325ed7b1a0ea7980c47e6c9f062db0c3060b

Ignore AnimationFinishedCallback when we call Animation::Clear() hard + Resolve bug when we call Play multiple times

There was several bugs at AnimationPlayList.

1. Core::ProcessCoreEvents sequence is like this :
     SceneEvent -> NotificationManager -> Relayout -> Flush.
   If we call Animation::Clear() at SceneEvent timing, Notfication will send animation finished signal
   what we must not send.

   To avoid this system, let we store the cleared animations at AnimationPlayList.

   Since AnimationFinished callback required at least 1 frame times after flush,
   we can ignore whole notify events before Flush done.

2. Scene mPlayList keep the reference of Animation whenever we call Play, there
   were several problems. For example, If we call Play() 2 times and Clear(),
   that Dali::Animation never be removed.

Change-Id: Ie9d5298311b9e5318fecad21b1124c896fcf3a8e
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Remove render once trace log

Since it print too much logs, let we remove it

Change-Id: I8987d8fe6b184938780f268f0349702456dd0433
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Add FittingMode for visual fittingMode" into devel/master

Merge "Apply RenderUntil() on sorted RenderList" into devel/master

Apply RenderUntil() on sorted RenderList

Change-Id: I64a3b567984494d1f284095d68e44b7e318a04d5

Implement ProcessorOnce

Since unregister processor during Process will give overhead,
we'd better make another container to process only once time
during ProcessCoreEvent.

Change-Id: I0b38490ccdbd7da4f7ec1acf80253a1aee8c42ad
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Optimize Add/Remove SG::RenderTask from SG::RenderTaskList

Let we make render task list use OrderedSet instead of vector container.

Vector container is slow if we call RemoveRenderTask.

TODO : Optimize event side render task list too.
But since it is quite complex, we'd better optimize that points
in another patch

Change-Id: Icd6f2e7a24964eb05c92fa9a5ce0e306814445ee
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

DALi Version 2.3.25

Change-Id: Ib32a2c7cb1d61bff3955047cb03be7142d14693f

Control core policy as one flag

Let we collect core creation status, instead of parameter increasement.

It will be useful when we add more features of Core.

Change-Id: I746bdb714eac5dcbd792a1a33a93abd2b96fe4cb
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Performance trace for some bottleneck of update/render manager

Let we check trace the top 5 area of update : update itself / animation /
frame update callback / update renderer / transform
and top 2 area of render : render instruction process / flush

TODO : Since the name of environment need to be sync with peformance
trace tool, we'd better change it after the system prepared.

Change-Id: I4a9e38b6b957508138737ee225c0cfa9ad94a8f9
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge changes Ib3f51bd8,Ibfbeca64 into devel/master

* changes:
  Pint log if animation play / stop / pause / clear / destroy
  Print top5 longest time spend AnimationFinished & FrameUpdate callbacks

Pint log if animation play / stop / pause / clear / destroy

Let we know the animation's state by id, if user want to know.

Change-Id: Ib3f51bd8035ae6958f37f2340b1dbbe10cfaf029
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Print top5 longest time spend AnimationFinished & FrameUpdate callbacks

Very rarely, we got some report from performance team that
"DALI_NOTIFICATION_NOTIFY_COMPLETED" and "DALI_FRAME_CALLBACK_UPDATE" is bottleneck.

To escape the false accusation, let we collect the App side callback
execute time and print it.

This time checker already merged at tizen_8.0 branch.

Change-Id: Ibfbeca6440f1b9c46ebb2f71200139de611a2b4a
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Even when multi-touching, the captured actor is used without a hittest every time.

Change-Id: Ib934b4b5626747a9ed7036d7489f82c94e7b4b16

Merge "Remove an unneccessary argument at Process()" into devel/master