no virtual destructor for point/size/rect

ui/gfx/point.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 UI_GFX_POINT_H_
 #define UI_GFX_POINT_H_
 #pragma once
 
 #include "ui/base/ui_export.h"
 #include "ui/gfx/point_base.h"
@@ skipping 16 matching lines @@
   // |point| is a DWORD value that contains a coordinate.  The x-coordinate is
   // the low-order short and the y-coordinate is the high-order short.  This
   // value is commonly acquired from GetMessagePos/GetCursorPos.
   explicit Point(DWORD point);
   explicit Point(const POINT& point);
   Point& operator=(const POINT& point);
 #elif defined(OS_MACOSX)
   explicit Point(const CGPoint& point);
 #endif
 
-  virtual ~Point() {}
+  ~Point() {}

[sky, 2012/04/24 17:41:14]

Move the destructors to be protected so we know no one is trying to delete
through the base classes.

[jbates, 2012/04/24 17:44:47]

The PointBase destructor is protected. Might be helpful to have some comments to
explain that it's not allowed to derive from Point though.

[oshima, 2012/04/24 17:54:10]

Yes, this dtor needs to be still public. All base classes' dtor are protected.
I added comment to make it clear.

On 2012/04/24 17:44:47, jbates wrote:

 
 #if defined(OS_WIN)
   POINT ToPOINT() const;
 #elif defined(OS_MACOSX)
   CGPoint ToCGPoint() const;
 #endif
 
   // Returns a string representation of point.
   std::string ToString() const;
 };
 
 }  // namespace gfx
 
 #endif  // UI_GFX_POINT_H_