Fixed javascript_dialog_manager to compile when extensions are disabled.

chrome/browser/ui/app_modal_dialogs/javascript_dialog_manager.cc

Index: chrome/browser/ui/app_modal_dialogs/javascript_dialog_manager.cc
diff --git a/chrome/browser/ui/app_modal_dialogs/javascript_dialog_manager.cc b/chrome/browser/ui/app_modal_dialogs/javascript_dialog_manager.cc
index 417a2898d0d5c5ca24a6a3f7a3de5ec48c670b12..fd7f9a6ec3e3fa2f14ab574be38bca6c5cb2e749 100644
--- a/chrome/browser/ui/app_modal_dialogs/javascript_dialog_manager.cc
+++ b/chrome/browser/ui/app_modal_dialogs/javascript_dialog_manager.cc
@@ -58,22 +58,26 @@ const Extension* GetExtensionForWebContents(WebContents* web_contents) {
 // dialog closes.
 void IncrementLazyKeepaliveCount(const Extension* extension,
                                  WebContents* web_contents) {
+#if defined(ENABLE_EXTENSIONS)
   DCHECK(extension);
   DCHECK(web_contents);
   extensions::ProcessManager* pm = GetExtensionsProcessManager(web_contents);
   if (pm)
     pm->IncrementLazyKeepaliveCount(extension);
+#endif  // defined(ENABLE_EXTENSIONS)

[msw, 2014/08/22 22:21:50]

nit: "// ENABLE_EXTENSIONS" would follow the more common general commenting
pattern for pre-processor block termination. That said, apparently this
particular pre-processor definition more commonly follows your pattern here.

[David Yen, 2014/08/22 22:28:25]

Done. I went ahead and fixed all the other #endif // defined(ENABLE_EXTENSIONS)"
in this file to match the general pattern.

 }
 
 // Allows an |extension| to shut down its lazy background page after a dialog
 // closes (if nothing else is keeping it open).
 void DecrementLazyKeepaliveCount(const Extension* extension,
                                  WebContents* web_contents) {
+#if defined(ENABLE_EXTENSIONS)
   DCHECK(extension);
   DCHECK(web_contents);
   extensions::ProcessManager* pm = GetExtensionsProcessManager(web_contents);
   if (pm)
     pm->DecrementLazyKeepaliveCount(extension);
+#endif  // defined(ENABLE_EXTENSIONS)
 }
 
 class ChromeJavaScriptDialogManager : public JavaScriptDialogManager {