[sugar] web-activity fix
Tomeu Vizoso
tomeu at tomeuvizoso.net
Mon May 12 04:46:33 EDT 2008
On Sun, May 11, 2008 at 7:56 PM, Dan Krejsa <dan.krejsa at alamedanet.net> wrote:
> Hi,
>
> The attached change in web-activity/downloadmanager.py fix a
> problem causing Browse downloads to fail with an exception in
> onProgressChange64() due to _create_journal_object() having failed
> to be called earlier from onStateChange() when state_flags contained
> the bit STATE_START, but also other bits as corresponding to
> State Type Flags from nsIWebProgressListener.idl.
>
> I'm pretty confident about the changes in onStateChange(), but
> the change (eliminating a trailing '"' character) in the
> components.registrar.registerFactory() calls are fairly gratuitous,
> and should be checked by someone who is familiar with that API.
r+ to me.
Thanks,
Tomeu
More information about the Sugar
mailing list