Have Chrome Frame build its .idl artifacts into a sub-folder of SHARED_INTERMEDIATE_DIR, a global...

chrome_frame/chrome_frame_launcher.gyp

 # 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.
 
 {
   'variables': {
     'chromium_code': 1,
 
     # Keep the archive builder happy.
     'chrome_personalization%': 1,
@@ skipping 52 matching lines @@
       'target_name': 'chrome_launcher',
       'type': 'executable',
       'dependencies': [
         '../breakpad/breakpad.gyp:breakpad_handler',
         '../chrome/chrome.gyp:chrome_version_header',
         '../google_update/google_update.gyp:google_update',
         'chrome_frame.gyp:chrome_frame_utils',
       ],
       'resource_include_dirs': [
         '<(INTERMEDIATE_DIR)',
         '<(SHARED_INTERMEDIATE_DIR)',

[grt (UTC plus 2), 2011/10/18 15:56:24]

i think this is here for version.h.  could you add a comment to that effect?

[robertshield, 2011/10/18 17:31:10]

Done.

       ],
       'sources': [
         'chrome_launcher_main.cc',
         'chrome_launcher_version.rc',
         'chrome_launcher.cc',
         'chrome_launcher.h',
         'update_launcher.cc',
         'update_launcher.h'
       ],
       'msvs_settings': {
@@ skipping 11 matching lines @@
       'type': 'executable',
       'dependencies': [
         '../breakpad/breakpad.gyp:breakpad_handler',
         '../chrome/chrome.gyp:chrome_version_header',
         'chrome_frame.gyp:chrome_frame_utils',
         'chrome_frame_helper_dll',
         'chrome_frame_helper_lib',
       ],
       'resource_include_dirs': [
         '<(INTERMEDIATE_DIR)',
         '<(SHARED_INTERMEDIATE_DIR)',

[grt (UTC plus 2), 2011/10/18 15:56:24]

same comment: i think these are all here because of the chrome_version_header
dependency.  it'd be cleaner if that target added '<(SHARED_INTERMEDIATE_DIR)'
as a resource_include_dir for direct dependents.  sigh.

[robertshield, 2011/10/18 17:31:10]

Yes, but as discussed that is not in the scope of this CL.

       ],
-      'include_dirs': [
-        # To allow including "chrome_tab.h"
-        '<(INTERMEDIATE_DIR)',
-        '<(INTERMEDIATE_DIR)/../chrome_frame',
-      ],
       'sources': [
         'chrome_frame_helper_main.cc',
         'chrome_frame_helper_version.rc',
       ],
       'msvs_settings': {
         'VCLinkerTool': {
           'OutputFile':
               '$(OutDir)\\$(ProjectName).exe',
           # Set /SUBSYSTEM:WINDOWS since this is not a command-line program.
           'SubSystem': '2',
         },
       },
     },
     {
       'target_name': 'chrome_frame_helper_dll',
       'type': 'shared_library',
       'dependencies': [
         '../chrome/chrome.gyp:chrome_version_header',
+        'chrome_frame.gyp:chrome_tab_idl',
         'chrome_frame_helper_lib',
       ],
+      'include_dirs': [

[grt (UTC plus 2), 2011/10/18 15:56:24]

this shouldn't be needed since chrome_tab_idl specifies it as a
direct_dependent_setting.

[robertshield, 2011/10/18 17:31:10]

Removed.

+        '<(SHARED_INTERMEDIATE_DIR)',
+      ],
       'resource_include_dirs': [
         '<(INTERMEDIATE_DIR)',
         '<(SHARED_INTERMEDIATE_DIR)',

[grt (UTC plus 2), 2011/10/18 15:56:24]

sigh

[robertshield, 2011/10/18 17:31:10]

I can make you a cake. Maybe that will make you feel better?

       ],
-      'include_dirs': [
-        # To allow including "chrome_tab.h"
-        '<(INTERMEDIATE_DIR)',
-        '<(INTERMEDIATE_DIR)/../chrome_frame',
-      ],
       'sources': [
         'bho_loader.cc',
         'bho_loader.h',
         'chrome_frame_helper_dll.cc',
         'chrome_frame_helper_dll.def',
         'chrome_frame_helper_version.rc',
-        '<(SHARED_INTERMEDIATE_DIR)/chrome_tab.h',
-        'chrome_tab.idl',
+        '<(SHARED_INTERMEDIATE_DIR)/chrome_frame/chrome_tab.h',
         'event_hooker.cc',
         'event_hooker.h',
         'iids.cc',
       ],
       'msvs_settings': {
         'VCLinkerTool': {
           'OutputFile': '$(OutDir)\\chrome_frame_helper.dll',
           'ProgramDatabaseFile': '$(OutDir)\\chrome_frame_helper_dll.pdb',
           # Set /SUBSYSTEM:WINDOWS since this is not a command-line program.
           'SubSystem': '2',
         },
       },
     },
     {
       'target_name': 'chrome_frame_helper_lib',
       'type': 'static_library',
       'dependencies': [
         '../chrome/chrome.gyp:chrome_version_header',
+        'chrome_frame.gyp:chrome_tab_idl',
+      ],
+      'include_dirs': [

[grt (UTC plus 2), 2011/10/18 15:56:24]

remove for same reason as above

[robertshield, 2011/10/18 17:31:10]

Done.

+        '<(SHARED_INTERMEDIATE_DIR)',
       ],
       'resource_include_dirs': [
         '<(INTERMEDIATE_DIR)',
         '<(SHARED_INTERMEDIATE_DIR)',

[grt (UTC plus 2), 2011/10/18 15:56:24]

more sighing

[robertshield, 2011/10/18 17:31:10]

The cake was a lie.

       ],
-      'include_dirs': [
-        # To allow including "chrome_tab.h"
-        '<(INTERMEDIATE_DIR)',
-        '<(INTERMEDIATE_DIR)/../chrome_frame',
-      ],
       'sources': [
         'chrome_frame_helper_util.cc',
         'chrome_frame_helper_util.h',
         'registry_watcher.cc',
         'registry_watcher.h',
-        '<(SHARED_INTERMEDIATE_DIR)/chrome_tab.h',
-        'chrome_tab.idl',
+        '<(SHARED_INTERMEDIATE_DIR)/chrome_frame/chrome_tab.h',
         'iids.cc',
       ],
       'msvs_settings': {
         'VCLinkerTool': {
           'AdditionalDependencies': [
             'shlwapi.lib',
           ],
         },
       },
     },
   ],
 }