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

Side by Side Diff: chrome/test/base/testing_browser_process.cc

Issue 1814923002: Nuke NotificationUIManager from PlatformNotificationServiceImpl (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@profile_manager_load
Patch Set: Review comments + unique_ptr rename 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 "chrome/test/base/testing_browser_process.h" 5 #include "chrome/test/base/testing_browser_process.h"
6 6
7 #include "base/strings/string_util.h" 7 #include "base/strings/string_util.h"
8 #include "base/time/default_clock.h" 8 #include "base/time/default_clock.h"
9 #include "base/time/default_tick_clock.h" 9 #include "base/time/default_tick_clock.h"
10 #include "build/build_config.h" 10 #include "build/build_config.h"
(...skipping 196 matching lines...) Expand 10 before | Expand all | Expand 10 after
207 BrowserProcessPlatformPart* TestingBrowserProcess::platform_part() { 207 BrowserProcessPlatformPart* TestingBrowserProcess::platform_part() {
208 return platform_part_.get(); 208 return platform_part_.get();
209 } 209 }
210 210
211 extensions::EventRouterForwarder* 211 extensions::EventRouterForwarder*
212 TestingBrowserProcess::extension_event_router_forwarder() { 212 TestingBrowserProcess::extension_event_router_forwarder() {
213 return nullptr; 213 return nullptr;
214 } 214 }
215 215
216 NotificationUIManager* TestingBrowserProcess::notification_ui_manager() { 216 NotificationUIManager* TestingBrowserProcess::notification_ui_manager() {
217 #if defined(ENABLE_NOTIFICATIONS) 217 #if defined(ENABLE_NOTIFICATIONS) && !defined(OS_ANDROID)
218 if (!notification_ui_manager_.get()) 218 if (!notification_ui_manager_.get())
219 notification_ui_manager_.reset( 219 notification_ui_manager_.reset(
220 NotificationUIManager::Create(local_state())); 220 NotificationUIManager::Create(local_state()));
221 return notification_ui_manager_.get(); 221 return notification_ui_manager_.get();
222 #else 222 #else
223 NOTIMPLEMENTED(); 223 NOTIMPLEMENTED();
224 return nullptr; 224 return nullptr;
225 #endif 225 #endif
226 } 226 }
227 227
228 NotificationBridge* TestingBrowserProcess::notification_bridge() {
229 return nullptr;
230 }
231
228 message_center::MessageCenter* TestingBrowserProcess::message_center() { 232 message_center::MessageCenter* TestingBrowserProcess::message_center() {
229 return message_center::MessageCenter::Get(); 233 return message_center::MessageCenter::Get();
230 } 234 }
231 235
232 IntranetRedirectDetector* TestingBrowserProcess::intranet_redirect_detector() { 236 IntranetRedirectDetector* TestingBrowserProcess::intranet_redirect_detector() {
233 return nullptr; 237 return nullptr;
234 } 238 }
235 239
236 void TestingBrowserProcess::CreateDevToolsHttpProtocolHandler( 240 void TestingBrowserProcess::CreateDevToolsHttpProtocolHandler(
237 const std::string& ip, 241 const std::string& ip,
(...skipping 181 matching lines...) Expand 10 before | Expand all | Expand 10 after
419 423
420 /////////////////////////////////////////////////////////////////////////////// 424 ///////////////////////////////////////////////////////////////////////////////
421 425
422 TestingBrowserProcessInitializer::TestingBrowserProcessInitializer() { 426 TestingBrowserProcessInitializer::TestingBrowserProcessInitializer() {
423 TestingBrowserProcess::CreateInstance(); 427 TestingBrowserProcess::CreateInstance();
424 } 428 }
425 429
426 TestingBrowserProcessInitializer::~TestingBrowserProcessInitializer() { 430 TestingBrowserProcessInitializer::~TestingBrowserProcessInitializer() {
427 TestingBrowserProcess::DeleteInstance(); 431 TestingBrowserProcess::DeleteInstance();
428 } 432 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698