--downloads-new-ui completely disables the download shelf.

chrome/browser/ui/browser.cc

 // Copyright (c) 2011 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/ui/browser.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #include <shellapi.h>
 #endif  // OS_WIN
@@ skipping 3361 matching lines @@
   if (constrained != wrapper) {
     // Download in a constrained popup is shown in the tab that opened it.
     TabContents* constrained_tab = constrained->tab_contents();
     constrained_tab->delegate()->OnStartDownload(constrained_tab, download);
     return;
   }
 
   if (!window())
     return;
 
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(

[cbentzel, 2011/08/11 20:08:29]

I'd rather wrap this in a helper function such as bool DisplayDownloadShelf()
which hides the command line itself. 

[benjhayden, 2011/08/15 14:57:15]

Done.

+        switches::kDownloadsNewUI)) {
 #if defined(OS_CHROMEOS)
-  // Don't show content browser for extension/theme downloads from gallery.
-  if (download->is_extension_install()) {
+    // Don't show content browser for extension/theme downloads from gallery.
     ExtensionService* service = profile_->GetExtensionService();
-    if (service && service->IsDownloadFromGallery(download->GetURL(),
-                                                  download->referrer_url())) {
-      return;
+    if (!download->is_extension_install() ||
+        (service == NULL) ||
+        !service->IsDownloadFromGallery(download->GetURL(),
+                                        download->referrer_url())) {
+      // Open the Active Downloads ui for chromeos.
+      ActiveDownloadsUI::OpenPopup(profile_);
     }
+#else
+    // GetDownloadShelf creates the download shelf if it was not yet created.
+    DownloadShelf* shelf = window()->GetDownloadShelf();
+    shelf->AddDownload(new DownloadItemModel(download));
+
+    // Don't show the animation for "Save file" downloads.
+    if (download->total_bytes() > 0) {
+      // For non-theme extensions, we don't show the download animation.
+      if (!download->is_extension_install() ||
+          ExtensionService::IsDownloadFromMiniGallery(download->GetURL())) {
+        // Show animation in same window as the download shelf. Download shelf
+        // may not be in the same window that initiated the download, e.g.
+        // Panels.
+        TabContents* shelf_tab = shelf->browser()->GetSelectedTabContents();
+
+        // We make this check for the case of minimized windows, unit tests,
+        // etc.
+        if (platform_util::IsVisible(shelf_tab->GetNativeView()) &&
+            ui::Animation::ShouldRenderRichAnimation()) {
+          DownloadStartedAnimation::Show(shelf_tab);
+        }
+      }
+    }
+#endif
   }
-  // Open the Active Downloads ui for chromeos.
-  ActiveDownloadsUI::OpenPopup(profile_);
-#else
-  // GetDownloadShelf creates the download shelf if it was not yet created.
-  DownloadShelf* shelf = window()->GetDownloadShelf();
-  shelf->AddDownload(new DownloadItemModel(download));
-
-  // Don't show the animation for "Save file" downloads.
-  if (download->total_bytes() <= 0)
-    return;
-
-  // For non-theme extensions, we don't show the download animation.
-  if (download->is_extension_install() &&
-      !ExtensionService::IsDownloadFromMiniGallery(download->GetURL()))
-    return;
-
-  // Show animation in same window as the download shelf. Download shelf
-  // may not be in the same window that initiated the download, e.g. Panels.
-  TabContents* shelf_tab = shelf->browser()->GetSelectedTabContents();
-
-  // We make this check for the case of minimized windows, unit tests, etc.
-  if (platform_util::IsVisible(shelf_tab->GetNativeView()) &&
-      ui::Animation::ShouldRenderRichAnimation()) {
-    DownloadStartedAnimation::Show(shelf_tab);
-  }
-#endif
 
   // If the download occurs in a new tab, close it.
   if (source->controller().IsInitialNavigation() && tab_count() > 1)
     CloseContents(source);
 }
 
 void Browser::ShowPageInfo(content::BrowserContext* browser_context,
                            const GURL& url,
                            const NavigationEntry::SSLStatus& ssl,
                            bool show_history) {
@@ skipping 1413 matching lines @@
 }
 
 void Browser::ShowSyncSetup() {
   ProfileSyncService* service =
       profile()->GetOriginalProfile()->GetProfileSyncService();
   if (service->HasSyncSetupCompleted())
     ShowOptionsTab(chrome::kSyncSetupSubPage);
   else
     service->ShowLoginDialog();
 }