[webkit-reviews] review requested: [Bug 46284] [GTK] Replace instances of g_timeout_add() with g_idle_add() in testatk.c : [Attachment 68430] Patch proposal

[bugzilla-daemon at webkit.org]

Mario Sanchez Prada <msanchez at igalia.com> has asked  for review:
Bug 46284: [GTK] Replace instances of g_timeout_add() with g_idle_add() in
testatk.c
https://bugs.webkit.org/show_bug.cgi?id=46284

Attachment 68430: Patch proposal
https://bugs.webkit.org/attachment.cgi?id=68430&action=review

------- Additional Comments from Mario Sanchez Prada <msanchez at igalia.com>
(In reply to comment #2)
> (From update of attachment 68419 [details])
> View in context:
https://bugs.webkit.org/attachment.cgi?id=68419&action=review
> 
> > WebKit/gtk/ChangeLog:8
> > +	     Use g_idle_add() whenever g_timeout_add() was being used
> 
> You should explain here why this conversion is a good thing. In the future,
other programmers will see this commit and wonder what the explanation was. We
should be able to concisely describe to people why we prefer this style.