MD Downloads: "Clear All" should remove dangerous downloads for good

chrome/browser/ui/webui/md_downloads/md_downloads_dom_handler.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_WEBUI_MD_DOWNLOADS_MD_DOWNLOADS_DOM_HANDLER_H_
 #define CHROME_BROWSER_UI_WEBUI_MD_DOWNLOADS_MD_DOWNLOADS_DOM_HANDLER_H_
 
 #include <stdint.h>
 
 #include <set>
@@ skipping 78 matching lines @@
 
  protected:
   // These methods are for mocking so that most of this class does not actually
   // depend on WebUI. The other methods that depend on WebUI are
   // RegisterMessages() and HandleDrag().
   virtual content::WebContents* GetWebUIWebContents();
 
   // Actually remove downloads with an ID in |removals_|. This cannot be undone.
   void FinalizeRemovals();
 
+  using DownloadVector = std::vector<content::DownloadItem*>;
+
+  // Remove all downloads in |to_remove|. Safe downloads can be revived,
+  // dangerous ones are immediately removed. Protected for testing.
+  void RemoveDownloads(const DownloadVector& to_remove);
+
  private:
   using IdSet = std::set<uint32_t>;
-  using DownloadVector = std::vector<content::DownloadItem*>;
 
   // Convenience method to call |main_notifier_->GetManager()| while
   // null-checking |main_notifier_|.
   content::DownloadManager* GetMainNotifierManager() const;
 
   // Convenience method to call |original_notifier_->GetManager()| while
   // null-checking |original_notifier_|.
   content::DownloadManager* GetOriginalNotifierManager() const;
 
   // Displays a native prompt asking the user for confirmation after accepting
@@ skipping 11 matching lines @@
   // Returns true if the records of any downloaded items are allowed (and able)
   // to be deleted.
   bool IsDeletingHistoryAllowed();
 
   // Returns the download that is referred to in a given value.
   content::DownloadItem* GetDownloadByValue(const base::ListValue* args);
 
   // Returns the download with |id| or NULL if it doesn't exist.
   content::DownloadItem* GetDownloadById(uint32_t id);
 
-  // Remove all downloads in |to_remove| with the ability to undo removal later.
-  void RemoveDownloads(const DownloadVector& to_remove);
+  // Removes the download specified by an ID from JavaScript in |args|.
+  void RemoveDownloadInArgs(const base::ListValue* args);
 
   // Checks whether a download's file was removed from its original location.
   void CheckForRemovedFiles();
 
   DownloadsListTracker list_tracker_;
 
   // IDs of downloads to remove when this handler gets deleted.
   std::vector<IdSet> removals_;
 
   base::WeakPtrFactory<MdDownloadsDOMHandler> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(MdDownloadsDOMHandler);
 };
 
 #endif  // CHROME_BROWSER_UI_WEBUI_MD_DOWNLOADS_MD_DOWNLOADS_DOM_HANDLER_H_