Fixed Build breaks while notification is enabled and extension is disabled.

chrome/browser/notifications/desktop_notification_service.h

Index: chrome/browser/notifications/desktop_notification_service.h
diff --git a/chrome/browser/notifications/desktop_notification_service.h b/chrome/browser/notifications/desktop_notification_service.h
index ca71f572935e3bb6841c20a7dfbecbd2626cf105..096fcfa8ea5b37e314ef4b31c3d787721249f552 100644
--- a/chrome/browser/notifications/desktop_notification_service.h
+++ b/chrome/browser/notifications/desktop_notification_service.h
@@ -58,9 +58,12 @@ typedef base::Callback<void(blink::WebNotificationPermission)>
 
 // The DesktopNotificationService is an object, owned by the Profile,
 // which provides the creation of desktop "toasts" to web pages and workers.
-class DesktopNotificationService
-    : public PermissionContextBase,
-      public extensions::ExtensionRegistryObserver {
+class DesktopNotificationService : public PermissionContextBase
+#if defined(ENABLE_EXTENSION)

[dewittj, 2014/09/19 17:33:05]

This should be ENABLE_EXTENSIONS

[Jitu( very slow this week), 2014/09/22 05:49:44]

Done.

+                                   ,
+                                   public extensions::ExtensionRegistryObserver

[stevenjb, 2014/09/18 17:56:08]

Ugh, this sort of #ifdef tends to be error prone.
I notice that we are still including  #include
"extensions/browser/extension_registry_observer.h".
Can we leave the class dependency and just #ifdef out the code inside the
implementation in the .cc file?

[Jun Mukai, 2014/09/18 18:07:13]

I am okay with this.  Factoring-out the observer would be cleaner for the coding
though.

[dewittj, 2014/09/18 19:24:45]

Seems to me that this is a temporary state of affairs anyway, since Peter is
planning to move the extension code into a separate service.

+#endif
+                                   {
  public:
   // Register profile-specific prefs of notifications.
   static void RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* prefs);
@@ -130,11 +133,13 @@ class DesktopNotificationService
       const message_center::NotifierId& notifier_id,
       bool enabled);
 
+#if defined(ENABLED_EXTENSIONS)

[dewittj, 2014/09/19 17:33:05]

This should be ENABLE_EXTENSIONS

[Jitu( very slow this week), 2014/09/22 05:49:44]

Done.

   // extensions::ExtensionRegistryObserver:
   virtual void OnExtensionUninstalled(
       content::BrowserContext* browser_context,
       const extensions::Extension* extension,
       extensions::UninstallReason reason) OVERRIDE;
+#endif
 
   // The profile which owns this object.
   Profile* profile_;
@@ -151,10 +156,12 @@ class DesktopNotificationService
   // On-memory data for the availability of system_component.
   std::set<std::string> disabled_system_component_ids_;
 
+#if defined(ENABLED_EXTENSIONS)

[dewittj, 2014/09/19 17:33:05]

This should be ENABLE_EXTENSIONS

[Jitu( very slow this week), 2014/09/22 05:49:44]

Done.

   // An observer to listen when extension is uninstalled.
   ScopedObserver<extensions::ExtensionRegistry,
                  extensions::ExtensionRegistryObserver>
       extension_registry_observer_;
+#endif
 
   // Welcome Notification
   scoped_ptr<ExtensionWelcomeNotification> chrome_now_welcome_notification_;