Fix some memory leaks.

ui/views/view_unittest.cc

 // 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.
 
 #include <map>
 
 #include "base/memory/scoped_ptr.h"
 #include "base/rand_util.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
@@ skipping 2818 matching lines @@
 
   content_view->RemoveChildView(v1);
   EXPECT_FALSE(LayerIsAncestor(widget()->GetCompositor()->root_layer(),
                                v2->layer()));
 
   // Reparent |v2|.
   content_view->AddChildView(v2);
   EXPECT_TRUE(LayerIsAncestor(widget()->GetCompositor()->root_layer(),
                               v2->layer()));
   EXPECT_TRUE(v2->layer()->IsDrawn());
+
+  delete v1;

[sky, 2012/02/06 17:55:47]

Can we delete this right after we remove it from the view hierarchy (and set it
to NULL)?

[sadrul, 2012/02/06 19:48:30]

Done.

 }
 
 class PaintTrackingView : public View {
  public:
   PaintTrackingView() : painted_(false) {
   }
 
   bool painted() const { return painted_; }
   void set_painted(bool value) { painted_ = value; }
 
@@ skipping 117 matching lines @@
 
   // Move c1 to the front. The layers should have moved too.
   content->ReorderChildView(c1, -1);
   EXPECT_EQ(c1->layer(), parent_layer->children()[1]);
   EXPECT_EQ(c2->layer(), parent_layer->children()[0]);
 }
 
 #endif  // USE_AURA
 
 }  // namespace views