Put the pop-up app launcher behind a compile-time flag, and provide defines.

chrome/chrome_browser_ui.gypi

 # 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.
 
 {
   'targets': [
     {
       'target_name': 'browser_ui',
       'type': 'static_library',
@@ skipping 2261 matching lines @@
             '../printing/printing.gyp:cups',
           ],
         }],
         ['use_gconf==0', {
           'sources!': [
             'browser/ui/gtk/gconf_titlebar_listener.cc',
             'browser/ui/gtk/gconf_titlebar_listener.h',
           ],
         }],
         ['use_ash==1', {
-          'sources': [
-            'browser/ui/app_list/app_list_view_delegate.cc',
-            'browser/ui/app_list/app_list_view_delegate.h',
-            'browser/ui/app_list/apps_model_builder.cc',
-            'browser/ui/app_list/apps_model_builder.h',
-            'browser/ui/app_list/chrome_app_list_item.h',
-            'browser/ui/app_list/extension_app_item.cc',
-            'browser/ui/app_list/extension_app_item.h',
-            'browser/ui/app_list/search_builder.cc',
-            'browser/ui/app_list/search_builder.h',
-          ],
           'dependencies': [
             '../ash/ash.gyp:ash',
             '../ash/ash_strings.gyp:ash_strings',
-            '../ui/app_list/app_list.gyp:app_list',
             '../ui/message_center/message_center.gyp:message_center',
           ],
         }],
         ['use_aura==1 and use_ash==0 and OS=="linux"', {
           'dependencies': [
             # gtk2 is the only component that can interact with gtk2 in our new
             # world.
             'browser/ui/libgtk2ui/libgtk2ui.gyp:gtk2ui',
           ],
         }],
@@ skipping 334 matching lines @@
             '../third_party/isimpledom/isimpledom.gyp:isimpledom',
             '../ui/app_list/app_list.gyp:app_list',
             '../ui/views/controls/webview/webview.gyp:webview',
             '../ui/views/views.gyp:views',
           ],
           'export_dependent_settings': [
             '../ui/views/controls/webview/webview.gyp:webview',
             '../ui/views/views.gyp:views',
           ],
           'sources': [
-            'browser/ui/app_list/app_list_view_delegate.cc',
-            'browser/ui/app_list/app_list_view_delegate.h',
-            'browser/ui/app_list/apps_model_builder.cc',
-            'browser/ui/app_list/apps_model_builder.h',
-            'browser/ui/app_list/chrome_app_list_item.h',
-            'browser/ui/app_list/extension_app_item.cc',
-            'browser/ui/app_list/extension_app_item.h',
-            'browser/ui/app_list/search_builder.cc',
-            'browser/ui/app_list/search_builder.h',
             'browser/ui/views/network_profile_bubble_view.cc',
             'browser/ui/views/network_profile_bubble_view.h',
             'browser/ui/webui/conflicts_ui.cc',
             'browser/ui/webui/conflicts_ui.h',
           ],
           'sources!': [
             'browser/ui/certificate_dialogs.cc',
             'browser/ui/certificate_dialogs.h',
             'browser/ui/crypto_module_password_dialog.cc',
             'browser/ui/crypto_module_password_dialog_nss.cc',
@@ skipping 79 matching lines @@
             ['include', '^browser/ui/views/select_file_dialog_extension_factory.cc'],
             ['include', '^browser/ui/views/select_file_dialog_extension_factory.h'],
           ]}, {
           'sources/': [
             ['exclude', '^browser/ui/views/select_file_dialog_extension.cc'],
             ['exclude', '^browser/ui/views/select_file_dialog_extension.h'],
             ['exclude', '^browser/ui/views/select_file_dialog_extension_factory.cc'],
             ['exclude', '^browser/ui/views/select_file_dialog_extension_factory.h'],
           ]}
         ],
+        ['enable_app_list==1', {
+          'dependencies': [
+            '../ui/app_list/app_list.gyp:app_list',
+          ],
+          'sources': [
+            'browser/ui/app_list/app_list_view_delegate.cc',

[sky, 2012/11/14 15:54:57]

How come we don't do the normal pattern of listing the files with all other
files and excluding where not appropriate? See comment on line 67.

[tapted, 2012/11/14 22:35:37]

I flip-flopped around this a bit in earlier patchsets, since it wasn't the prior
method for app_list.. But with changes to startup_browser_creator_impl.cc the
excludes are much cleaner, so I'm fully sold. Done.

+            'browser/ui/app_list/app_list_view_delegate.h',
+            'browser/ui/app_list/apps_model_builder.cc',
+            'browser/ui/app_list/apps_model_builder.h',
+            'browser/ui/app_list/chrome_app_list_item.h',
+            'browser/ui/app_list/extension_app_item.cc',
+            'browser/ui/app_list/extension_app_item.h',
+            'browser/ui/app_list/search_builder.cc',
+            'browser/ui/app_list/search_builder.h',
+          ]
+        }, { # else: enable_app_list==0
+          'sources!': [
+            'browser/ui/views/app_list/app_list_controller_win.cc'
+          ]
+        }],
       ],
     },
   ],
 }