From c15199b814c394eea98c6f89e0a0f4fa7e316f56 Mon Sep 17 00:00:00 2001
From: Jason Jarvis <jazzay@gmail.com>
Date: Sun, 8 May 2016 15:17:43 -0700
Subject: [PATCH 1/3] Upgraded to latest SkiaSharp nuget.

---
 src/Skia/Perspex.Skia.Android/Perspex.Skia.Android.csproj | 2 +-
 src/Skia/Perspex.Skia.Android/packages.config             | 2 +-
 src/Skia/Perspex.Skia.Desktop/Perspex.Skia.Desktop.csproj | 6 +++---
 src/Skia/Perspex.Skia.Desktop/packages.config             | 2 +-
 src/Skia/Perspex.Skia.iOS/Perspex.Skia.iOS.csproj         | 2 +-
 src/Skia/Perspex.Skia.iOS/packages.config                 | 2 +-
 6 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/Skia/Perspex.Skia.Android/Perspex.Skia.Android.csproj b/src/Skia/Perspex.Skia.Android/Perspex.Skia.Android.csproj
index 129da56ff8..469fcddc79 100644
--- a/src/Skia/Perspex.Skia.Android/Perspex.Skia.Android.csproj
+++ b/src/Skia/Perspex.Skia.Android/Perspex.Skia.Android.csproj
@@ -40,7 +40,7 @@
     <Reference Include="Mono.Android" />
     <Reference Include="mscorlib" />
     <Reference Include="SkiaSharp, Version=1.49.0.0, Culture=neutral, processorArchitecture=MSIL">
-      <HintPath>..\..\..\packages\SkiaSharp.1.49.2.0-beta\lib\MonoAndroid\SkiaSharp.dll</HintPath>
+      <HintPath>..\..\..\packages\SkiaSharp.1.49.2.1-beta\lib\MonoAndroid\SkiaSharp.dll</HintPath>
       <Private>True</Private>
     </Reference>
     <Reference Include="System" />
diff --git a/src/Skia/Perspex.Skia.Android/packages.config b/src/Skia/Perspex.Skia.Android/packages.config
index c0f3883e68..050c885479 100644
--- a/src/Skia/Perspex.Skia.Android/packages.config
+++ b/src/Skia/Perspex.Skia.Android/packages.config
@@ -1,4 +1,4 @@
 <?xml version="1.0" encoding="utf-8"?>
 <packages>
-  <package id="SkiaSharp" version="1.49.2.0-beta" targetFramework="monoandroid60" />
+  <package id="SkiaSharp" version="1.49.2.1-beta" targetFramework="monoandroid50" />
 </packages>
\ No newline at end of file
diff --git a/src/Skia/Perspex.Skia.Desktop/Perspex.Skia.Desktop.csproj b/src/Skia/Perspex.Skia.Desktop/Perspex.Skia.Desktop.csproj
index 499963c1d2..5ea8021734 100644
--- a/src/Skia/Perspex.Skia.Desktop/Perspex.Skia.Desktop.csproj
+++ b/src/Skia/Perspex.Skia.Desktop/Perspex.Skia.Desktop.csproj
@@ -57,7 +57,7 @@
   </PropertyGroup>
   <ItemGroup>
     <Reference Include="SkiaSharp, Version=1.49.0.0, Culture=neutral, processorArchitecture=MSIL">
-      <HintPath>..\..\..\packages\SkiaSharp.1.49.2.0-beta\lib\net45\SkiaSharp.dll</HintPath>
+      <HintPath>..\..\..\packages\SkiaSharp.1.49.2.1-beta\lib\net45\SkiaSharp.dll</HintPath>
       <Private>True</Private>
     </Reference>
     <Reference Include="System" />
@@ -116,12 +116,12 @@
   <Import Project="..\Perspex.Skia\Perspex.Skia.projitems" Label="Shared" />
   <Import Project="..\..\Shared\RenderHelpers\RenderHelpers.projitems" Label="Shared" />
   <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
-  <Import Project="..\..\..\packages\SkiaSharp.1.49.2.0-beta\build\net45\SkiaSharp.targets" Condition="Exists('..\..\..\packages\SkiaSharp.1.49.2.0-beta\build\net45\SkiaSharp.targets')" />
+  <Import Project="..\..\..\packages\SkiaSharp.1.49.2.1-beta\build\net45\SkiaSharp.targets" Condition="Exists('..\..\..\packages\SkiaSharp.1.49.2.1-beta\build\net45\SkiaSharp.targets')" />
   <Target Name="EnsureNuGetPackageBuildImports" BeforeTargets="PrepareForBuild">
     <PropertyGroup>
       <ErrorText>This project references NuGet package(s) that are missing on this computer. Use NuGet Package Restore to download them.  For more information, see http://go.microsoft.com/fwlink/?LinkID=322105. The missing file is {0}.</ErrorText>
     </PropertyGroup>
-    <Error Condition="!Exists('..\..\..\packages\SkiaSharp.1.49.2.0-beta\build\net45\SkiaSharp.targets')" Text="$([System.String]::Format('$(ErrorText)', '..\..\..\packages\SkiaSharp.1.49.2.0-beta\build\net45\SkiaSharp.targets'))" />
+    <Error Condition="!Exists('..\..\..\packages\SkiaSharp.1.49.2.1-beta\build\net45\SkiaSharp.targets')" Text="$([System.String]::Format('$(ErrorText)', '..\..\..\packages\SkiaSharp.1.49.2.1-beta\build\net45\SkiaSharp.targets'))" />
   </Target>
   <!-- To modify your build process, add your task inside one of the targets below and uncomment it. 
        Other similar extension points exist, see Microsoft.Common.targets.
diff --git a/src/Skia/Perspex.Skia.Desktop/packages.config b/src/Skia/Perspex.Skia.Desktop/packages.config
index aa192941cd..a2d6225904 100644
--- a/src/Skia/Perspex.Skia.Desktop/packages.config
+++ b/src/Skia/Perspex.Skia.Desktop/packages.config
@@ -1,4 +1,4 @@
 <?xml version="1.0" encoding="utf-8"?>
 <packages>
-  <package id="SkiaSharp" version="1.49.2.0-beta" targetFramework="net45" />
+  <package id="SkiaSharp" version="1.49.2.1-beta" targetFramework="net45" />
 </packages>
\ No newline at end of file
diff --git a/src/Skia/Perspex.Skia.iOS/Perspex.Skia.iOS.csproj b/src/Skia/Perspex.Skia.iOS/Perspex.Skia.iOS.csproj
index 572d466de5..07c96d5904 100644
--- a/src/Skia/Perspex.Skia.iOS/Perspex.Skia.iOS.csproj
+++ b/src/Skia/Perspex.Skia.iOS/Perspex.Skia.iOS.csproj
@@ -41,7 +41,7 @@
   </ItemGroup>
   <ItemGroup>
     <Reference Include="SkiaSharp, Version=1.49.0.0, Culture=neutral, processorArchitecture=MSIL">
-      <HintPath>..\..\..\packages\SkiaSharp.1.49.2.0-beta\lib\XamariniOS\SkiaSharp.dll</HintPath>
+      <HintPath>..\..\..\packages\SkiaSharp.1.49.2.1-beta\lib\XamariniOS\SkiaSharp.dll</HintPath>
       <Private>True</Private>
     </Reference>
     <Reference Include="System" />
diff --git a/src/Skia/Perspex.Skia.iOS/packages.config b/src/Skia/Perspex.Skia.iOS/packages.config
index 33db31775b..cad65f14ba 100644
--- a/src/Skia/Perspex.Skia.iOS/packages.config
+++ b/src/Skia/Perspex.Skia.iOS/packages.config
@@ -1,4 +1,4 @@
 <?xml version="1.0" encoding="utf-8"?>
 <packages>
-  <package id="SkiaSharp" version="1.49.2.0-beta" targetFramework="xamarinios10" />
+  <package id="SkiaSharp" version="1.49.2.1-beta" targetFramework="xamarinios10" />
 </packages>
\ No newline at end of file

From ef28479b935af0897c618ecb308928874f5db2af Mon Sep 17 00:00:00 2001
From: Jason Jarvis <jazzay@gmail.com>
Date: Sun, 8 May 2016 15:21:12 -0700
Subject: [PATCH 2/3] Use DrawRoundRect on Skia now that SkiaSharp provides it.

---
 src/Skia/Perspex.Skia/DrawingContextImpl.cs | 10 ++--------
 src/Skia/Perspex.Skia/readme.md             |  1 -
 2 files changed, 2 insertions(+), 9 deletions(-)

diff --git a/src/Skia/Perspex.Skia/DrawingContextImpl.cs b/src/Skia/Perspex.Skia/DrawingContextImpl.cs
index 49d627442b..970ecfbb2f 100644
--- a/src/Skia/Perspex.Skia/DrawingContextImpl.cs
+++ b/src/Skia/Perspex.Skia/DrawingContextImpl.cs
@@ -229,10 +229,7 @@ namespace Perspex.Skia
                 }
                 else
                 {
-                    // TODO: DrawRRect (ore DrawRoundedRect) is not accesible in SkiaSharp yet. We should add that
-                    // to SkiaSharp and initiate a PR....
-                    Canvas.DrawRect(rc, paint.Paint);
-                    //Canvas.DrawRoundedRect(rc, cornerRadius, cornerRadius, paint);
+                    Canvas.DrawRoundRect(rc, cornerRadius, cornerRadius, paint.Paint);
                 }
             }
         }
@@ -248,10 +245,7 @@ namespace Perspex.Skia
                 }
                 else
                 {
-                    // TODO: this does not exist in SkiaSharp yet
-                    //throw new NotImplementedException();
-                    //Canvas.DrawRoundedRect(rc, cornerRadius, cornerRadius, paint);
-                    Canvas.DrawRect(rc, paint.Paint);
+                    Canvas.DrawRoundRect(rc, cornerRadius, cornerRadius, paint.Paint);
                 }
             }
         }
diff --git a/src/Skia/Perspex.Skia/readme.md b/src/Skia/Perspex.Skia/readme.md
index 1c923df3c5..615564b17f 100644
--- a/src/Skia/Perspex.Skia/readme.md
+++ b/src/Skia/Perspex.Skia/readme.md
@@ -5,7 +5,6 @@ BitmapImpl
 - Save
 
 DrawingContextImpl
-- DrawRoundRect is not properly implemented due to lack of support in SkiaSharp
 - Alpha support missing as SkiaSharp does not expose this
 - Gradient Shader caching?
 - TileBrushes

From 5d8e339744018cb99ff43838d35a1440b6234c17 Mon Sep 17 00:00:00 2001
From: Jason Jarvis <jazzay@gmail.com>
Date: Sun, 8 May 2016 22:41:27 -0700
Subject: [PATCH 3/3] Fixed up Skia's StreamGeometryImpl now that SkiaSharp
 contains SKPath cloning and transforming.

---
 src/Skia/Perspex.Skia/SkiaSharpExtensions.cs |  5 +++
 src/Skia/Perspex.Skia/StreamGeometryImpl.cs  | 40 +++++++-------------
 src/Skia/Perspex.Skia/readme.md              |  6 ---
 3 files changed, 18 insertions(+), 33 deletions(-)

diff --git a/src/Skia/Perspex.Skia/SkiaSharpExtensions.cs b/src/Skia/Perspex.Skia/SkiaSharpExtensions.cs
index a46f37f6ab..1cb0c710be 100644
--- a/src/Skia/Perspex.Skia/SkiaSharpExtensions.cs
+++ b/src/Skia/Perspex.Skia/SkiaSharpExtensions.cs
@@ -65,5 +65,10 @@ namespace Perspex.Skia
                 case TextAlignment.Right: return SKTextAlign.Right;
             }
         }
+
+        public static SKPath Clone(this SKPath src)
+        {
+            return src != null ? new SKPath(src) : null;
+        }
     }
 }
diff --git a/src/Skia/Perspex.Skia/StreamGeometryImpl.cs b/src/Skia/Perspex.Skia/StreamGeometryImpl.cs
index b5a7f9e3c9..ea704f7e38 100644
--- a/src/Skia/Perspex.Skia/StreamGeometryImpl.cs
+++ b/src/Skia/Perspex.Skia/StreamGeometryImpl.cs
@@ -50,28 +50,23 @@ namespace Perspex.Skia
                 _transformedPath.Dispose();
                 _transformedPath = null;
             }
-            
-            // TODO: SkiaSharp does not expose Transform yet!!!
-            //if (!Transform.IsIdentity)
-            //{
-            //	_transformedPath = new SKPath();
-            //	_path.Transform(Transform.ToSKMatrix(), _transformedPath);
-            //}
+
+            if (!Transform.IsIdentity)
+            {
+                _transformedPath = new SKPath(_path);
+                _transformedPath.Transform(Transform.ToSKMatrix());
+            }
         }
 
         public IStreamGeometryImpl Clone()
         {
-            // TODO: there is no SKPath.Clone yet!!!!!!!!!!!!!
-            //
-            //return new StreamGeometryImpl
-            //{
-            //	_path = _path?.Clone(),
-            //	_transformedPath = _transformedPath?.Clone(),
-            //	_transform = Transform,
-            //	Bounds = Bounds
-            //};
-
-            return this;        // this will probably end bad!!
+            return new StreamGeometryImpl
+            {
+                _path = _path?.Clone(),
+                _transformedPath = _transformedPath?.Clone(),
+                _transform = Transform,
+                Bounds = Bounds
+            };
         }
 
         public IStreamGeometryContextImpl Open()
@@ -94,15 +89,6 @@ namespace Perspex.Skia
 
             public void Dispose()
             {
-                // TODO: Not sure what we need to do here. This code left here for reference.
-                //
-                //	var arr = _elements.ToArray();
-                //             SkRect rc;
-                //             _path?.Dispose();
-                //             _path = new SKPath(new SkPath(MethodTable.Instance.CreatePath(arr, arr.Length, out rc)));
-                //             _geometryImpl.ApplyTransform();
-                //             _geometryImpl.Bounds = rc.ToRect();
-
                 SKRect rc;
                 _path.GetBounds(out rc);
                 _geometryImpl.ApplyTransform();
diff --git a/src/Skia/Perspex.Skia/readme.md b/src/Skia/Perspex.Skia/readme.md
index 615564b17f..7c20cf6c2e 100644
--- a/src/Skia/Perspex.Skia/readme.md
+++ b/src/Skia/Perspex.Skia/readme.md
@@ -17,12 +17,6 @@ RenderTarget
 - Figure out a cleaner implementation across all platforms
 - HW acceleration
 
-StreamGeometry
-- Paths within Paths may not work right
-- Paths cannot be Cloned (lack of SkiaSupport)
-- Paths cannot be transformed (lack of SkiaSupport)
-- ArcTo
-
 App Bootstrapping
 - Cleanup the testapplications across all platforms
 - Add a cleaner Fluent API for the subsystems