Changed mac and win Official Chrome build to drop default app CRXs in the build/installer output.

chrome/chrome_dll.gypi

Index: chrome/chrome_dll.gypi
===================================================================
--- chrome/chrome_dll.gypi	(revision 99174)
+++ chrome/chrome_dll.gypi	(working copy)
@@ -163,6 +163,17 @@
                 },
               },
               'conditions': [
+                ['buildtype=="Official" and branding=="Chrome"', {
+                  'copies': [
+                    {
+                      'destination': '<(PRODUCT_DIR)/default_apps',

[Mark Mentovai, 2011/09/01 20:53:38]

Why do you need to copy things to <(PRODUCT_DIR)/default_apps? I don’t think you
ever reference anything in this location.

[SteveT, 2011/09/01 21:19:57]

This is the expected location for win, according to 7811020. I'll add a comment
here for Robert about how these are pulled in by chrome.release, and also
explicitly mention that this is for Windows.

+                      'files': [
+                        'browser/resources/default_apps/gmail.crx',
+                        'browser/resources/default_apps/youtube.crx',
+                      ],
+                    },
+                  ],
+                }],
                 ['incremental_chrome_dll==1 and optimize_with_syzygy==0', {
                   # Linking to a different directory and then hardlinking back
                   # to OutDir is a workaround to avoid having the .ilk for
@@ -590,6 +601,18 @@
                 },
               ],
               'conditions': [
+                ['buildtype=="Official" and branding=="Chrome"', {
+                  'copies': [
+                    {
+                      'destination':
+                          '<(PRODUCT_DIR)/$(CONTENTS_FOLDER_PATH)/Default Apps',

[Mark Mentovai, 2011/09/01 20:53:38]

I commented in 7811020 about this. I don’t think the path you’re using there is
correct.

[SteveT, 2011/09/01 21:19:57]

Discussed with Cait. I thought they had implemented that bit with the correct
location, but that's not the case yet. They'll address that before committing.

+                      'files': [
+                        'browser/resources/default_apps/gmail.crx',
+                        'browser/resources/default_apps/youtube.crx',
+                      ],
+                    },
+                  ],
+                }],
                 ['mac_breakpad==1', {
                   'variables': {
                     # A real .dSYM is needed for dump_syms to operate on.