Changeset 138673 in webkit

Timestamp:

Jan 2, 2013, 4:08:50 PM (13 years ago)

Author:

pilgrim@chromium.org

Message:

[Chromium] Add Platform/chromium to include_dirs
​https://bugs.webkit.org/show_bug.cgi?id=105955

Reviewed by James Robinson.

Currently the headers in WebKit/chromium/public/Web*.h can not
include headers from the new Platform/chromium/ directory with the
"#include <public/WebFoo.h>" syntax, because the include_dirs for
dependent builds (like Chromium-within-WebKit) do not include the
new Platform/chromium/ directory. This patch adds that capability
without managing to break anything else.

• WebKit.gyp:

Location:

trunk/Source/WebKit/chromium

Files:

• ChangeLog (modified) (1 diff)
• WebKit.gyp (modified) (2 diffs)

trunk/Source/WebKit/chromium/ChangeLog

@@ +1 @@
+2013-01-02  Mark Pilgrim  <pilgrim@chromium.org>
+
+        [Chromium] Add Platform/chromium to include_dirs
+        https://bugs.webkit.org/show_bug.cgi?id=105955
+
+        Reviewed by James Robinson.
+
+        Currently the headers in WebKit/chromium/public/Web*.h can not
+        include headers from the new Platform/chromium/ directory with the
+        "#include <public/WebFoo.h>" syntax, because the include_dirs for
+        dependent builds (like Chromium-within-WebKit) do not include the
+        new Platform/chromium/ directory. This patch adds that capability
+        without managing to break anything else.
+
+        * WebKit.gyp:
+
 2013-01-02  Alec Flett  <alecflett@chromium.org>
 

trunk/Source/WebKit/chromium/WebKit.gyp

@@ -855 +855 @@
                 }],
             ],
+            'direct_dependent_settings': {
+                'include_dirs': [
+                    '../../Platform/chromium/',
+                ],
+            },
             'target_conditions': [
                 ['OS=="android"', {
@@ -903 +908 @@
                         'public',
                         '../../WebCore/testing/v8', # for WebCoreTestSupport.h, needed to link in window.internals code.
+                        '../../Platform/chromium/',
                     ],
                     'sources': [