Update min Vulkan version to 1.0.8.0, and fix various bugs

src/gpu/vk/GrVkImage.h

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef GrVkImage_DEFINED
 #define GrVkImage_DEFINED
 
@@ skipping 10 matching lines @@
     class Resource : public GrVkResource {
     public:
         enum Flags {
             kNo_Flags = 0,
             kLinearTiling_Flag = 0x01
         };
 
         VkImage        fImage;
         VkDeviceMemory fAlloc;
         Flags          fFlags;
+        VkFormat       fFormat;
 
-        Resource() : INHERITED(), fImage(VK_NULL_HANDLE), fAlloc(VK_NULL_HANDLE), fFlags(kNo_Flags) {}
+        Resource()
+            : INHERITED(),
+            fImage(VK_NULL_HANDLE),

[jvanverth1, 2016/04/21 13:25:24]

Nit: usually we put the commas before the variable, lined up with the colon.

[egdaniel, 2016/04/21 14:09:20]

Woops no idea why my mind did this opposite here.

+            fAlloc(VK_NULL_HANDLE),
+            fFlags(kNo_Flags),
+            fFormat(VK_FORMAT_UNDEFINED) {}
 
-        Resource(VkImage image, VkDeviceMemory alloc, Flags flags)
-            : fImage(image), fAlloc(alloc), fFlags(flags) {}
+        Resource(VkImage image, VkDeviceMemory alloc, Flags flags, VkFormat format)
+            : fImage(image), fAlloc(alloc), fFlags(flags), fFormat(format) {}
 
         ~Resource() override {}
     private:
         void freeGPUData(const GrVkGpu* gpu) const override;
 
         typedef GrVkResource INHERITED;
     };
 
     // for wrapped textures
     class BorrowedResource : public Resource {
     public:
-        BorrowedResource(VkImage image, VkDeviceMemory alloc, Flags flags)
-            : Resource(image, alloc, flags) {}
+        BorrowedResource(VkImage image, VkDeviceMemory alloc, Flags flags, VkFormat format)
+            : Resource(image, alloc, flags, format) {}
     private:
         void freeGPUData(const GrVkGpu* gpu) const override;
     };
 
     GrVkImage(const Resource* imageResource) : fResource(imageResource) {
         if (imageResource->fFlags & Resource::kLinearTiling_Flag) {
             fCurrentLayout = VK_IMAGE_LAYOUT_PREINITIALIZED;
         } else {
             fCurrentLayout = VK_IMAGE_LAYOUT_UNDEFINED;
         }
         imageResource->ref();
     }
 
     virtual ~GrVkImage();
 
     VkImage textureImage() const { return fResource->fImage; }
     VkDeviceMemory textureMemory() const { return fResource->fAlloc; }
     const Resource* resource() const { return fResource; }
     bool isLinearTiled() const {
         return SkToBool(fResource->fFlags & Resource::kLinearTiling_Flag);
     }
 
     VkImageLayout currentLayout() const { return fCurrentLayout; }
 
-    void setImageLayout(const GrVkGpu* gpu, VkImageLayout newLayout,
+    void setImageLayout(const GrVkGpu* gpu,
+                        VkImageLayout newLayout,
                         VkAccessFlags srcAccessMask,
                         VkAccessFlags dstAccessMask,
                         VkPipelineStageFlags srcStageMask,
                         VkPipelineStageFlags dstStageMask,
                         bool byRegion);
 
     struct ImageDesc {
         VkImageType         fImageType;
         VkFormat            fFormat;
         uint32_t            fWidth;
@@ skipping 23 matching lines @@
     void releaseImage(const GrVkGpu* gpu);
     void abandonImage();
 
     const Resource* fResource;
 
     VkImageLayout   fCurrentLayout;
 
 };
 
 #endif