| Index: chrome/browser/ui/webui/media_router/media_router_ui_unittest.cc
|
| diff --git a/chrome/browser/ui/webui/media_router/media_router_ui_unittest.cc b/chrome/browser/ui/webui/media_router/media_router_ui_unittest.cc
|
| index 8183aa653ef012002b34fc17200c7a2afef265ad..6e8d17ae2858cb4a7f8bd6d08ce482401ca2881e 100644
|
| --- a/chrome/browser/ui/webui/media_router/media_router_ui_unittest.cc
|
| +++ b/chrome/browser/ui/webui/media_router/media_router_ui_unittest.cc
|
| @@ -2,14 +2,16 @@
|
| // Use of this source code is governed by a BSD-style license that can be
|
| // found in the LICENSE file.
|
|
|
| +#include "chrome/browser/ui/webui/media_router/media_router_ui.h"
|
| +
|
| #include "base/bind.h"
|
| +#include "base/memory/ptr_util.h"
|
| #include "base/strings/utf_string_conversions.h"
|
| #include "chrome/browser/media/router/media_route.h"
|
| #include "chrome/browser/media/router/mock_media_router.h"
|
| #include "chrome/browser/media/router/route_request_result.h"
|
| #include "chrome/browser/media/router/test_helper.h"
|
| #include "chrome/browser/sessions/session_tab_helper.h"
|
| -#include "chrome/browser/ui/webui/media_router/media_router_ui.h"
|
| #include "chrome/browser/ui/webui/media_router/media_router_webui_message_handler.h"
|
| #include "chrome/grit/generated_resources.h"
|
| #include "chrome/test/base/testing_profile.h"
|
| @@ -70,11 +72,11 @@ class MediaRouterUITest : public ::testing::Test {
|
| MockMediaRouter mock_router_;
|
| content::TestBrowserThreadBundle thread_bundle_;
|
| TestingProfile profile_;
|
| - scoped_ptr<content::WebContents> initiator_;
|
| + std::unique_ptr<content::WebContents> initiator_;
|
| content::TestWebUI web_ui_;
|
| - scoped_ptr<content::WebContents> web_contents_;
|
| - scoped_ptr<MediaRouterUI> media_router_ui_;
|
| - scoped_ptr<MediaRouterWebUIMessageHandler> message_handler_;
|
| + std::unique_ptr<content::WebContents> web_contents_;
|
| + std::unique_ptr<MediaRouterUI> media_router_ui_;
|
| + std::unique_ptr<MediaRouterWebUIMessageHandler> message_handler_;
|
| };
|
|
|
| TEST_F(MediaRouterUITest, RouteCreationTimeoutForTab) {
|
| @@ -89,7 +91,7 @@ TEST_F(MediaRouterUITest, RouteCreationTimeoutForTab) {
|
| std::string expected_title = l10n_util::GetStringUTF8(
|
| IDS_MEDIA_ROUTER_ISSUE_CREATE_ROUTE_TIMEOUT_FOR_TAB);
|
| EXPECT_CALL(mock_router_, AddIssue(IssueTitleEquals(expected_title)));
|
| - scoped_ptr<RouteRequestResult> result =
|
| + std::unique_ptr<RouteRequestResult> result =
|
| RouteRequestResult::FromError("Timed out", RouteRequestResult::TIMED_OUT);
|
| for (const auto& callback : callbacks)
|
| callback.Run(*result);
|
| @@ -107,7 +109,7 @@ TEST_F(MediaRouterUITest, RouteCreationTimeoutForDesktop) {
|
| std::string expected_title = l10n_util::GetStringUTF8(
|
| IDS_MEDIA_ROUTER_ISSUE_CREATE_ROUTE_TIMEOUT_FOR_DESKTOP);
|
| EXPECT_CALL(mock_router_, AddIssue(IssueTitleEquals(expected_title)));
|
| - scoped_ptr<RouteRequestResult> result =
|
| + std::unique_ptr<RouteRequestResult> result =
|
| RouteRequestResult::FromError("Timed out", RouteRequestResult::TIMED_OUT);
|
| for (const auto& callback : callbacks)
|
| callback.Run(*result);
|
| @@ -130,7 +132,7 @@ TEST_F(MediaRouterUITest, RouteCreationTimeoutForPresentation) {
|
| l10n_util::GetStringFUTF8(IDS_MEDIA_ROUTER_ISSUE_CREATE_ROUTE_TIMEOUT,
|
| base::UTF8ToUTF16("frameurl.fakeurl"));
|
| EXPECT_CALL(mock_router_, AddIssue(IssueTitleEquals(expected_title)));
|
| - scoped_ptr<RouteRequestResult> result =
|
| + std::unique_ptr<RouteRequestResult> result =
|
| RouteRequestResult::FromError("Timed out", RouteRequestResult::TIMED_OUT);
|
| for (const auto& callback : callbacks)
|
| callback.Run(*result);
|
| @@ -182,7 +184,7 @@ TEST_F(MediaRouterUITest, UIMediaRoutesObserverFiltersNonDisplayRoutes) {
|
| EXPECT_CALL(mock_router_, RegisterMediaRoutesObserver(_)).Times(1);
|
| MediaSource media_source("mediaSource");
|
| MockRoutesUpdatedCallback mock_callback;
|
| - scoped_ptr<MediaRouterUI::UIMediaRoutesObserver> observer(
|
| + std::unique_ptr<MediaRouterUI::UIMediaRoutesObserver> observer(
|
| new MediaRouterUI::UIMediaRoutesObserver(
|
| &mock_router_, media_source.id(),
|
| base::Bind(&MockRoutesUpdatedCallback::OnRoutesUpdated,
|
| @@ -220,7 +222,7 @@ TEST_F(MediaRouterUITest,
|
| EXPECT_CALL(mock_router_, RegisterMediaRoutesObserver(_)).Times(1);
|
| MediaSource media_source("mediaSource");
|
| MockRoutesUpdatedCallback mock_callback;
|
| - scoped_ptr<MediaRouterUI::UIMediaRoutesObserver> observer(
|
| + std::unique_ptr<MediaRouterUI::UIMediaRoutesObserver> observer(
|
| new MediaRouterUI::UIMediaRoutesObserver(
|
| &mock_router_, media_source.id(),
|
| base::Bind(&MockRoutesUpdatedCallback::OnRoutesUpdated,
|
| @@ -260,8 +262,8 @@ TEST_F(MediaRouterUITest,
|
| TEST_F(MediaRouterUITest, GetExtensionNameExtensionPresent) {
|
| std::string id = "extensionid";
|
| GURL url = GURL("chrome-extension://" + id);
|
| - scoped_ptr<extensions::ExtensionRegistry> registry =
|
| - make_scoped_ptr(new extensions::ExtensionRegistry(nullptr));
|
| + std::unique_ptr<extensions::ExtensionRegistry> registry =
|
| + base::WrapUnique(new extensions::ExtensionRegistry(nullptr));
|
| scoped_refptr<extensions::Extension> app =
|
| extensions::test_util::BuildApp(extensions::ExtensionBuilder())
|
| .MergeManifest(extensions::DictionaryBuilder()
|
| @@ -278,16 +280,16 @@ TEST_F(MediaRouterUITest, GetExtensionNameExtensionPresent) {
|
| TEST_F(MediaRouterUITest, GetExtensionNameEmptyWhenNotInstalled) {
|
| std::string id = "extensionid";
|
| GURL url = GURL("chrome-extension://" + id);
|
| - scoped_ptr<extensions::ExtensionRegistry> registry =
|
| - make_scoped_ptr(new extensions::ExtensionRegistry(nullptr));
|
| + std::unique_ptr<extensions::ExtensionRegistry> registry =
|
| + base::WrapUnique(new extensions::ExtensionRegistry(nullptr));
|
|
|
| EXPECT_EQ("", MediaRouterUI::GetExtensionName(url, registry.get()));
|
| }
|
|
|
| TEST_F(MediaRouterUITest, GetExtensionNameEmptyWhenNotExtensionURL) {
|
| GURL url = GURL("https://www.google.com");
|
| - scoped_ptr<extensions::ExtensionRegistry> registry =
|
| - make_scoped_ptr(new extensions::ExtensionRegistry(nullptr));
|
| + std::unique_ptr<extensions::ExtensionRegistry> registry =
|
| + base::WrapUnique(new extensions::ExtensionRegistry(nullptr));
|
|
|
| EXPECT_EQ("", MediaRouterUI::GetExtensionName(url, registry.get()));
|
| }
|
|
|