Bug 842185 - "Remove From History" doesn't work r=mak
authorChristian Sonne <csonne@mozilla.com>
Mon, 18 Feb 2013 01:31:04 -0800
changeset 122222 673dd484f5b5eaecbcb45e8da3d0a07ac82017b1
parent 122218 524e7bc6743104e4bbc6f1a0eb520e23b1fe4abd
child 122223 b4a233c10cc9c24bd9370c517b393fe757d90de3
push id24327
push usergszorc@mozilla.com
push dateTue, 19 Feb 2013 05:22:32 +0000
treeherdermozilla-central@e8f8a3f6f1f6 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmak
bugs842185
milestone21.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 842185 - "Remove From History" doesn't work r=mak
browser/components/downloads/src/DownloadsCommon.jsm
--- a/browser/components/downloads/src/DownloadsCommon.jsm
+++ b/browser/components/downloads/src/DownloadsCommon.jsm
@@ -908,17 +908,17 @@ DownloadsDataCtor.prototype = {
   //// nsIObserver
 
   observe: function DD_observe(aSubject, aTopic, aData)
   {
     switch (aTopic) {
       case "download-manager-remove-download-guid":
         // If a single download was removed, remove the corresponding data item.
         if (aSubject) {
-            let downloadGuid = aSubject.data.QueryInterface(Ci.nsISupportsCString);
+            let downloadGuid = aSubject.QueryInterface(Ci.nsISupportsCString).data;
             DownloadsCommon.log("A single download with id",
                                 downloadGuid, "was removed.");
           this._removeDataItem(downloadGuid);
           break;
         }
 
         // Multiple downloads have been removed.  Iterate over known downloads
         // and remove those that don't exist anymore.