vulkaninfo: Fix quoting strings in json output

When adding a value_description, the json output would incorrectly quote
the PrintKeyString output, leading to invalid json being generated.

icd: Set MockICD driver version to header version

build: Update to header 1.3.235

- Update known-good
- Generate source

cmake: Use new VulkanHeaders cmake support

build: Update to header 1.3.234

- Update known-good
- Generate source

build: Compile vkcube and other demos, even when cross-compiling

When building software for a non-x86 processor on an x86 build machine,
or building 32-bit software on a 64-bit build machine, it's still useful
to compile demo programs that can be copied into the target environment
and run there, even if they won't necessarily be runnable on the build
machine.

This also addresses a build failure when cross-compiling and attempting
to leave the demos enabled:

    CMake Error at cube/CMakeLists.txt:274 (install):
      install TARGETS given target "vkcube" which does not exist.

(and similar for the other demos)

Resolves: https://github.com/KhronosGroup/Vulkan-Tools/issues/592
Signed-off-by: Simon McVittie <smcv@collabora.com>

build: Update to header 1.3.233

- Update known-good
- Generate source

vulkaninfo: Fix VkShaderStageFlagBits expansion

Fix bug where SHADER_STAGE_ALL_GRAPHICS and SHADER_STAGE_ALL were
showing up in the list of set bits for
VkPhysicalDeviceSubgroupProperties::supportedStages. When expanding a
FlagBits value we should only consider the single-bit options.

vulkaninfo: Add Driver Version handling

Previously, driverVersion fields were treated as hex values, when in
reality each vendor has their own specific format. This commit adds
parsers for Nvidia, Intel (windows only), and relies on the Vulkan
API format for everyone else. AMD uses the Vulkan API format. This
change applies only to the summary, text, and html output.

Add ostream<< operators for VulkanVersion and VkConformanceVersion,
rather than having bespoke to_string functions.

When printing versions, use the major.minor.patch as the primary output
and put the uint32_t form of the version in parenthesis. This moves the
more useful version info into the spotlight, putting the less useful in
parenthesis.

Rework the way secondary values (stuff in parenthesis after a value)
are printed.

vulkaninfo: Improve array printing

Make the autogen use for loops instead of outputing printers for each
array element, since its needlessly verbose.

Make UUID and LUID printers use operator<< instead of to_string. This
simplifies the logic and makes sure that the array is printed correctly
everywhere, rather than requiring calling code to use to_string_XX()
where needed.

vulkaninfo: Escape strings in JSON output

JSON output breaks if escape sequences are in the various strings
output by drivers but aren't properly handles. This commit scans
over each string and inserts backslashes when necessary.

build: Update to header 1.3.232

- Update known-good
- Generate source

cmake: Sync FindVulkanHeaders.cmake with VVL

This picks up a few fixes, notably
https://github.com/KhronosGroup/Vulkan-ValidationLayers/pull/1393 which
helps with cross-compiling.

cmake: Remove illegal CMake code

include(FindPkgConfig) technically works but is very wrong.

cmake: Suppress Wayland warnings

The warnings are purely semantic.

docs: Explain CMAKE_EXPORT_COMPILE_COMMANDS

cmake: Remove USE_CCACHE

Update docs with modern CMake approach

cmake: Remove outdate code

Setting CMAKE_MACOS_RPATH is not needed given our cmake minimum

vulkaninfo: Fix validation errors

Required checking for surface support of a given physical device as well as
adding the correct Image usage for transient images.

ci: Use MACOS_DEPLOYMENT_TARGET

- This approach matches what our SDK release process expect
  - Since setting CMAKE_MACOS_DEPLOYMENT_TARGET ignores env variable which SDK
release process uses, which could will cause issues in the future, if
the version is changed
- CMAKE_MACOS_DEPLOYMENT_TARGET doesn't propogate to dependencies via
  update_deps.py

Updated MoltenVK to 1.2.0

cubepp: Fix presentKHR assert

Currently cubepp fails on Debug builds and triggers an assert.

This avoids the assert by calling the appropriate presentKHR.

build: Update to header 1.3.231

- Update known-good
- Generate source

cube: silence -DNDEBUG warning for KHR_display after 27c21248221f

cube/cube.c:3122:14: error: variable 'result' set but not used [-Werror,-Wunused-but-set-variable]
    VkResult result;
             ^

build: Update to header 1.3.230

- Update known-good
- Generate source

build: Update to header 1.3.229

- Update known-good
- Generate source

build: Update to header 1.3.228

- Update known-good
- Generate source

build: Update to header 1.3.227

- Update known-good
- Generate source

build: Also include XCB header path

Move BUILD.gn values into the scope that uses them

There are several values that in the build file that are only used when
the platform is not Android, but which are declared outside the
`if(!is_android)` block. This causes unused variable errors when
attempting to build chromium on Android with a dependency on this repo.

By moving the definitions into the conditional block the error is
resolved.

build: Update to header 1.3.226

- Update known-good
- Generate source

vulkaninfo/cube: added support for OpenBSD

build: Update to header 1.3.225

- Update known-good
- VkVideoProfileKHR -> VkVideoProfileInfoKHR
- VkVideoProfilesKHR -> VkVideoProfileListInfoKHR
- Generate source

Updated known good version of MoltenVK

build: Update to header 1.3.224

- Update known-good
- Generate source

vulkaninfo: Prevent infinite loops

GetVectorInit could get stuck in an infinite loop if the underlying API call eternally returned
VK_INCOMPLETE. This commit makes GetVectorInit stop after 3 tries.

build: Update to header 1.3.223

- Update known-good
- Generate source

ci: Fix github android build

The ANDROID_NDK_HOME env var seems to have disappeared with the
deprecation of ndk-bundle:
https://github.com/actions/virtual-environments/issues/5879

vulkaninfo: include wayland-client because its functions are used

"Include what you use": vulkaninfo.h calls wl_registry_bind, thus
ought to include wayland-client.h.

Set usage on transient image based on format

Avoids setting the color attachment bit for depth stencil formats to prevent crash

build: Update to header 1.3.222

- Update known-good
- Generate source

build: Update to header 1.3.221

- Update known-good
- Generate source

build: Update to header 1.3.220

- Update known-good
- Generate source

build: Update to header 1.3.219

- Update mock_icd generator create function detection to only look for
  Create/Allocate at beginning of function name
- Update known-good
- Generate source

cmake: Use ${VULKAN_SDK} environment variable paths prior to the system default paths.

Log Surface Extension Error, but don't fail on it.

win32: use _set_abort_behavior from ucrt

build: Update to header 1.3.218

- Update known-good
- Generate source

build: Update to header 1.3.217

- Update known-good
- Generate source

Updated MoltenVK known good to 1.1.10

build: Update to header 1.3.216

- Update known-good
- Generate source

vkcubepp: Use correct extension for portability enumeration

vulkaninfo: Set sType in PhysDevGroupProps

This was recently discovered by new validation messages. The
VkPhysicalDeviceGroupProperties sType member needs to be initialized properly.

build: Update to header 1.3.215

- Update known-good
- Generate source

cube: add null check on wl_*_destroy functions

Encountered the crash on a compositor without keyboard, this seems to
fix it.

Avoid _set_abort_behavior on MINGW

build: Update to header 1.3.214

- Update known-good
- Generate source

ci: Build cube with multiple WSI options

cubepp: Fix VK_KHR_display compile error

Vulkan-Hpp Implicit-cast operators on vk::ResultValue were previously
deprecated and finally removed in the v1.3.213 headers. Fix the
resulting compiler error in cubepp by explicitly using the value member
of vk:ResultValue.

Change-Id: Ie441bd56dfb9dfe26e966e0ec3fb3139712a351a

build: Update to header 1.3.213

- Update known-good
- Generate source

ci: Remove fbactions/setup-winsdk

The default version 18362 started 404'ing and a new enough version is
present in the base image.

build: Update to header 1.3.212

- Update known-good
- Generate source

Updated MoltenVK tag

Now at latest known good release

docs: Replace references to DevSim with Profiles

Updated the docs to better reflect the behavior of vulkaninfo and
remove references to DevSim.

Guard chromium-specific configs in BUILD.gn

Skia is going to start using these GN files, but does not have chromium's //build. To get them to build, guard those references using `build_with_chromium`. A similar mechanism (with the same argument) is used throughout other third party dependencies.

mock_icd: Remove fallthroughs in switches

They cause unecessary warnings in some compilers and are not necessary for the
code to function. Replaced with if statements.

cube,vulkaninfo: Add portability_enumeration if present

Make vulkaninfo and vkcube/vkcubepp enable the portability enumeration bit if it is
available.

Show surface information on GGP platform

Factor out typemap_helper header in BUILD rules for reusability in Dawn without depending on entire icd lib.

build: Update to header 1.3.211

- Update known-good
- Generate source

build: Update to header 1.3.210

- Update known-good
- Generate source

cubepp: Fix wayland regression

cubepp: Fix regression in VK_KHR_display

build: Update to header 1.3.209

- Update known-good
- Generate source

vulkaninfo: Make default json filename unique

Use the device name and driver version in the default filename for profile
schema vulkaninfo outputs. Example output would be:
"VP_VULKANINFO_Driver_Name_100_3_4.json"

vulkaninfo: Replace devsim with Profile json output

Make vulkaninfo capable of printing Vulkan Profiles with the json output. This
replaces the current DevSim output and folds the portability schema output into
the regular JSON output.

Changes include:
* Printing to file by default if json output is chosen
* Cleaning up codegen for extension structs
* Fixing up the help message
* Consolidating the portability schema output into just json

cubepp: Add debug utils messenger

cubepp: Use uint32_t and Demo&

The width and height parameters don't need to be ints, use uint32_t for consistency.

Demo& is more idiomatic C++ than Demo*.

vkcubepp: Address feedback from code-review

cubepp: Copy cubes swapchain format selection

The differing logic led to vkcube choosing a different swapchain image format
to vkcubepp.

cubepp: Cleanup usage of C++ and vulkan.hpp

Many changes to better utilize C++ and vulkan.hpp's features.
Includes:
* range-for loop usage where appropriate
* References instead of pointers
* using std::vector and std::array
* Using C++11's default struct values
* Making use of vulkan.hpp functions which return vectors
* Passing objects by reference to vulkan.hpp instead of by pointer
* Rename in application title to vkcubepp

build: Update to header 1.3.208

- Update known-good
- Generate source

build: Update to header 1.3.207

- Update known-good
- Generate source

Update known_good.json

Updated MoltenVK to 1.1.8

build: Add install target for vkcube-wayland

build: Update to header 1.3.206

- Update known-good
- Generate source

Make device selection output constistent

Make vkcube print a string for device type, which is consistent with
vkcubepp.

vulkaninfo: Enable portability extension for device groups

Expand previous change to include device group creation. Also added some
VK_ENABLE_BETA_EXTENSIONS guards around the portability identifiers.

vulkaninfo: Enable portability subset extension if present

If this extension is present, it must be enabled when creating a VkDevice.

build: Update to header 1.3.205

- Update known-good
- Generate source

Updated MoltenVK known good tag to 1.1.7

vulkaninfo: remove redundant str() in script

vulkaninfo: Use """ where appropriate in script

vulkaninfo: Use single quote in generation script

vulkaninfo: Use constants in pythong script

vulkaninfo: Update generated copyright

vulkaninfo: Support structs from multiple extensions

VkPhysicalDeviceIDProperties can come from 1.1 but also from support for
multiple extensions. This makes the autogen properly check that one of the
required extensions is supported.

vulkaninfo: Make chain iterators optional

Some of the pNext chains didn't make use of the generic chain printers. Since they aren't
being used in code, it is good to disable the generation of those printers.

vulkaninfo: Remove set width in json functions

Set width does nothing inside json printers, removing it since it isn't necessary.

vulkaninfo: bitmask printers check for 0 value

Only generate the early exit if there doesn't exist an bitmask which has a
value of 0. For example, VK_RESOLVE_MODE_NONE has a value of 0, but it would
never be printed without this special case.