Don't compile download notification code on !CrOS.

chrome/browser/download/download_ui_controller.cc

 // Copyright 2013 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/browser/download/download_ui_controller.h"
 
 #include "base/callback.h"
 #include "base/stl_util.h"
 #include "chrome/browser/download/download_item_model.h"
 #include "chrome/browser/download/download_shelf.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/browser_tabstrip.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "content/public/browser/download_item.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_delegate.h"
 
 #if defined(OS_ANDROID)
 #include "content/public/browser/android/download_controller_android.h"
 #else
-#include "chrome/browser/download/notification/download_notification_manager.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/host_desktop.h"
 #endif
 
+#if defined(OS_CHROMEOS)
+#include "chrome/browser/download/notification/download_notification_manager.h"
+#endif
+
 namespace {
 
 // DownloadShelfUIControllerDelegate{Android,} is used when a
 // DownloadUIController is
 // constructed without specifying an explicit Delegate.
 #if defined(OS_ANDROID)
 
 class AndroidUIControllerDelegate : public DownloadUIController::Delegate {
  public:
   AndroidUIControllerDelegate() {}
@@ skipping 59 matching lines @@
 
 } // namespace
 
 DownloadUIController::Delegate::~Delegate() {
 }
 
 DownloadUIController::DownloadUIController(content::DownloadManager* manager,
                                            scoped_ptr<Delegate> delegate)
     : download_notifier_(manager, this),
       delegate_(delegate.Pass()) {
-  if (!delegate_) {
 #if defined(OS_ANDROID)
+  if (!delegate_)
     delegate_.reset(new AndroidUIControllerDelegate());
 #else
-    // The delegate should not be invoked after the profile has gone away. This
-    // should be the case since DownloadUIController is owned by
-    // DownloadService, which in turn is a profile keyed service.
-    if (DownloadNotificationManager::IsEnabled()) {
-      delegate_.reset(new DownloadNotificationManager(
-          Profile::FromBrowserContext(manager->GetBrowserContext())));
-    } else {
-      delegate_.reset(new DownloadShelfUIControllerDelegate(
-          Profile::FromBrowserContext(manager->GetBrowserContext())));
-    }
-#endif
+#if defined(OS_CHROMEOS)
+  if (!delegate_ && DownloadNotificationManager::IsEnabled()) {
+    // The Profile is guaranteed to be valid since DownloadUIController is owned
+    // by DownloadService, which in turn is a profile keyed service.
+    delegate_.reset(new DownloadNotificationManager(
+        Profile::FromBrowserContext(manager->GetBrowserContext())));
   }
+#endif  // defined(OS_CHROMEOS)
+  if (!delegate_) {
+    delegate_.reset(new DownloadShelfUIControllerDelegate(
+        Profile::FromBrowserContext(manager->GetBrowserContext())));
+  }
+#endif  // defined(OS_ANDROID)
 }
 
 DownloadUIController::~DownloadUIController() {
 }
 
 void DownloadUIController::OnDownloadCreated(content::DownloadManager* manager,
                                              content::DownloadItem* item) {
   // SavePackage downloads are created in a state where they can be shown in the
   // browser. Call OnDownloadUpdated() once to notify the UI immediately.
   OnDownloadUpdated(manager, item);
@@ skipping 28 matching lines @@
         browser->tab_strip_model()->GetIndexOfWebContents(web_contents) !=
             TabStripModel::kNoTab &&
         !item->IsSavePackageDownload()) {
       web_contents->Close();
     }
   }
 #endif
 
   delegate_->OnNewDownloadReady(item);
 }