Nuke NotificationUIManager from PlatformNotificationServiceImpl

chrome/test/base/testing_browser_process.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/test/base/testing_browser_process.h"
 
 #include "base/strings/string_util.h"
 #include "base/time/default_clock.h"
 #include "base/time/default_tick_clock.h"
 #include "build/build_config.h"
@@ skipping 196 matching lines @@
 BrowserProcessPlatformPart* TestingBrowserProcess::platform_part() {
   return platform_part_.get();
 }
 
 extensions::EventRouterForwarder*
 TestingBrowserProcess::extension_event_router_forwarder() {
   return nullptr;
 }
 
 NotificationUIManager* TestingBrowserProcess::notification_ui_manager() {
+#if defined(OS_ANDROID)

[Peter Beverloo, 2016/04/18 14:57:10]

Why not modify the #ifdef on line 221? Displaying the NOTIMPLEMENTED() should be
fine because we shouldn't be calling this in the first place.

[Miguel Garcia, 2016/04/19 14:24:58]

SG

+  return nullptr;
+#endif
+
 #if defined(ENABLE_NOTIFICATIONS)
   if (!notification_ui_manager_.get())
     notification_ui_manager_.reset(
         NotificationUIManager::Create(local_state()));
   return notification_ui_manager_.get();
 #else
   NOTIMPLEMENTED();
   return nullptr;
 #endif
 }
 
+NotificationBridge* TestingBrowserProcess::notification_bridge() {
+  return nullptr;
+}
+
 message_center::MessageCenter* TestingBrowserProcess::message_center() {
   return message_center::MessageCenter::Get();
 }
 
 IntranetRedirectDetector* TestingBrowserProcess::intranet_redirect_detector() {
   return nullptr;
 }
 
 void TestingBrowserProcess::CreateDevToolsHttpProtocolHandler(
     const std::string& ip,
@@ skipping 181 matching lines @@
 
 ///////////////////////////////////////////////////////////////////////////////
 
 TestingBrowserProcessInitializer::TestingBrowserProcessInitializer() {
   TestingBrowserProcess::CreateInstance();
 }
 
 TestingBrowserProcessInitializer::~TestingBrowserProcessInitializer() {
   TestingBrowserProcess::DeleteInstance();
 }