Remove DownloadItem::Is*() in favor of DI::GetState()

content/public/browser/download_item.h

Index: content/public/browser/download_item.h
diff --git a/content/public/browser/download_item.h b/content/public/browser/download_item.h
index 11b2b16c108172fb6f53ccca9b9e1eef63a89788..4c6d0b735add46d72689fc04aba8a730ac61e8b7 100644
--- a/content/public/browser/download_item.h
+++ b/content/public/browser/download_item.h
@@ -175,20 +175,6 @@ class CONTENT_EXPORT DownloadItem : public base::SupportsUserData {
   // can't be resumed.
   virtual bool IsDone() const = 0;
 
-  //    Convenience routines for accessing GetState() results conceptually -----
-
-  // Returns true if the download is still receiving data.
-  virtual bool IsInProgress() const = 0;
-
-  // Returns true if the download has been cancelled or was interrupted.
-  virtual bool IsCancelled() const = 0;
-
-  // Returns true if the download was interrupted.
-  virtual bool IsInterrupted() const = 0;
-
-  // Returns true if we have all the data and know the final file name.
-  virtual bool IsComplete() const = 0;
-
   //    Origin State accessors -------------------------------------------------
 
   virtual const GURL& GetURL() const = 0;