Have jni_generator.gypi specify include_dirs to dependent targets.

chrome/chrome_browser.gypi

Index: chrome/chrome_browser.gypi
diff --git a/chrome/chrome_browser.gypi b/chrome/chrome_browser.gypi
index bdda04b9a42c3912bd1334f74015a9ba1244faa3..e0c6fa3c2ec7ebe3d76247331b98000d10e65a50 100644
--- a/chrome/chrome_browser.gypi
+++ b/chrome/chrome_browser.gypi
@@ -3156,6 +3156,9 @@
             '../components/components.gyp:web_modal',
             '../third_party/libusb/libusb.gyp:libusb',
           ],
+          'export_dependent_settings': [

[digit1, 2013/12/16 16:25:09]

Interesting, can you add a comment explaining why this is needed here. I must
say it's a bit surprising to see here.

[blundell, 2013/12/16 16:32:47]

I included it because chrome_android_core (in chrome_android.gypi) depends on
this target and I assumed that chrome_android_core included the headers from
chrome_browser_jni_headers (since chrome_android_core had previously declared
the jni gen dir from chrome_browser_jni_headers as an include_dir). Now that I
look at the files in chrome_android_core, however, I don't think that they make
any such inclusions...probably those include_dirs are just unnecessary at this
point. I'll try another build without this export.

On 2013/12/16 16:25:09, digit1 wrote:

+            'chrome_browser_jni_headers',
+          ],
           'sources': [
             'browser/sessions/in_memory_tab_restore_service.cc',
             'browser/sessions/in_memory_tab_restore_service.h',