Fix object ownership in ui/base/models.

ui/app_list/views/app_list_view_unittest.cc

 // Copyright 2014 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 "ui/app_list/views/app_list_view.h"
 
 #include <stddef.h>
 
 #include <string>
 #include <vector>
@@ skipping 396 matching lines @@
   SimulateClick(all_apps_button);
   main_view->contents_view()->Layout();
   EXPECT_TRUE(IsStateShown(AppListModel::STATE_APPS));
 
   // Hiding and showing the search box should not affect the app list's
   // preferred size. This is a regression test for http://crbug.com/386912.
   EXPECT_EQ(view_size.ToString(), view_->GetPreferredSize().ToString());
 
   // Check tiles hide and show on deletion and addition.
   EXPECT_TRUE(SetAppListState(AppListModel::STATE_START));
-  model->results()->Add(new TestStartPageSearchResult());
+  model->results()->Add(base::MakeUnique<TestStartPageSearchResult>());
   start_page_view->UpdateForTesting();
   EXPECT_EQ(1u, GetVisibleViews(start_page_view->tile_views()));
   model->results()->DeleteAll();
   start_page_view->UpdateForTesting();
   EXPECT_EQ(0u, GetVisibleViews(start_page_view->tile_views()));
 
   // Tiles should not update when the start page is not active but should be
   // correct once the start page is shown.
   EXPECT_TRUE(SetAppListState(AppListModel::STATE_APPS));
-  model->results()->Add(new TestStartPageSearchResult());
+  model->results()->Add(base::MakeUnique<TestStartPageSearchResult>());
   start_page_view->UpdateForTesting();
   EXPECT_EQ(0u, GetVisibleViews(start_page_view->tile_views()));
   EXPECT_TRUE(SetAppListState(AppListModel::STATE_START));
   EXPECT_EQ(1u, GetVisibleViews(start_page_view->tile_views()));
 
   Close();
 }
 
 void AppListViewTestContext::RunPageSwitchingAnimationTest() {
   Show();
@@ skipping 43 matching lines @@
 
   // The original ContentsView is destroyed here.
   view_->SetProfileByPath(base::FilePath());
   EXPECT_EQ(1, GetPaginationModel()->total_pages());
 
   StartPageView* start_page_view =
       view_->app_list_main_view()->contents_view()->start_page_view();
   EXPECT_NO_FATAL_FAILURE(CheckView(start_page_view));
 
   // New model updates should be processed by the start page view.
-  delegate_->GetTestModel()->results()->Add(new TestStartPageSearchResult());
+  delegate_->GetTestModel()->results()->Add(
+      base::MakeUnique<TestStartPageSearchResult>());
   start_page_view->UpdateForTesting();
   EXPECT_EQ(1u, GetVisibleViews(start_page_view->tile_views()));
 
   // Old model updates should be ignored.
-  original_test_model->results()->Add(new TestStartPageSearchResult());
-  original_test_model->results()->Add(new TestStartPageSearchResult());
+  original_test_model->results()->Add(
+      base::MakeUnique<TestStartPageSearchResult>());
+  original_test_model->results()->Add(
+      base::MakeUnique<TestStartPageSearchResult>());
   start_page_view->UpdateForTesting();
   EXPECT_EQ(1u, GetVisibleViews(start_page_view->tile_views()));
 
   Close();
 }
 
 void AppListViewTestContext::RunSearchResultsTest() {
   EXPECT_FALSE(view_->GetWidget()->IsVisible());
   EXPECT_EQ(-1, GetPaginationModel()->total_pages());
   AppListTestModel* model = delegate_->GetTestModel();
@@ skipping 240 matching lines @@
 TEST_F(AppListViewTestAura, AppListOverlayTest) {
   EXPECT_NO_FATAL_FAILURE(test_context_->RunAppListOverlayTest());
 }
 
 TEST_F(AppListViewTestDesktop, AppListOverlayTest) {
   EXPECT_NO_FATAL_FAILURE(test_context_->RunAppListOverlayTest());
 }
 
 }  // namespace test
 }  // namespace app_list