From: djsollen <djsollen@google.com>
Date: Wed, 6 Jul 2016 16:17:35 +0000 (-0700)
Subject: Compile Vulkan when building for the Android framework.
X-Git-Tag: accepted/tizen/5.0/unified/20181102.025319~116^2~965
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d5fd2ff49067efebcd7e28fdc787ec21b2176152;p=platform%2Fupstream%2FlibSkiaSharp.git

Compile Vulkan when building for the Android framework.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2129523002

Review-Url: https://codereview.chromium.org/2129523002
---

diff --git a/gyp/common_variables.gypi b/gyp/common_variables.gypi
index 969e484..5163d88 100644
--- a/gyp/common_variables.gypi
+++ b/gyp/common_variables.gypi
@@ -82,6 +82,7 @@
           'skia_chrome_utils%': 0,
           'skia_use_android_framework_defines%': 1,
           'skia_use_system_json%': 1,
+          'skia_vulkan%': 1,
         }, {
           'skia_os%': '<(skia_os)',
           'skia_chrome_utils%': 1,
diff --git a/gyp/gpu.gyp b/gyp/gpu.gyp
index 874db5c..c17789a 100644
--- a/gyp/gpu.gyp
+++ b/gyp/gpu.gyp
@@ -181,6 +181,12 @@
           },
         }],
         [ 'skia_vulkan', {
+          'link_settings': {
+            'libraries': [ '<(vulkan_lib_name)', ],
+          },
+          'dependencies': [
+            'shaderc.gyp:shaderc_combined',
+          ],
           'conditions': [
             [ 'skia_os == "win"', {
              'variables': {
@@ -225,21 +231,32 @@
                 'library_dirs': [ '<(vulkan_sdk_path)/lib', ],
               },
             }],
+            [ 'skia_android_framework', {
+              'include_dirs': [
+                'frameworks/native/vulkan/include',
+                '../tools/viewer/sk_app',
+                '../tools/viewer/sk_app/android',
+              ],
+              'direct_dependent_settings': {
+                'include_dirs': [
+                  'frameworks/native/vulkan/include',
+                  '../tools/viewer/sk_app',
+                  '../tools/viewer/sk_app/android',
+                ],
+              },
+              'dependencies!': [
+                'shaderc.gyp:shaderc_combined',
+              ],
+              'sources': [
+                # the gyp -> android.mk generator doesn't seem to like cpp files
+                # in directories outside of src, bench, or dm.  Until this gets fixed
+                # I just start the path in the src directory.
+                '<(skia_src_path)/../tools/viewer/sk_app/WindowContext.cpp', 
+                '<(skia_src_path)/../tools/viewer/sk_app/VulkanWindowContext.cpp',
+                '<(skia_src_path)/../tools/viewer/sk_app/android/VulkanWindowContext_android.cpp'
+              ],
+            }],
           ],
-          'dependencies': [
-            'shaderc.gyp:shaderc_combined',
-          ],
-          'include_dirs': [
-            '../third_party/externals/shaderc2/libshaderc/include',
-          ],
-          'direct_dependent_settings': {
-            'include_dirs': [
-              '../third_party/externals/shaderc2/libshaderc/include',
-            ],
-          },
-          'link_settings': {
-            'libraries': [ '<(vulkan_lib_name)', ],
-          },
         }, {
           'sources!': [
             '<@(skgpu_vk_sources)',
diff --git a/gyp/shaderc.gyp b/gyp/shaderc.gyp
index 31fd897..4a85411 100644
--- a/gyp/shaderc.gyp
+++ b/gyp/shaderc.gyp
@@ -17,6 +17,11 @@
         'libglslangosdependent',
         'libglslang',
       ],
+      'direct_dependent_settings': {
+        'include_dirs': [
+          '../third_party/externals/shaderc2/libshaderc/include',
+        ],
+      },
     },
     {
       'target_name': 'libshaderc',
diff --git a/platform_tools/android/gyp_gen/gypd_parser.py b/platform_tools/android/gyp_gen/gypd_parser.py
index e992637..082d264 100644
--- a/platform_tools/android/gyp_gen/gypd_parser.py
+++ b/platform_tools/android/gyp_gen/gypd_parser.py
@@ -89,7 +89,7 @@ def parse_dictionary(var_dict, d, current_target_name, dest_dir):
     var_dict.LOCAL_CPPFLAGS.add(flag)
 
   for include in d.get('include_dirs', []):
-    if include.startswith('external'):
+    if include.startswith('external') or include.startswith('frameworks'):
       # This path is relative to the Android root. Leave it alone.
       rel_include = include
     else:
diff --git a/src/gpu/vk/GrVkPipelineStateBuilder.h b/src/gpu/vk/GrVkPipelineStateBuilder.h
index 5a4cfb8..2f78116 100644
--- a/src/gpu/vk/GrVkPipelineStateBuilder.h
+++ b/src/gpu/vk/GrVkPipelineStateBuilder.h
@@ -15,7 +15,6 @@
 #include "GrVkUniformHandler.h"
 #include "GrVkVaryingHandler.h"
 
-#include "shaderc/shaderc.h"
 #include "vk/GrVkDefines.h"
 
 class GrVkGpu;
diff --git a/src/gpu/vk/GrVkProgramDesc.cpp b/src/gpu/vk/GrVkProgramDesc.cpp
index 4619bcf..2154af6 100644
--- a/src/gpu/vk/GrVkProgramDesc.cpp
+++ b/src/gpu/vk/GrVkProgramDesc.cpp
@@ -17,8 +17,6 @@
 #include "glsl/GrGLSLFragmentShaderBuilder.h"
 #include "glsl/GrGLSLCaps.h"
 
-#include "shaderc/shaderc.h"
-
 static void add_texture_key(GrProcessorKeyBuilder* b, const GrProcessor& proc,
                             const GrGLSLCaps& caps) {
     int numTextures = proc.numTextures();