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

chrome_frame/chrome_frame.gyp

Index: chrome_frame/chrome_frame.gyp
===================================================================
--- chrome_frame/chrome_frame.gyp	(revision 105823)
+++ chrome_frame/chrome_frame.gyp	(working copy)
@@ -62,7 +62,7 @@
       'type': 'none',
       'msvs_settings': {
         'VCMIDLTool': {
-          'OutputDirectory': '<(SHARED_INTERMEDIATE_DIR)',
+          'OutputDirectory': '<(SHARED_INTERMEDIATE_DIR)/chrome_frame',
         },
       },
       'sources': [
@@ -89,7 +89,7 @@
         'locales/locales.gyp:*',
       ],
       'sources': [
-        '<(SHARED_INTERMEDIATE_DIR)/chrome_tab.h',
+        '<(SHARED_INTERMEDIATE_DIR)/chrome_frame/chrome_tab.h',
         'chrome_frame_unittest_main.cc',
         'chrome_launcher.cc',
         'chrome_launcher.h',
@@ -249,7 +249,7 @@
         'test/win_event_receiver.cc',
         'test/win_event_receiver.h',
         'chrome_launcher_version.rc',
-        '<(SHARED_INTERMEDIATE_DIR)/chrome_tab.h',
+        '<(SHARED_INTERMEDIATE_DIR)/chrome_frame/chrome_tab.h',
         'test_utils.cc',
         'test_utils.h',
       ],
@@ -324,7 +324,7 @@
         '../chrome/test/base/chrome_process_util.cc',
         '../chrome/test/base/chrome_process_util.h',
         '../chrome/test/ui/ui_test.cc',
-        '<(SHARED_INTERMEDIATE_DIR)/chrome_tab.h',
+        '<(SHARED_INTERMEDIATE_DIR)/chrome_frame/chrome_tab.h',
         'test/chrome_frame_test_utils.cc',
         'test/chrome_frame_test_utils.h',
         'test/perf/chrome_frame_perftest.cc',
@@ -414,7 +414,7 @@
         'test/net/test_automation_provider.h',
         'test/net/test_automation_resource_message_filter.cc',
         'test/net/test_automation_resource_message_filter.h',
-        '<(SHARED_INTERMEDIATE_DIR)/chrome_tab.h',
+        '<(SHARED_INTERMEDIATE_DIR)/chrome_frame/chrome_tab.h',
         '<(SHARED_INTERMEDIATE_DIR)/ui/ui_resources/ui_resources.rc',
         'test_utils.cc',
         'test_utils.h',
@@ -485,7 +485,7 @@
         'test/simulate_input.h',
         'test/win_event_receiver.cc',
         'test/win_event_receiver.h',
-        '<(SHARED_INTERMEDIATE_DIR)/chrome_tab.h',
+        '<(SHARED_INTERMEDIATE_DIR)/chrome_frame/chrome_tab.h',
         '../base/test/test_file_util_win.cc',
         '../chrome/test/automation/proxy_launcher.cc',
         '../chrome/test/automation/proxy_launcher.h',
@@ -561,7 +561,7 @@
         'test/test_with_web_server.h',
         'test/win_event_receiver.cc',
         'test/win_event_receiver.h',
-        '<(SHARED_INTERMEDIATE_DIR)/chrome_tab.h',
+        '<(SHARED_INTERMEDIATE_DIR)/chrome_frame/chrome_tab.h',
         'chrome_tab.idl',
         'renderer_glue.cc',
         'test_utils.cc',
@@ -570,8 +570,6 @@
       'include_dirs': [
         '<(DEPTH)/third_party/wtl/include',
         '<(DEPTH)/breakpad/src',
-        # To allow including "chrome_tab.h"
-        '<(INTERMEDIATE_DIR)',
       ],
       'resource_include_dirs': [
         '<(INTERMEDIATE_DIR)',
@@ -627,6 +625,7 @@
       ],
       'include_dirs': [
         # To allow including "version.h"
+        # TODO(grt): remove this as per http://crbug.com/99368
         '<(SHARED_INTERMEDIATE_DIR)',
       ],
       'sources': [
@@ -675,7 +674,7 @@
         'chrome_protocol.cc',
         'chrome_protocol.h',
         'chrome_protocol.rgs',
-        '<(SHARED_INTERMEDIATE_DIR)/chrome_tab.h',
+        '<(SHARED_INTERMEDIATE_DIR)/chrome_frame/chrome_tab.h',
         'com_message_event.cc',
         'com_message_event.h',
         'com_type_info_holder.cc',
@@ -745,7 +744,6 @@
         '../third_party/active_doc/ole_document_impl.h',
       ],
       'include_dirs': [
-        '<(INTERMEDIATE_DIR)/../chrome_frame',
         '<(DEPTH)/third_party/wtl/include',
       ],
       'conditions': [
@@ -859,7 +857,7 @@
         'chrome_frame_reporting.h',
         'chrome_tab.cc',
         'chrome_tab.def',
-        '<(SHARED_INTERMEDIATE_DIR)/chrome_tab.h',
+        '<(SHARED_INTERMEDIATE_DIR)/chrome_frame/chrome_tab.h',
         # FIXME(slightlyoff): For chrome_tab.tlb. Giant hack until we can
         #   figure out something more gyp-ish.
         'resources/tlb_resource.rc',
@@ -869,10 +867,12 @@
         'resource.h',
       ],
       'include_dirs': [

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

this shouldn't be here.  ideally, there should be no occurrences of
'<(SHARED_INTERMEDIATE_DIR)/chrome_frame' in this file aside from setting the
output dir for midl.exe, since all consumers use "chrome_frame/" as part of the
include path.  this specific include_dirs section shouldn't be needed on account
of the direct_dependent_settings on chrome_tab_idl.

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

Done.

-        # For chrome_tab.h
-        '<(SHARED_INTERMEDIATE_DIR)',
-        '<(INTERMEDIATE_DIR)/../npchrome_frame',
+        '<(SHARED_INTERMEDIATE_DIR)/chrome_frame',
       ],
+      'resource_include_dirs': [
+        # To pick up chrome_tab.tlb
+        '<(SHARED_INTERMEDIATE_DIR)/chrome_frame'

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

how about editing resources/tlb_resource.rc to change the TYPELIB to use
"chrome_frame/chrome_tab.tlb" and either remove "/chrome_frame" from this
string, or remove resource_include_dirs altogether and switch to using
direct_dependent_settings to set this dir way up in chrome_tab_idl?

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

Done (the former).

+      ],
       'conditions': [
         ['OS=="win"', {
           # NOTE(slightlyoff):