Revert "Fix a ton of compiler warnings."

chrome/browser/download/download_request_manager.h

-// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// Copyright (c) 2006-2008 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_DOWNLOAD_DOWNLOAD_REQUEST_MANAGER_H_
 #define CHROME_BROWSER_DOWNLOAD_DOWNLOAD_REQUEST_MANAGER_H_
 
 #include <map>
 #include <string>
 #include <vector>
 
@@ skipping 38 matching lines @@
     PROMPT_BEFORE_DOWNLOAD,
     ALLOW_ALL_DOWNLOADS,
     DOWNLOADS_NOT_ALLOWED
   };
 
   // The callback from CanDownloadOnIOThread. This is invoked on the io thread.
   class Callback {
    public:
     virtual void ContinueDownload() = 0;
     virtual void CancelDownload() = 0;
-   protected:
-    ~Callback() {}
   };
 
   // TabDownloadState maintains the download state for a particular tab.
   // TabDownloadState installs observers to update the download status
   // appropriately. Additionally TabDownloadState prompts the user as necessary.
   // TabDownloadState deletes itself (by invoking
   // DownloadRequestManager::Remove) as necessary.
   class TabDownloadState : public NotificationObserver {
    public:
     // Creates a new TabDownloadState. |controller| is the controller the
@@ skipping 92 matching lines @@
   void OnUserGesture(TabContents* tab);
 
  private:
   friend class DownloadRequestManagerTest;
   friend class TabDownloadState;
 
   // For unit tests. If non-null this is used instead of creating a dialog.
   class TestingDelegate {
    public:
     virtual bool ShouldAllowDownload() = 0;
-   protected:
-    ~TestingDelegate() {}
   };
   static void SetTestingDelegate(TestingDelegate* delegate);
 
   // Gets the download state for the specified controller. If the
   // TabDownloadState does not exist and |create| is true, one is created.
   // See TabDownloadState's constructor description for details on the two
   // controllers.
   //
   // The returned TabDownloadState is owned by the DownloadRequestManager and
   // deleted when no longer needed (the Remove method is invoked).
@@ skipping 36 matching lines @@
   // the TabDownloadState is removed and deleted (by way of Remove).
   typedef std::map<NavigationController*, TabDownloadState*> StateMap;
   StateMap state_map_;
 
   static TestingDelegate* delegate_;
 
   DISALLOW_COPY_AND_ASSIGN(DownloadRequestManager);
 };
 
 #endif  // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_REQUEST_MANAGER_H_