Correcting comments a little
authormozilla.mano@sent.com
Mon, 17 Mar 2008 04:38:17 -0700
changeset 13170 150736fc1f19a6f3ea4ad42b11822d895bd2220b
parent 13169 373de5f8625ec1623be04107034564089b7695a7
child 13171 662ab32fb982b5602e16679bf9107c705c654746
push idunknown
push userunknown
push dateunknown
milestone1.9b5pre
Correcting comments a little
browser/base/content/browser-places.js
browser/components/places/content/menu.xml
--- a/browser/base/content/browser-places.js
+++ b/browser/base/content/browser-places.js
@@ -595,17 +595,17 @@ var BookmarksEventHandler = {
 #else
     var modifKey = aEvent.ctrlKey || aEvent.shiftKey;
 #endif
     if (aEvent.button == 2 || (aEvent.button == 0 && !modifKey))
       return;
 
     var target = aEvent.originalTarget;
     // If this event bubbled up from a menu or menuitem, close the menus.
-    // Do this before opening tabs, to avoid hiding the open tabs confirm.
+    // Do this before opening tabs, to avoid hiding the open tabs confirm-dialog.
     if (target.localName == "menu" || target.localName == "menuitem") {
       for (node = target.parentNode; node; node = node.parentNode) {
         if (node.localName == "menupopup")
           node.hidePopup();
         else if (node.localName != "menu")
           break;
       }
     }
--- a/browser/components/places/content/menu.xml
+++ b/browser/components/places/content/menu.xml
@@ -945,17 +945,17 @@
           ]]>
         </body>
       </method>
     </implementation>
     <handlers>
       <handler event="popupshowing" phase="capturing"><![CDATA[
         this._ensureInitialized();
         var popup = event.target;
-        // We should avoid to handle events of inner views
+        // Avoid handling popupshowing of inner views
         if (!popup._resultNode || PlacesUIUtils.getViewForNode(popup) != this)
           return;
 
         this.onPopupShowing(event);
       ]]></handler>
 
       <handler event="popuphidden"><![CDATA[
         var popup = event.target;