[Pkg-mozext-commits] [itsalltext] 306/459: The new monitor code seem to be working in all the test cases I've thrown at it....time to find guinea pigs....

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository itsalltext.

commit 75209a1c9761e095bedc9f711e42803d6416e8b6
Author: docwhat at gerf.org <docwhat at gerf.org>
Date:   Mon Jan 28 23:48:03 2008 -0500

    The new monitor code seem to be working in all the test cases I've
    thrown at it....time to find guinea pigs....
---
 src/chrome/content/cacheobj.js   | 1 +
 src/chrome/content/itsalltext.js | 3 +++
 src/chrome/content/monitor.js    | 4 +++-
 3 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/chrome/content/cacheobj.js b/src/chrome/content/cacheobj.js
index 5ffe6b2..f2fb1c2 100644
--- a/src/chrome/content/cacheobj.js
+++ b/src/chrome/content/cacheobj.js
@@ -483,6 +483,7 @@ CacheObj.prototype.fade = function(steps, delay) {
  */
 CacheObj.prototype.update = function() {
     var value;
+    ItsAllText.debug('narf update', this.file.path);
     if (this.hasChanged()) {
         value = this.read();
         if (value !== null) {
diff --git a/src/chrome/content/itsalltext.js b/src/chrome/content/itsalltext.js
index 5a48e47..76ea1f6 100644
--- a/src/chrome/content/itsalltext.js
+++ b/src/chrome/content/itsalltext.js
@@ -821,7 +821,9 @@ Line 0
 
 
     // Do the startup when things are loaded.
+    // TODONOW: move to separate function
     that.listen(window, 'load', function (event) {
+        that.debug('!!load', event);
         if (typeof(gBrowser) === 'undefined') {
             that.new_monitor.registerPage(event);
         } else {
@@ -841,6 +843,7 @@ Line 0
         }
     }, false);
 
+    // TODONOW: move to separate function
     that.listen(window, 'unload', function (event) {
         if (typeof(gBrowser) === 'undefined') {
             that.new_monitor.stopPage(event);
diff --git a/src/chrome/content/monitor.js b/src/chrome/content/monitor.js
index 5fb41ad..ffd0e30 100644
--- a/src/chrome/content/monitor.js
+++ b/src/chrome/content/monitor.js
@@ -67,12 +67,14 @@ new_monitor.prototype.hitched_registerPage = function (event) {
         this.iat.listen(appContent, 'DOMContentLoaded', this.startPage, true);
         this.iat.listen(document, 'unload', this.stopPage, true);
         this.iat.listen(gBrowser.tabContainer, 'TabSelect', this.watcher, true);
+        this.startPage({originalTarget: doc});
         this.iat.debug('RegisterPage: END');
     }
 };
 
 new_monitor.prototype.hitched_watcher = function (offset, init) {
-    if (offset.type === 'TabSelect') {
+    if (typeof(offset) === 'number' &&
+        offset.type === 'TabSelect') {
         init = true;
     }
     var rate = this.iat.getRefresh();

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-mozext/itsalltext.git