[webkit-reviews] review granted: [Bug 24221] [GTK] Do not emit load-finished when being disposed : [Attachment 28067] disposing.patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Feb 27 00:57:56 PST 2009
Alexey Proskuryakov <ap at webkit.org> has granted Xan Lopez
<xan.lopez at gmail.com>'s request for review:
Bug 24221: [GTK] Do not emit load-finished when being disposed
https://bugs.webkit.org/show_bug.cgi?id=24221
Attachment 28067: disposing.patch
https://bugs.webkit.org/attachment.cgi?id=28067&action=review
------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
r=me
> + /*
> + * We can get a stopLoad() from dispose when the object is being
> + * destroyed, don't emit the signal in that case.
> + */
We prefer C++-style comments in C++ files.
Is this check needed before other g_signal_emit_by_name() calls?
More information about the webkit-reviews
mailing list