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

Side by Side Diff: chrome/browser/extensions/api/notifications/notifications_apitest.cc

Issue 1871713002: Convert //chrome/browser/extensions from scoped_ptr to std::unique_ptr (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase and fix header 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/strings/string_number_conversions.h" 5 #include "base/strings/string_number_conversions.h"
6 #include "base/strings/stringprintf.h" 6 #include "base/strings/stringprintf.h"
7 #include "base/strings/utf_string_conversions.h" 7 #include "base/strings/utf_string_conversions.h"
8 #include "chrome/browser/browser_process.h" 8 #include "chrome/browser/browser_process.h"
9 #include "chrome/browser/extensions/api/notifications/notifications_api.h" 9 #include "chrome/browser/extensions/api/notifications/notifications_api.h"
10 #include "chrome/browser/extensions/extension_apitest.h" 10 #include "chrome/browser/extensions/extension_apitest.h"
(...skipping 178 matching lines...) Expand 10 before | Expand all | Expand 10 after
189 189
190 // Get permission level for the extension whose notifications are enabled. 190 // Get permission level for the extension whose notifications are enabled.
191 { 191 {
192 scoped_refptr<extensions::NotificationsGetPermissionLevelFunction> 192 scoped_refptr<extensions::NotificationsGetPermissionLevelFunction>
193 notification_function( 193 notification_function(
194 new extensions::NotificationsGetPermissionLevelFunction()); 194 new extensions::NotificationsGetPermissionLevelFunction());
195 195
196 notification_function->set_extension(empty_extension.get()); 196 notification_function->set_extension(empty_extension.get());
197 notification_function->set_has_callback(true); 197 notification_function->set_has_callback(true);
198 198
199 scoped_ptr<base::Value> result(utils::RunFunctionAndReturnSingleResult( 199 std::unique_ptr<base::Value> result(utils::RunFunctionAndReturnSingleResult(
200 notification_function.get(), 200 notification_function.get(), "[]", browser(), utils::NONE));
201 "[]",
202 browser(),
203 utils::NONE));
204 201
205 EXPECT_EQ(base::Value::TYPE_STRING, result->GetType()); 202 EXPECT_EQ(base::Value::TYPE_STRING, result->GetType());
206 std::string permission_level; 203 std::string permission_level;
207 EXPECT_TRUE(result->GetAsString(&permission_level)); 204 EXPECT_TRUE(result->GetAsString(&permission_level));
208 EXPECT_EQ("granted", permission_level); 205 EXPECT_EQ("granted", permission_level);
209 } 206 }
210 207
211 // Get permission level for the extension whose notifications are disabled. 208 // Get permission level for the extension whose notifications are disabled.
212 { 209 {
213 scoped_refptr<extensions::NotificationsGetPermissionLevelFunction> 210 scoped_refptr<extensions::NotificationsGetPermissionLevelFunction>
214 notification_function( 211 notification_function(
215 new extensions::NotificationsGetPermissionLevelFunction()); 212 new extensions::NotificationsGetPermissionLevelFunction());
216 213
217 notification_function->set_extension(empty_extension.get()); 214 notification_function->set_extension(empty_extension.get());
218 notification_function->set_has_callback(true); 215 notification_function->set_has_callback(true);
219 216
220 message_center::NotifierId notifier_id( 217 message_center::NotifierId notifier_id(
221 message_center::NotifierId::APPLICATION, 218 message_center::NotifierId::APPLICATION,
222 empty_extension->id()); 219 empty_extension->id());
223 message_center::Notifier notifier(notifier_id, base::string16(), true); 220 message_center::Notifier notifier(notifier_id, base::string16(), true);
224 g_browser_process->message_center()->GetNotifierSettingsProvider()-> 221 g_browser_process->message_center()->GetNotifierSettingsProvider()->
225 SetNotifierEnabled(notifier, false); 222 SetNotifierEnabled(notifier, false);
226 223
227 scoped_ptr<base::Value> result(utils::RunFunctionAndReturnSingleResult( 224 std::unique_ptr<base::Value> result(utils::RunFunctionAndReturnSingleResult(
228 notification_function.get(), 225 notification_function.get(), "[]", browser(), utils::NONE));
229 "[]",
230 browser(),
231 utils::NONE));
232 226
233 EXPECT_EQ(base::Value::TYPE_STRING, result->GetType()); 227 EXPECT_EQ(base::Value::TYPE_STRING, result->GetType());
234 std::string permission_level; 228 std::string permission_level;
235 EXPECT_TRUE(result->GetAsString(&permission_level)); 229 EXPECT_TRUE(result->GetAsString(&permission_level));
236 EXPECT_EQ("denied", permission_level); 230 EXPECT_EQ("denied", permission_level);
237 } 231 }
238 } 232 }
239 233
240 IN_PROC_BROWSER_TEST_F(NotificationsApiTest, TestOnPermissionLevelChanged) { 234 IN_PROC_BROWSER_TEST_F(NotificationsApiTest, TestOnPermissionLevelChanged) {
241 const extensions::Extension* extension = 235 const extensions::Extension* extension =
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
301 ASSERT_TRUE(extension) << message_; 295 ASSERT_TRUE(extension) << message_;
302 296
303 const message_center::NotificationList::Notifications& notifications = 297 const message_center::NotificationList::Notifications& notifications =
304 g_browser_process->message_center()->GetVisibleNotifications(); 298 g_browser_process->message_center()->GetVisibleNotifications();
305 ASSERT_EQ(1u, notifications.size()); 299 ASSERT_EQ(1u, notifications.size());
306 message_center::Notification* notification = *(notifications.begin()); 300 message_center::Notification* notification = *(notifications.begin());
307 ASSERT_EQ(extension->url(), notification->origin_url()); 301 ASSERT_EQ(extension->url(), notification->origin_url());
308 302
309 EXPECT_TRUE(notification->never_timeout()); 303 EXPECT_TRUE(notification->never_timeout());
310 } 304 }
OLDNEW
« no previous file with comments | « chrome/browser/extensions/api/notifications/notifications_api.cc ('k') | chrome/browser/extensions/api/omnibox/omnibox_api.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698