An experimental fix for Bug 64726...

An experimental fix for Bug 64726
This change just adds a virtual destructor to TestRenderViewHost to see we can delete the objects created in its constructor.

BUG=64726
TEST=make the "Chromium Mac (valgrind)" bot green.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=67977

[Elliot Glaysher, 2010-11-30 17:31:55]

LGTM

On Mon, Nov 29, 2010 at 9:45 PM,  <hbono@chromium.org> wrote:
> Reviewers: Lei Zhang, Elliot Glaysher,
>
> Description:
> An experimental fix for Bug 64726
> This change just adds a virtual destructor to TestRenderViewHost to see we
> can
> delete the objects created in its constructor.
>
> BUG=64726
> TEST=make the "Chromium Mac (valgrind)" bot green.
>
> Please review this at http://codereview.chromium.org/5273011/
>
> SVN Base: svn://chrome-svn/chrome/trunk/src/
>
> Affected files:
>  M     chrome/browser/renderer_host/test/test_render_view_host.h
>  M     chrome/browser/renderer_host/test/test_render_view_host.cc
>
>
> Index: chrome/browser/renderer_host/test/test_render_view_host.cc
> ===================================================================
> --- chrome/browser/renderer_host/test/test_render_view_host.cc  (revision
> 67654)
> +++ chrome/browser/renderer_host/test/test_render_view_host.cc  (working
> copy)
> @@ -105,6 +105,9 @@
>       is_showing_(false) {
>  }
>
> +TestRenderWidgetHostView::~TestRenderWidgetHostView() {
> +}
> +
>  gfx::Rect TestRenderWidgetHostView::GetViewBounds() const {
>   return gfx::Rect();
>  }
> Index: chrome/browser/renderer_host/test/test_render_view_host.h
> ===================================================================
> --- chrome/browser/renderer_host/test/test_render_view_host.h   (revision
> 67654)
> +++ chrome/browser/renderer_host/test/test_render_view_host.h   (working
> copy)
> @@ -49,6 +49,7 @@
>  class TestRenderWidgetHostView : public RenderWidgetHostView {
>  public:
>   explicit TestRenderWidgetHostView(RenderWidgetHost* rwh);
> +  virtual ~TestRenderWidgetHostView();
>
>   virtual void InitAsPopup(RenderWidgetHostView* parent_host_view,
>                            const gfx::Rect& pos) {}
>
>
>