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

chrome/browser/notifications/desktop_notification_service.h

 // 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.
 
 #ifndef CHROME_BROWSER_NOTIFICATIONS_DESKTOP_NOTIFICATION_SERVICE_H_
 #define CHROME_BROWSER_NOTIFICATIONS_DESKTOP_NOTIFICATION_SERVICE_H_
 
 #include <set>
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/callback_forward.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "base/prefs/pref_member.h"
 #include "base/scoped_observer.h"
 #include "base/strings/string16.h"
 #include "chrome/browser/content_settings/content_settings_provider.h"
 #include "chrome/browser/content_settings/permission_context_base.h"
 #include "chrome/browser/notifications/extension_welcome_notification.h"
 #include "components/content_settings/core/common/content_settings.h"
 #include "components/keyed_service/core/keyed_service.h"
+#if defined(ENABLE_EXTENSIONS)

[Jun Mukai, 2014/09/22 07:22:47]

Please move this below. Our style is:

#include ...
#include ...

#if defined(...)
#include ...
#endif

refer desktop_notification_service.cc which has this section.

 #include "extensions/browser/extension_registry_observer.h"
+#endif
 #include "third_party/WebKit/public/platform/WebNotificationPermission.h"
 #include "third_party/WebKit/public/web/WebTextDirection.h"
 #include "ui/message_center/notifier_settings.h"
 #include "url/gurl.h"
 
 class ContentSettingsPattern;
 class Notification;
 class NotificationDelegate;
 class NotificationUIManager;
 class Profile;
 
 namespace content {
 class DesktopNotificationDelegate;
 class RenderFrameHost;
 struct ShowDesktopNotificationHostMsgParams;
 }
 
+#if defined(ENABLE_EXTENSIONS)
 namespace extensions {
 class ExtensionRegistry;
 }
+#endif
 
 namespace gfx {
 class Image;
 }
 
 namespace user_prefs {
 class PrefRegistrySyncable;
 }
 
 // Callback to be invoked when the result of a permission request is known.
 typedef base::Callback<void(blink::WebNotificationPermission)>
     NotificationPermissionCallback;
 
 // 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_EXTENSIONS)
+                                   ,
+                                   public extensions::ExtensionRegistryObserver
+#endif
+                                   {
  public:
   // Register profile-specific prefs of notifications.
   static void RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* prefs);
 
     // Add a desktop notification.
   static std::string AddIconNotification(const GURL& origin_url,
                                          const base::string16& title,
                                          const base::string16& message,
                                          const gfx::Image& icon,
                                          const base::string16& replace_id,
@@ skipping 49 matching lines @@
   // Used as a callback once a permission has been decided to convert |allowed|
   // to one of the blink::WebNotificationPermission values.
   void OnNotificationPermissionRequested(
       const base::Callback<void(blink::WebNotificationPermission)>& callback,
       bool allowed);
 
   void FirePermissionLevelChangedEvent(
       const message_center::NotifierId& notifier_id,
       bool enabled);
 
+#if defined(ENABLE_EXTENSIONS)
   // 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_;
 
   // Prefs listener for disabled_extension_id.
   StringListPrefMember disabled_extension_id_pref_;
 
   // Prefs listener for disabled_system_component_id.
   StringListPrefMember disabled_system_component_id_pref_;
 
   // On-memory data for the availability of extensions.
   std::set<std::string> disabled_extension_ids_;
 
   // On-memory data for the availability of system_component.
   std::set<std::string> disabled_system_component_ids_;
 
+#if defined(ENABLE_EXTENSIONS)
   // 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_;
 
   base::WeakPtrFactory<DesktopNotificationService> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(DesktopNotificationService);
 };
 
 #endif  // CHROME_BROWSER_NOTIFICATIONS_DESKTOP_NOTIFICATION_SERVICE_H_