update gif fix files

Author: guanghuispark

com.unity.uiwidgets/Runtime/rendering/layer.cs

@@ -59,7 +59,7 @@ public abstract class Layer : AbstractNodeMixinDiagnosticableTree {
             get { return (ContainerLayer) base.parent; }
         }
 
-        public virtual void DisposeCPter()
+        public virtual void DisposeCPtr()
         {
             if(_engineLayer != null) {
                 _engineLayer.DisposeCPtr();
@@ -102,7 +102,7 @@ bool  debugSubtreeNeedsAddToScene {
             }
         }
 
-        public EngineLayer engineLayer {
+        protected EngineLayer engineLayer {
             get { return _engineLayer; }
             set {
                 if(_engineLayer != null){
@@ -254,9 +254,9 @@ public PictureLayer(Rect canvasBounds) {
             this.canvasBounds = canvasBounds;
         }
 
-        public override void DisposeCPter()
+        public override void DisposeCPtr()
         {
-            base.DisposeCPter();
+            base.DisposeCPtr();
             if(_picture != null){
                  _picture.DisposeCPtr();
             }
@@ -504,16 +504,12 @@ protected List<PictureLayer> _debugCheckElevations() {
             return addedLayers;
         }
 
-        public override void DisposeCPter() {
-            base.DisposeCPter();
-            DisposeAllChildren();
-        }
-
-        private void DisposeAllChildren(){
+        public override void DisposeCPtr() {
+            base.DisposeCPtr();
             Layer child = firstChild;
             while (child != null) {
                 Layer next = child.nextSibling;
-                child.DisposeCPter();
+                child.DisposeCPtr();
                 child = next;
             }
         }
@@ -633,7 +629,7 @@ internal void _removeChild(Layer child) {
             child._nextSibling = null;
             child._previousSibling = null;
             dropChild(child);
-            child.DisposeCPter();
+            child.DisposeCPtr();
             D.assert(!child.attached);
         }
 
@@ -645,7 +641,7 @@ public void removeAllChildren() {
                 child._nextSibling = null;
                 D.assert(child.attached == attached);
                 dropChild(child);
-                child.DisposeCPter();
+                child.DisposeCPtr();
                 child = next;
             }
 

com.unity.uiwidgets/Runtime/rendering/view.cs

@@ -133,9 +133,8 @@ public void compositeFrame() {
             var builder = new SceneBuilder();
             using (var scene = layer.buildScene(builder)) {
                 Window.instance.render(scene);
-                builder.DisposeCPtr();
             }
-
+            builder.DisposeCPtr();
             D.assert(() => {
                 if (D.debugRepaintRainbowEnabled || D.debugRepaintTextRainbowEnabled) {
                     D.debugCurrentRepaintColor =

com.unity.uiwidgets/Runtime/ui/native_bindings.cs

@@ -64,4 +64,14 @@ public void Dispose() {
             _dispose();
         }
     }
+
+    public abstract class NativeWrapperCPtrDisposable : NativeWrapperDisposable {
+        public bool isDisposed = false;
+
+        protected NativeWrapperCPtrDisposable() {
+        }
+
+        protected NativeWrapperCPtrDisposable(IntPtr ptr) : base(ptr) {
+        }
+    }
 }

com.unity.uiwidgets/Runtime/ui/painting.cs

@@ -763,9 +763,7 @@ internal _ImageInfo(int width, int height, int format, int? rowBytes = null) {
         public int rowBytes;
     }
 
-    public class Image : NativeWrapperDisposable, IEquatable<Image> {
-
-        private bool isDisposed = false;
+    public class Image : NativeWrapperCPtrDisposable, IEquatable<Image> {
 
         internal Image(IntPtr ptr) : base(ptr) {
         }
@@ -895,8 +893,7 @@ public override int GetHashCode() {
 
     public delegate void ImageDecoderCallback(Image result);
 
-    public class FrameInfo : NativeWrapper {
-        bool isDisposed = false;
+    public class FrameInfo : NativeWrapperCPtrDisposable {
 
         internal FrameInfo(IntPtr ptr) : base(ptr) {
         }
@@ -1097,8 +1094,7 @@ public enum PathOperation {
         reverseDifference,
     }
 
-    public abstract class EngineLayer : NativeWrapper {
-        private bool isDisposed = false;
+    public abstract class EngineLayer : NativeWrapperCPtrDisposable {
         protected EngineLayer(IntPtr ptr) : base(ptr) {
         }
 
@@ -2900,9 +2896,8 @@ static extern void Canvas_drawShadow(IntPtr ptr, IntPtr path, uint color, float
             bool transparentOccluder);
     }
 
-    public class Picture : NativeWrapperDisposable {
+    public class Picture : NativeWrapperCPtrDisposable {
 
-        private bool isDisposed = false;
         internal Picture(IntPtr ptr) : base(ptr) {
         }
 

com.unity.uiwidgets/Runtime/widgets/widget_inspector.cs

@@ -2444,7 +2444,7 @@ Rect targetRect
         }
 
 
-        public override void DisposeCPter() {
+        public override void DisposeCPtr() {
             if(_picture != null){
                 _picture.DisposeCPtr();
             }