[webkit-reviews] review requested: [Bug 62380] [EFL] correct sharedTimer value in SharedTimerEfl.cpp : [Attachment 96651] remove _sharedTimer = 0 in timerEvent()

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jun 9 15:15:01 PDT 2011


Henry Song <hsong at sisa.samsung.com> has asked  for review:
Bug 62380: [EFL] correct sharedTimer value in SharedTimerEfl.cpp
https://bugs.webkit.org/show_bug.cgi?id=62380

Attachment 96651: remove _sharedTimer = 0 in timerEvent()
https://bugs.webkit.org/attachment.cgi?id=96651&action=review

------- Additional Comments from Henry Song <hsong at sisa.samsung.com>
Both stopSharedTimer() and addNewTimer() can be called within timerFunction()
in timerEvent().  If addNewTimer() is called, the static _sharedTimer object is
replaced with new timer object.  If set it to 0 after timerFunction() returns,
the new timer object is lost and cannot be deleted in later stopSharedTimer(). 
This is similar to how it is implemented in SharedTimerGtk.cpp for GTK port.


More information about the webkit-reviews mailing list