Download manager. Not part of build. a=asa

xpfe/components/download-manager/resources/content/downloadmanager.js

@@ -163,12 +163,10 @@ var downloadViewController = {
       // XXX we should probably prompt the user
       for (i = 0; i < selection.length; ++i)
         gDownloadManager.cancelDownload(selection[i].id);
-
       break;
     case "cmd_remove":
       for (i = 0; i < selection.length; ++i)
         gDownloadManager.removeDownload(selection[i].id);
-      
       break;
     case "cmd_selectAll":
       gDownloadView.selectAll();

xpfe/components/download-manager/resources/content/downloadmanager.xul

@@ -113,10 +113,7 @@
          which is good, but we need to specify it ourselves here -->
     <key id="key_selectAll" command="cmd_selectAll"/>
 
-    <!-- We need to provide our own delete key binding because the key_delete
-         handler in platformGlobalOverlay.xul maps command to "cmd_delete" which
-         is NOT what we want! -->
-    <key id="key_delete" command="cmd_remove"/>
+    <key id="key_remove" keycode="VK_DELETE" command="cmd_remove"/>
 
     <key id="key_properties" key="&cmd.properties.keybinding;"
          command="cmd_properties" modifiers="accel"/>

xpfe/components/download-manager/src/nsDownloadManager.cpp

@@ -961,7 +961,7 @@ nsDownload::OnStateChange(nsIWebProgress* aWebProgress,
     mDialogListener->OnStateChange(aWebProgress, aRequest, aStateFlags, aStatus);
 
   if (aStateFlags & STATE_STOP) {
-    if (mDownloadState == DOWNLOADING && mPercentComplete == 100)
+    if ((mDownloadState == DOWNLOADING && mPercentComplete == 100) || mDownloadState == NOTSTARTED)
       mDownloadState = FINISHED;
 
     char* persistentDescriptor;