717081 - Get rid of extra onLoad during cached calendar sync

Status: RESOLVED FIXED

(Calendar :: Internal Components, defect)

Description

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: Fix - v1

This might reduce the flickering a bit during cached calendar operations.

Before, the offline items were played back and then an onLoad event was fired on completion. During async operations inbetween, the playbackAddedItems function returns, causing emptyQueue to end the internal sync state, which again fires an onLoad due to how the synchronize() function is called.

This change postpones the emptyQueue call (which will also fire an onLoad) to after the offline items were reconciled and drops the extra onLoad.

Mohit, was there any specific reason you called emptyQueue before reconciliation ends? Do you think this patch will work out without side effects?

Comment 1

[Mohit Kanwal [:redDragon]]

Comment on attachment 587513 [details] [diff] [review]
Fix - v1

Review of attachment 587513 [details] [diff] [review]:
-----------------------------------------------------------------

Yup looks good! 

I can't remember why I did that, but I think I had adapted the code from the old completelistener which might have had the emptyQueue.
I tested it out and i can add/modify/delete as well as handle recurrences in the offline/online mode.

And yes the flicker is gone :)

Comment 2

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Pushed to comm-central changeset 5b6cf9b3be52

Comment 3

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Backported to releases/comm-aurora changeset 98f65cb2e5cb

Comment 4

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Backported to releases/comm-beta changeset 977925b81e00