Revert "Fix a ton of compiler warnings."

chrome/browser/blocked_popup_container.h

 // Copyright (c) 2009 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.
 
 // Defines the public interface for the blocked popup notifications. This
 // interface should only be used by TabContents. Users and subclasses of
 // TabContents should use the appropriate methods on TabContents to access
 // information about blocked popups.
 
 // TODO(idanan): Rename class to BlockedContentContainer.
@@ skipping 34 matching lines @@
 
   // Sets the text in the blocked popup label.
   virtual void UpdateLabel() = 0;
 
   // Hides the blocked popup view / Animates the blocked popup view out.
   virtual void HideView() = 0;
 
   // Called by the BlockedPopupContainer that owns us. Destroys the view or
   // starts a delayed Task to destroy the View at some later time.
   virtual void Destroy() = 0;
- protected:
-  ~BlockedPopupContainerView() {}
 };
 
 // Takes ownership of TabContents that are unrequested popup windows and
 // presents an interface to the user for launching them. (Or never showing them
 // again). This class contains all the cross-platform bits that can be used in
 // all ports.
 //
 // +- BlockedPopupContainer ---+         +- BlockedPopupContainerView -----+
 // | All model logic           |    +--->| Abstract cross platform         |
 // |                           |    |    | interface                       |
@@ skipping 74 matching lines @@
   // Overridden from TabContentsDelegate:
 
   // Forwards OpenURLFromTab to our |owner_|.
   virtual void OpenURLFromTab(TabContents* source,
                               const GURL& url, const GURL& referrer,
                               WindowOpenDisposition disposition,
                               PageTransition::Type transition);
 
   // Ignored; BlockedPopupContainer doesn't display a throbber.
   virtual void NavigationStateChanged(const TabContents* source,
-                                      unsigned changed_flags) {}
+                                      unsigned changed_flags) { }
 
   // Forwards AddNewContents to our |owner_|.
   virtual void AddNewContents(TabContents* source,
                               TabContents* new_contents,
                               WindowOpenDisposition disposition,
                               const gfx::Rect& initial_position,
                               bool user_gesture);
 
   // Ignore activation requests from the TabContents we're blocking.
-  virtual void ActivateContents(TabContents* contents) {}
+  virtual void ActivateContents(TabContents* contents) { }
 
   // Ignored; BlockedPopupContainer doesn't display a throbber.
-  virtual void LoadingStateChanged(TabContents* source) {}
+  virtual void LoadingStateChanged(TabContents* source) { }
 
   // Removes |source| from our internal list of blocked popups.
   virtual void CloseContents(TabContents* source);
 
   // Changes the opening rectangle associated with |source|.
   virtual void MoveContents(TabContents* source, const gfx::Rect& new_bounds);
 
   // Always returns true.
   virtual bool IsPopup(TabContents* source);
 
   // Returns our |owner_|.
   virtual TabContents* GetConstrainingContents(TabContents* source);
 
   // Ignored; BlockedPopupContainer doesn't display a toolbar.
-  virtual void ToolbarSizeChanged(TabContents* source, bool is_animating) {}
+  virtual void ToolbarSizeChanged(TabContents* source, bool is_animating) { }
 
   // Ignored; BlockedPopupContainer doesn't display a bookmarking star.
-  virtual void URLStarredChanged(TabContents* source, bool starred) {}
+  virtual void URLStarredChanged(TabContents* source, bool starred) { }
 
   // Ignored; BlockedPopupContainer doesn't display a URL bar.
-  virtual void UpdateTargetURL(TabContents* source, const GURL& url) {}
+  virtual void UpdateTargetURL(TabContents* source, const GURL& url) { }
 
   // A number larger than the internal popup count on the Renderer; meant for
   // preventing a compromised renderer from exhausting GDI memory by spawning
   // infinite windows.
   static const size_t kImpossibleNumberOfPopups = 30;
 
  protected:
   struct BlockedPopup {
     BlockedPopup(TabContents* tab_contents,
                  const gfx::Rect& bounds,
@@ skipping 82 matching lines @@
   // Our platform specific view.
   BlockedPopupContainerView* view_;
 
   // The profile for the browser associated with the container.
   Profile* profile_;
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(BlockedPopupContainer);
 };
 
 #endif  // CHROME_BROWSER_BLOCKED_POPUP_CONTAINER_H_