Fix BrowserAccessibilityManager valgrind errors.

chrome/browser/accessibility/browser_accessibility_manager_unittest.cc

 // Copyright (c) 2011 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 "base/string16.h"
 #include "base/utf_string_conversions.h"
 #include "content/browser/accessibility/browser_accessibility.h"
 #include "content/browser/accessibility/browser_accessibility_manager.h"
 #include "content/common/view_messages.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 536 matching lines @@
   // There should be 4 objects now.
   EXPECT_EQ(4, CountedBrowserAccessibility::global_obj_count_);
 
   // Delete the manager and make sure all memory is cleaned up.
   delete manager;
   ASSERT_EQ(0, CountedBrowserAccessibility::global_obj_count_);
 }
 
 TEST(BrowserAccessibilityManagerTest, TestCreateEmptyDocument) {
   // Try creating an empty document with busy state.
-  BrowserAccessibilityManager* manager =
-      BrowserAccessibilityManager::CreateEmptyDocument(
-          NULL,
-          WebAccessibility::STATE_BUSY,
-          NULL,
-          new CountedBrowserAccessibilityFactory());
+  scoped_ptr<BrowserAccessibilityManager> manager;
+  manager.reset(BrowserAccessibilityManager::CreateEmptyDocument(
+      NULL,
+      WebAccessibility::STATE_BUSY,
+      NULL,
+      new CountedBrowserAccessibilityFactory()));
 
   // Verify the root is as we expect by default.
   BrowserAccessibility* root = manager->GetRoot();
   EXPECT_EQ(1000, root->renderer_id());
   EXPECT_EQ(WebAccessibility::ROLE_WEB_AREA, root->role());
   EXPECT_EQ(WebAccessibility::STATE_BUSY, root->state());
 
   // Tree with a child textfield.
   WebAccessibility tree1_1;
   tree1_1.id = 1000;
@@ skipping 45 matching lines @@
 
   // Verify the root has not changed.
   EXPECT_EQ(root, manager->GetRoot());
 
   // And the proper child remains.
   EXPECT_EQ(WebAccessibility::ROLE_BUTTON, acc2_2->role());
   EXPECT_EQ(1001, acc2_2->renderer_id());
 
   // Verify we don't reuse objects that have changed roles.
   EXPECT_NE(acc1_2, acc2_2);
+
+  // Ensure we properly cleaned up.
+  manager.reset();
+  ASSERT_EQ(0, CountedBrowserAccessibility::global_obj_count_);
 }