Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(136)

Unified Diff: chrome/browser/chromeos/ui/idle_app_name_notification_view_unittest.cc

Issue 1870793002: Convert //chrome/browser/chromeos from scoped_ptr to std::unique_ptr (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/ui/idle_app_name_notification_view_unittest.cc
diff --git a/chrome/browser/chromeos/ui/idle_app_name_notification_view_unittest.cc b/chrome/browser/chromeos/ui/idle_app_name_notification_view_unittest.cc
index 01bf50d2528e42da364f20fee6a968d7b793e238..cdb9204e83f5d00b3d8dd1651c11b7b558ebf8e7 100644
--- a/chrome/browser/chromeos/ui/idle_app_name_notification_view_unittest.cc
+++ b/chrome/browser/chromeos/ui/idle_app_name_notification_view_unittest.cc
@@ -86,14 +86,14 @@ class IdleAppNameNotificationViewTest : public BrowserWithTestWindowTest {
// message).
TEST_F(IdleAppNameNotificationViewTest, CheckTooEarlyDestruction) {
// Create a message which is visible for 10ms and fades in/out for 5ms.
- scoped_ptr<chromeos::IdleAppNameNotificationView> message(
+ std::unique_ptr<chromeos::IdleAppNameNotificationView> message(
new chromeos::IdleAppNameNotificationView(10, 5, correct_extension()));
}
// Check that the message gets created and it destroys itself after time.
TEST_F(IdleAppNameNotificationViewTest, CheckSelfDestruction) {
// Create a message which is visible for 10ms and fades in/out for 5ms.
- scoped_ptr<chromeos::IdleAppNameNotificationView> message(
+ std::unique_ptr<chromeos::IdleAppNameNotificationView> message(
new chromeos::IdleAppNameNotificationView(10, 5, correct_extension()));
EXPECT_TRUE(message->IsVisible());
@@ -108,7 +108,7 @@ TEST_F(IdleAppNameNotificationViewTest, CheckSelfDestruction) {
// Check that the shown text for a correct application is correct.
TEST_F(IdleAppNameNotificationViewTest, CheckCorrectApp) {
// Create a message which is visible for 10ms and fades in/out for 5ms.
- scoped_ptr<chromeos::IdleAppNameNotificationView> message(
+ std::unique_ptr<chromeos::IdleAppNameNotificationView> message(
new chromeos::IdleAppNameNotificationView(10, 5, correct_extension()));
base::string16 text = message->GetShownTextForTest();
// Check that the string is the application name.
@@ -119,7 +119,7 @@ TEST_F(IdleAppNameNotificationViewTest, CheckCorrectApp) {
// Check that an invalid app gets shown accordingly.
TEST_F(IdleAppNameNotificationViewTest, CheckInvalidApp) {
// Create a message which is visible for 10ms and fades in/out for 5ms.
- scoped_ptr<chromeos::IdleAppNameNotificationView> message(
+ std::unique_ptr<chromeos::IdleAppNameNotificationView> message(
new chromeos::IdleAppNameNotificationView(10, 5, NULL));
base::string16 text = message->GetShownTextForTest();
base::string16 error = l10n_util::GetStringUTF16(

Powered by Google App Engine
This is Rietveld 408576698