Move navigation interception component to content/components

content/components/navigation_interception/navigation_interception.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': 'navigation_interception',

[Jói, 2012/11/05 16:02:01]

There was a recent discussion on chromium-dev@
(https://groups.google.com/a/chromium.org/forum/?fromgroups=#!topic/chromium-d...)
and I think the consensus was that until we have better tooling support, .gypi
files should be colocated with the .gyp file that includes them.

jam@, do you agree that is the consensus for now?  If so we should move this to
content/ at least until the tools are better, and for now can add per-file
OWNERS for this .gypi to content/OWNERS.

[John Knottenbelt, 2012/11/05 16:15:14]

I've been struggling with the pathname relativization a fair bit, so this sounds
good to me, if it's ok to have another gypi in content/.

Please advise how I should rename this file, I'm thinking something like
content_components_navigation_interception.gypi ?

On 2012/11/05 16:02:01, Jói wrote:

[jam, 2012/11/05 20:33:13]

sgtm

       'type': 'static_library',
       'dependencies': [
-        '../../../base/base.gyp:base',
-        '../../../content/content.gyp:content_browser',
-        '../../../content/content.gyp:content_common',
-        '../../../net/net.gyp:net',
+        '../base/base.gyp:base',
+        '../net/net.gyp:net',
+        'content_browser',
+        'content_common',
       ],
       'include_dirs': [
-        '../../../..',
-        '../../../../skia/config',
-        '<(SHARED_INTERMEDIATE_DIR)/chrome/browser_component',
+        '../../..',
+        '../../../skia/config',
+        '<(SHARED_INTERMEDIATE_DIR)/navigation_interception',
 
       ],
       'sources': [
         'intercept_navigation_resource_throttle.cc',
         'intercept_navigation_resource_throttle.h',
       ],
       'conditions': [
         ['OS=="android"', {
           'dependencies': [
-            'browser_component_jni_headers',
+              'navigation_interception_jni_headers',
           ],
           'sources': [
             'component_jni_registrar.cc',
             'component_jni_registrar.h',
             'intercept_navigation_delegate.cc',
             'intercept_navigation_delegate.h',
           ],
         }],
       ],
     },
   ],
   'conditions': [
     ['OS=="android"', {
       'targets': [
         {
           'target_name': 'navigation_interception_java',
           'type': 'none',
           'dependencies': [
-            '../../../base/base.gyp:base',
+            '../base/base.gyp:base',
           ],
           'variables': {
             'package_name': 'navigation_interception',
             'java_in_dir': 'java',
           },
-          'includes': [ '../../../../build/java.gypi' ],
+          'includes': [ '../../../build/java.gypi' ],
+        },
+        {
+          'target_name': 'navigation_interception_jni_headers',
+          'type': 'none',
+          'sources': [
+            'java/src/org/chromium/content/components/navigation_interception/InterceptNavigationDelegate.java',
+          ],
+          'variables': {
+            # We need ../.. due to path name relativization of jni_gen_dir.
+            'jni_gen_dir': '../../navigation_interception',
+          },
+          'includes': [ '../../../build/jni_generator.gypi' ],
         },
       ],
     }],
   ],
 }