From 03f3900d7553f9d764006709585684c7170e3455 Mon Sep 17 00:00:00 2001 From: Marshall Greenblatt Date: Mon, 14 Nov 2016 13:03:12 -0500 Subject: [PATCH] Remove usage of base::WrapUnique in unit tests (issue #1632) --- tests/unittests/frame_unittest.cc | 25 ++++++++++++------------- 1 file changed, 12 insertions(+), 13 deletions(-) diff --git a/tests/unittests/frame_unittest.cc b/tests/unittests/frame_unittest.cc index 47115e5bb..06f1a0e19 100644 --- a/tests/unittests/frame_unittest.cc +++ b/tests/unittests/frame_unittest.cc @@ -9,7 +9,6 @@ #include "tests/cefclient/renderer/client_app_renderer.h" #include "tests/unittests/test_handler.h" -#include "base/memory/ptr_util.h" #include "testing/gtest/include/gtest/gtest.h" using client::ClientAppBrowser; @@ -866,7 +865,7 @@ class FrameNavExpectationsFactoryBrowserTestSingleNavHarness : std::unique_ptr Create(int nav) override { EXPECT_FALSE(got_create_); got_create_.yes(); - return base::WrapUnique( + return std::unique_ptr( new FrameNavExpectationsBrowserTestSingleNavHarness(nav)); } @@ -887,7 +886,7 @@ class FrameNavExpectationsFactoryRendererTestSingleNavHarness : protected: std::unique_ptr Create(int nav) override { - return base::WrapUnique( + return std::unique_ptr( new FrameNavExpectationsRendererTestSingleNavHarness(nav)); } }; @@ -1129,7 +1128,7 @@ class FrameNavExpectationsFactoryBrowserTestSingleNav : protected: std::unique_ptr Create(int nav) override { - return base::WrapUnique( + return std::unique_ptr( new FrameNavExpectationsBrowserTestSingleNav(nav)); } }; @@ -1145,7 +1144,7 @@ class FrameNavExpectationsFactoryRendererTestSingleNav : protected: std::unique_ptr Create(int nav) override { - return base::WrapUnique( + return std::unique_ptr( new FrameNavExpectationsRendererTestSingleNav(nav)); } }; @@ -1465,7 +1464,7 @@ class FrameNavExpectationsFactoryBrowserTestMultiNavHarness : protected: std::unique_ptr Create(int nav) override { create_count_++; - return base::WrapUnique( + return std::unique_ptr( new FrameNavExpectationsBrowserTestMultiNavHarness(nav)); } @@ -1486,7 +1485,7 @@ class FrameNavExpectationsFactoryRendererTestMultiNavHarness : protected: std::unique_ptr Create(int nav) override { - return base::WrapUnique( + return std::unique_ptr( new FrameNavExpectationsRendererTestMultiNavHarness(nav)); } }; @@ -1724,7 +1723,7 @@ class FrameNavExpectationsFactoryBrowserTestMultiNav : protected: std::unique_ptr Create(int nav) override { nav_count_++; - return base::WrapUnique( + return std::unique_ptr( new FrameNavExpectationsBrowserTestMultiNav(nav)); } @@ -1743,7 +1742,7 @@ class FrameNavExpectationsFactoryRendererTestMultiNav : protected: std::unique_ptr Create(int nav) override { - return base::WrapUnique( + return std::unique_ptr( new FrameNavExpectationsRendererTestMultiNav(nav)); } }; @@ -2178,7 +2177,7 @@ class FrameNavExpectationsFactoryBrowserTestNestedIframesSameOrigin : protected: std::unique_ptr Create(int nav) override { create_count_++; - return base::WrapUnique( + return std::unique_ptr( new FrameNavExpectationsBrowserTestNestedIframes(nav, true)); } @@ -2197,7 +2196,7 @@ class FrameNavExpectationsFactoryRendererTestNestedIframesSameOrigin : protected: std::unique_ptr Create(int nav) override { - return base::WrapUnique( + return std::unique_ptr( new FrameNavExpectationsRendererTestNestedIframes(nav, true)); } }; @@ -2233,7 +2232,7 @@ class FrameNavExpectationsFactoryBrowserTestNestedIframesDiffOrigin : protected: std::unique_ptr Create(int nav) override { create_count_++; - return base::WrapUnique( + return std::unique_ptr( new FrameNavExpectationsBrowserTestNestedIframes(nav, false)); } @@ -2252,7 +2251,7 @@ class FrameNavExpectationsFactoryRendererTestNestedIframesDiffOrigin : protected: std::unique_ptr Create(int nav) override { - return base::WrapUnique( + return std::unique_ptr( new FrameNavExpectationsRendererTestNestedIframes(nav, false)); } };