[Conkeror] [PATCH 1/2] Set buffer.loading false before content_buffer_finished_loading_hook

David Kettler kettler at internode.on.net
Fri Feb 6 20:49:08 PST 2009


Hook functions may want to check that flag, so we now set it prior to
calling the hook functions.

The (equivalent) flag was previously set first, but that was changed,
perhaps inadvertently, in 61d83c5 (Implemented tab bar module,
2008-01-13).
---
 modules/content-buffer.js |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/modules/content-buffer.js b/modules/content-buffer.js
index 880f306..43d61f3 100644
--- a/modules/content-buffer.js
+++ b/modules/content-buffer.js
@@ -202,8 +202,8 @@ content_buffer.prototype = {
                  aStateFlags & Ci.nsIWebProgressListener.STATE_IS_NETWORK) {
             if (this.loading == true)  {
                 //dumpln("*** finished loading");
-                content_buffer_finished_loading_hook.run(this);
                 this.loading = false;
+                content_buffer_finished_loading_hook.run(this);
             }
         }
 
-- 
1.5.6.5



More information about the Conkeror mailing list