Bug 637630 Compose throbber doesn't r=Mnyromyr
authorNeil Rashbrook <neil@parkwaycc.co.uk>
Wed, 09 Mar 2011 23:55:24 +0000
changeset 7295 27f43f536cb220cac51eebda75687f53cbf1b348
parent 7294 675814419cd87399d0a12a1568cdb9f49a7e1249
child 7296 a5df7cbbe13f1ea59979e825fe01cf9122ed161e
push idunknown
push userunknown
push dateunknown
reviewersMnyromyr
bugs637630
Bug 637630 Compose throbber doesn't r=Mnyromyr
suite/mailnews/compose/MsgComposeCommands.js
suite/mailnews/compose/messengercompose.xul
--- a/suite/mailnews/compose/MsgComposeCommands.js
+++ b/suite/mailnews/compose/MsgComposeCommands.js
@@ -322,23 +322,25 @@ var stateListener = {
 };
 
 // all progress notifications are done through the nsIWebProgressListener implementation...
 var progressListener = {
     onStateChange: function(aWebProgress, aRequest, aStateFlags, aStatus)
     {
       if (aStateFlags & Components.interfaces.nsIWebProgressListener.STATE_START)
       {
+        document.getElementById('navigator-throbber').setAttribute("busy", "true");
         document.getElementById('compose-progressmeter').setAttribute( "mode", "undetermined" );
         document.getElementById("statusbar-progresspanel").collapsed = false;
       }
 
       if (aStateFlags & Components.interfaces.nsIWebProgressListener.STATE_STOP)
       {
         gSendOrSaveOperationInProgress = false;
+        document.getElementById('navigator-throbber').removeAttribute("busy");
         document.getElementById('compose-progressmeter').setAttribute( "mode", "normal" );
         document.getElementById('compose-progressmeter').setAttribute( "value", 0 );
         document.getElementById("statusbar-progresspanel").collapsed = true;
         document.getElementById('statusText').setAttribute('label', '');
       }
     },
 
     onProgressChange: function(aWebProgress, aRequest, aCurSelfProgress, aMaxSelfProgress, aCurTotalProgress, aMaxTotalProgress)
--- a/suite/mailnews/compose/messengercompose.xul
+++ b/suite/mailnews/compose/messengercompose.xul
@@ -736,19 +736,17 @@
         </menupopup>
       </toolbarbutton>
 
       <toolbaritem id="throbber-box"
                    align="center">
           <button id="navigator-throbber"
                   oncommand="goClickThrobber('compose.throbber.url', event)"
                   onclick="checkForMiddleClick(this, event);"
-                  tooltiptext="&throbber.tooltip;">
-            <observes element="broadcaster_throbber" attribute="busy"/>
-          </button>
+                  tooltiptext="&throbber.tooltip;"/>
       </toolbaritem>
     </toolbarpalette>
 
   </toolbox>
 
   <splitter id="compose-toolbar-sizer"
             resizeafter="grow"
             onmousedown="awSizerListen();"