Changeset 7385

Timestamp:

02/20/07 16:31:55 (23 months ago)

Author:

BradNeuberg

Message:

Cleaning up debug output of Moxie

Location:

trunk

Files:

• demos/storage/editor.html (modified) (4 diffs)
• demos/storage/editor.js (modified) (5 diffs)
• src/off.js (modified) (2 diffs)
• src/sync.js (modified) (22 diffs)

trunk/demos/storage/editor.html

@@ -10 +10 @@
                 var djConfig = 
                                                 { 
-                                                        isDebug: true, 
+                                                        isDebug: false, 
                                                         parseWidgets: false, 
                                                         searchIds: ["storageValue"] 
@@ -41 +41 @@
                 #storageKey, #directory { margin-top: 0.3em; }
                 #saveButton { margin-left: 0.2em; }
-                /*#dot-widget-container { height: auto; }*/
+                #dot-widget-container { height: auto; }
                 
                 .widget { border: 2px solid #CDDDE9; }
@@ -90 +90 @@
                                 filled into any element with ID "dot-widget".
                          -->
-                        
+                        <!--
                         <div id="dot-widget"></div>
-                        
+                        -->
                         
                         <!-- Customize what parts of the Offline Widget we see -->
-                        <!--
                         <div id="dot-widget-container" style="visibility: hidden;">
                                 <div id="dot-widget-title-bar">
@@ -121 +120 @@
                                 </div>
                         </div>
-                        -->
                 </div>
                 

trunk/demos/storage/editor.js

@@ -391 +391 @@
         
         _downloadData: function(){
-                dojo.debug("Moxie._downloadData");
                 var self = this;
                 // FIXME: Use Dojo's internal cache busting mechanisms
@@ -400 +399 @@
                         headers:        { "Accept" : "text/javascript" },
                         error:          function(type, errObj){
-                                dojo.debug("Moxie._downloadData.error, type="+type+", errObj="+errObj.message);
+                                //dojo.debug("Moxie._downloadData.error, type="+type+", errObj="+errObj.message);
                                 var message = "Unable to download our documents from server: "
                                                                 + errObj.message;
@@ -406 +405 @@
                         },
                         load:           function(type, data, evt){
-                                dojo.debug("Moxie._downloadData.load, type="+type+", evt="+evt);        
+                                //dojo.debug("Moxie._downloadData.load, type="+type+", evt="+evt);      
                                 self._saveDownloadedData(data);
                         }
@@ -416 +415 @@
         
         _saveDownloadedData: function(data){
-                dojo.debug("Moxie.saveDownloadedData");
                 // persist the data into Dojo Storage, with the key
                 // "documents". 'data'
@@ -425 +423 @@
                 try{
                         dojo.storage.put("documents", data, function(status, key, message){
-                                dojo.debug("_saveDownloadedData.resultHandler, status="+status+", key="+key+", message="+message);
+                                //dojo.debug("_saveDownloadedData.resultHandler, status="+status+", key="+key+", message="+message);
                                 if(status == dojo.storage.SUCCESS){
                                         // update our list of available keys

trunk/src/off.js

@@ -239 +239 @@
         
         _onLoad: function(){
-                dojo.debug("dojo.off._onLoad");
+                //dojo.debug("dojo.off._onLoad");
                 // both local storage and the page are finished loading
                 
@@ -251 +251 @@
                 var self = this;
                 this.load(function(){
-                        dojo.debug("Load finished callback inside of dojo.off._onLoad");
                         // kick off a thread to check network status on
                         // a regular basis

trunk/src/sync.js

@@ -63 +63 @@
         
         replay: function(){ /* void */
-                dojo.debug("replay");
                 // summary:
                 //      Replays all of the commands that have been
                 //      cached in this command log when we go back online;
                 //      onCommand will be called for each command we have
-                dojo.debug("isReplaying="+this.isReplaying);
                 
                 if(this.isReplaying == true){
@@ -77 +75 @@
                 
                 if(this.commands.length == 0){
-                        dojo.debug("no commands to run");
                         this.onReplayFinished();
                         return;
@@ -85 +82 @@
                 
                 var nextCommand = this.commands[0];
-                dojo.debug("nextCommand="+nextCommand);
                 this.onCommand(nextCommand);
         },
@@ -115 +111 @@
                 //      the network, and you should call one of these two methods based on 
                 //      the result of your network call.
-                dojo.debug("default onCommand");
         },
         
         add: function(command /* Object */){ /* void */
-                dojo.debug("CommandLog.add");
                 // summary:
                 //      Adds an action to our command log
@@ -146 +140 @@
                                                                 + "we are replaying a command log");
                 }
-                dojo.debug("this.commands="+this.commands);
-                dojo.debug("this.commands.push="+this.commands.push);
-                for(var i in this.commands){
-                        dojo.debug(i + "=" + this.commands[i]);
-                }
+                
                 this.commands.push(command);
                 
@@ -182 +172 @@
                 //      are not rolled back), while the command entry that was halted
                 //      stays in our list of commands to later be replayed.     
-                dojo.debug("commandlog.haltReplay");
                 if(this.isReplaying == false){
                         return;
@@ -207 +196 @@
         
         continueReplay: function(){ /* void */
-                dojo.debug("commandlog.continueReplay");
                 // summary:
                 //      Indicates that we should continue processing out list
@@ -224 +212 @@
                 this.commands.shift();
                 
-                dojo.debug("this.commands.length after shifting="+this.commands.length);
-                
                 // are we done?
                 if(this.commands.length == 0){
-                        dojo.debug("no more length");
                         // save the state of our command log, then
                         // tell anyone who is interested that we are
@@ -235 +220 @@
                                 var self = this;
                                 this.save(function(){
-                                        dojo.debug("finished saving command log");
                                         self.isReplaying = false;
                                         self.onReplayFinished();
@@ -249 +233 @@
                 // get the next command
                 var nextCommand = this.commands[0];
-                dojo.debug("nextCommand="+nextCommand);
                 this.onCommand(nextCommand);
         },
@@ -271 +254 @@
         
         save: function(finishedCallback){ /* void */
-                dojo.debug("commandlog.save");
                 // summary:
                 //      Saves this command log to persistent, client-side storage
@@ -306 +288 @@
         
         load: function(finishedCallback){ /* void */
-                dojo.debug("CommandLog load!");
                 // summary:
                 //      Loads our command log from reliable, persistent local storage;
@@ -443 +424 @@
         
         synchronize: function(){ /* void */
-                dojo.debug("dojo.sync.synchronize");
                 // summary:
                 //      Begin a synchronization session.
@@ -476 +456 @@
         
         start: function(){ /* void */
-                dojo.debug("dojo.sync.start");
                 if(this.cancelled == true){
                         this.finished();
@@ -490 +469 @@
         
         refreshUI: function(){ /* void */
-                dojo.debug("dojo.sync.refreshUI");
                 if(this.cancelled == true){
                         this.finished();
@@ -515 +493 @@
         
         upload: function(){ /* void */
-                dojo.debug("dojo.sync.upload");
                 if(this.cancelled == true){
                         this.finished();
@@ -531 +508 @@
                 if(this.log.onReplayFinished == null){
                         this.log.onReplayFinished = function(){
-                                dojo.debug("onReplayFinished");
                                 self.download();
                         }
@@ -541 +517 @@
         
         download: function(){ /* void */
-                dojo.debug("dojo.sync.download");
                 if(this.cancelled == true){
                         this.finished();
@@ -558 +533 @@
         
         doDownload: function(){ /* void */
-                dojo.debug("dojo.sync.doDownload");
                 // summary:
                 //      Actually downloads the data we need to
@@ -571 +545 @@
         finishedDownloading: function(successful /* boolean */, 
                                                                         errorMessage /* String */){
-                dojo.debug("dojo.sync.finishedDownloading, successful="+successful+", errorMessage="+errorMessage);
                 // summary:
                 //      Applications call this method from their
@@ -591 +564 @@
         
         finished: function(){ /* void */
-                dojo.debug("dojo.sync.finished");
                 this.isSyncing = false;
                 
@@ -650 +622 @@
                 //      as it is handle automatically by the Dojo Offline
                 //      framework.
-                dojo.debug("dojo.sync.load, log="+this.log);
                 this.log.load(function(){
-                        dojo.debug("!!!Loaded log, log="+dojo.sync.log);
                         finishedCallback();
                 });