diff --git a/.editorconfig b/.editorconfig
index 3620896f34..eac5870f96 100644
--- a/.editorconfig
+++ b/.editorconfig
@@ -137,6 +137,9 @@ space_within_single_line_array_initializer_braces = true
#Net Analyzer
dotnet_analyzer_diagnostic.category-Performance.severity = none #error - Uncomment when all violations are fixed.
+# CS0649: Field 'field' is never assigned to, and will always have its default value 'value'
+dotnet_diagnostic.CS0649.severity = error
+
# CS1591: Missing XML comment for publicly visible type or member
dotnet_diagnostic.CS1591.severity = suggestion
diff --git a/build/Base.props b/build/Base.props
index c2fc11c94b..2d50a7eae0 100644
--- a/build/Base.props
+++ b/build/Base.props
@@ -1,5 +1,6 @@
-
+
+
diff --git a/build/System.Memory.props b/build/System.Memory.props
index a413e18927..35a87a3a2f 100644
--- a/build/System.Memory.props
+++ b/build/System.Memory.props
@@ -1,5 +1,6 @@
-
+
+
diff --git a/samples/ControlCatalog.NetCore/Program.cs b/samples/ControlCatalog.NetCore/Program.cs
index d5e5cb14dc..e55f003133 100644
--- a/samples/ControlCatalog.NetCore/Program.cs
+++ b/samples/ControlCatalog.NetCore/Program.cs
@@ -55,8 +55,7 @@ namespace ControlCatalog.NetCore
return builder
.UseHeadless(new AvaloniaHeadlessPlatformOptions
{
- UseHeadlessDrawing = true,
- UseCompositor = true
+ UseHeadlessDrawing = true
})
.AfterSetup(_ =>
{
diff --git a/samples/ControlCatalog/Pages/DataGridPage.xaml b/samples/ControlCatalog/Pages/DataGridPage.xaml
index 4c3c211ca5..c39e9f0a81 100644
--- a/samples/ControlCatalog/Pages/DataGridPage.xaml
+++ b/samples/ControlCatalog/Pages/DataGridPage.xaml
@@ -1,7 +1,9 @@
+ xmlns:pages="clr-namespace:ControlCatalog.Pages"
+ x:Class="ControlCatalog.Pages.DataGridPage"
+ x:DataType="pages:DataGridPage">
@@ -33,7 +35,7 @@
-
+
-
+
+
-
-
-
-
+
+
+
+
-
+
-
+
diff --git a/samples/ControlCatalog/Pages/DataGridPage.xaml.cs b/samples/ControlCatalog/Pages/DataGridPage.xaml.cs
index 3565d113bc..b0c3e3a553 100644
--- a/samples/ControlCatalog/Pages/DataGridPage.xaml.cs
+++ b/samples/ControlCatalog/Pages/DataGridPage.xaml.cs
@@ -1,5 +1,6 @@
using System.Collections;
using System.Collections.Generic;
+using System.Collections.ObjectModel;
using System.ComponentModel;
using System.Linq;
using Avalonia.Controls;
@@ -48,20 +49,22 @@ namespace ControlCatalog.Pages
var dg3 = this.Get("dataGridEdit");
dg3.IsReadOnly = false;
- var items = new List
+ var list = new ObservableCollection
{
new Person { FirstName = "John", LastName = "Doe" , Age = 30},
new Person { FirstName = "Elizabeth", LastName = "Thomas", IsBanned = true , Age = 40 },
new Person { FirstName = "Zack", LastName = "Ward" , Age = 50 }
};
- var collectionView3 = new DataGridCollectionView(items);
-
- dg3.Items = collectionView3;
+ DataGrid3Source = list;
var addButton = this.Get
diff --git a/src/Avalonia.Base/Input/DragEventArgs.cs b/src/Avalonia.Base/Input/DragEventArgs.cs
index 7a27c53023..403dd6f23e 100644
--- a/src/Avalonia.Base/Input/DragEventArgs.cs
+++ b/src/Avalonia.Base/Input/DragEventArgs.cs
@@ -1,5 +1,6 @@
using System;
using Avalonia.Interactivity;
+using Avalonia.Metadata;
using Avalonia.VisualTree;
namespace Avalonia.Input
@@ -32,7 +33,9 @@ namespace Avalonia.Input
return point;
}
- internal DragEventArgs(RoutedEvent routedEvent, IDataObject data, Interactive target, Point targetLocation, KeyModifiers keyModifiers)
+ [Unstable]
+ [Obsolete("This constructor might be removed in 12.0. For unit testing, consider using DragDrop.DoDragDrop or IHeadlessWindow.DragDrop.")]
+ public DragEventArgs(RoutedEvent routedEvent, IDataObject data, Interactive target, Point targetLocation, KeyModifiers keyModifiers)
: base(routedEvent)
{
Data = data;
diff --git a/src/Avalonia.Base/Input/GotFocusEventArgs.cs b/src/Avalonia.Base/Input/GotFocusEventArgs.cs
index f3de55ebae..8d15c3f9ec 100644
--- a/src/Avalonia.Base/Input/GotFocusEventArgs.cs
+++ b/src/Avalonia.Base/Input/GotFocusEventArgs.cs
@@ -7,19 +7,18 @@ namespace Avalonia.Input
///
public class GotFocusEventArgs : RoutedEventArgs
{
- internal GotFocusEventArgs()
+ public GotFocusEventArgs() : base(InputElement.GotFocusEvent)
{
-
}
///
/// Gets or sets a value indicating how the change in focus occurred.
///
- public NavigationMethod NavigationMethod { get; set; }
+ public NavigationMethod NavigationMethod { get; init; }
///
/// Gets or sets any key modifiers active at the time of focus.
///
- public KeyModifiers KeyModifiers { get; set; }
+ public KeyModifiers KeyModifiers { get; init; }
}
}
diff --git a/src/Avalonia.Base/Input/KeyEventArgs.cs b/src/Avalonia.Base/Input/KeyEventArgs.cs
index 35fa549995..9fa097c4b1 100644
--- a/src/Avalonia.Base/Input/KeyEventArgs.cs
+++ b/src/Avalonia.Base/Input/KeyEventArgs.cs
@@ -5,15 +5,10 @@ namespace Avalonia.Input
{
public class KeyEventArgs : RoutedEventArgs
{
- public KeyEventArgs()
- {
+ public IKeyboardDevice? Device { get; init; }
- }
+ public Key Key { get; init; }
- public IKeyboardDevice? Device { get; set; }
-
- public Key Key { get; set; }
-
- public KeyModifiers KeyModifiers { get; set; }
+ public KeyModifiers KeyModifiers { get; init; }
}
}
diff --git a/src/Avalonia.Base/Input/KeyboardDevice.cs b/src/Avalonia.Base/Input/KeyboardDevice.cs
index 26ff71a4e7..c46834fff4 100644
--- a/src/Avalonia.Base/Input/KeyboardDevice.cs
+++ b/src/Avalonia.Base/Input/KeyboardDevice.cs
@@ -156,7 +156,6 @@ namespace Avalonia.Input
interactive?.RaiseEvent(new GotFocusEventArgs
{
- RoutedEvent = InputElement.GotFocusEvent,
NavigationMethod = method,
KeyModifiers = keyModifiers,
});
diff --git a/src/Avalonia.Base/Input/PointerDeltaEventArgs.cs b/src/Avalonia.Base/Input/PointerDeltaEventArgs.cs
index af0fa83382..c405cdfacd 100644
--- a/src/Avalonia.Base/Input/PointerDeltaEventArgs.cs
+++ b/src/Avalonia.Base/Input/PointerDeltaEventArgs.cs
@@ -1,13 +1,17 @@
+using System;
using Avalonia.Interactivity;
+using Avalonia.Metadata;
using Avalonia.VisualTree;
namespace Avalonia.Input
{
public class PointerDeltaEventArgs : PointerEventArgs
{
- public Vector Delta { get; set; }
+ public Vector Delta { get; }
- internal PointerDeltaEventArgs(RoutedEvent routedEvent, object? source,
+ [Unstable]
+ [Obsolete("This constructor might be removed in 12.0.")]
+ public PointerDeltaEventArgs(RoutedEvent routedEvent, object? source,
IPointer pointer, Visual rootVisual, Point rootVisualPosition, ulong timestamp,
PointerPointProperties properties, KeyModifiers modifiers, Vector delta)
: base(routedEvent, source, pointer, rootVisual, rootVisualPosition,
diff --git a/src/Avalonia.Base/Input/PointerEventArgs.cs b/src/Avalonia.Base/Input/PointerEventArgs.cs
index d736253728..50d7cc5dc5 100644
--- a/src/Avalonia.Base/Input/PointerEventArgs.cs
+++ b/src/Avalonia.Base/Input/PointerEventArgs.cs
@@ -2,6 +2,7 @@ using System;
using System.Collections.Generic;
using Avalonia.Input.Raw;
using Avalonia.Interactivity;
+using Avalonia.Metadata;
using Avalonia.VisualTree;
namespace Avalonia.Input
@@ -13,7 +14,9 @@ namespace Avalonia.Input
private readonly PointerPointProperties _properties;
private readonly Lazy?>? _previousPoints;
- internal PointerEventArgs(RoutedEvent routedEvent,
+ [Unstable]
+ [Obsolete("This constructor might be removed in 12.0. For unit testing, consider using IHeadlessWindow mouse methods.")]
+ public PointerEventArgs(RoutedEvent routedEvent,
object? source,
IPointer pointer,
Visual? rootVisual, Point rootVisualPosition,
@@ -124,7 +127,9 @@ namespace Avalonia.Input
public class PointerPressedEventArgs : PointerEventArgs
{
- internal PointerPressedEventArgs(
+ [Unstable]
+ [Obsolete("This constructor might be removed in 12.0. For unit testing, consider using IHeadlessWindow mouse methods.")]
+ public PointerPressedEventArgs(
object source,
IPointer pointer,
Visual rootVisual, Point rootVisualPosition,
@@ -143,7 +148,9 @@ namespace Avalonia.Input
public class PointerReleasedEventArgs : PointerEventArgs
{
- internal PointerReleasedEventArgs(
+ [Unstable]
+ [Obsolete("This constructor might be removed in 12.0. For unit testing, consider using IHeadlessWindow mouse methods.")]
+ public PointerReleasedEventArgs(
object source, IPointer pointer,
Visual rootVisual, Point rootVisualPosition, ulong timestamp,
PointerPointProperties properties, KeyModifiers modifiers,
@@ -164,7 +171,9 @@ namespace Avalonia.Input
{
public IPointer Pointer { get; }
- internal PointerCaptureLostEventArgs(object source, IPointer pointer) : base(InputElement.PointerCaptureLostEvent)
+ [Unstable]
+ [Obsolete("This constructor might be removed in 12.0. If you need to remove capture, use stable methods on the IPointer instance.,")]
+ public PointerCaptureLostEventArgs(object source, IPointer pointer) : base(InputElement.PointerCaptureLostEvent)
{
Pointer = pointer;
Source = source;
diff --git a/src/Avalonia.Base/Input/PointerWheelEventArgs.cs b/src/Avalonia.Base/Input/PointerWheelEventArgs.cs
index a3de0eaaea..903019d85d 100644
--- a/src/Avalonia.Base/Input/PointerWheelEventArgs.cs
+++ b/src/Avalonia.Base/Input/PointerWheelEventArgs.cs
@@ -1,13 +1,17 @@
+using System;
using Avalonia.Interactivity;
+using Avalonia.Metadata;
using Avalonia.VisualTree;
namespace Avalonia.Input
{
public class PointerWheelEventArgs : PointerEventArgs
{
- public Vector Delta { get; set; }
+ public Vector Delta { get; }
- internal PointerWheelEventArgs(object source, IPointer pointer, Visual rootVisual,
+ [Unstable]
+ [Obsolete("This constructor might be removed in 12.0. For unit testing, consider using IHeadlessWindow.MouseWheel.")]
+ public PointerWheelEventArgs(object source, IPointer pointer, Visual rootVisual,
Point rootVisualPosition, ulong timestamp,
PointerPointProperties properties, KeyModifiers modifiers, Vector delta)
: base(InputElement.PointerWheelChangedEvent, source, pointer, rootVisual, rootVisualPosition,
diff --git a/src/Avalonia.Base/Input/Raw/RawTextInputEventArgs.cs b/src/Avalonia.Base/Input/Raw/RawTextInputEventArgs.cs
index 48c882197f..cd1cf29bcf 100644
--- a/src/Avalonia.Base/Input/Raw/RawTextInputEventArgs.cs
+++ b/src/Avalonia.Base/Input/Raw/RawTextInputEventArgs.cs
@@ -12,6 +12,6 @@ namespace Avalonia.Input.Raw
Text = text;
}
- public string Text { get; set; }
+ public string Text { get; }
}
}
diff --git a/src/Avalonia.Base/Input/ScrollGestureEventArgs.cs b/src/Avalonia.Base/Input/ScrollGestureEventArgs.cs
index 55aaadff71..dd78080708 100644
--- a/src/Avalonia.Base/Input/ScrollGestureEventArgs.cs
+++ b/src/Avalonia.Base/Input/ScrollGestureEventArgs.cs
@@ -14,7 +14,7 @@ namespace Avalonia.Input
public static int GetNextFreeId() => _nextId++;
- internal ScrollGestureEventArgs(int id, Vector delta) : base(Gestures.ScrollGestureEvent)
+ public ScrollGestureEventArgs(int id, Vector delta) : base(Gestures.ScrollGestureEvent)
{
Id = id;
Delta = delta;
@@ -25,7 +25,7 @@ namespace Avalonia.Input
{
public int Id { get; }
- internal ScrollGestureEndedEventArgs(int id) : base(Gestures.ScrollGestureEndedEvent)
+ public ScrollGestureEndedEventArgs(int id) : base(Gestures.ScrollGestureEndedEvent)
{
Id = id;
}
diff --git a/src/Avalonia.Base/Input/TappedEventArgs.cs b/src/Avalonia.Base/Input/TappedEventArgs.cs
index 3e15c4843a..663207a104 100644
--- a/src/Avalonia.Base/Input/TappedEventArgs.cs
+++ b/src/Avalonia.Base/Input/TappedEventArgs.cs
@@ -1,3 +1,4 @@
+using System;
using Avalonia.Interactivity;
using Avalonia.VisualTree;
@@ -7,7 +8,7 @@ namespace Avalonia.Input
{
private readonly PointerEventArgs lastPointerEventArgs;
- internal TappedEventArgs(RoutedEvent routedEvent, PointerEventArgs lastPointerEventArgs)
+ public TappedEventArgs(RoutedEvent routedEvent, PointerEventArgs lastPointerEventArgs)
: base(routedEvent)
{
this.lastPointerEventArgs = lastPointerEventArgs;
diff --git a/src/Avalonia.Base/Input/TextInput/TransformTrackingHelper.cs b/src/Avalonia.Base/Input/TextInput/TransformTrackingHelper.cs
index 09716b4246..36ee1fff27 100644
--- a/src/Avalonia.Base/Input/TextInput/TransformTrackingHelper.cs
+++ b/src/Avalonia.Base/Input/TextInput/TransformTrackingHelper.cs
@@ -105,5 +105,13 @@ namespace Avalonia.Input.TextInput
UnsubscribeFromParents();
UpdateMatrix();
}
+
+ public static IDisposable Track(Visual visual, Action cb)
+ {
+ var rv = new TransformTrackingHelper();
+ rv.MatrixChanged += () => cb(visual, rv.Matrix);
+ rv.SetVisual(visual);
+ return rv;
+ }
}
}
diff --git a/src/Avalonia.Base/Input/TextInputEventArgs.cs b/src/Avalonia.Base/Input/TextInputEventArgs.cs
index a027bec0c6..cda0103749 100644
--- a/src/Avalonia.Base/Input/TextInputEventArgs.cs
+++ b/src/Avalonia.Base/Input/TextInputEventArgs.cs
@@ -4,10 +4,6 @@ namespace Avalonia.Input
{
public class TextInputEventArgs : RoutedEventArgs
{
- public TextInputEventArgs()
- {
-
- }
public IKeyboardDevice? Device { get; set; }
public string? Text { get; set; }
diff --git a/src/Avalonia.Base/Input/VectorEventArgs.cs b/src/Avalonia.Base/Input/VectorEventArgs.cs
index 3e8098f904..2ce95cf35a 100644
--- a/src/Avalonia.Base/Input/VectorEventArgs.cs
+++ b/src/Avalonia.Base/Input/VectorEventArgs.cs
@@ -5,11 +5,6 @@ namespace Avalonia.Input
{
public class VectorEventArgs : RoutedEventArgs
{
- internal VectorEventArgs()
- {
-
- }
-
- public Vector Vector { get; set; }
+ public Vector Vector { get; init; }
}
}
diff --git a/src/Avalonia.Base/Interactivity/CancelRoutedEventArgs.cs b/src/Avalonia.Base/Interactivity/CancelRoutedEventArgs.cs
new file mode 100644
index 0000000000..b6913939ab
--- /dev/null
+++ b/src/Avalonia.Base/Interactivity/CancelRoutedEventArgs.cs
@@ -0,0 +1,39 @@
+namespace Avalonia.Interactivity
+{
+ ///
+ /// Provides state information and data specific to a cancelable routed event.
+ ///
+ public class CancelRoutedEventArgs : RoutedEventArgs
+ {
+ ///
+ /// Initializes a new instance of the class.
+ ///
+ public CancelRoutedEventArgs()
+ {
+ }
+
+ ///
+ /// Initializes a new instance of the class.
+ ///
+ /// The routed event associated with these event args.
+ public CancelRoutedEventArgs(RoutedEvent? routedEvent)
+ : base(routedEvent)
+ {
+ }
+
+ ///
+ /// Initializes a new instance of the class.
+ ///
+ /// The routed event associated with these event args.
+ /// The source object that raised the routed event.
+ public CancelRoutedEventArgs(RoutedEvent? routedEvent, object? source)
+ : base(routedEvent, source)
+ {
+ }
+
+ ///
+ /// Gets or sets a value indicating whether the routed event should be canceled.
+ ///
+ public bool Cancel { get; set; } = false;
+ }
+}
diff --git a/src/Avalonia.Base/Layout/EffectiveViewportChangedEventArgs.cs b/src/Avalonia.Base/Layout/EffectiveViewportChangedEventArgs.cs
index 749d2ecc2b..1cdc775b13 100644
--- a/src/Avalonia.Base/Layout/EffectiveViewportChangedEventArgs.cs
+++ b/src/Avalonia.Base/Layout/EffectiveViewportChangedEventArgs.cs
@@ -7,7 +7,7 @@ namespace Avalonia.Layout
///
public class EffectiveViewportChangedEventArgs : EventArgs
{
- internal EffectiveViewportChangedEventArgs(Rect effectiveViewport)
+ public EffectiveViewportChangedEventArgs(Rect effectiveViewport)
{
EffectiveViewport = effectiveViewport;
}
diff --git a/src/Avalonia.Base/Media/Imaging/Bitmap.cs b/src/Avalonia.Base/Media/Imaging/Bitmap.cs
index ce38fc5abc..6577532891 100644
--- a/src/Avalonia.Base/Media/Imaging/Bitmap.cs
+++ b/src/Avalonia.Base/Media/Imaging/Bitmap.cs
@@ -1,5 +1,7 @@
using System;
using System.IO;
+using System.Runtime.CompilerServices;
+using System.Runtime.InteropServices;
using Avalonia.Platform;
using Avalonia.Utilities;
@@ -10,6 +12,7 @@ namespace Avalonia.Media.Imaging
///
public class Bitmap : IBitmap
{
+ private bool _isTranscoded;
///
/// Loads a Bitmap from a stream and decodes at the desired width. Aspect ratio is maintained.
/// This is more efficient than loading and then resizing.
@@ -100,7 +103,28 @@ namespace Avalonia.Media.Imaging
/// The number of bytes per row.
public Bitmap(PixelFormat format, AlphaFormat alphaFormat, IntPtr data, PixelSize size, Vector dpi, int stride)
{
- PlatformImpl = RefCountable.Create(GetFactory().LoadBitmap(format, alphaFormat, data, size, dpi, stride));
+ var factory = GetFactory();
+ if (factory.IsSupportedBitmapPixelFormat(format))
+ PlatformImpl = RefCountable.Create(factory.LoadBitmap(format, alphaFormat, data, size, dpi, stride));
+ else
+ {
+ var transcoded = Marshal.AllocHGlobal(size.Width * size.Height * 4);
+ var transcodedStride = size.Width * 4;
+ try
+ {
+ PixelFormatReader.Transcode(transcoded, data, size, stride, transcodedStride, format);
+ var transcodedAlphaFormat = format.HasAlpha ? alphaFormat : AlphaFormat.Opaque;
+
+ PlatformImpl = RefCountable.Create(factory.LoadBitmap(PixelFormat.Rgba8888, transcodedAlphaFormat,
+ transcoded, size, dpi, transcodedStride));
+ }
+ finally
+ {
+ Marshal.FreeHGlobal(transcoded);
+ }
+
+ _isTranscoded = true;
+ }
}
///
@@ -145,6 +169,57 @@ namespace Avalonia.Media.Imaging
PlatformImpl.Item.Save(stream, quality);
}
+ public virtual PixelFormat? Format => (PlatformImpl.Item as IReadableBitmapImpl)?.Format;
+
+ protected internal unsafe void CopyPixelsCore(PixelRect sourceRect, IntPtr buffer, int bufferSize, int stride,
+ ILockedFramebuffer fb)
+ {
+ if ((sourceRect.Width <= 0 || sourceRect.Height <= 0) && (sourceRect.X != 0 || sourceRect.Y != 0))
+ throw new ArgumentOutOfRangeException(nameof(sourceRect));
+
+ if (sourceRect.X < 0 || sourceRect.Y < 0)
+ throw new ArgumentOutOfRangeException(nameof(sourceRect));
+
+ if (sourceRect.Width <= 0)
+ sourceRect = sourceRect.WithWidth(PixelSize.Width);
+ if (sourceRect.Height <= 0)
+ sourceRect = sourceRect.WithHeight(PixelSize.Height);
+
+ if (sourceRect.Right > PixelSize.Width || sourceRect.Bottom > PixelSize.Height)
+ throw new ArgumentOutOfRangeException(nameof(sourceRect));
+
+ int minStride = checked(((sourceRect.Width * fb.Format.BitsPerPixel) + 7) / 8);
+ if (stride < minStride)
+ throw new ArgumentOutOfRangeException(nameof(stride));
+
+ var minBufferSize = stride * sourceRect.Height;
+ if (minBufferSize > bufferSize)
+ throw new ArgumentOutOfRangeException(nameof(bufferSize));
+
+ for (var y = 0; y < sourceRect.Height; y++)
+ {
+ var srcAddress = fb.Address + fb.RowBytes * y;
+ var dstAddress = buffer + stride * y;
+ Unsafe.CopyBlock(dstAddress.ToPointer(), srcAddress.ToPointer(), (uint)minStride);
+ }
+ }
+
+ public virtual void CopyPixels(PixelRect sourceRect, IntPtr buffer, int bufferSize, int stride)
+ {
+ if (
+ Format == null
+ || PlatformImpl.Item is not IReadableBitmapImpl readable
+ || Format != readable.Format
+ )
+ throw new NotSupportedException("CopyPixels is not supported for this bitmap type");
+
+ if (_isTranscoded)
+ throw new NotSupportedException("CopyPixels is not supported for transcoded bitmaps");
+
+ using (var fb = readable.Lock())
+ CopyPixelsCore(sourceRect, buffer, bufferSize, stride, fb);
+ }
+
///
void IImage.Draw(
DrawingContext context,
diff --git a/src/Avalonia.Base/Media/Imaging/BitmapMemory.cs b/src/Avalonia.Base/Media/Imaging/BitmapMemory.cs
new file mode 100644
index 0000000000..68ae2e37a5
--- /dev/null
+++ b/src/Avalonia.Base/Media/Imaging/BitmapMemory.cs
@@ -0,0 +1,51 @@
+using System;
+using System.Runtime.InteropServices;
+using System.Threading;
+using Avalonia.Platform;
+
+namespace Avalonia.Media.Imaging;
+
+internal class BitmapMemory : IDisposable
+{
+ private readonly int _memorySize;
+
+ public BitmapMemory(PixelFormat format, PixelSize size)
+ {
+ Format = format;
+ Size = size;
+ RowBytes = (size.Width * format.BitsPerPixel + 7) / 8;
+ _memorySize = RowBytes * size.Height;
+ Address = Marshal.AllocHGlobal(_memorySize);
+ GC.AddMemoryPressure(_memorySize);
+ }
+
+ private void ReleaseUnmanagedResources()
+ {
+ if (Address != IntPtr.Zero)
+ {
+ GC.RemoveMemoryPressure(_memorySize);
+ Marshal.FreeHGlobal(Address);
+ }
+ }
+
+ public void Dispose()
+ {
+ ReleaseUnmanagedResources();
+ GC.SuppressFinalize(this);
+ }
+
+ ~BitmapMemory()
+ {
+ ReleaseUnmanagedResources();
+ }
+
+ public IntPtr Address { get; private set; }
+ public PixelSize Size { get; }
+ public int RowBytes { get; }
+ public PixelFormat Format { get; }
+
+
+
+ public void CopyToRgba(IntPtr buffer, int rowBytes) =>
+ PixelFormatReader.Transcode(buffer, Address, Size, RowBytes, rowBytes, Format);
+}
\ No newline at end of file
diff --git a/src/Avalonia.Base/Media/Imaging/PixelFormatReaders.cs b/src/Avalonia.Base/Media/Imaging/PixelFormatReaders.cs
new file mode 100644
index 0000000000..fc7c174ed6
--- /dev/null
+++ b/src/Avalonia.Base/Media/Imaging/PixelFormatReaders.cs
@@ -0,0 +1,280 @@
+using System;
+using Avalonia.Platform;
+namespace Avalonia.Media.Imaging;
+
+internal struct Rgba8888Pixel
+{
+ public byte R;
+ public byte G;
+ public byte B;
+ public byte A;
+}
+
+static unsafe class PixelFormatReader
+{
+ public interface IPixelFormatReader
+ {
+ Rgba8888Pixel ReadNext();
+ void Reset(IntPtr address);
+ }
+
+ private static readonly Rgba8888Pixel s_white = new Rgba8888Pixel
+ {
+ A = 255,
+ B = 255,
+ G = 255,
+ R = 255
+ };
+
+ private static readonly Rgba8888Pixel s_black = new Rgba8888Pixel
+ {
+ A = 255,
+ B = 0,
+ G = 0,
+ R = 0
+ };
+
+ public unsafe struct BlackWhitePixelReader : IPixelFormatReader
+ {
+ private int _bit;
+ private byte* _address;
+
+ public void Reset(IntPtr address)
+ {
+ _address = (byte*)address;
+ _bit = 0;
+ }
+
+ public Rgba8888Pixel ReadNext()
+ {
+ var shift = 7 - _bit;
+ var value = (*_address >> shift) & 1;
+ _bit++;
+ if (_bit == 8)
+ {
+ _address++;
+ _bit = 0;
+ }
+ return value == 1 ? s_white : s_black;
+ }
+ }
+
+ public unsafe struct Gray2PixelReader : IPixelFormatReader
+ {
+ private int _bit;
+ private byte* _address;
+
+ public void Reset(IntPtr address)
+ {
+ _address = (byte*)address;
+ _bit = 0;
+ }
+
+ private static Rgba8888Pixel[] Palette = new[]
+ {
+ s_black,
+ new Rgba8888Pixel
+ {
+ A = 255, B = 0x55, G = 0x55, R = 0x55
+ },
+ new Rgba8888Pixel
+ {
+ A = 255, B = 0xAA, G = 0xAA, R = 0xAA
+ },
+ s_white
+ };
+
+ public Rgba8888Pixel ReadNext()
+ {
+ var shift = 6 - _bit;
+ var value = (byte)((*_address >> shift));
+ value = (byte)((value & 3));
+ _bit += 2;
+ if (_bit == 8)
+ {
+ _address++;
+ _bit = 0;
+ }
+
+ return Palette[value];
+ }
+ }
+
+ public unsafe struct Gray4PixelReader : IPixelFormatReader
+ {
+ private int _bit;
+ private byte* _address;
+
+ public void Reset(IntPtr address)
+ {
+ _address = (byte*)address;
+ _bit = 0;
+ }
+
+ public Rgba8888Pixel ReadNext()
+ {
+ var shift = 4 - _bit;
+ var value = (byte)((*_address >> shift));
+ value = (byte)((value & 0xF));
+ value = (byte)(value | (value << 4));
+ _bit += 4;
+ if (_bit == 8)
+ {
+ _address++;
+ _bit = 0;
+ }
+
+ return new Rgba8888Pixel
+ {
+ A = 255,
+ B = value,
+ G = value,
+ R = value
+ };
+ }
+ }
+
+ public unsafe struct Gray8PixelReader : IPixelFormatReader
+ {
+ private byte* _address;
+ public void Reset(IntPtr address)
+ {
+ _address = (byte*)address;
+ }
+
+ public Rgba8888Pixel ReadNext()
+ {
+ var value = *_address;
+ _address++;
+
+ return new Rgba8888Pixel
+ {
+ A = 255,
+ B = value,
+ G = value,
+ R = value
+ };
+ }
+ }
+
+ public unsafe struct Gray16PixelReader : IPixelFormatReader
+ {
+ private ushort* _address;
+ public Rgba8888Pixel ReadNext()
+ {
+ var value16 = *_address;
+ _address++;
+ var value8 = (byte)(value16 >> 8);
+ return new Rgba8888Pixel
+ {
+ A = 255,
+ B = value8,
+ G = value8,
+ R = value8
+ };
+ }
+
+ public void Reset(IntPtr address) => _address = (ushort*)address;
+ }
+
+ public unsafe struct Gray32FloatPixelReader : IPixelFormatReader
+ {
+ private byte* _address;
+ public Rgba8888Pixel ReadNext()
+ {
+ var f = *(float*)_address;
+ var srgb = Math.Pow(f, 1 / 2.2);
+ var value = (byte)(srgb * 255);
+
+ _address += 4;
+ return new Rgba8888Pixel
+ {
+ A = 255,
+ B = value,
+ G = value,
+ R = value
+ };
+ }
+
+ public void Reset(IntPtr address) => _address = (byte*)address;
+ }
+
+ struct Rgba64
+ {
+#pragma warning disable CS0649
+ public ushort R;
+ public ushort G;
+ public ushort B;
+ public ushort A;
+#pragma warning restore CS0649
+ }
+
+ public unsafe struct Rgba64PixelFormatReader : IPixelFormatReader
+ {
+ private Rgba64* _address;
+ public Rgba8888Pixel ReadNext()
+ {
+ var value = *_address;
+
+ _address++;
+ return new Rgba8888Pixel
+ {
+ A = (byte)(value.A >> 8),
+ B = (byte)(value.B >> 8),
+ G = (byte)(value.G >> 8),
+ R = (byte)(value.R >> 8),
+ };
+ }
+
+ public void Reset(IntPtr address) => _address = (Rgba64*)address;
+ }
+
+ public static void Transcode(IntPtr dst, IntPtr src, PixelSize size, int strideSrc, int strideDst,
+ PixelFormat format)
+ {
+ if (format == PixelFormats.BlackWhite)
+ Transcode(dst, src, size, strideSrc, strideDst);
+ else if (format == PixelFormats.Gray2)
+ Transcode(dst, src, size, strideSrc, strideDst);
+ else if (format == PixelFormats.Gray4)
+ Transcode(dst, src, size, strideSrc, strideDst);
+ else if (format == PixelFormats.Gray8)
+ Transcode(dst, src, size, strideSrc, strideDst);
+ else if (format == PixelFormats.Gray16)
+ Transcode(dst, src, size, strideSrc, strideDst);
+ else if (format == PixelFormats.Gray32Float)
+ Transcode(dst, src, size, strideSrc, strideDst);
+ else if (format == PixelFormats.Rgba64)
+ Transcode(dst, src, size, strideSrc, strideDst);
+ else
+ throw new NotSupportedException($"Pixel format {format} is not supported");
+ }
+
+ public static bool SupportsFormat(PixelFormat format)
+ {
+ return format == PixelFormats.BlackWhite
+ || format == PixelFormats.Gray2
+ || format == PixelFormats.Gray4
+ || format == PixelFormats.Gray8
+ || format == PixelFormats.Gray16
+ || format == PixelFormats.Gray32Float
+ || format == PixelFormats.Rgba64;
+ }
+
+ public static void Transcode(IntPtr dst, IntPtr src, PixelSize size, int strideSrc, int strideDst) where TReader : struct, IPixelFormatReader
+ {
+ var w = size.Width;
+ var h = size.Height;
+ TReader reader = default;
+ for (var y = 0; y < h; y++)
+ {
+ reader.Reset(src + strideSrc * y);
+ var dstRow = (Rgba8888Pixel*)(dst + strideDst * y);
+ for (var x = 0; x < w; x++)
+ {
+ *dstRow = reader.ReadNext();
+ dstRow++;
+ }
+ }
+ }
+}
\ No newline at end of file
diff --git a/src/Avalonia.Base/Media/Imaging/WriteableBitmap.cs b/src/Avalonia.Base/Media/Imaging/WriteableBitmap.cs
index 1aac8efac7..868f4439c4 100644
--- a/src/Avalonia.Base/Media/Imaging/WriteableBitmap.cs
+++ b/src/Avalonia.Base/Media/Imaging/WriteableBitmap.cs
@@ -1,5 +1,6 @@
using System;
using System.IO;
+using System.Runtime.CompilerServices;
using Avalonia.Platform;
namespace Avalonia.Media.Imaging
@@ -9,7 +10,9 @@ namespace Avalonia.Media.Imaging
///
public class WriteableBitmap : Bitmap
{
-
+ // Holds a buffer with pixel format that requires transcoding
+ private BitmapMemory? _pixelFormatMemory = null;
+
///
/// Initializes a new instance of the class.
///
@@ -19,16 +22,67 @@ namespace Avalonia.Media.Imaging
/// The alpha format (optional).
/// An .
public WriteableBitmap(PixelSize size, Vector dpi, PixelFormat? format = null, AlphaFormat? alphaFormat = null)
- : base(CreatePlatformImpl(size, dpi, format, alphaFormat))
+ : this(CreatePlatformImpl(size, dpi, format, alphaFormat))
{
}
- private WriteableBitmap(IWriteableBitmapImpl impl) : base(impl)
+ private WriteableBitmap((IBitmapImpl impl, BitmapMemory? mem) bitmapWithMem) : this(bitmapWithMem.impl,
+ bitmapWithMem.mem)
{
}
+
+ private WriteableBitmap(IBitmapImpl impl, BitmapMemory? pixelFormatMemory = null) : base(impl)
+ {
+ _pixelFormatMemory = pixelFormatMemory;
+ }
+
+ ///
+ /// Initializes a new instance of the class with existing pixel data
+ /// The data is copied to the bitmap
+ ///
+ /// The pixel format.
+ /// The alpha format.
+ /// The pointer to the source bytes.
+ /// The size of the bitmap in device pixels.
+ /// The DPI of the bitmap.
+ /// The number of bytes per row.
+ public unsafe WriteableBitmap(PixelFormat format, AlphaFormat alphaFormat, IntPtr data, PixelSize size, Vector dpi, int stride)
+ : this(size, dpi, format, alphaFormat)
+ {
+ var minStride = (format.BitsPerPixel * size.Width + 7) / 8;
+ if (minStride > stride)
+ throw new ArgumentOutOfRangeException(nameof(stride));
- public ILockedFramebuffer Lock() => ((IWriteableBitmapImpl) PlatformImpl.Item).Lock();
+ using (var locked = Lock())
+ {
+ for (var y = 0; y < size.Height; y++)
+ Unsafe.CopyBlock((locked.Address + locked.RowBytes * y).ToPointer(),
+ (data + y * stride).ToPointer(), (uint)minStride);
+ }
+ }
+
+ public override PixelFormat? Format => _pixelFormatMemory?.Format ?? base.Format;
+
+ public ILockedFramebuffer Lock()
+ {
+ if (_pixelFormatMemory == null)
+ return ((IWriteableBitmapImpl)PlatformImpl.Item).Lock();
+
+ return new LockedFramebuffer(_pixelFormatMemory.Address, _pixelFormatMemory.Size,
+ _pixelFormatMemory.RowBytes,
+ Dpi, _pixelFormatMemory.Format, () =>
+ {
+ using var inner = ((IWriteableBitmapImpl)PlatformImpl.Item).Lock();
+ _pixelFormatMemory.CopyToRgba(inner.Address, inner.RowBytes);
+ });
+ }
+
+ public override void CopyPixels(PixelRect sourceRect, IntPtr buffer, int bufferSize, int stride)
+ {
+ using (var fb = Lock())
+ CopyPixelsCore(sourceRect, buffer, bufferSize, stride, fb);
+ }
public static WriteableBitmap Decode(Stream stream)
{
@@ -67,14 +121,25 @@ namespace Avalonia.Media.Imaging
return new WriteableBitmap(ri.LoadWriteableBitmapToHeight(stream, height, interpolationMode));
}
- private static IBitmapImpl CreatePlatformImpl(PixelSize size, in Vector dpi, PixelFormat? format, AlphaFormat? alphaFormat)
+ private static (IBitmapImpl, BitmapMemory?) CreatePlatformImpl(PixelSize size, in Vector dpi, PixelFormat? format, AlphaFormat? alphaFormat)
{
+ if (size.Width <= 0 || size.Height <= 0)
+ throw new ArgumentException("Size should be >= (1,1)", nameof(size));
+
var ri = GetFactory();
PixelFormat finalFormat = format ?? ri.DefaultPixelFormat;
AlphaFormat finalAlphaFormat = alphaFormat ?? ri.DefaultAlphaFormat;
- return ri.CreateWriteableBitmap(size, dpi, finalFormat, finalAlphaFormat);
+ if (ri.IsSupportedBitmapPixelFormat(finalFormat))
+ return (ri.CreateWriteableBitmap(size, dpi, finalFormat, finalAlphaFormat), null);
+
+ if (!PixelFormatReader.SupportsFormat(finalFormat))
+ throw new NotSupportedException($"Pixel format {finalFormat} is not supported");
+
+ var impl = ri.CreateWriteableBitmap(size, dpi, PixelFormat.Rgba8888,
+ finalFormat.HasAlpha ? finalAlphaFormat : AlphaFormat.Opaque);
+ return (impl, new BitmapMemory(finalFormat, size));
}
private static IPlatformRenderInterface GetFactory()
diff --git a/src/Avalonia.Base/Metadata/InheritDataTypeFromItemsAttribute.cs b/src/Avalonia.Base/Metadata/InheritDataTypeFromItemsAttribute.cs
new file mode 100644
index 0000000000..6bb820d214
--- /dev/null
+++ b/src/Avalonia.Base/Metadata/InheritDataTypeFromItemsAttribute.cs
@@ -0,0 +1,34 @@
+using System;
+
+namespace Avalonia.Metadata;
+
+///
+/// Instructs the compiler to resolve the compiled bindings data type for the item-specific properties of collection-like controls.
+///
+///
+/// A typical usage example is a ListBox control, where is defined on the ItemTemplate property,
+/// allowing the template to inherit the data type from the Items collection binding.
+///
+[AttributeUsage(AttributeTargets.Property, AllowMultiple = false, Inherited = true)]
+public sealed class InheritDataTypeFromItemsAttribute : Attribute
+{
+ ///
+ /// Initializes a new instance of the class.
+ ///
+ /// The name of the property whose item type should be used on the target property.
+ public InheritDataTypeFromItemsAttribute(string ancestorItemsProperty)
+ {
+ AncestorItemsProperty = ancestorItemsProperty;
+ }
+
+ ///
+ /// The name of the property whose item type should be used on the target property.
+ ///
+ public string AncestorItemsProperty { get; }
+
+ ///
+ /// The ancestor type to be used in a lookup for the .
+ /// If null, the declaring type of the target property is used.
+ ///
+ public Type? AncestorType { get; set; }
+}
diff --git a/src/Avalonia.Base/PixelRect.cs b/src/Avalonia.Base/PixelRect.cs
index 469f33e7fd..ef207a3dae 100644
--- a/src/Avalonia.Base/PixelRect.cs
+++ b/src/Avalonia.Base/PixelRect.cs
@@ -351,7 +351,7 @@ namespace Avalonia
/// The new .
public PixelRect WithHeight(int height)
{
- return new PixelRect(X, Y, Width, Height);
+ return new PixelRect(X, Y, Width, height);
}
///
diff --git a/src/Avalonia.Base/Platform/IOptionalFeatureProvider.cs b/src/Avalonia.Base/Platform/IOptionalFeatureProvider.cs
index 66d4f083c2..1778031e5b 100644
--- a/src/Avalonia.Base/Platform/IOptionalFeatureProvider.cs
+++ b/src/Avalonia.Base/Platform/IOptionalFeatureProvider.cs
@@ -23,5 +23,4 @@ public static class OptionalFeatureProviderExtensions
rv = provider.TryGetFeature();
return rv != null;
}
-
}
diff --git a/src/Avalonia.Base/Platform/IPlatformRenderInterface.cs b/src/Avalonia.Base/Platform/IPlatformRenderInterface.cs
index 41e792d58e..cfc7fac3ea 100644
--- a/src/Avalonia.Base/Platform/IPlatformRenderInterface.cs
+++ b/src/Avalonia.Base/Platform/IPlatformRenderInterface.cs
@@ -197,6 +197,8 @@ namespace Avalonia.Platform
/// Default used on this platform.
///
public PixelFormat DefaultPixelFormat { get; }
+
+ bool IsSupportedBitmapPixelFormat(PixelFormat format);
}
[Unstable]
diff --git a/src/Avalonia.Base/Platform/IReadableBitmapImpl.cs b/src/Avalonia.Base/Platform/IReadableBitmapImpl.cs
new file mode 100644
index 0000000000..acf1801e0a
--- /dev/null
+++ b/src/Avalonia.Base/Platform/IReadableBitmapImpl.cs
@@ -0,0 +1,7 @@
+namespace Avalonia.Platform;
+
+public interface IReadableBitmapImpl
+{
+ PixelFormat? Format { get; }
+ ILockedFramebuffer Lock();
+}
\ No newline at end of file
diff --git a/src/Avalonia.Base/Platform/IWriteableBitmapImpl.cs b/src/Avalonia.Base/Platform/IWriteableBitmapImpl.cs
index fa1e1862b7..3284d34a0a 100644
--- a/src/Avalonia.Base/Platform/IWriteableBitmapImpl.cs
+++ b/src/Avalonia.Base/Platform/IWriteableBitmapImpl.cs
@@ -6,8 +6,7 @@ namespace Avalonia.Platform
/// Defines the platform-specific interface for a .
///
[Unstable]
- public interface IWriteableBitmapImpl : IBitmapImpl
+ public interface IWriteableBitmapImpl : IBitmapImpl, IReadableBitmapImpl
{
- ILockedFramebuffer Lock();
}
}
diff --git a/src/Avalonia.Base/Platform/PixelFormat.cs b/src/Avalonia.Base/Platform/PixelFormat.cs
index 526303ebb1..99fe17055d 100644
--- a/src/Avalonia.Base/Platform/PixelFormat.cs
+++ b/src/Avalonia.Base/Platform/PixelFormat.cs
@@ -1,9 +1,74 @@
-namespace Avalonia.Platform
+using System;
+
+namespace Avalonia.Platform
{
- public enum PixelFormat
+ internal enum PixelFormatEnum
{
Rgb565,
Rgba8888,
- Bgra8888
+ Bgra8888,
+ BlackWhite,
+ Gray2,
+ Gray4,
+ Gray8,
+ Gray16,
+ Gray32Float,
+ Rgba64
+ }
+
+ public record struct PixelFormat
+ {
+ internal PixelFormatEnum FormatEnum;
+
+ public int BitsPerPixel
+ {
+ get
+ {
+ if (FormatEnum == PixelFormatEnum.BlackWhite)
+ return 1;
+ else if (FormatEnum == PixelFormatEnum.Gray2)
+ return 2;
+ else if (FormatEnum == PixelFormatEnum.Gray4)
+ return 4;
+ else if (FormatEnum == PixelFormatEnum.Gray8)
+ return 8;
+ else if (FormatEnum == PixelFormatEnum.Rgb565
+ || FormatEnum == PixelFormatEnum.Gray16)
+ return 16;
+ else if (FormatEnum == PixelFormatEnum.Rgba64)
+ return 64;
+
+ return 32;
+ }
+ }
+
+ internal bool HasAlpha => FormatEnum == PixelFormatEnum.Rgba8888
+ || FormatEnum == PixelFormatEnum.Bgra8888
+ || FormatEnum == PixelFormatEnum.Rgba64;
+
+ internal PixelFormat(PixelFormatEnum format)
+ {
+ FormatEnum = format;
+ }
+
+ public static PixelFormat Rgb565 => PixelFormats.Rgb565;
+ public static PixelFormat Rgba8888 => PixelFormats.Rgba8888;
+ public static PixelFormat Bgra8888 => PixelFormats.Bgra8888;
+
+ public override string ToString() => FormatEnum.ToString();
+ }
+
+ public static class PixelFormats
+ {
+ public static PixelFormat Rgb565 { get; } = new PixelFormat(PixelFormatEnum.Rgb565);
+ public static PixelFormat Rgba8888 { get; } = new PixelFormat(PixelFormatEnum.Rgba8888);
+ public static PixelFormat Rgba64 { get; } = new PixelFormat(PixelFormatEnum.Rgba64);
+ public static PixelFormat Bgra8888 { get; } = new PixelFormat(PixelFormatEnum.Bgra8888);
+ public static PixelFormat BlackWhite { get; } = new PixelFormat(PixelFormatEnum.BlackWhite);
+ public static PixelFormat Gray2 { get; } = new PixelFormat(PixelFormatEnum.Gray2);
+ public static PixelFormat Gray4 { get; } = new PixelFormat(PixelFormatEnum.Gray4);
+ public static PixelFormat Gray8 { get; } = new PixelFormat(PixelFormatEnum.Gray8);
+ public static PixelFormat Gray16 { get; } = new PixelFormat(PixelFormatEnum.Gray16);
+ public static PixelFormat Gray32Float { get; } = new PixelFormat(PixelFormatEnum.Gray32Float);
}
}
diff --git a/src/Avalonia.Base/Platform/Storage/FileIO/BclStorageFile.cs b/src/Avalonia.Base/Platform/Storage/FileIO/BclStorageFile.cs
index cec20678cf..a4005d4f5f 100644
--- a/src/Avalonia.Base/Platform/Storage/FileIO/BclStorageFile.cs
+++ b/src/Avalonia.Base/Platform/Storage/FileIO/BclStorageFile.cs
@@ -1,50 +1,65 @@
using System;
-using System.Diagnostics.CodeAnalysis;
using System.IO;
using System.Security;
using System.Threading.Tasks;
-using Avalonia.Metadata;
namespace Avalonia.Platform.Storage.FileIO;
-[Unstable]
-public class BclStorageFile : IStorageBookmarkFile
+internal class BclStorageFile : IStorageBookmarkFile
{
- private readonly FileInfo _fileInfo;
-
public BclStorageFile(string fileName)
{
- _fileInfo = new FileInfo(fileName);
+ FileInfo = new FileInfo(fileName);
}
public BclStorageFile(FileInfo fileInfo)
{
- _fileInfo = fileInfo ?? throw new ArgumentNullException(nameof(fileInfo));
+ FileInfo = fileInfo ?? throw new ArgumentNullException(nameof(fileInfo));
}
+ public FileInfo FileInfo { get; }
+
public bool CanOpenRead => true;
public bool CanOpenWrite => true;
- public string Name => _fileInfo.Name;
+ public string Name => FileInfo.Name;
public virtual bool CanBookmark => true;
+ public Uri Path
+ {
+ get
+ {
+ try
+ {
+ if (FileInfo.Directory is not null)
+ {
+ return StorageProviderHelpers.FilePathToUri(FileInfo.FullName);
+ }
+ }
+ catch (SecurityException)
+ {
+ }
+ return new Uri(FileInfo.Name, UriKind.Relative);
+ }
+ }
+
public Task GetBasicPropertiesAsync()
{
- if (_fileInfo.Exists)
+ if (FileInfo.Exists)
{
return Task.FromResult(new StorageItemProperties(
- (ulong)_fileInfo.Length,
- _fileInfo.CreationTimeUtc,
- _fileInfo.LastAccessTimeUtc));
+ (ulong)FileInfo.Length,
+ FileInfo.CreationTimeUtc,
+ FileInfo.LastAccessTimeUtc));
}
return Task.FromResult(new StorageItemProperties());
}
public Task GetParentAsync()
{
- if (_fileInfo.Directory is { } directory)
+ if (FileInfo.Directory is { } directory)
{
return Task.FromResult(new BclStorageFolder(directory));
}
@@ -53,17 +68,17 @@ public class BclStorageFile : IStorageBookmarkFile
public Task OpenReadAsync()
{
- return Task.FromResult(_fileInfo.OpenRead());
+ return Task.FromResult(FileInfo.OpenRead());
}
public Task OpenWriteAsync()
{
- return Task.FromResult(_fileInfo.OpenWrite());
+ return Task.FromResult(FileInfo.OpenWrite());
}
public virtual Task SaveBookmarkAsync()
{
- return Task.FromResult(_fileInfo.FullName);
+ return Task.FromResult(FileInfo.FullName);
}
public Task ReleaseBookmarkAsync()
@@ -72,28 +87,6 @@ public class BclStorageFile : IStorageBookmarkFile
return Task.CompletedTask;
}
- public bool TryGetUri([NotNullWhen(true)] out Uri? uri)
- {
- try
- {
- if (_fileInfo.Directory is not null)
- {
- uri = Path.IsPathRooted(_fileInfo.FullName) ?
- new Uri(new Uri("file://"), _fileInfo.FullName) :
- new Uri(_fileInfo.FullName, UriKind.Relative);
- return true;
- }
-
- uri = null;
- return false;
- }
- catch (SecurityException)
- {
- uri = null;
- return false;
- }
- }
-
protected virtual void Dispose(bool disposing)
{
}
diff --git a/src/Avalonia.Base/Platform/Storage/FileIO/BclStorageFolder.cs b/src/Avalonia.Base/Platform/Storage/FileIO/BclStorageFolder.cs
index b91e910777..1e21c197bb 100644
--- a/src/Avalonia.Base/Platform/Storage/FileIO/BclStorageFolder.cs
+++ b/src/Avalonia.Base/Platform/Storage/FileIO/BclStorageFolder.cs
@@ -1,23 +1,18 @@
using System;
using System.Collections.Generic;
-using System.Diagnostics.CodeAnalysis;
using System.IO;
using System.Linq;
using System.Security;
using System.Threading.Tasks;
-using Avalonia.Metadata;
namespace Avalonia.Platform.Storage.FileIO;
-[Unstable]
-public class BclStorageFolder : IStorageBookmarkFolder
+internal class BclStorageFolder : IStorageBookmarkFolder
{
- private readonly DirectoryInfo _directoryInfo;
-
public BclStorageFolder(string path)
{
- _directoryInfo = new DirectoryInfo(path);
- if (!_directoryInfo.Exists)
+ DirectoryInfo = new DirectoryInfo(path);
+ if (!DirectoryInfo.Exists)
{
throw new ArgumentException("Directory must exist");
}
@@ -25,29 +20,46 @@ public class BclStorageFolder : IStorageBookmarkFolder
public BclStorageFolder(DirectoryInfo directoryInfo)
{
- _directoryInfo = directoryInfo ?? throw new ArgumentNullException(nameof(directoryInfo));
- if (!_directoryInfo.Exists)
+ DirectoryInfo = directoryInfo ?? throw new ArgumentNullException(nameof(directoryInfo));
+ if (!DirectoryInfo.Exists)
{
throw new ArgumentException("Directory must exist", nameof(directoryInfo));
}
}
- public string Name => _directoryInfo.Name;
+ public string Name => DirectoryInfo.Name;
+
+ public DirectoryInfo DirectoryInfo { get; }
public bool CanBookmark => true;
+ public Uri Path
+ {
+ get
+ {
+ try
+ {
+ return StorageProviderHelpers.FilePathToUri(DirectoryInfo.FullName);
+ }
+ catch (SecurityException)
+ {
+ return new Uri(DirectoryInfo.Name, UriKind.Relative);
+ }
+ }
+ }
+
public Task GetBasicPropertiesAsync()
{
var props = new StorageItemProperties(
null,
- _directoryInfo.CreationTimeUtc,
- _directoryInfo.LastAccessTimeUtc);
+ DirectoryInfo.CreationTimeUtc,
+ DirectoryInfo.LastAccessTimeUtc);
return Task.FromResult(props);
}
public Task GetParentAsync()
{
- if (_directoryInfo.Parent is { } directory)
+ if (DirectoryInfo.Parent is { } directory)
{
return Task.FromResult(new BclStorageFolder(directory));
}
@@ -56,9 +68,9 @@ public class BclStorageFolder : IStorageBookmarkFolder
public Task> GetItemsAsync()
{
- var items = _directoryInfo.GetDirectories()
+ var items = DirectoryInfo.GetDirectories()
.Select(d => (IStorageItem)new BclStorageFolder(d))
- .Concat(_directoryInfo.GetFiles().Select(f => new BclStorageFile(f)))
+ .Concat(DirectoryInfo.GetFiles().Select(f => new BclStorageFile(f)))
.ToArray();
return Task.FromResult>(items);
@@ -66,7 +78,7 @@ public class BclStorageFolder : IStorageBookmarkFolder
public virtual Task SaveBookmarkAsync()
{
- return Task.FromResult(_directoryInfo.FullName);
+ return Task.FromResult(DirectoryInfo.FullName);
}
public Task ReleaseBookmarkAsync()
@@ -75,23 +87,6 @@ public class BclStorageFolder : IStorageBookmarkFolder
return Task.CompletedTask;
}
- public bool TryGetUri([NotNullWhen(true)] out Uri? uri)
- {
- try
- {
- uri = Path.IsPathRooted(_directoryInfo.FullName) ?
- new Uri(new Uri("file://"), _directoryInfo.FullName) :
- new Uri(_directoryInfo.FullName, UriKind.Relative);
-
- return true;
- }
- catch (SecurityException)
- {
- uri = null;
- return false;
- }
- }
-
protected virtual void Dispose(bool disposing)
{
}
diff --git a/src/Avalonia.Base/Platform/Storage/FileIO/BclStorageProvider.cs b/src/Avalonia.Base/Platform/Storage/FileIO/BclStorageProvider.cs
index 469388021e..ee169d62a5 100644
--- a/src/Avalonia.Base/Platform/Storage/FileIO/BclStorageProvider.cs
+++ b/src/Avalonia.Base/Platform/Storage/FileIO/BclStorageProvider.cs
@@ -1,12 +1,13 @@
-using System.Collections.Generic;
+using System;
+using System.Collections.Generic;
using System.IO;
+using System.Runtime.InteropServices;
using System.Threading.Tasks;
-using Avalonia.Metadata;
+using Avalonia.Compatibility;
namespace Avalonia.Platform.Storage.FileIO;
-[Unstable]
-public abstract class BclStorageProvider : IStorageProvider
+internal abstract class BclStorageProvider : IStorageProvider
{
public abstract bool CanOpen { get; }
public abstract Task> OpenFilePickerAsync(FilePickerOpenOptions options);
@@ -32,4 +33,98 @@ public abstract class BclStorageProvider : IStorageProvider
? Task.FromResult(new BclStorageFolder(folder))
: Task.FromResult(null);
}
+
+ public virtual Task TryGetFileFromPath(Uri filePath)
+ {
+ if (filePath.IsAbsoluteUri)
+ {
+ var file = new FileInfo(filePath.LocalPath);
+ if (file.Exists)
+ {
+ return Task.FromResult(new BclStorageFile(file));
+ }
+ }
+
+ return Task.FromResult(null);
+ }
+
+ public virtual Task TryGetFolderFromPath(Uri folderPath)
+ {
+ if (folderPath.IsAbsoluteUri)
+ {
+ var directory = new DirectoryInfo(folderPath.LocalPath);
+ if (directory.Exists)
+ {
+ return Task.FromResult(new BclStorageFolder(directory));
+ }
+ }
+
+ return Task.FromResult(null);
+ }
+
+ public virtual Task TryGetWellKnownFolder(WellKnownFolder wellKnownFolder)
+ {
+ // Note, this BCL API returns different values depending on the .NET version.
+ // We should also document it.
+ // https://github.com/dotnet/docs/issues/31423
+ // For pre-breaking change values see table:
+ // https://johnkoerner.com/csharp/special-folder-values-on-windows-versus-mac/
+ var folderPath = wellKnownFolder switch
+ {
+ WellKnownFolder.Desktop => GetFromSpecialFolder(Environment.SpecialFolder.Desktop),
+ WellKnownFolder.Documents => GetFromSpecialFolder(Environment.SpecialFolder.MyDocuments),
+ WellKnownFolder.Downloads => GetDownloadsWellKnownFolder(),
+ WellKnownFolder.Music => GetFromSpecialFolder(Environment.SpecialFolder.MyMusic),
+ WellKnownFolder.Pictures => GetFromSpecialFolder(Environment.SpecialFolder.MyPictures),
+ WellKnownFolder.Videos => GetFromSpecialFolder(Environment.SpecialFolder.MyVideos),
+ _ => throw new ArgumentOutOfRangeException(nameof(wellKnownFolder), wellKnownFolder, null)
+ };
+
+ if (folderPath is null)
+ {
+ return Task.FromResult(null);
+ }
+
+ var directory = new DirectoryInfo(folderPath);
+ if (!directory.Exists)
+ {
+ return Task.FromResult(null);
+ }
+
+ return Task.FromResult(new BclStorageFolder(directory));
+
+ string GetFromSpecialFolder(Environment.SpecialFolder folder) =>
+ Environment.GetFolderPath(folder, Environment.SpecialFolderOption.Create);
+ }
+
+ // TODO, replace with https://github.com/dotnet/runtime/issues/70484 when implemented.
+ // Normally we want to avoid platform specific code in the Avalonia.Base assembly.
+ protected static string? GetDownloadsWellKnownFolder()
+ {
+ if (OperatingSystemEx.IsWindows())
+ {
+ return Environment.OSVersion.Version.Major < 6 ? null :
+ SHGetKnownFolderPath(s_folderDownloads, 0, IntPtr.Zero);
+ }
+
+ if (OperatingSystemEx.IsLinux())
+ {
+ var envDir = Environment.GetEnvironmentVariable("XDG_DOWNLOAD_DIR");
+ if (envDir != null && Directory.Exists(envDir))
+ {
+ return envDir;
+ }
+ }
+
+ if (OperatingSystemEx.IsLinux() || OperatingSystemEx.IsMacOS())
+ {
+ return "~/Downloads";
+ }
+
+ return null;
+ }
+
+ private static readonly Guid s_folderDownloads = new Guid("374DE290-123F-4565-9164-39C4925E467B");
+ [DllImport("shell32.dll", CharSet = CharSet.Unicode, ExactSpelling = true, PreserveSig = false)]
+ private static extern string SHGetKnownFolderPath([MarshalAs(UnmanagedType.LPStruct)] Guid id, int flags, IntPtr token);
}
diff --git a/src/Avalonia.Base/Platform/Storage/FileIO/StorageProviderHelpers.cs b/src/Avalonia.Base/Platform/Storage/FileIO/StorageProviderHelpers.cs
index f90d0a5a2f..55e84ee937 100644
--- a/src/Avalonia.Base/Platform/Storage/FileIO/StorageProviderHelpers.cs
+++ b/src/Avalonia.Base/Platform/Storage/FileIO/StorageProviderHelpers.cs
@@ -1,13 +1,23 @@
using System;
using System.IO;
using System.Linq;
-using Avalonia.Metadata;
+using System.Text;
namespace Avalonia.Platform.Storage.FileIO;
-[Unstable]
-public static class StorageProviderHelpers
+internal static class StorageProviderHelpers
{
+ public static Uri FilePathToUri(string path)
+ {
+ var uriPath = new StringBuilder(path)
+ .Replace("%", $"%{(int)'%':X2}")
+ .Replace("[", $"%{(int)'[':X2}")
+ .Replace("]", $"%{(int)']':X2}")
+ .ToString();
+
+ return new UriBuilder("file", string.Empty) { Path = uriPath }.Uri;
+ }
+
public static string NameWithExtension(string path, string? defaultExtension, FilePickerFileType? filter)
{
var name = Path.GetFileName(path);
diff --git a/src/Avalonia.Base/Platform/Storage/IStorageItem.cs b/src/Avalonia.Base/Platform/Storage/IStorageItem.cs
index f5469d31c9..c2c93400b0 100644
--- a/src/Avalonia.Base/Platform/Storage/IStorageItem.cs
+++ b/src/Avalonia.Base/Platform/Storage/IStorageItem.cs
@@ -20,13 +20,13 @@ public interface IStorageItem : IDisposable
string Name { get; }
///
- /// Gets the full file-system path of the item, if the item has a path.
+ /// Gets the file-system path of the item.
///
///
/// Android backend might return file path with "content:" scheme.
/// Browser and iOS backends might return relative uris.
///
- bool TryGetUri([NotNullWhen(true)] out Uri? uri);
+ Uri Path { get; }
///
/// Gets the basic properties of the current item.
diff --git a/src/Avalonia.Base/Platform/Storage/IStorageProvider.cs b/src/Avalonia.Base/Platform/Storage/IStorageProvider.cs
index 0f5cf931d9..6922151e02 100644
--- a/src/Avalonia.Base/Platform/Storage/IStorageProvider.cs
+++ b/src/Avalonia.Base/Platform/Storage/IStorageProvider.cs
@@ -1,4 +1,6 @@
-using System.Collections.Generic;
+using System;
+using System.Collections.Generic;
+using System.Runtime.Versioning;
using System.Threading.Tasks;
using Avalonia.Metadata;
@@ -53,4 +55,35 @@ public interface IStorageProvider
/// Bookmark ID.
/// Bookmarked folder or null if OS denied request.
Task OpenFolderBookmarkAsync(string bookmark);
+
+ ///
+ /// Attempts to read file from the file-system by its path.
+ ///
+ /// The path of the item to retrieve in Uri format.
+ ///
+ /// Uri path is usually expected to be an absolute path with "file" scheme.
+ /// But it can be an uri with "content" scheme on the Android.
+ /// It also might ask user for the permission, and throw an exception if it was denied.
+ ///
+ /// File or null if it doesn't exist.
+ Task TryGetFileFromPath(Uri filePath);
+
+ ///
+ /// Attempts to read folder from the file-system by its path.
+ ///
+ /// The path of the folder to retrieve in Uri format.
+ ///
+ /// Uri path is usually expected to be an absolute path with "file" scheme.
+ /// But it can be an uri with "content" scheme on the Android.
+ /// It also might ask user for the permission, and throw an exception if it was denied.
+ ///
+ /// Folder or null if it doesn't exist.
+ Task TryGetFolderFromPath(Uri folderPath);
+
+ ///
+ /// Attempts to read folder from the file-system by its path
+ ///
+ /// Well known folder identifier.
+ /// Folder or null if it doesn't exist.
+ Task TryGetWellKnownFolder(WellKnownFolder wellKnownFolder);
}
diff --git a/src/Avalonia.Base/Platform/Storage/NameCollisionOption.cs b/src/Avalonia.Base/Platform/Storage/NameCollisionOption.cs
new file mode 100644
index 0000000000..4a0d0c634f
--- /dev/null
+++ b/src/Avalonia.Base/Platform/Storage/NameCollisionOption.cs
@@ -0,0 +1,6 @@
+namespace Avalonia.Platform.Storage;
+
+public class NameCollisionOption
+{
+
+}
diff --git a/src/Avalonia.Base/Platform/Storage/StorageProviderExtensions.cs b/src/Avalonia.Base/Platform/Storage/StorageProviderExtensions.cs
new file mode 100644
index 0000000000..c7772d1196
--- /dev/null
+++ b/src/Avalonia.Base/Platform/Storage/StorageProviderExtensions.cs
@@ -0,0 +1,55 @@
+using System.Threading.Tasks;
+using Avalonia.Platform.Storage.FileIO;
+
+namespace Avalonia.Platform.Storage;
+
+///
+/// Group of public extensions for class.
+///
+public static class StorageProviderExtensions
+{
+ ///
+ public static Task TryGetFileFromPath(this IStorageProvider provider, string filePath)
+ {
+ return provider.TryGetFileFromPath(StorageProviderHelpers.FilePathToUri(filePath));
+ }
+
+ ///
+ public static Task TryGetFolderFromPath(this IStorageProvider provider, string folderPath)
+ {
+ return provider.TryGetFolderFromPath(StorageProviderHelpers.FilePathToUri(folderPath));
+ }
+
+ internal static string? TryGetFullPath(this IStorageFolder folder)
+ {
+ // We can avoid double escaping of the path by checking for BclStorageFolder.
+ // Ideally, `folder.Path.LocalPath` should also work, as that's only available way for the users.
+ if (folder is BclStorageFolder storageFolder)
+ {
+ return storageFolder.DirectoryInfo.FullName;
+ }
+
+ if (folder.Path is { IsAbsoluteUri: true, Scheme: "file" } absolutePath)
+ {
+ return absolutePath.LocalPath;
+ }
+
+ // android "content:", browser and ios relative links go here.
+ return null;
+ }
+
+ internal static string? TryGetFullPath(this IStorageFile file)
+ {
+ if (file is BclStorageFile storageFolder)
+ {
+ return storageFolder.FileInfo.FullName;
+ }
+
+ if (file.Path is { IsAbsoluteUri: true, Scheme: "file" } absolutePath)
+ {
+ return absolutePath.LocalPath;
+ }
+
+ return null;
+ }
+}
diff --git a/src/Avalonia.Base/Platform/Storage/WellKnownFolder.cs b/src/Avalonia.Base/Platform/Storage/WellKnownFolder.cs
new file mode 100644
index 0000000000..fcc933bc53
--- /dev/null
+++ b/src/Avalonia.Base/Platform/Storage/WellKnownFolder.cs
@@ -0,0 +1,37 @@
+namespace Avalonia.Platform.Storage;
+
+///
+/// Specifies enumerated constants used to retrieve directory paths to system well known folders.
+///
+public enum WellKnownFolder
+{
+ ///
+ /// Current user desktop folder.
+ ///
+ Desktop,
+
+ ///
+ /// Current user documents folder.
+ ///
+ Documents,
+
+ ///
+ /// Current user downloads folder.
+ ///
+ Downloads,
+
+ ///
+ /// Current user music folder
+ ///
+ Music,
+
+ ///
+ /// Current user pictures folder
+ ///
+ Pictures,
+
+ ///
+ /// Current user videos folder
+ ///
+ Videos,
+}
diff --git a/src/Avalonia.Base/Platform/SystemNavigationManager.cs b/src/Avalonia.Base/Platform/SystemNavigationManagerImpl.cs
similarity index 50%
rename from src/Avalonia.Base/Platform/SystemNavigationManager.cs
rename to src/Avalonia.Base/Platform/SystemNavigationManagerImpl.cs
index 6165b2bb77..080cd8083e 100644
--- a/src/Avalonia.Base/Platform/SystemNavigationManager.cs
+++ b/src/Avalonia.Base/Platform/SystemNavigationManagerImpl.cs
@@ -5,13 +5,7 @@ using Avalonia.Metadata;
namespace Avalonia.Platform
{
[Unstable]
- public interface ITopLevelWithSystemNavigationManager
- {
- ISystemNavigationManager SystemNavigationManager { get; }
- }
-
- [Unstable]
- public interface ISystemNavigationManager
+ public interface ISystemNavigationManagerImpl
{
public event EventHandler? BackRequested;
}
diff --git a/src/Avalonia.Base/Point.cs b/src/Avalonia.Base/Point.cs
index 0c789ff20f..86f7adf0d1 100644
--- a/src/Avalonia.Base/Point.cs
+++ b/src/Avalonia.Base/Point.cs
@@ -251,6 +251,12 @@ namespace Avalonia
/// The transform.
/// The transformed point.
public Point Transform(Matrix transform) => transform.Transform(this);
+
+ internal Point Transform(Matrix4x4 matrix)
+ {
+ var vec = Vector2.Transform(new Vector2((float)X, (float)Y), matrix);
+ return new Point(vec.X, vec.Y);
+ }
///
/// Returns a new point with the specified X coordinate.
diff --git a/src/Avalonia.Base/Rect.cs b/src/Avalonia.Base/Rect.cs
index 831ab28adc..49dc087933 100644
--- a/src/Avalonia.Base/Rect.cs
+++ b/src/Avalonia.Base/Rect.cs
@@ -1,5 +1,6 @@
using System;
using System.Globalization;
+using System.Numerics;
using Avalonia.Animation.Animators;
using Avalonia.Utilities;
@@ -441,6 +442,32 @@ namespace Avalonia
return new Rect(new Point(left, top), new Point(right, bottom));
}
+
+ internal Rect TransformToAABB(Matrix4x4 matrix)
+ {
+ ReadOnlySpan points = stackalloc Point[4]
+ {
+ TopLeft.Transform(matrix),
+ TopRight.Transform(matrix),
+ BottomRight.Transform(matrix),
+ BottomLeft.Transform(matrix)
+ };
+
+ var left = double.MaxValue;
+ var right = double.MinValue;
+ var top = double.MaxValue;
+ var bottom = double.MinValue;
+
+ foreach (var p in points)
+ {
+ if (p.X < left) left = p.X;
+ if (p.X > right) right = p.X;
+ if (p.Y < top) top = p.Y;
+ if (p.Y > bottom) bottom = p.Y;
+ }
+
+ return new Rect(new Point(left, top), new Point(right, bottom));
+ }
///
/// Translates the rectangle by an offset.
diff --git a/src/Avalonia.Base/Rendering/Composition/CompositingRenderer.cs b/src/Avalonia.Base/Rendering/Composition/CompositingRenderer.cs
index a2209db09f..c5d7ec61e0 100644
--- a/src/Avalonia.Base/Rendering/Composition/CompositingRenderer.cs
+++ b/src/Avalonia.Base/Rendering/Composition/CompositingRenderer.cs
@@ -83,8 +83,16 @@ public class CompositingRenderer : IRendererWithCompositor
}
///
- public IEnumerable HitTest(Point p, Visual root, Func? filter)
+ public IEnumerable HitTest(Point p, Visual? root, Func? filter)
{
+ CompositionVisual? rootVisual = null;
+ if (root != null)
+ {
+ if (root.CompositionVisual == null)
+ yield break;
+ rootVisual = root.CompositionVisual;
+ }
+
Func? f = null;
if (filter != null)
f = v =>
@@ -93,8 +101,8 @@ public class CompositingRenderer : IRendererWithCompositor
return filter(dlv.Visual);
return true;
};
-
- var res = CompositionTarget.TryHitTest(p, f);
+
+ var res = CompositionTarget.TryHitTest(p, rootVisual, f);
if(res == null)
yield break;
foreach(var v in res)
@@ -257,6 +265,7 @@ public class CompositingRenderer : IRendererWithCompositor
_recalculateChildren.Clear();
CompositionTarget.Size = _root.ClientSize;
CompositionTarget.Scaling = _root.RenderScaling;
+ TriggerSceneInvalidatedOnBatchCompletion(_compositor.RequestCommitAsync());
}
private async void TriggerSceneInvalidatedOnBatchCompletion(Task batchCompletion)
diff --git a/src/Avalonia.Base/Rendering/Composition/CompositionDrawListVisual.cs b/src/Avalonia.Base/Rendering/Composition/CompositionDrawListVisual.cs
index b019d1792b..f1905fec08 100644
--- a/src/Avalonia.Base/Rendering/Composition/CompositionDrawListVisual.cs
+++ b/src/Avalonia.Base/Rendering/Composition/CompositionDrawListVisual.cs
@@ -61,9 +61,6 @@ internal class CompositionDrawListVisual : CompositionContainerVisual
return false;
if (custom != null)
{
- // Simulate the old behavior
- // TODO: Change behavior once legacy renderers are removed
- pt += new Point(Offset.X, Offset.Y);
return custom.HitTest(pt);
}
diff --git a/src/Avalonia.Base/Rendering/Composition/CompositionDrawingSurface.cs b/src/Avalonia.Base/Rendering/Composition/CompositionDrawingSurface.cs
index 2f7e261250..ab4329df62 100644
--- a/src/Avalonia.Base/Rendering/Composition/CompositionDrawingSurface.cs
+++ b/src/Avalonia.Base/Rendering/Composition/CompositionDrawingSurface.cs
@@ -1,6 +1,7 @@
using System;
using System.Threading.Tasks;
using Avalonia.Rendering.Composition.Server;
+using Avalonia.Threading;
namespace Avalonia.Rendering.Composition;
@@ -55,6 +56,6 @@ public class CompositionDrawingSurface : CompositionSurface
~CompositionDrawingSurface()
{
- Dispose();
+ Dispatcher.UIThread.Post(Dispose);
}
}
diff --git a/src/Avalonia.Base/Rendering/Composition/CompositionTarget.cs b/src/Avalonia.Base/Rendering/Composition/CompositionTarget.cs
index 0c24d6cd44..f2fa846205 100644
--- a/src/Avalonia.Base/Rendering/Composition/CompositionTarget.cs
+++ b/src/Avalonia.Base/Rendering/Composition/CompositionTarget.cs
@@ -28,16 +28,15 @@ namespace Avalonia.Rendering.Composition
///
/// Attempts to perform a hit-tst
///
- ///
- ///
///
- public PooledList? TryHitTest(Point point, Func? filter)
+ public PooledList? TryHitTest(Point point, CompositionVisual? root, Func? filter)
{
Server.Readback.NextRead();
- if (Root == null)
+ root ??= Root;
+ if (root == null)
return null;
var res = new PooledList();
- HitTestCore(Root, point, res, filter);
+ HitTestCore(root, point, res, filter);
return res;
}
diff --git a/src/Avalonia.Base/Rendering/Composition/Compositor.cs b/src/Avalonia.Base/Rendering/Composition/Compositor.cs
index bdcbe65403..aea4df525d 100644
--- a/src/Avalonia.Base/Rendering/Composition/Compositor.cs
+++ b/src/Avalonia.Base/Rendering/Composition/Compositor.cs
@@ -37,6 +37,8 @@ namespace Avalonia.Rendering.Composition
private List _pendingServerCompositorJobs = new();
internal IEasing DefaultEasing { get; }
+
+ internal event Action? AfterCommit;
///
@@ -88,6 +90,7 @@ namespace Avalonia.Rendering.Composition
{
if (_invokeBeforeCommitWrite.Count > 0)
RequestCommitAsync();
+ AfterCommit?.Invoke();
}
}
diff --git a/src/Avalonia.Base/Rendering/DeferredRenderer.cs b/src/Avalonia.Base/Rendering/DeferredRenderer.cs
deleted file mode 100644
index f0b993a2b0..0000000000
--- a/src/Avalonia.Base/Rendering/DeferredRenderer.cs
+++ /dev/null
@@ -1,780 +0,0 @@
-using System;
-using System.Collections.Generic;
-using System.Diagnostics.CodeAnalysis;
-using System.IO;
-using System.Linq;
-using System.Threading.Tasks;
-using Avalonia.Logging;
-using Avalonia.Media;
-using Avalonia.Media.Immutable;
-using Avalonia.Platform;
-using Avalonia.Rendering.SceneGraph;
-using Avalonia.Threading;
-using Avalonia.Utilities;
-using Avalonia.VisualTree;
-
-namespace Avalonia.Rendering
-{
- ///
- /// A renderer which renders the state of the visual tree to an intermediate scene graph
- /// representation which is then rendered on a rendering thread.
- ///
- public class DeferredRenderer : RendererBase, IRenderer, IRenderLoopTask, IVisualBrushRenderer
- {
- private readonly IDispatcher? _dispatcher;
- private readonly IRenderLoop? _renderLoop;
- private readonly Func? _renderTargetFactory;
- private readonly PlatformRenderInterfaceContextManager? _renderInterface;
- private readonly Visual _root;
- private readonly ISceneBuilder _sceneBuilder;
-
- private bool _running;
- private bool _disposed;
- private volatile IRef? _scene;
- private DirtyVisuals? _dirty;
- private HashSet? _recalculateChildren;
- private IRef? _overlay;
- private int _lastSceneId = -1;
- private DisplayDirtyRects _dirtyRectsDisplay = new DisplayDirtyRects();
- private IRef? _currentDraw;
- private readonly IDeferredRendererLock _lock;
- private readonly object _sceneLock = new object();
- private readonly object _startStopLock = new object();
- private readonly object _renderLoopIsRenderingLock = new object();
- private readonly Action _updateSceneIfNeededDelegate;
- private List? _pendingRenderThreadJobs;
-
- ///
- /// Initializes a new instance of the class.
- ///
- /// The control to render.
- /// The render loop.
- /// The target render factory.
- /// The Platform Render Context.
- /// The scene builder to use. Optional.
- /// The dispatcher to use. Optional.
- /// Lock object used before trying to access render target
- public DeferredRenderer(
- IRenderRoot root,
- IRenderLoop renderLoop,
- Func renderTargetFactory,
- PlatformRenderInterfaceContextManager? renderInterface = null,
- ISceneBuilder? sceneBuilder = null,
- IDispatcher? dispatcher = null,
- IDeferredRendererLock? rendererLock = null) : base(true)
- {
- _dispatcher = dispatcher ?? Dispatcher.UIThread;
- _root = root as Visual ?? throw new ArgumentNullException(nameof(root));
- _sceneBuilder = sceneBuilder ?? new SceneBuilder();
- Layers = new RenderLayers();
- _renderLoop = renderLoop;
- _renderTargetFactory = renderTargetFactory;
- _renderInterface = renderInterface;
- _lock = rendererLock ?? new ManagedDeferredRendererLock();
- _updateSceneIfNeededDelegate = UpdateSceneIfNeeded;
- }
-
- ///
- /// Initializes a new instance of the class.
- ///
- /// The control to render.
- /// The render target.
- /// The scene builder to use. Optional.
- ///
- /// This constructor is intended to be used for unit testing.
- ///
- public DeferredRenderer(
- Visual root,
- IRenderTarget renderTarget,
- ISceneBuilder? sceneBuilder = null) : base(true)
- {
- _root = root ?? throw new ArgumentNullException(nameof(root));
- RenderTarget = renderTarget ?? throw new ArgumentNullException(nameof(renderTarget));
- _sceneBuilder = sceneBuilder ?? new SceneBuilder();
- Layers = new RenderLayers();
- _lock = new ManagedDeferredRendererLock();
- _updateSceneIfNeededDelegate = UpdateSceneIfNeeded;
- }
-
- ///
- public bool DrawFps { get; set; }
-
- ///
- public bool DrawDirtyRects { get; set; }
-
- ///
- /// Gets or sets a path to which rendered frame should be rendered for debugging.
- ///
- public string? DebugFramesPath { get; set; }
-
- ///
- /// Forces the renderer to only draw frames on the render thread. Makes Paint to wait until frame is rendered
- ///
- public bool RenderOnlyOnRenderThread { get; set; }
-
- ///
- public event EventHandler? SceneInvalidated;
-
- ///
- /// Gets the render layers.
- ///
- internal RenderLayers Layers { get; }
-
- ///
- /// Gets the current render target.
- ///
- internal IRenderTarget? RenderTarget { get; private set; }
-
- ///
- public void AddDirty(Visual visual)
- {
- _dirty?.Add(visual);
- }
-
- ///
- /// Disposes of the renderer and detaches from the render loop.
- ///
- public void Dispose()
- {
- lock (_sceneLock)
- {
- if (_disposed)
- return;
- _disposed = true;
- var scene = _scene;
- _scene = null;
- scene?.Dispose();
- }
-
- Stop();
- // Wait for any in-progress rendering to complete
- lock(_renderLoopIsRenderingLock){}
- DisposeRenderTarget();
- }
-
- public void RecalculateChildren(Visual visual) => _recalculateChildren?.Add(visual);
-
- void DisposeRenderTarget()
- {
- using (var l = _lock.TryLock())
- {
- if(l == null)
- {
- // We are still trying to render on the render thread, try again a bit later
- DispatcherTimer.RunOnce(DisposeRenderTarget, TimeSpan.FromMilliseconds(50),
- DispatcherPriority.Background);
- return;
- }
-
- Layers.Clear();
- RenderTarget?.Dispose();
- RenderTarget = null;
- }
- }
-
- ///
- public IEnumerable HitTest(Point p, Visual root, Func? filter)
- {
- EnsureCanHitTest();
-
- //It's safe to access _scene here without a lock since
- //it's only changed from UI thread which we are currently on
- return _scene?.Item.HitTest(p, root, filter) ?? Enumerable.Empty();
- }
-
- ///
- public Visual? HitTestFirst(Point p, Visual root, Func? filter)
- {
- EnsureCanHitTest();
-
- //It's safe to access _scene here without a lock since
- //it's only changed from UI thread which we are currently on
- return _scene?.Item.HitTestFirst(p, root, filter);
- }
-
- ///
- public void Paint(Rect rect)
- {
- if (RenderOnlyOnRenderThread)
- {
- // Renderer is stopped and doesn't tick on the render thread
- // This indicates a bug somewhere in our code
- // (currently happens when a window gets minimized with Show desktop on Windows 10)
- if(!_running)
- return;
-
- while (true)
- {
- Scene? scene;
- bool? updated;
- lock (_sceneLock)
- {
- updated = UpdateScene();
- scene = _scene?.Item;
- }
-
- // Renderer is in invalid state, skip drawing
- if(updated == null)
- return;
-
- // Wait for the scene to be rendered or disposed
- scene?.Rendered.Wait();
-
- // That was an up-to-date scene, we can return immediately
- if (updated == true)
- return;
- }
- }
- else
- {
- var t = (IRenderLoopTask)this;
- if (t.NeedsUpdate)
- UpdateScene();
- if (_scene?.Item != null)
- Render(true);
- }
- }
-
- ///
- public void Resized(Size size)
- {
- }
-
- ///
- public void Start()
- {
- lock (_startStopLock)
- {
- if (!_running && _renderLoop != null)
- {
- _renderLoop.Add(this);
- _running = true;
- }
- }
- }
-
- ///
- public void Stop()
- {
- lock (_startStopLock)
- {
- if (_running && _renderLoop != null)
- {
- _renderLoop.Remove(this);
- _running = false;
- }
- }
- }
-
- public ValueTask TryGetRenderInterfaceFeature(Type featureType)
- {
- if (_renderInterface == null)
- return new((object?)null);
-
- var tcs = new TaskCompletionSource();
- _pendingRenderThreadJobs ??= new();
- _pendingRenderThreadJobs.Add(() =>
- {
- try
- {
- using (_renderInterface.EnsureCurrent())
- {
- tcs.TrySetResult(_renderInterface.Value.TryGetFeature(featureType));
- }
- }
- catch (Exception e)
- {
- tcs.TrySetException(e);
- }
- });
- return new ValueTask(tcs.Task);
- }
-
- bool NeedsUpdate => _dirty == null || _dirty.Count > 0;
- bool IRenderLoopTask.NeedsUpdate => NeedsUpdate;
-
- void IRenderLoopTask.Update(TimeSpan time) => UpdateScene();
-
- void IRenderLoopTask.Render()
- {
- lock (_renderLoopIsRenderingLock)
- {
- lock(_startStopLock)
- if(!_running)
- return;
- Render(false);
- }
- }
-
- static Scene? TryGetChildScene(IRef? op) => (op?.Item as BrushDrawOperation)?.Aux as Scene;
-
- ///
- Size IVisualBrushRenderer.GetRenderTargetSize(IVisualBrush brush)
- {
- return TryGetChildScene(_currentDraw)?.Size ?? default;
- }
-
- ///
- void IVisualBrushRenderer.RenderVisualBrush(IDrawingContextImpl context, IVisualBrush brush)
- {
- var childScene = TryGetChildScene(_currentDraw);
-
- if (childScene != null)
- {
- Render(context, (VisualNode)childScene.Root, null, new Rect(childScene.Size));
- }
- }
-
- internal void UnitTestUpdateScene() => UpdateScene();
-
- internal void UnitTestRender() => Render(false);
-
- internal Scene? UnitTestScene() => _scene?.Item;
-
- private void EnsureCanHitTest()
- {
- if (_renderLoop == null && (_dirty == null || _dirty.Count > 0))
- {
- // When unit testing the renderLoop may be null, so update the scene manually.
- UpdateScene();
- }
- }
-
- internal void Render(bool forceComposite)
- {
- using (var l = _lock.TryLock())
- {
- if (l == null)
- return;
-
- IDrawingContextImpl? context = null;
- try
- {
- try
- {
- var (scene, updated) = UpdateRenderLayersAndConsumeSceneIfNeeded(ref context);
- if (updated)
- FpsTick();
- using (scene)
- {
- if (scene?.Item != null)
- {
- try
- {
- var overlay = DrawDirtyRects || DrawFps;
- if (DrawDirtyRects)
- _dirtyRectsDisplay.Tick();
- if (overlay)
- RenderOverlay(scene.Item, ref context);
- if (updated || forceComposite || overlay)
- RenderComposite(scene.Item, ref context);
- }
- finally
- {
- try
- {
- if(scene.Item.RenderThreadJobs!=null)
- foreach (var job in scene.Item.RenderThreadJobs)
- job();
- }
- finally
- {
- scene.Item.MarkAsRendered();
- }
- }
- }
- }
- }
- finally
- {
- context?.Dispose();
- }
- }
- catch (RenderTargetCorruptedException ex)
- {
- Logger.TryGet(LogEventLevel.Information, LogArea.Animations)?.Log(this, "Render target was corrupted. Exception: {0}", ex);
- RenderTarget?.Dispose();
- RenderTarget = null;
- }
- }
- }
-
- private (IRef? scene, bool updated) UpdateRenderLayersAndConsumeSceneIfNeeded(ref IDrawingContextImpl? context,
- bool recursiveCall = false)
- {
- IRef? sceneRef;
- lock (_sceneLock)
- sceneRef = _scene?.Clone();
- if (sceneRef == null)
- return (null, false);
- using (sceneRef)
- {
- var scene = sceneRef.Item;
- if (scene.Generation != _lastSceneId)
- {
- EnsureDrawingContext(ref context);
-
- Layers.Update(scene, context);
-
- RenderToLayers(scene);
-
- if (DebugFramesPath != null)
- {
- SaveDebugFrames(scene.Generation);
- }
-
- lock (_sceneLock)
- _lastSceneId = scene.Generation;
-
-
- var isUiThread = Dispatcher.UIThread.CheckAccess();
- // We have consumed the previously available scene, but there might be some dirty
- // rects since the last update. *If* we are on UI thread, we can force immediate scene
- // rebuild before rendering anything on-screen
- // We are calling the same method recursively here
- if (!recursiveCall && isUiThread && NeedsUpdate)
- {
- UpdateScene();
- var (rs, _) = UpdateRenderLayersAndConsumeSceneIfNeeded(ref context, true);
- return (rs, true);
- }
-
- // We are rendering a new scene version, so it's highly likely
- // that there is already a pending update for animations
- // So we are scheduling an update call so UI thread could prepare a scene before
- // the next render timer tick
- if (!recursiveCall && !isUiThread)
- Dispatcher.UIThread.Post(_updateSceneIfNeededDelegate, DispatcherPriority.Render);
-
- // Indicate that we have updated the layers
- return (sceneRef.Clone(), true);
- }
-
- // Just return scene, layers weren't updated
- return (sceneRef.Clone(), false);
- }
-
- }
-
-
- private void Render(IDrawingContextImpl context, VisualNode node, Visual? layer, Rect clipBounds)
- {
- if (layer == null || node.LayerRoot == layer)
- {
- clipBounds = node.ClipBounds.Intersect(clipBounds);
-
- if (!clipBounds.IsDefault && node.Opacity > 0)
- {
- var isLayerRoot = node.Visual == layer;
-
- node.BeginRender(context, isLayerRoot);
-
- var drawOperations = node.DrawOperations;
- var drawOperationsCount = drawOperations.Count;
- for (int i = 0; i < drawOperationsCount; i++)
- {
- var operation = drawOperations[i];
- _currentDraw = operation;
- operation.Item.Render(context);
- _currentDraw = null;
- }
-
- var children = node.Children;
- var childrenCount = children.Count;
- for (int i = 0; i < childrenCount; i++)
- {
- var child = children[i];
- Render(context, (VisualNode)child, layer, clipBounds);
- }
-
- node.EndRender(context, isLayerRoot);
- }
- }
- }
-
- private void RenderToLayers(Scene scene)
- {
- foreach (var layer in scene.Layers)
- {
- var renderLayer = Layers[layer.LayerRoot];
- if (layer.Dirty.IsEmpty && !renderLayer.IsEmpty)
- continue;
- var renderTarget = renderLayer.Bitmap;
- var node = (VisualNode?)scene.FindNode(layer.LayerRoot);
-
- if (node != null)
- {
- using (var context = renderTarget.Item.CreateDrawingContext(this))
- {
- if (renderLayer.IsEmpty)
- {
- // Render entire layer root node
- context.Clear(Colors.Transparent);
- context.Transform = Matrix.Identity;
- context.PushClip(node.ClipBounds);
- Render(context, node, layer.LayerRoot, node.ClipBounds);
- context.PopClip();
- if (DrawDirtyRects)
- {
- _dirtyRectsDisplay.Add(node.ClipBounds);
- }
-
- renderLayer.IsEmpty = false;
- }
- else
- {
- var scale = scene.Scaling;
-
- foreach (var rect in layer.Dirty)
- {
- var snappedRect = SnapToDevicePixels(rect, scale);
- context.Transform = Matrix.Identity;
- context.PushClip(snappedRect);
- context.Clear(Colors.Transparent);
- Render(context, node, layer.LayerRoot, snappedRect);
- context.PopClip();
-
- if (DrawDirtyRects)
- {
- _dirtyRectsDisplay.Add(snappedRect);
- }
- }
- }
- }
- }
- }
- }
-
- private static Rect SnapToDevicePixels(Rect rect, double scale)
- {
- return new Rect(
- new Point(
- Math.Floor(rect.X * scale) / scale,
- Math.Floor(rect.Y * scale) / scale),
- new Point(
- Math.Ceiling(rect.Right * scale) / scale,
- Math.Ceiling(rect.Bottom * scale) / scale));
- }
-
- private void RenderOverlay(Scene scene, ref IDrawingContextImpl? parentContent)
- {
- EnsureDrawingContext(ref parentContent);
-
- if (DrawDirtyRects)
- {
- var overlay = GetOverlay(parentContent, scene.Size, scene.Scaling);
-
- using (var context = overlay.Item.CreateDrawingContext(this))
- {
- context.Clear(Colors.Transparent);
- RenderDirtyRects(context);
- }
- }
- else
- {
- _overlay?.Dispose();
- _overlay = null;
- }
- }
-
- private void RenderDirtyRects(IDrawingContextImpl context)
- {
- foreach (var r in _dirtyRectsDisplay)
- {
- var brush = new ImmutableSolidColorBrush(Colors.Magenta, r.Opacity);
- context.DrawRectangle(brush,null, r.Rect);
- }
- }
-
- private void RenderComposite(Scene scene, ref IDrawingContextImpl? context)
- {
- EnsureDrawingContext(ref context);
-
- context.Clear(Colors.Transparent);
-
- var clientRect = new Rect(scene.Size);
-
- var firstLayer = true;
- foreach (var layer in scene.Layers)
- {
- var bitmap = Layers[layer.LayerRoot].Bitmap;
- var sourceRect = new Rect(0, 0, bitmap.Item.PixelSize.Width, bitmap.Item.PixelSize.Height);
-
- if (layer.GeometryClip != null)
- {
- context.PushGeometryClip(layer.GeometryClip);
- }
-
- if (layer.OpacityMask == null)
- {
- if (firstLayer && bitmap.Item.CanBlit)
- bitmap.Item.Blit(context);
- else
- context.DrawBitmap(bitmap, layer.Opacity, sourceRect, clientRect);
- }
- else
- {
- context.DrawBitmap(bitmap, layer.OpacityMask, layer.OpacityMaskRect, sourceRect);
- }
-
- if (layer.GeometryClip != null)
- {
- context.PopGeometryClip();
- }
-
- firstLayer = false;
- }
-
- if (_overlay != null)
- {
- var sourceRect = new Rect(0, 0, _overlay.Item.PixelSize.Width, _overlay.Item.PixelSize.Height);
- context.DrawBitmap(_overlay, 0.5, sourceRect, clientRect);
- }
-
- if (DrawFps)
- {
- using (var c = new DrawingContext(context, false))
- {
- RenderFps(c, clientRect, scene.Layers.Count);
- }
- }
- }
-
- private void EnsureDrawingContext([NotNull] ref IDrawingContextImpl? context)
- {
- if (context != null)
- {
- return;
- }
-
- if (RenderTarget?.IsCorrupted == true)
- {
- RenderTarget!.Dispose();
- RenderTarget = null;
- }
-
- if (RenderTarget == null)
- {
- RenderTarget = _renderTargetFactory!();
- }
-
- context = RenderTarget.CreateDrawingContext(this);
- }
-
- private void UpdateSceneIfNeeded()
- {
- if(NeedsUpdate)
- UpdateScene();
- }
-
- private bool? UpdateScene()
- {
- Dispatcher.UIThread.VerifyAccess();
- using var noPump = NonPumpingLockHelper.Use();
- lock (_sceneLock)
- {
- if (_disposed)
- return null;
- if (_scene?.Item.Generation > _lastSceneId)
- return false;
- }
- if (_root.IsVisible)
- {
- var sceneRef = RefCountable.Create(_scene?.Item.CloneScene() ?? new Scene(_root)
- {
- RenderThreadJobs = _pendingRenderThreadJobs
- });
- _pendingRenderThreadJobs = null;
- var scene = sceneRef.Item;
-
- if (_dirty == null)
- {
- _dirty = new DirtyVisuals();
- _recalculateChildren = new HashSet();
- _sceneBuilder.UpdateAll(scene);
- }
- else
- {
- foreach (var visual in _recalculateChildren!)
- {
- var node = scene.FindNode(visual);
- ((VisualNode?)node)?.SortChildren(scene);
- }
-
- _recalculateChildren.Clear();
-
- foreach (var visual in _dirty)
- {
- _sceneBuilder.Update(scene, visual);
- }
- }
-
- lock (_sceneLock)
- {
- var oldScene = _scene;
- _scene = sceneRef;
- oldScene?.Dispose();
- }
-
- _dirty.Clear();
-
- if (SceneInvalidated != null)
- {
- var rect = new Rect();
-
- foreach (var layer in scene.Layers)
- {
- foreach (var dirty in layer.Dirty)
- {
- rect = rect.Union(dirty);
- }
- }
-
- SceneInvalidated(this, new SceneInvalidatedEventArgs((IRenderRoot)_root, rect));
- }
-
- return true;
- }
- else
- {
- lock (_sceneLock)
- {
- var oldScene = _scene;
- _scene = null;
- oldScene?.Dispose();
- }
-
- return null;
- }
- }
-
- private IRef GetOverlay(
- IDrawingContextImpl parentContext,
- Size size,
- double scaling)
- {
- var pixelSize = size * scaling;
-
- if (_overlay == null ||
- _overlay.Item.PixelSize.Width != pixelSize.Width ||
- _overlay.Item.PixelSize.Height != pixelSize.Height)
- {
- _overlay?.Dispose();
- _overlay = RefCountable.Create(parentContext.CreateLayer(size));
- }
-
- return _overlay;
- }
-
- private void SaveDebugFrames(int id)
- {
- var index = 0;
-
- foreach (var layer in Layers)
- {
- var fileName = Path.Combine(DebugFramesPath ?? string.Empty, FormattableString.Invariant($"frame-{id}-layer-{index++}.png"));
- layer.Bitmap.Item.Save(fileName);
- }
- }
- }
-}
diff --git a/src/Avalonia.Base/Rendering/ICustomHitTest.cs b/src/Avalonia.Base/Rendering/ICustomHitTest.cs
new file mode 100644
index 0000000000..65bb24b4b3
--- /dev/null
+++ b/src/Avalonia.Base/Rendering/ICustomHitTest.cs
@@ -0,0 +1,16 @@
+using System.Collections.Generic;
+using System.Linq;
+using Avalonia.VisualTree;
+
+namespace Avalonia.Rendering
+{
+
+ ///
+ /// Allows customization of hit-testing
+ ///
+ public interface ICustomHitTest
+ {
+ /// The point to hit test in global coordinate space.
+ bool HitTest(Point point);
+ }
+}
diff --git a/src/Avalonia.Base/Rendering/ICustomSimpleHitTest.cs b/src/Avalonia.Base/Rendering/ICustomSimpleHitTest.cs
deleted file mode 100644
index fae595413f..0000000000
--- a/src/Avalonia.Base/Rendering/ICustomSimpleHitTest.cs
+++ /dev/null
@@ -1,33 +0,0 @@
-using System.Collections.Generic;
-using System.Linq;
-using Avalonia.VisualTree;
-
-namespace Avalonia.Rendering
-{
- ///
- /// An interface to allow non-templated controls to customize their hit-testing
- /// when using a renderer with a simple hit-testing algorithm without a scene graph,
- /// such as
- ///
- public interface ICustomSimpleHitTest
- {
- /// The point to hit test in global coordinate space.
- bool HitTest(Point point);
- }
-
- ///
- /// Allows customization of hit-testing for all renderers.
- ///
- public interface ICustomHitTest : ICustomSimpleHitTest
- {
- }
-
- public static class CustomSimpleHitTestExtensions
- {
- public static bool HitTestCustom(this Visual visual, Point point)
- => (visual as ICustomSimpleHitTest)?.HitTest(point) ?? visual.TransformedBounds?.Contains(point) == true;
-
- public static bool HitTestCustom(this IEnumerable children, Point point)
- => children.Any(ctrl => ctrl.HitTestCustom(point));
- }
-}
diff --git a/src/Avalonia.Base/Rendering/IRenderRoot.cs b/src/Avalonia.Base/Rendering/IRenderRoot.cs
index cabb1302cd..fa3260ffb4 100644
--- a/src/Avalonia.Base/Rendering/IRenderRoot.cs
+++ b/src/Avalonia.Base/Rendering/IRenderRoot.cs
@@ -25,12 +25,6 @@ namespace Avalonia.Rendering
///
double RenderScaling { get; }
- ///
- /// Adds a rectangle to the window's dirty region.
- ///
- /// The rectangle.
- void Invalidate(Rect rect);
-
///
/// Converts a point from screen to client coordinates.
///
diff --git a/src/Avalonia.Base/Rendering/IRendererFactory.cs b/src/Avalonia.Base/Rendering/IRendererFactory.cs
deleted file mode 100644
index 11d89ef94c..0000000000
--- a/src/Avalonia.Base/Rendering/IRendererFactory.cs
+++ /dev/null
@@ -1,16 +0,0 @@
-
-namespace Avalonia.Rendering
-{
- ///
- /// Defines the interface for a renderer factory.
- ///
- public interface IRendererFactory
- {
- ///
- /// Creates a renderer.
- ///
- /// The root visual.
- /// The render loop.
- IRenderer Create(IRenderRoot root, IRenderLoop renderLoop);
- }
-}
diff --git a/src/Avalonia.Base/Rendering/ImmediateRenderer.cs b/src/Avalonia.Base/Rendering/ImmediateRenderer.cs
index 60b144e806..c67ac7057d 100644
--- a/src/Avalonia.Base/Rendering/ImmediateRenderer.cs
+++ b/src/Avalonia.Base/Rendering/ImmediateRenderer.cs
@@ -10,102 +10,12 @@ using Avalonia.VisualTree;
namespace Avalonia.Rendering
{
///
- /// A renderer which renders the state of the visual tree without an intermediate scene graph
- /// representation.
+ /// This class is used to render the visual tree into a DrawingContext by doing
+ /// a simple tree traversal.
+ /// It's currently used mostly for RenderTargetBitmap.Render and VisualBrush
///
- ///
- /// The immediate renderer supports only clip-bound-based hit testing; a control's geometry is
- /// not taken into account.
- ///
- public class ImmediateRenderer : RendererBase, IRenderer, IVisualBrushRenderer
+ internal class ImmediateRenderer : IVisualBrushRenderer//, IRenderer
{
- private readonly Visual _root;
- private readonly Func _renderTargetFactory;
- private readonly PlatformRenderInterfaceContextManager? _renderContext;
- private readonly IRenderRoot? _renderRoot;
- private bool _updateTransformedBounds = true;
- private IRenderTarget? _renderTarget;
-
- ///
- /// Initializes a new instance of the class.
- ///
- /// The control to render.
- /// The target render factory.
- /// The render contex.
- public ImmediateRenderer(Visual root, Func renderTargetFactory,
- PlatformRenderInterfaceContextManager? renderContext = null)
- {
- _root = root ?? throw new ArgumentNullException(nameof(root));
- _renderTargetFactory = renderTargetFactory;
- _renderContext = renderContext;
- _renderRoot = root as IRenderRoot;
- }
-
- private ImmediateRenderer(Visual root, Func renderTargetFactory, bool updateTransformedBounds)
- {
- _root = root ?? throw new ArgumentNullException(nameof(root));
- _renderTargetFactory = renderTargetFactory;
- _renderRoot = root as IRenderRoot;
- _updateTransformedBounds = updateTransformedBounds;
- }
-
- ///
- public bool DrawFps { get; set; }
-
- ///
- public bool DrawDirtyRects { get; set; }
-
- ///
- public event EventHandler? SceneInvalidated;
-
- ///
- public void Paint(Rect rect)
- {
- if (_renderTarget == null)
- {
- _renderTarget = _renderTargetFactory();
- }
-
- try
- {
- using (var context = new DrawingContext(_renderTarget.CreateDrawingContext(this)))
- {
- context.PlatformImpl.Clear(Colors.Transparent);
-
- using (context.PushTransformContainer())
- {
- Render(context, _root, _root.Bounds);
- }
-
- if (DrawDirtyRects)
- {
- var color = (uint)new Random().Next(0xffffff) | 0x44000000;
- context.FillRectangle(
- new SolidColorBrush(color),
- rect);
- }
-
- if (DrawFps)
- {
- RenderFps(context, _root.Bounds, null);
- }
- }
- }
- catch (RenderTargetCorruptedException ex)
- {
- Logger.TryGet(LogEventLevel.Information, LogArea.Animations)?.Log(this, "Render target was corrupted. Exception: {0}", ex);
- _renderTarget.Dispose();
- _renderTarget = null;
- }
-
- SceneInvalidated?.Invoke(this, new SceneInvalidatedEventArgs((IRenderRoot)_root, rect));
- }
-
- ///
- public void Resized(Size size)
- {
- }
-
///
/// Renders a visual to a render target.
///
@@ -113,11 +23,8 @@ namespace Avalonia.Rendering
/// The render target.
public static void Render(Visual visual, IRenderTarget target)
{
- using (var renderer = new ImmediateRenderer(visual, () => target, updateTransformedBounds: false))
- using (var context = new DrawingContext(target.CreateDrawingContext(renderer)))
- {
- renderer.Render(context, visual, visual.Bounds);
- }
+ using var context = new DrawingContext(target.CreateDrawingContext(new ImmediateRenderer()));
+ Render(context, visual, visual.Bounds);
}
///
@@ -127,77 +34,9 @@ namespace Avalonia.Rendering
/// The drawing context.
public static void Render(Visual visual, DrawingContext context)
{
- using (var renderer = new ImmediateRenderer(visual,
- () => throw new InvalidOperationException("This is not supposed to be called"),
- updateTransformedBounds: false))
- {
- renderer.Render(context, visual, visual.Bounds);
- }
+ Render(context, visual, visual.Bounds);
}
-
- ///
- public void AddDirty(Visual visual)
- {
- if (!visual.Bounds.IsDefault)
- {
- var m = visual.TransformToVisual(_root);
-
- if (m.HasValue)
- {
- var bounds = new Rect(visual.Bounds.Size).TransformToAABB(m.Value);
-
- //use transformedbounds as previous render state of the visual bounds
- //so we can invalidate old and new bounds of a control in case it moved/shrinked
- if (visual.TransformedBounds.HasValue)
- {
- var trb = visual.TransformedBounds.Value;
- var trBounds = trb.Bounds.TransformToAABB(trb.Transform);
-
- if (trBounds != bounds)
- {
- _renderRoot?.Invalidate(trBounds);
- }
- }
-
- _renderRoot?.Invalidate(bounds);
- }
- }
- }
-
- ///
- /// Ends the operation of the renderer.
- ///
- public void Dispose()
- {
- _renderTarget?.Dispose();
- }
-
- ///
- public IEnumerable HitTest(Point p, Visual root, Func filter)
- {
- return HitTest(root, p, filter);
- }
-
- public Visual? HitTestFirst(Point p, Visual root, Func filter)
- {
- return HitTest(root, p, filter).FirstOrDefault();
- }
-
- ///
- public void RecalculateChildren(Visual visual) => AddDirty(visual);
-
- ///
- public void Start()
- {
- }
-
- ///
- public void Stop()
- {
- }
-
- public ValueTask TryGetRenderInterfaceFeature(Type featureType) =>
- new(_renderContext?.Value?.TryGetFeature(featureType));
+
///
Size IVisualBrushRenderer.GetRenderTargetSize(IVisualBrush brush)
@@ -215,18 +54,7 @@ namespace Avalonia.Rendering
internal static void Render(Visual visual, DrawingContext context, bool updateTransformedBounds)
{
- using var renderer = new ImmediateRenderer(visual,
- () => throw new InvalidOperationException("This is not supposed to be called"),
- updateTransformedBounds);
- renderer.Render(context, visual, visual.Bounds);
- }
-
- private static void ClearTransformedBounds(Visual visual)
- {
- foreach (var e in visual.GetSelfAndVisualDescendants())
- {
- visual.SetTransformedBounds(null);
- }
+ Render(context, visual, visual.Bounds);
}
private static Rect GetTransformedBounds(Visual visual)
@@ -244,45 +72,8 @@ namespace Avalonia.Rendering
}
}
- private static IEnumerable HitTest(
- Visual visual,
- Point p,
- Func? filter)
- {
- _ = visual ?? throw new ArgumentNullException(nameof(visual));
-
- if (filter?.Invoke(visual) != false)
- {
- bool containsPoint;
-
- if (visual is ICustomSimpleHitTest custom)
- {
- containsPoint = custom.HitTest(p);
- }
- else
- {
- containsPoint = visual.TransformedBounds?.Contains(p) == true;
- }
-
- if ((containsPoint || !visual.ClipToBounds) && visual.VisualChildren.Count > 0)
- {
- foreach (var child in visual.VisualChildren.SortByZIndex())
- {
- foreach (var result in HitTest(child, p, filter))
- {
- yield return result;
- }
- }
- }
- if (containsPoint)
- {
- yield return visual;
- }
- }
- }
-
- private void Render(DrawingContext context, Visual visual, Rect clipRect)
+ private static void Render(DrawingContext context, Visual visual, Rect clipRect)
{
var opacity = visual.Opacity;
var clipToBounds = visual.ClipToBounds;
@@ -338,15 +129,7 @@ namespace Avalonia.Rendering
using (context.PushTransformContainer())
{
visual.Render(context);
-
-#pragma warning disable 0618
- var transformed =
- new TransformedBounds(bounds, new Rect(), context.CurrentContainerTransform);
-#pragma warning restore 0618
-
- if (_updateTransformedBounds)
- visual.SetTransformedBounds(transformed);
-
+
var childrenEnumerable = visual.HasNonUniformZIndexChildren
? visual.VisualChildren.OrderBy(x => x, ZIndexComparer.Instance)
: (IEnumerable)visual.VisualChildren;
@@ -362,18 +145,9 @@ namespace Avalonia.Rendering
: clipRect;
Render(context, child, childClipRect);
}
- else if (_updateTransformedBounds)
- {
- ClearTransformedBounds(child);
- }
}
}
}
-
- if (!visual.IsVisible && _updateTransformedBounds)
- {
- ClearTransformedBounds(visual);
- }
}
}
}
diff --git a/src/Avalonia.Base/Rendering/PlatformRenderInterfaceContextManager.cs b/src/Avalonia.Base/Rendering/PlatformRenderInterfaceContextManager.cs
index c13bcfda96..82dcd8f184 100644
--- a/src/Avalonia.Base/Rendering/PlatformRenderInterfaceContextManager.cs
+++ b/src/Avalonia.Base/Rendering/PlatformRenderInterfaceContextManager.cs
@@ -6,9 +6,7 @@ using Avalonia.Reactive;
namespace Avalonia.Rendering;
-[Unstable]
-// TODO: Make it internal once legacy renderers are removed
-public class PlatformRenderInterfaceContextManager
+internal class PlatformRenderInterfaceContextManager
{
private readonly IPlatformGraphics? _graphics;
private IPlatformRenderInterfaceContext? _backend;
diff --git a/src/Avalonia.Base/Rendering/RenderLayer.cs b/src/Avalonia.Base/Rendering/RenderLayer.cs
deleted file mode 100644
index d1e3fcafb1..0000000000
--- a/src/Avalonia.Base/Rendering/RenderLayer.cs
+++ /dev/null
@@ -1,48 +0,0 @@
-using Avalonia.Media;
-using Avalonia.Platform;
-using Avalonia.Utilities;
-using Avalonia.VisualTree;
-
-namespace Avalonia.Rendering
-{
- public class RenderLayer
- {
- public RenderLayer(
- IDrawingContextImpl drawingContext,
- Size size,
- double scaling,
- Visual layerRoot)
- {
- Bitmap = RefCountable.Create(drawingContext.CreateLayer(size));
- Size = size;
- Scaling = scaling;
- LayerRoot = layerRoot;
- IsEmpty = true;
- }
-
- public IRef Bitmap { get; private set; }
- public bool IsEmpty { get; set; }
- public double Scaling { get; private set; }
- public Size Size { get; private set; }
- public Visual LayerRoot { get; }
-
- public void RecreateBitmap(IDrawingContextImpl drawingContext, Size size, double scaling)
- {
- if (Size != size || Scaling != scaling)
- {
- var resized = RefCountable.Create(drawingContext.CreateLayer(size));
-
- using (var context = resized.Item.CreateDrawingContext(null))
- {
- Bitmap.Dispose();
- context.Clear(default);
-
- Bitmap = resized;
- Scaling = scaling;
- Size = size;
- IsEmpty = true;
- }
- }
- }
- }
-}
diff --git a/src/Avalonia.Base/Rendering/RenderLayers.cs b/src/Avalonia.Base/Rendering/RenderLayers.cs
deleted file mode 100644
index eff81e6bbf..0000000000
--- a/src/Avalonia.Base/Rendering/RenderLayers.cs
+++ /dev/null
@@ -1,69 +0,0 @@
-using System.Collections;
-using System.Collections.Generic;
-using System.Diagnostics.CodeAnalysis;
-using Avalonia.Platform;
-using Avalonia.Rendering.SceneGraph;
-
-namespace Avalonia.Rendering
-{
- public class RenderLayers : IEnumerable
- {
- private readonly List _inner = new List();
- private readonly Dictionary _index = new Dictionary();
-
- public int Count => _inner.Count;
- public RenderLayer this[Visual layerRoot] => _index[layerRoot];
-
- public void Update(Scene scene, IDrawingContextImpl context)
- {
- for (var i = scene.Layers.Count - 1; i >= 0; --i)
- {
- var src = scene.Layers[i];
-
- if (!_index.TryGetValue(src.LayerRoot, out var layer))
- {
- layer = new RenderLayer(context, scene.Size, scene.Scaling, src.LayerRoot);
- _inner.Add(layer);
- _index.Add(src.LayerRoot, layer);
- }
- else
- {
- layer.RecreateBitmap(context, scene.Size, scene.Scaling);
- }
- }
-
- for (var i = 0; i < _inner.Count;)
- {
- var layer = _inner[i];
-
- if (!scene.Layers.Exists(layer.LayerRoot))
- {
- layer.Bitmap.Dispose();
- _inner.RemoveAt(i);
- _index.Remove(layer.LayerRoot);
- }
- else
- i++;
- }
- }
-
- public void Clear()
- {
- foreach (var layer in _index.Values)
- {
- layer.Bitmap.Dispose();
- }
-
- _index.Clear();
- _inner.Clear();
- }
-
- public bool TryGetValue(Visual layerRoot, [NotNullWhen(true)] out RenderLayer? value)
- {
- return _index.TryGetValue(layerRoot, out value);
- }
-
- public IEnumerator GetEnumerator() => _inner.GetEnumerator();
- IEnumerator IEnumerable.GetEnumerator() => GetEnumerator();
- }
-}
diff --git a/src/Avalonia.Base/Rendering/SceneGraph/DeferredDrawingContextImpl.cs b/src/Avalonia.Base/Rendering/SceneGraph/DeferredDrawingContextImpl.cs
deleted file mode 100644
index b1d8301557..0000000000
--- a/src/Avalonia.Base/Rendering/SceneGraph/DeferredDrawingContextImpl.cs
+++ /dev/null
@@ -1,482 +0,0 @@
-using System;
-using System.Collections.Generic;
-using System.Numerics;
-using Avalonia.Media;
-using Avalonia.Platform;
-using Avalonia.Utilities;
-using Avalonia.Media.Imaging;
-using Avalonia.VisualTree;
-
-namespace Avalonia.Rendering.SceneGraph
-{
- ///
- /// A drawing context which builds a scene graph.
- ///
- internal class DeferredDrawingContextImpl : IDrawingContextImpl, IDrawingContextWithAcrylicLikeSupport
- {
- private readonly ISceneBuilder _sceneBuilder;
- private VisualNode? _node;
- private int _childIndex;
- private int _drawOperationindex;
-
- ///
- /// Initializes a new instance of the class.
- ///
- ///
- /// A scene builder used for constructing child scenes for visual brushes.
- ///
- /// The scene layers.
- public DeferredDrawingContextImpl(ISceneBuilder sceneBuilder, SceneLayers layers)
- {
- _sceneBuilder = sceneBuilder;
- Layers = layers;
- }
-
- ///
- public Matrix Transform { get; set; } = Matrix.Identity;
-
- ///
- /// Gets the layers in the scene being built.
- ///
- public SceneLayers Layers { get; }
-
- ///
- /// Informs the drawing context of the visual node that is about to be rendered.
- ///
- /// The visual node.
- ///
- /// An object which when disposed will commit the changes to visual node.
- ///
- public UpdateState BeginUpdate(VisualNode node)
- {
- _ = node ?? throw new ArgumentNullException(nameof(node));
-
- if (_node != null)
- {
- if (_childIndex < _node.Children.Count)
- {
- _node.ReplaceChild(_childIndex, node);
- }
- else
- {
- _node.AddChild(node);
- }
-
- ++_childIndex;
- }
-
- var state = new UpdateState(this, _node, _childIndex, _drawOperationindex);
- _node = node;
- _childIndex = _drawOperationindex = 0;
- return state;
- }
-
- ///
- public void Clear(Color color)
- {
- // Cannot clear a deferred scene.
- }
-
- ///
- public void Dispose()
- {
- // Nothing to do here since we allocate no unmanaged resources.
- }
-
- ///
- /// Removes any remaining drawing operations from the visual node.
- ///
- ///
- /// Drawing operations are updated in place, overwriting existing drawing operations if
- /// they are different. Once drawing has completed for the current visual node, it is
- /// possible that there are stale drawing operations at the end of the list. This method
- /// trims these stale drawing operations.
- ///
- public void TrimChildren()
- {
- _node!.TrimChildren(_childIndex);
- }
-
- ///
- public void DrawGeometry(IBrush? brush, IPen? pen, IGeometryImpl geometry)
- {
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(Transform, brush, pen, geometry))
- {
- Add(new GeometryNode(Transform, brush, pen, geometry, CreateChildScene(brush)));
- }
- else
- {
- ++_drawOperationindex;
- }
- }
-
- ///
- public void DrawBitmap(IRef source, double opacity, Rect sourceRect, Rect destRect, BitmapInterpolationMode bitmapInterpolationMode)
- {
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(Transform, source, opacity, sourceRect, destRect, bitmapInterpolationMode))
- {
- Add(new ImageNode(Transform, source, opacity, sourceRect, destRect, bitmapInterpolationMode));
- }
- else
- {
- ++_drawOperationindex;
- }
- }
-
- ///
- public void DrawBitmap(IRef source, IBrush opacityMask, Rect opacityMaskRect, Rect sourceRect)
- {
- // This method is currently only used to composite layers so shouldn't be called here.
- throw new NotSupportedException();
- }
-
- ///
- public void DrawLine(IPen pen, Point p1, Point p2)
- {
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(Transform, pen, p1, p2))
- {
- Add(new LineNode(Transform, pen, p1, p2, CreateChildScene(pen.Brush)));
- }
- else
- {
- ++_drawOperationindex;
- }
- }
-
- ///
- public void DrawRectangle(IBrush? brush, IPen? pen, RoundedRect rect,
- BoxShadows boxShadows = default)
- {
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(Transform, brush, pen, rect, boxShadows))
- {
- Add(new RectangleNode(Transform, brush, pen, rect, boxShadows, CreateChildScene(brush)));
- }
- else
- {
- ++_drawOperationindex;
- }
- }
-
- ///
- public void DrawRectangle(IExperimentalAcrylicMaterial material, RoundedRect rect)
- {
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(Transform, material, rect))
- {
- Add(new ExperimentalAcrylicNode(Transform, material, rect));
- }
- else
- {
- ++_drawOperationindex;
- }
- }
-
- public void DrawEllipse(IBrush? brush, IPen? pen, Rect rect)
- {
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(Transform, brush, pen, rect))
- {
- Add(new EllipseNode(Transform, brush, pen, rect, CreateChildScene(brush)));
- }
- else
- {
- ++_drawOperationindex;
- }
- }
-
- public void Custom(ICustomDrawOperation custom)
- {
- var next = NextDrawAs();
- if (next == null || !next.Item.Equals(Transform, custom))
- Add(new CustomDrawOperation(custom, Transform));
- else
- ++_drawOperationindex;
- }
-
- public object? GetFeature(Type t) => null;
-
- ///
- public void DrawGlyphRun(IBrush foreground, IRef glyphRun)
- {
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(Transform, foreground, glyphRun))
- {
- Add(new GlyphRunNode(Transform, foreground, glyphRun, CreateChildScene(foreground)));
- }
-
- else
- {
- ++_drawOperationindex;
- }
- }
- public IDrawingContextLayerImpl CreateLayer(Size size)
- {
- throw new NotSupportedException("Creating layers on a deferred drawing context not supported");
- }
-
- ///
- public void PopClip()
- {
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(null))
- {
- Add(new ClipNode());
- }
- else
- {
- ++_drawOperationindex;
- }
- }
-
- ///
- public void PopGeometryClip()
- {
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(null))
- {
- Add(new GeometryClipNode());
- }
- else
- {
- ++_drawOperationindex;
- }
- }
-
- ///
- public void PopBitmapBlendMode()
- {
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(null))
- {
- Add(new BitmapBlendModeNode());
- }
- else
- {
- ++_drawOperationindex;
- }
- }
-
- ///
- public void PopOpacity()
- {
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(null))
- {
- Add(new OpacityNode());
- }
- else
- {
- ++_drawOperationindex;
- }
- }
-
- ///
- public void PopOpacityMask()
- {
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(null, null))
- {
- Add(new OpacityMaskNode());
- }
- else
- {
- ++_drawOperationindex;
- }
- }
-
- ///
- public void PushClip(Rect clip)
- {
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(Transform, clip))
- {
- Add(new ClipNode(Transform, clip));
- }
- else
- {
- ++_drawOperationindex;
- }
- }
-
- ///
- public void PushClip(RoundedRect clip)
- {
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(Transform, clip))
- {
- Add(new ClipNode(Transform, clip));
- }
- else
- {
- ++_drawOperationindex;
- }
- }
-
- ///
- public void PushGeometryClip(IGeometryImpl? clip)
- {
- if (clip is null)
- return;
-
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(Transform, clip))
- {
- Add(new GeometryClipNode(Transform, clip));
- }
- else
- {
- ++_drawOperationindex;
- }
- }
-
- ///
- public void PushOpacity(double opacity)
- {
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(opacity))
- {
- Add(new OpacityNode(opacity));
- }
- else
- {
- ++_drawOperationindex;
- }
- }
-
- ///
- public void PushOpacityMask(IBrush mask, Rect bounds)
- {
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(mask, bounds))
- {
- Add(new OpacityMaskNode(mask, bounds, CreateChildScene(mask)));
- }
- else
- {
- ++_drawOperationindex;
- }
- }
-
- ///
- public void PushBitmapBlendMode(BitmapBlendingMode blendingMode)
- {
- var next = NextDrawAs();
-
- if (next == null || !next.Item.Equals(blendingMode))
- {
- Add(new BitmapBlendModeNode(blendingMode));
- }
- else
- {
- ++_drawOperationindex;
- }
- }
-
- public readonly struct UpdateState : IDisposable
- {
- public UpdateState(
- DeferredDrawingContextImpl owner,
- VisualNode? node,
- int childIndex,
- int drawOperationIndex)
- {
- Owner = owner;
- Node = node;
- ChildIndex = childIndex;
- DrawOperationIndex = drawOperationIndex;
- }
-
- public void Dispose()
- {
- Owner._node!.TrimDrawOperations(Owner._drawOperationindex);
-
- var dirty = Owner.Layers.GetOrAdd(Owner._node.LayerRoot!).Dirty;
-
- var drawOperations = Owner._node.DrawOperations;
- var drawOperationsCount = drawOperations.Count;
-
- for (var i = 0; i < drawOperationsCount; i++)
- {
- dirty.Add(drawOperations[i].Item.Bounds);
- }
-
- Owner._node = Node;
- Owner._childIndex = ChildIndex;
- Owner._drawOperationindex = DrawOperationIndex;
- }
-
- public DeferredDrawingContextImpl Owner { get; }
- public VisualNode? Node { get; }
- public int ChildIndex { get; }
- public int DrawOperationIndex { get; }
- }
-
- private void Add(T node) where T : class, IDrawOperation
- {
- using (var refCounted = RefCountable.Create(node))
- {
- Add(refCounted);
- }
- }
-
- private void Add(IRef node)
- {
- if (_drawOperationindex < _node!.DrawOperations.Count)
- {
- _node.ReplaceDrawOperation(_drawOperationindex, node);
- }
- else
- {
- _node.AddDrawOperation(node);
- }
-
- ++_drawOperationindex;
- }
-
- private IRef? NextDrawAs() where T : class, IDrawOperation
- {
- return _drawOperationindex < _node!.DrawOperations.Count ? _node.DrawOperations[_drawOperationindex] as IRef : null;
- }
-
- private IDisposable? CreateChildScene(IBrush? brush)
- {
- var visualBrush = brush as VisualBrush;
-
- if (visualBrush != null)
- {
- var visual = visualBrush.Visual;
-
- if (visual != null)
- {
- (visual as IVisualBrushInitialize)?.EnsureInitialized();
- var scene = new Scene(visual);
- _sceneBuilder.UpdateAll(scene);
- return scene;
- }
- }
-
- return null;
- }
- }
-}
diff --git a/src/Avalonia.Base/Rendering/SceneGraph/ISceneBuilder.cs b/src/Avalonia.Base/Rendering/SceneGraph/ISceneBuilder.cs
deleted file mode 100644
index f469fdbfe8..0000000000
--- a/src/Avalonia.Base/Rendering/SceneGraph/ISceneBuilder.cs
+++ /dev/null
@@ -1,24 +0,0 @@
-using Avalonia.VisualTree;
-
-namespace Avalonia.Rendering.SceneGraph
-{
- ///
- /// Builds a scene graph from a visual tree.
- ///
- public interface ISceneBuilder
- {
- ///
- /// Builds the initial scene graph for a visual tree.
- ///
- /// The scene to build.
- void UpdateAll(Scene scene);
-
- ///
- /// Updates the visual (and potentially its children) in a scene.
- ///
- /// The scene.
- /// The visual to update.
- /// True if changes were made, otherwise false.
- bool Update(Scene scene, Visual visual);
- }
-}
diff --git a/src/Avalonia.Base/Rendering/SceneGraph/IVisualNode.cs b/src/Avalonia.Base/Rendering/SceneGraph/IVisualNode.cs
deleted file mode 100644
index 59a032748d..0000000000
--- a/src/Avalonia.Base/Rendering/SceneGraph/IVisualNode.cs
+++ /dev/null
@@ -1,105 +0,0 @@
-using System;
-using System.Collections.Generic;
-using Avalonia.Platform;
-using Avalonia.Utilities;
-
-namespace Avalonia.Rendering.SceneGraph
-{
- ///
- /// Represents a node in the low-level scene graph representing a .
- ///
- public interface IVisualNode : IDisposable
- {
- ///
- /// Gets the visual to which the node relates.
- ///
- Visual Visual { get; }
-
- ///
- /// Gets the parent scene graph node.
- ///
- IVisualNode? Parent { get; }
-
- ///
- /// Gets the transform for the node from global to control coordinates.
- ///
- Matrix Transform { get; }
-
- ///
- /// Gets the corner radius of visual. Contents are clipped to this radius.
- ///
- CornerRadius ClipToBoundsRadius { get; }
-
- ///
- /// Gets the bounds of the node's geometry in global coordinates.
- ///
- Rect Bounds { get; }
-
- ///
- /// Gets the clip bounds for the node in global coordinates.
- ///
- Rect ClipBounds { get; }
-
- ///
- /// Gets the layout bounds for the node in global coordinates.
- ///
- Rect LayoutBounds { get; }
-
- ///
- /// Whether the node is clipped to .
- ///
- bool ClipToBounds { get; }
-
- ///
- /// Gets the node's clip geometry, if any.
- ///
- IGeometryImpl? GeometryClip { get; set; }
-
- ///
- /// Gets a value indicating whether one of the node's ancestors has a geometry clip.
- ///
- bool HasAncestorGeometryClip { get; }
-
- ///
- /// Gets the child scene graph nodes.
- ///
- IReadOnlyList Children { get; }
-
- ///
- /// Gets the drawing operations for the visual.
- ///
- IReadOnlyList> DrawOperations { get; }
-
- ///
- /// Gets the opacity of the scene graph node.
- ///
- double Opacity { get; }
-
- ///
- /// Sets up the drawing context for rendering the node's geometry.
- ///
- /// The drawing context.
- /// Whether to skip pushing the control's opacity.
- void BeginRender(IDrawingContextImpl context, bool skipOpacity);
-
- ///
- /// Resets the drawing context after rendering the node's geometry.
- ///
- /// The drawing context.
- /// Whether to skip popping the control's opacity.
- void EndRender(IDrawingContextImpl context, bool skipOpacity);
-
- ///
- /// Hit test the geometry in this node.
- ///
- /// The point in global coordinates.
- /// True if the point hits the node's geometry; otherwise false.
- ///
- /// This method does not recurse to child s, if you want
- /// to hit test children they must be hit tested manually.
- ///
- bool HitTest(Point p);
-
- bool Disposed { get; }
- }
-}
diff --git a/src/Avalonia.Base/Rendering/SceneGraph/Scene.cs b/src/Avalonia.Base/Rendering/SceneGraph/Scene.cs
deleted file mode 100644
index 735eb3bb3f..0000000000
--- a/src/Avalonia.Base/Rendering/SceneGraph/Scene.cs
+++ /dev/null
@@ -1,352 +0,0 @@
-using System;
-using System.Collections;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
-using Avalonia.Collections.Pooled;
-using Avalonia.VisualTree;
-
-namespace Avalonia.Rendering.SceneGraph
-{
- ///
- /// Represents a scene graph used by the .
- ///
- public class Scene : IDisposable
- {
- private readonly Dictionary _index;
- private readonly TaskCompletionSource _rendered = new TaskCompletionSource();
-
- ///
- /// Initializes a new instance of the class.
- ///
- /// The root visual to draw.
- public Scene(Visual rootVisual)
- : this(
- new VisualNode(rootVisual, null),
- new Dictionary(),
- new SceneLayers(rootVisual),
- 0)
- {
- _index.Add(rootVisual, Root);
- }
-
- private Scene(VisualNode root, Dictionary index, SceneLayers layers, int generation)
- {
- _ = root ?? throw new ArgumentNullException(nameof(root));
-
- var renderRoot = root.Visual as IRenderRoot;
-
- _index = index;
- Root = root;
- Layers = layers;
- Generation = generation;
- root.LayerRoot = root.Visual;
- }
-
- public Task Rendered => _rendered.Task;
-
- ///
- /// Gets a value identifying the scene's generation. This is incremented each time the scene is cloned.
- ///
- public int Generation { get; }
-
- ///
- /// Gets the layers for the scene.
- ///
- public SceneLayers Layers { get; }
-
- ///
- /// Gets the root node of the scene graph.
- ///
- public IVisualNode Root { get; }
-
- ///
- /// Gets or sets the size of the scene in device independent pixels.
- ///
- public Size Size { get; set; }
-
- ///
- /// Gets or sets the scene scaling.
- ///
- public double Scaling { get; set; } = 1;
-
- ///
- /// Adds a node to the scene index.
- ///
- /// The node.
- public void Add(IVisualNode node)
- {
- _ = node ?? throw new ArgumentNullException(nameof(node));
-
- _index.Add(node.Visual, node);
- }
-
- ///
- /// Clones the scene.
- ///
- /// The cloned scene.
- public Scene CloneScene()
- {
- var index = new Dictionary(_index.Count);
- var root = Clone((VisualNode)Root, null, index);
-
- var result = new Scene(root, index, Layers.Clone(), Generation + 1)
- {
- Size = Size,
- Scaling = Scaling,
- };
-
- return result;
- }
-
- public void Dispose()
- {
- _rendered.TrySetResult(false);
- foreach (var node in _index.Values)
- {
- node.Dispose();
- }
- }
-
- ///
- /// Tries to find a node in the scene graph representing the specified visual.
- ///
- /// The visual.
- ///
- /// The node representing the visual or null if it could not be found.
- ///
- public IVisualNode? FindNode(Visual visual)
- {
- _index.TryGetValue(visual, out var node);
- return node;
- }
-
- ///
- /// Gets the visuals at a point in the scene.
- ///
- /// The point.
- /// The root of the subtree to search.
- /// A filter. May be null.
- /// The visuals at the specified point.
- public IEnumerable HitTest(Point p, Visual root, Func? filter)
- {
- var node = FindNode(root);
- return (node != null) ? new HitTestEnumerable(node, filter, p, Root) : Enumerable.Empty();
- }
-
- ///
- /// Gets the visual at a point in the scene.
- ///
- /// The point.
- /// The root of the subtree to search.
- /// A filter. May be null.
- /// The visual at the specified point.
- public Visual? HitTestFirst(Point p, Visual root, Func? filter)
- {
- var node = FindNode(root);
- return (node != null) ? HitTestFirst(node, p, filter) : null;
- }
-
- ///
- /// Removes a node from the scene index.
- ///
- /// The node.
- public void Remove(IVisualNode node)
- {
- _ = node ?? throw new ArgumentNullException(nameof(node));
-
- _index.Remove(node.Visual);
-
- node.Dispose();
- }
-
- private VisualNode Clone(VisualNode source, IVisualNode? parent, Dictionary index)
- {
- var result = source.Clone(parent);
-
- index.Add(result.Visual, result);
-
- var children = source.Children;
- var childrenCount = children.Count;
-
- if (childrenCount > 0)
- {
- result.TryPreallocateChildren(childrenCount);
-
- for (var i = 0; i < childrenCount; i++)
- {
- var child = children[i];
-
- result.AddChild(Clone((VisualNode)child, result, index));
- }
- }
-
- return result;
- }
-
- private Visual HitTestFirst(IVisualNode root, Point p, Func? filter)
- {
- using var enumerator = new HitTestEnumerator(root, filter, p, Root);
-
- enumerator.MoveNext();
-
- return enumerator.Current;
- }
-
- private class HitTestEnumerable : IEnumerable
- {
- private readonly IVisualNode _root;
- private readonly Func? _filter;
- private readonly IVisualNode _sceneRoot;
- private readonly Point _point;
-
- public HitTestEnumerable(IVisualNode root, Func? filter, Point point, IVisualNode sceneRoot)
- {
- _root = root;
- _filter = filter;
- _point = point;
- _sceneRoot = sceneRoot;
- }
-
- public IEnumerator GetEnumerator()
- {
- return new HitTestEnumerator(_root, _filter, _point, _sceneRoot);
- }
-
- IEnumerator IEnumerable.GetEnumerator()
- {
- return GetEnumerator();
- }
- }
-
- private struct HitTestEnumerator : IEnumerator
- {
- private readonly PooledStack _nodeStack;
- private readonly Func? _filter;
- private readonly IVisualNode _sceneRoot;
- private Visual? _current;
- private readonly Point _point;
-
- public HitTestEnumerator(IVisualNode root, Func? filter, Point point, IVisualNode sceneRoot)
- {
- _nodeStack = new PooledStack();
- _nodeStack.Push(new Entry(root, false, null, true));
-
- _filter = filter;
- _point = point;
- _sceneRoot = sceneRoot;
-
- _current = null;
- }
-
- public bool MoveNext()
- {
- while (_nodeStack.Count > 0)
- {
- (var wasVisited, var isRoot, IVisualNode node, Rect? clip) = _nodeStack.Pop();
-
- if (wasVisited && isRoot)
- {
- break;
- }
-
- var children = node.Children;
- int childCount = children.Count;
-
- if (childCount == 0 || wasVisited)
- {
- if ((wasVisited || FilterAndClip(node, ref clip)) &&
- (node.Visual is ICustomHitTest custom ? custom.HitTest(_point) : node.HitTest(_point)))
- {
- _current = node.Visual;
-
- return true;
- }
- }
- else if (FilterAndClip(node, ref clip))
- {
- _nodeStack.Push(new Entry(node, true, null));
-
- for (var i = 0; i < childCount; i++)
- {
- _nodeStack.Push(new Entry(children[i], false, clip));
- }
- }
- }
-
- return false;
- }
-
- public void Reset()
- {
- throw new NotSupportedException();
- }
-
- public Visual Current => _current!;
-
- object IEnumerator.Current => Current;
-
- public void Dispose()
- {
- _nodeStack.Dispose();
- }
-
- private bool FilterAndClip(IVisualNode node, ref Rect? clip)
- {
- if (_filter?.Invoke(node.Visual) != false && node.Visual.IsAttachedToVisualTree)
- {
- var clipped = false;
-
- if (node.ClipToBounds)
- {
- clip = clip == null ? node.ClipBounds : clip.Value.Intersect(node.ClipBounds);
- clipped = !clip.Value.ContainsExclusive(_point);
- }
-
- if (node.GeometryClip != null)
- {
- var controlPoint = _sceneRoot.Visual.TranslatePoint(_point, node.Visual);
- clipped = !node.GeometryClip.FillContains(controlPoint!.Value);
- }
-
- if (!clipped && node.Visual is ICustomHitTest custom)
- {
- clipped = !custom.HitTest(_point);
- }
-
- return !clipped;
- }
-
- return false;
- }
-
- private readonly struct Entry
- {
- public readonly bool WasVisited;
- public readonly bool IsRoot;
- public readonly IVisualNode Node;
- public readonly Rect? Clip;
-
- public Entry(IVisualNode node, bool wasVisited, Rect? clip, bool isRoot = false)
- {
- Node = node;
- WasVisited = wasVisited;
- IsRoot = isRoot;
- Clip = clip;
- }
-
- public void Deconstruct(out bool wasVisited, out bool isRoot, out IVisualNode node, out Rect? clip)
- {
- wasVisited = WasVisited;
- isRoot = IsRoot;
- node = Node;
- clip = Clip;
- }
- }
- }
-
- public void MarkAsRendered() => _rendered.TrySetResult(true);
-
- public List? RenderThreadJobs { get; set; }
- }
-}
diff --git a/src/Avalonia.Base/Rendering/SceneGraph/SceneBuilder.cs b/src/Avalonia.Base/Rendering/SceneGraph/SceneBuilder.cs
deleted file mode 100644
index 55ff772772..0000000000
--- a/src/Avalonia.Base/Rendering/SceneGraph/SceneBuilder.cs
+++ /dev/null
@@ -1,485 +0,0 @@
-using System;
-using System.Collections.Generic;
-using Avalonia.Media;
-using Avalonia.Platform;
-using Avalonia.Threading;
-using Avalonia.VisualTree;
-
-namespace Avalonia.Rendering.SceneGraph
-{
- ///
- /// Builds a scene graph from a visual tree.
- ///
- public class SceneBuilder : ISceneBuilder
- {
- ///
- public void UpdateAll(Scene scene)
- {
- _ = scene ?? throw new ArgumentNullException(nameof(scene));
- Dispatcher.UIThread.VerifyAccess();
-
- UpdateSize(scene);
- scene.Layers.GetOrAdd(scene.Root.Visual);
-
- using (var impl = new DeferredDrawingContextImpl(this, scene.Layers))
- using (var context = new DrawingContext(impl))
- {
- var clip = new Rect(scene.Root.Visual.Bounds.Size);
- Update(context, scene, (VisualNode)scene.Root, clip, true);
- }
- }
-
- ///
- public bool Update(Scene scene, Visual visual)
- {
- _ = scene ?? throw new ArgumentNullException(nameof(scene));
- _ = visual ?? throw new ArgumentNullException(nameof(visual));
-
- Dispatcher.UIThread.VerifyAccess();
-
- if (!scene.Root.Visual.IsVisible)
- {
- throw new AvaloniaInternalException("Cannot update the scene for an invisible root visual.");
- }
-
- var node = (VisualNode?)scene.FindNode(visual);
-
- if (visual == scene.Root.Visual)
- {
- UpdateSize(scene);
- }
-
- if (visual.VisualRoot == scene.Root.Visual)
- {
- if (node?.Parent != null &&
- visual.VisualParent != null &&
- node.Parent.Visual != visual.VisualParent)
- {
- // The control has changed parents. Remove the node and recurse into the new parent node.
- ((VisualNode)node.Parent).RemoveChild(node);
- Deindex(scene, node);
- node = (VisualNode?)scene.FindNode(visual.VisualParent);
- }
-
- if (visual.IsVisible)
- {
- // If the node isn't yet part of the scene, find the nearest ancestor that is.
- node = node ?? FindExistingAncestor(scene, visual);
-
- // We don't need to do anything if this part of the tree has already been fully
- // updated.
- if (node != null && !node.SubTreeUpdated)
- {
- // If the control we've been asked to update isn't part of the scene then
- // we're carrying out an add operation, so recurse and add all the
- // descendents too.
- var recurse = node.Visual != visual;
-
- using (var impl = new DeferredDrawingContextImpl(this, scene.Layers))
- using (var context = new DrawingContext(impl))
- {
- var clip = new Rect(scene.Root.Visual.Bounds.Size);
-
- if (node.Parent != null)
- {
- context.PushPostTransform(node.Parent.Transform);
- clip = node.Parent.ClipBounds;
- }
-
- using (context.PushTransformContainer())
- {
- Update(context, scene, node, clip, recurse);
- }
- }
-
- return true;
- }
- }
- else
- {
- if (node != null)
- {
- // The control has been hidden so remove it from its parent and deindex the
- // node and its descendents.
- ((VisualNode?)node.Parent)?.RemoveChild(node);
- Deindex(scene, node);
- return true;
- }
- }
- }
- else if (node != null)
- {
- // The control has been removed so remove it from its parent and deindex the
- // node and its descendents.
- var trim = FindFirstDeadAncestor(scene, node);
- ((VisualNode)trim.Parent!).RemoveChild(trim);
- Deindex(scene, trim);
- return true;
- }
-
- return false;
- }
-
- private static VisualNode? FindExistingAncestor(Scene scene, Visual visual)
- {
- var node = scene.FindNode(visual);
-
- while (node == null && visual.IsVisible)
- {
- var parent = visual.VisualParent;
-
- if (parent is null)
- return null;
-
- visual = parent;
- node = scene.FindNode(visual);
- }
-
- return visual.IsVisible ? (VisualNode?)node : null;
- }
-
- private static VisualNode FindFirstDeadAncestor(Scene scene, IVisualNode node)
- {
- var parent = node.Parent;
-
- while (parent!.Visual.VisualRoot == null)
- {
- node = parent;
- parent = node.Parent;
- }
-
- return (VisualNode)node;
- }
-
- private static object GetOrCreateChildNode(Scene scene, Visual child, VisualNode parent)
- {
- var result = (VisualNode?)scene.FindNode(child);
-
- if (result != null && result.Parent != parent)
- {
- Deindex(scene, result);
- ((VisualNode?)result.Parent)?.RemoveChild(result);
- result = null;
- }
-
- return result ?? CreateNode(scene, child, parent);
- }
-
- private static void Update(DrawingContext context, Scene scene, VisualNode node, Rect clip, bool forceRecurse)
- {
- var visual = node.Visual;
- var opacity = visual.Opacity;
- var clipToBounds = visual.ClipToBounds;
-#pragma warning disable CS0618 // Type or member is obsolete
- var clipToBoundsRadius = visual is IVisualWithRoundRectClip roundRectClip ?
- roundRectClip.ClipToBoundsRadius :
- default;
-#pragma warning restore CS0618 // Type or member is obsolete
-
- var bounds = new Rect(visual.Bounds.Size);
- var contextImpl = (DeferredDrawingContextImpl)context.PlatformImpl;
-
- contextImpl.Layers.Find(node.LayerRoot!)?.Dirty.Add(node.Bounds);
-
- if (visual.IsVisible)
- {
- var m = node != scene.Root ?
- Matrix.CreateTranslation(visual.Bounds.Position) :
- Matrix.Identity;
-
- var renderTransform = Matrix.Identity;
-
- // this should be calculated BEFORE renderTransform
- if (visual.HasMirrorTransform)
- {
- var mirrorMatrix = new Matrix(-1.0, 0.0, 0.0, 1.0, visual.Bounds.Width, 0);
- renderTransform *= mirrorMatrix;
- }
-
- if (visual.RenderTransform != null)
- {
- var origin = visual.RenderTransformOrigin.ToPixels(new Size(visual.Bounds.Width, visual.Bounds.Height));
- var offset = Matrix.CreateTranslation(origin);
- var finalTransform = (-offset) * visual.RenderTransform.Value * (offset);
- renderTransform *= finalTransform;
- }
-
- m = renderTransform * m;
-
- using (contextImpl.BeginUpdate(node))
- using (context.PushPostTransform(m))
- using (context.PushTransformContainer())
- {
- var globalBounds = bounds.TransformToAABB(contextImpl.Transform);
- var clipBounds = clipToBounds ?
- globalBounds.Intersect(clip) :
- clip;
-
- forceRecurse = forceRecurse ||
- node.ClipBounds != clipBounds ||
- node.Opacity != opacity ||
- node.Transform != contextImpl.Transform;
-
- node.Transform = contextImpl.Transform;
- node.ClipBounds = clipBounds;
- node.ClipToBounds = clipToBounds;
- node.LayoutBounds = globalBounds;
- node.ClipToBoundsRadius = clipToBoundsRadius;
- node.GeometryClip = visual.Clip?.PlatformImpl;
- node.Opacity = opacity;
-
- // TODO: Check equality between node.OpacityMask and visual.OpacityMask before assigning.
- node.OpacityMask = visual.OpacityMask?.ToImmutable();
-
- if (ShouldStartLayer(visual))
- {
- if (node.LayerRoot != visual)
- {
- MakeLayer(scene, node);
- }
- else
- {
- UpdateLayer(node, scene.Layers[node.LayerRoot]);
- }
- }
- else if (node.LayerRoot == node.Visual && node.Parent != null)
- {
- ClearLayer(scene, node);
- }
-
- if (node.ClipToBounds)
- {
- clip = clip.Intersect(node.ClipBounds);
- }
-
- try
- {
- visual.Render(context);
- }
- catch { }
-
- var transformed = new TransformedBounds(new Rect(visual.Bounds.Size), clip, node.Transform);
- visual.SetTransformedBounds(transformed);
-
- if (forceRecurse)
- {
- var visualChildren = (IList) visual.VisualChildren;
-
- node.TryPreallocateChildren(visualChildren.Count);
-
- if (visualChildren.Count == 1)
- {
- var childNode = GetOrCreateChildNode(scene, visualChildren[0], node);
- Update(context, scene, (VisualNode)childNode, clip, forceRecurse);
- }
- else if (visualChildren.Count > 1)
- {
- var count = visualChildren.Count;
-
- if (visual.HasNonUniformZIndexChildren)
- {
- var sortedChildren = new (Visual visual, int index)[count];
-
- for (var i = 0; i < count; i++)
- {
- sortedChildren[i] = (visualChildren[i], i);
- }
-
- // Regular Array.Sort is unstable, we need to provide indices as well to avoid reshuffling elements.
- Array.Sort(sortedChildren, (lhs, rhs) =>
- {
- var result = ZIndexComparer.Instance.Compare(lhs.visual, rhs.visual);
-
- return result == 0 ? lhs.index.CompareTo(rhs.index) : result;
- });
-
- foreach (var child in sortedChildren)
- {
- var childNode = GetOrCreateChildNode(scene, child.Item1, node);
- Update(context, scene, (VisualNode)childNode, clip, forceRecurse);
- }
- }
- else
- foreach (var child in visualChildren)
- {
- var childNode = GetOrCreateChildNode(scene, child, node);
- Update(context, scene, (VisualNode)childNode, clip, forceRecurse);
- }
- }
-
- node.SubTreeUpdated = true;
- contextImpl.TrimChildren();
- }
- }
- }
- else
- {
- contextImpl.BeginUpdate(node).Dispose();
- }
- }
-
- private static void UpdateSize(Scene scene)
- {
- var renderRoot = scene.Root.Visual as IRenderRoot;
- var newSize = renderRoot?.ClientSize ?? scene.Root.Visual.Bounds.Size;
-
- scene.Scaling = renderRoot?.RenderScaling ?? 1;
-
- if (scene.Size != newSize)
- {
- var oldSize = scene.Size;
-
- scene.Size = newSize;
-
- Rect horizontalDirtyRect = default;
- Rect verticalDirtyRect = default;
-
- if (newSize.Width > oldSize.Width)
- {
- horizontalDirtyRect = new Rect(oldSize.Width, 0, newSize.Width - oldSize.Width, oldSize.Height);
- }
-
- if (newSize.Height > oldSize.Height)
- {
- verticalDirtyRect = new Rect(0, oldSize.Height, newSize.Width, newSize.Height - oldSize.Height);
- }
-
- foreach (var layer in scene.Layers)
- {
- layer.Dirty.Add(horizontalDirtyRect);
- layer.Dirty.Add(verticalDirtyRect);
- }
- }
- }
-
- private static VisualNode CreateNode(Scene scene, Visual visual, VisualNode parent)
- {
- var node = new VisualNode(visual, parent);
- node.LayerRoot = parent.LayerRoot;
- scene.Add(node);
- return node;
- }
-
- private static void Deindex(Scene scene, VisualNode node)
- {
- var nodeChildren = node.Children;
- var nodeChildrenCount = nodeChildren.Count;
-
- for (var i = 0; i < nodeChildrenCount; i++)
- {
- if (nodeChildren[i] is VisualNode visual)
- {
- Deindex(scene, visual);
- }
- }
-
- scene.Remove(node);
-
- node.SubTreeUpdated = true;
-
- scene.Layers[node.LayerRoot!].Dirty.Add(node.Bounds);
-
- node.Visual.SetTransformedBounds(null);
-
- if (node.LayerRoot == node.Visual && node.Visual != scene.Root.Visual)
- {
- scene.Layers.Remove(node.LayerRoot);
- }
- }
-
- private static void ClearLayer(Scene scene, VisualNode node)
- {
- var parent = (VisualNode)node.Parent!;
- var oldLayerRoot = node.LayerRoot;
- var newLayerRoot = parent.LayerRoot!;
- var existingDirtyRects = scene.Layers[node.LayerRoot!].Dirty;
- var newDirtyRects = scene.Layers[newLayerRoot].Dirty;
-
- existingDirtyRects.Coalesce();
-
- foreach (var r in existingDirtyRects)
- {
- newDirtyRects.Add(r);
- }
-
- var oldLayer = scene.Layers[oldLayerRoot!];
- PropagateLayer(node, scene.Layers[newLayerRoot], oldLayer);
- scene.Layers.Remove(oldLayer);
- }
-
- private static void MakeLayer(Scene scene, VisualNode node)
- {
- var oldLayerRoot = node.LayerRoot!;
- var layer = scene.Layers.Add(node.Visual);
- var oldLayer = scene.Layers[oldLayerRoot!];
-
- UpdateLayer(node, layer);
- PropagateLayer(node, layer, scene.Layers[oldLayerRoot]);
- }
-
- private static void UpdateLayer(VisualNode node, SceneLayer layer)
- {
- layer.Opacity = node.Visual.Opacity;
-
- if (node.Visual.OpacityMask != null)
- {
- layer.OpacityMask = node.Visual.OpacityMask?.ToImmutable();
- layer.OpacityMaskRect = node.ClipBounds;
- }
- else
- {
- layer.OpacityMask = null;
- layer.OpacityMaskRect = default;
- }
-
- layer.GeometryClip = node.HasAncestorGeometryClip ?
- CreateLayerGeometryClip(node) :
- null;
- }
-
- private static void PropagateLayer(VisualNode node, SceneLayer layer, SceneLayer oldLayer)
- {
- node.LayerRoot = layer.LayerRoot;
-
- layer.Dirty.Add(node.Bounds);
- oldLayer.Dirty.Add(node.Bounds);
-
- foreach (VisualNode child in node.Children)
- {
- // If the child is not the start of a new layer, recurse.
- if (child.LayerRoot != child.Visual)
- {
- PropagateLayer(child, layer, oldLayer);
- }
- }
- }
-
- // HACK: Disabled layers because they're broken in current renderer. See #2244.
- private static bool ShouldStartLayer(Visual visual) => false;
-
- private static IGeometryImpl? CreateLayerGeometryClip(VisualNode node)
- {
- IGeometryImpl? result = null;
- VisualNode? n = node;
-
- for (;;)
- {
- n = (VisualNode?)n!.Parent;
-
- if (n == null || (n.GeometryClip == null && !n.HasAncestorGeometryClip))
- {
- break;
- }
-
- if (n?.GeometryClip != null)
- {
- var transformed = n.GeometryClip.WithTransform(n.Transform);
-
- result = result == null ? transformed : result.Intersect(transformed);
- }
- }
-
- return result;
- }
- }
-}
diff --git a/src/Avalonia.Base/Rendering/SceneGraph/SceneLayer.cs b/src/Avalonia.Base/Rendering/SceneGraph/SceneLayer.cs
deleted file mode 100644
index a5e3b88188..0000000000
--- a/src/Avalonia.Base/Rendering/SceneGraph/SceneLayer.cs
+++ /dev/null
@@ -1,73 +0,0 @@
-using Avalonia.Media;
-using Avalonia.Platform;
-
-namespace Avalonia.Rendering.SceneGraph
-{
- ///
- /// Represents a layer in a .
- ///
- public class SceneLayer
- {
- ///
- /// Initializes a new instance of the class.
- ///
- /// The visual at the root of the layer.
- /// The distance from the scene root.
- public SceneLayer(Visual layerRoot, int distanceFromRoot)
- {
- LayerRoot = layerRoot;
- Dirty = new DirtyRects();
- DistanceFromRoot = distanceFromRoot;
- }
-
- ///
- /// Clones the layer.
- ///
- /// The cloned layer.
- public SceneLayer Clone()
- {
- return new SceneLayer(LayerRoot, DistanceFromRoot)
- {
- Opacity = Opacity,
- OpacityMask = OpacityMask,
- OpacityMaskRect = OpacityMaskRect,
- GeometryClip = GeometryClip,
- };
- }
-
- ///
- /// Gets the visual at the root of the layer.
- ///
- public Visual LayerRoot { get; }
-
- ///
- /// Gets the distance of the layer root from the root of the scene.
- ///
- public int DistanceFromRoot { get; }
-
- ///
- /// Gets or sets the opacity of the layer.
- ///
- public double Opacity { get; set; } = 1;
-
- ///
- /// Gets or sets the opacity mask for the layer.
- ///
- public IBrush? OpacityMask { get; set; }
-
- ///
- /// Gets or sets the target rectangle for the layer opacity mask.
- ///
- public Rect OpacityMaskRect { get; set; }
-
- ///
- /// Gets the layer's geometry clip.
- ///
- public IGeometryImpl? GeometryClip { get; set; }
-
- ///
- /// Gets the dirty rectangles for the layer.
- ///
- internal DirtyRects Dirty { get; }
- }
-}
diff --git a/src/Avalonia.Base/Rendering/SceneGraph/SceneLayers.cs b/src/Avalonia.Base/Rendering/SceneGraph/SceneLayers.cs
deleted file mode 100644
index 8a997c5ace..0000000000
--- a/src/Avalonia.Base/Rendering/SceneGraph/SceneLayers.cs
+++ /dev/null
@@ -1,206 +0,0 @@
-using System;
-using System.Collections;
-using System.Collections.Generic;
-using Avalonia.VisualTree;
-
-namespace Avalonia.Rendering.SceneGraph
-{
- ///
- /// Holds a collection of layers for a .
- ///
- public class SceneLayers : IEnumerable
- {
- private readonly Visual _root;
- private readonly List _inner;
- private readonly Dictionary _index;
-
- ///
- /// Initializes a new instance of the class.
- ///
- /// The scene's root visual.
- public SceneLayers(Visual root) : this(root, 0)
- {
- }
-
- ///
- /// Initializes a new instance of the class.
- ///
- /// The scene's root visual.
- /// Initial layer capacity.
- public SceneLayers(Visual root, int capacity)
- {
- _root = root;
-
- _inner = new List(capacity);
- _index = new Dictionary(capacity);
- }
-
- ///
- /// Gets the number of layers in the scene.
- ///
- public int Count => _inner.Count;
-
- ///
- /// Gets a value indicating whether any of the layers have a dirty region.
- ///
- public bool HasDirty
- {
- get
- {
- foreach (var layer in _inner)
- {
- if (!layer.Dirty.IsEmpty)
- {
- return true;
- }
- }
-
- return false;
- }
- }
-
- ///
- /// Gets a layer by index.
- ///
- /// The index of the layer.
- /// The layer.
- public SceneLayer this[int index] => _inner[index];
-
- ///
- /// Gets a layer by its root visual.
- ///
- /// The layer's root visual.
- /// The layer.
- public SceneLayer this[Visual visual] => _index[visual];
-
- ///
- /// Adds a layer to the scene.
- ///
- /// The root visual of the layer.
- /// The created layer.
- public SceneLayer Add(Visual layerRoot)
- {
- _ = layerRoot ?? throw new ArgumentNullException(nameof(layerRoot));
-
- var distance = layerRoot.CalculateDistanceFromAncestor(_root);
- var layer = new SceneLayer(layerRoot, distance);
- var insert = FindInsertIndex(layer);
- _index.Add(layerRoot, layer);
- _inner.Insert(insert, layer);
- return layer;
- }
-
- ///
- /// Makes a deep clone of the layers.
- ///
- /// The cloned layers.
- public SceneLayers Clone()
- {
- var result = new SceneLayers(_root, Count);
-
- foreach (var src in _inner)
- {
- var dest = src.Clone();
- result._index.Add(dest.LayerRoot, dest);
- result._inner.Add(dest);
- }
-
- return result;
- }
-
- ///
- /// Tests whether a layer exists with the specified root visual.
- ///
- /// The root visual.
- ///
- /// True if a layer exists with the specified root visual, otherwise false.
- ///
- public bool Exists(Visual layerRoot)
- {
- _ = layerRoot ?? throw new ArgumentNullException(nameof(layerRoot));
-
- return _index.ContainsKey(layerRoot);
- }
-
- ///
- /// Tries to find a layer with the specified root visual.
- ///
- /// The root visual.
- /// The layer if found, otherwise null.
- public SceneLayer? Find(Visual layerRoot)
- {
- _index.TryGetValue(layerRoot, out var result);
- return result;
- }
-
- ///
- /// Gets an existing layer or creates a new one if no existing layer is found.
- ///
- /// The root visual.
- /// The layer.
- public SceneLayer GetOrAdd(Visual layerRoot)
- {
- _ = layerRoot ?? throw new ArgumentNullException(nameof(layerRoot));
-
- if (!_index.TryGetValue(layerRoot, out var result))
- {
- result = Add(layerRoot);
- }
-
- return result;
- }
-
- ///
- /// Removes a layer from the scene.
- ///
- /// The root visual.
- /// True if a matching layer was removed, otherwise false.
- public bool Remove(Visual layerRoot)
- {
- _ = layerRoot ?? throw new ArgumentNullException(nameof(layerRoot));
-
- if (_index.TryGetValue(layerRoot, out var layer))
- {
- Remove(layer);
- }
-
- return layer != null;
- }
-
- ///
- /// Removes a layer from the scene.
- ///
- /// The layer.
- /// True if the layer was part of the scene, otherwise false.
- public bool Remove(SceneLayer layer)
- {
- _ = layer ?? throw new ArgumentNullException(nameof(layer));
-
- _index.Remove(layer.LayerRoot);
- return _inner.Remove(layer);
- }
-
- ///
- public IEnumerator GetEnumerator() => _inner.GetEnumerator();
-
- ///
- IEnumerator IEnumerable.GetEnumerator() => GetEnumerator();
-
- private int FindInsertIndex(SceneLayer insert)
- {
- var index = 0;
-
- foreach (var layer in _inner)
- {
- if (layer.DistanceFromRoot > insert.DistanceFromRoot)
- {
- break;
- }
-
- ++index;
- }
-
- return index;
- }
- }
-}
diff --git a/src/Avalonia.Base/Rendering/SceneGraph/VisualNode.cs b/src/Avalonia.Base/Rendering/SceneGraph/VisualNode.cs
deleted file mode 100644
index b9491e6cbd..0000000000
--- a/src/Avalonia.Base/Rendering/SceneGraph/VisualNode.cs
+++ /dev/null
@@ -1,448 +0,0 @@
-using System;
-using System.Collections.Generic;
-using System.Diagnostics.CodeAnalysis;
-using Avalonia.Reactive;
-using Avalonia.Media;
-using Avalonia.Platform;
-using Avalonia.Utilities;
-using Avalonia.VisualTree;
-
-namespace Avalonia.Rendering.SceneGraph
-{
- ///
- /// A node in the low-level scene graph representing an .
- ///
- internal class VisualNode : IVisualNode
- {
- private static readonly IReadOnlyList EmptyChildren = Array.Empty();
- private static readonly IReadOnlyList> EmptyDrawOperations = Array.Empty>();
-
- private Rect? _bounds;
- private double _opacity;
- private List? _children;
- private List>? _drawOperations;
- private IRef? _drawOperationsRefCounter;
- private bool _drawOperationsCloned;
- private Matrix transformRestore;
-
- ///
- /// Initializes a new instance of the class.
- ///
- /// The visual that this node represents.
- /// The parent scene graph node, if any.
- public VisualNode(Visual visual, IVisualNode? parent)
- {
- Visual = visual ?? throw new ArgumentNullException(nameof(visual));
- Parent = parent;
- HasAncestorGeometryClip = parent != null &&
- (parent.HasAncestorGeometryClip || parent.GeometryClip != null);
- }
-
- ///
- public Visual Visual { get; }
-
- ///
- public IVisualNode? Parent { get; }
-
- ///
- public CornerRadius ClipToBoundsRadius { get; set; }
-
- ///
- public Matrix Transform { get; set; }
-
- ///
- public Rect Bounds => _bounds ?? CalculateBounds();
-
- ///
- public Rect ClipBounds { get; set; }
-
- ///
- public Rect LayoutBounds { get; set; }
-
- ///
- public bool ClipToBounds { get; set; }
-
- ///
- public IGeometryImpl? GeometryClip { get; set; }
-
- ///
- public bool HasAncestorGeometryClip { get; }
-
- ///
- public double Opacity
- {
- get { return _opacity; }
- set
- {
- if (_opacity != value)
- {
- _opacity = value;
- OpacityChanged = true;
- }
- }
- }
-
- ///
- /// Gets or sets the opacity mask for the scene graph node.
- ///
- public IBrush? OpacityMask { get; set; }
-
- ///
- /// Gets a value indicating whether this node in the scene graph has already
- /// been updated in the current update pass.
- ///
- public bool SubTreeUpdated { get; set; }
-
- ///
- /// Gets a value indicating whether the property has changed.
- ///
- public bool OpacityChanged { get; private set; }
-
- public Visual? LayerRoot { get; set; }
-
- ///
- public IReadOnlyList Children => _children ?? EmptyChildren;
-
- ///
- public IReadOnlyList> DrawOperations => _drawOperations ?? EmptyDrawOperations;
-
- ///
- /// Adds a child to the collection.
- ///
- /// The child to add.
- public void AddChild(IVisualNode child)
- {
- if (child.Disposed)
- {
- throw new ObjectDisposedException("Visual node for {node.Visual}");
- }
-
- if (child.Parent != this)
- {
- throw new AvaloniaInternalException("VisualNode added to wrong parent.");
- }
-
- EnsureChildrenCreated();
- _children.Add(child);
- }
-
- ///
- /// Adds an operation to the collection.
- ///
- /// The operation to add.
- public void AddDrawOperation(IRef operation)
- {
- EnsureDrawOperationsCreated();
- _drawOperations.Add(operation.Clone());
- }
-
- ///
- /// Removes a child from the collection.
- ///
- /// The child to remove.
- public void RemoveChild(IVisualNode child)
- {
- EnsureChildrenCreated();
- _children.Remove(child);
- }
-
- ///
- /// Replaces a child in the collection.
- ///
- /// The child to be replaced.
- /// The child to add.
- public void ReplaceChild(int index, IVisualNode node)
- {
- if (node.Disposed)
- {
- throw new ObjectDisposedException("Visual node for {node.Visual}");
- }
-
- if (node.Parent != this)
- {
- throw new AvaloniaInternalException("VisualNode added to wrong parent.");
- }
-
- EnsureChildrenCreated();
- _children[index] = node;
- }
-
- ///
- /// Replaces an item in the collection.
- ///
- /// The operation to be replaced.
- /// The operation to add.
- public void ReplaceDrawOperation(int index, IRef operation)
- {
- EnsureDrawOperationsCreated();
- var old = _drawOperations[index];
- _drawOperations[index] = operation.Clone();
- old.Dispose();
- }
-
- ///
- /// Sorts the collection according to the order of the visual's
- /// children and their z-index.
- ///
- /// The scene that the node is a part of.
- public void SortChildren(Scene scene)
- {
- if (_children == null || _children.Count <= 1)
- {
- return;
- }
-
- var keys = new List(Visual.VisualChildren.Count);
-
- for (var i = 0; i < Visual.VisualChildren.Count; ++i)
- {
- var child = Visual.VisualChildren[i];
- var zIndex = child.ZIndex;
- keys.Add(((long)zIndex << 32) + i);
- }
-
- keys.Sort();
- _children.Clear();
-
- foreach (var i in keys)
- {
- var child = Visual.VisualChildren[(int)(i & 0xffffffff)];
- var node = scene.FindNode(child);
-
- if (node != null)
- {
- _children.Add(node);
- }
- }
- }
-
- ///
- /// Removes items in the collection from the specified index
- /// to the end.
- ///
- /// The index of the first child to be removed.
- public void TrimChildren(int first)
- {
- if (first < _children?.Count)
- {
- EnsureChildrenCreated();
- for (int i = first; i < _children.Count; i++)
- {
- _children[i].Dispose();
- }
- _children.RemoveRange(first, _children.Count - first);
- }
- }
-
- ///
- /// Removes items in the collection from the specified index
- /// to the end.
- ///
- /// The index of the first operation to be removed.
- public void TrimDrawOperations(int first)
- {
- if (first < _drawOperations?.Count)
- {
- EnsureDrawOperationsCreated();
- for (int i = first; i < _drawOperations.Count; i++)
- {
- _drawOperations[i].Dispose();
- }
- _drawOperations.RemoveRange(first, _drawOperations.Count - first);
- }
- }
-
- ///
- /// Makes a copy of the node
- ///
- /// The new parent node.
- /// A cloned node.
- public VisualNode Clone(IVisualNode? parent)
- {
- return new VisualNode(Visual, parent)
- {
- Transform = Transform,
- ClipBounds = ClipBounds,
- ClipToBoundsRadius = ClipToBoundsRadius,
- ClipToBounds = ClipToBounds,
- LayoutBounds = LayoutBounds,
- GeometryClip = GeometryClip,
- _opacity = Opacity,
- OpacityMask = OpacityMask,
- _drawOperations = _drawOperations,
- _drawOperationsRefCounter = _drawOperationsRefCounter?.Clone(),
- _drawOperationsCloned = true,
- LayerRoot= LayerRoot,
- };
- }
-
- ///
- public bool HitTest(Point p)
- {
- var drawOperations = DrawOperations;
- var drawOperationsCount = drawOperations.Count;
-
- for (var i = 0; i < drawOperationsCount; i++)
- {
- var operation = drawOperations[i];
-
- if (operation?.Item?.HitTest(p) == true)
- {
- return true;
- }
- }
-
- return false;
- }
-
- ///
- public void BeginRender(IDrawingContextImpl context, bool skipOpacity)
- {
- transformRestore = context.Transform;
-
- if (ClipToBounds)
- {
- context.Transform = Matrix.Identity;
- if (ClipToBoundsRadius.IsDefault)
- context.PushClip(ClipBounds);
- else
- context.PushClip(new RoundedRect(ClipBounds, ClipToBoundsRadius));
- }
-
- context.Transform = Transform;
-
- if (Opacity != 1 && !skipOpacity)
- {
- context.PushOpacity(Opacity);
- }
-
- if (GeometryClip != null)
- {
- context.PushGeometryClip(GeometryClip);
- }
-
- if (OpacityMask != null)
- {
- context.PushOpacityMask(OpacityMask, LayoutBounds);
- }
- }
-
- ///
- public void EndRender(IDrawingContextImpl context, bool skipOpacity)
- {
- if (OpacityMask != null)
- {
- context.PopOpacityMask();
- }
-
- if (GeometryClip != null)
- {
- context.PopGeometryClip();
- }
-
- if (Opacity != 1 && !skipOpacity)
- {
- context.PopOpacity();
- }
-
- if (ClipToBounds)
- {
- context.Transform = Matrix.Identity;
- context.PopClip();
- }
-
- context.Transform = transformRestore;
- }
-
- internal void TryPreallocateChildren(int count)
- {
- if (count == 0)
- {
- return;
- }
-
- EnsureChildrenCreated(count);
- }
-
- private Rect CalculateBounds()
- {
- var result = new Rect();
-
- if (_drawOperations != null)
- {
- foreach (var operation in _drawOperations)
- {
- result = result.Union(operation.Item.Bounds);
- }
- }
-
- _bounds = result;
- return result;
- }
-
- [MemberNotNull(nameof(_children))]
- private void EnsureChildrenCreated(int capacity = 0)
- {
- if (_children == null)
- {
- _children = new List(capacity);
- }
- }
-
- ///
- /// Ensures that this node draw operations have been created and are mutable (in case we are using cloned operations).
- ///
- [MemberNotNull(nameof(_drawOperations))]
- private void EnsureDrawOperationsCreated()
- {
- if (_drawOperations == null)
- {
- _drawOperations = new List>();
- _drawOperationsRefCounter = RefCountable.Create(CreateDisposeDrawOperations(_drawOperations));
- _drawOperationsCloned = false;
- }
- else if (_drawOperationsCloned)
- {
- var oldDrawOperations = _drawOperations;
-
- _drawOperations = new List>(oldDrawOperations.Count);
-
- foreach (var drawOperation in oldDrawOperations)
- {
- _drawOperations.Add(drawOperation.Clone());
- }
-
- _drawOperationsRefCounter?.Dispose();
- _drawOperationsRefCounter = RefCountable.Create(CreateDisposeDrawOperations(_drawOperations));
- _drawOperationsCloned = false;
- }
- }
-
- ///
- /// Creates disposable that will dispose all items in passed draw operations after being disposed.
- /// It is crucial that we don't capture current instance
- /// as draw operations can be cloned and may persist across subsequent scenes.
- ///
- /// Draw operations that need to be disposed.
- /// Disposable for given draw operations.
- private static IDisposable CreateDisposeDrawOperations(List> drawOperations)
- {
- return Disposable.Create(drawOperations, operations =>
- {
- foreach (var operation in operations)
- {
- operation.Dispose();
- }
- });
- }
-
- public bool Disposed { get; private set; }
-
- public void Dispose()
- {
- _drawOperationsRefCounter?.Dispose();
-
- Disposed = true;
- }
- }
-}
diff --git a/src/Avalonia.Base/Rendering/SceneInvalidatedEventArgs.cs b/src/Avalonia.Base/Rendering/SceneInvalidatedEventArgs.cs
index 73840376fe..cac4d1693a 100644
--- a/src/Avalonia.Base/Rendering/SceneInvalidatedEventArgs.cs
+++ b/src/Avalonia.Base/Rendering/SceneInvalidatedEventArgs.cs
@@ -12,7 +12,7 @@ namespace Avalonia.Rendering
///
/// The render root that has been updated.
/// The updated area.
- internal SceneInvalidatedEventArgs(
+ public SceneInvalidatedEventArgs(
IRenderRoot root,
Rect dirtyRect)
{
diff --git a/src/Avalonia.Base/Visual.cs b/src/Avalonia.Base/Visual.cs
index 7fcb53bcea..e6d7492c51 100644
--- a/src/Avalonia.Base/Visual.cs
+++ b/src/Avalonia.Base/Visual.cs
@@ -36,12 +36,7 @@ namespace Avalonia
///
public static readonly DirectProperty BoundsProperty =
AvaloniaProperty.RegisterDirect(nameof(Bounds), o => o.Bounds);
-
- public static readonly DirectProperty TransformedBoundsProperty =
- AvaloniaProperty.RegisterDirect(
- nameof(TransformedBounds),
- o => o.TransformedBounds);
-
+
///
/// Defines the property.
///
@@ -116,7 +111,6 @@ namespace Avalonia
(s, h) => s.Invalidated -= h);
private Rect _bounds;
- private TransformedBounds? _transformedBounds;
private IRenderRoot? _visualRoot;
private Visual? _visualParent;
private bool _hasMirrorTransform;
@@ -172,11 +166,6 @@ namespace Avalonia
protected set { SetAndRaise(BoundsProperty, ref _bounds, value); }
}
- ///
- /// Gets the bounds of the control relative to the window, accounting for rendering transforms.
- ///
- public TransformedBounds? TransformedBounds => _transformedBounds;
-
///
/// Gets or sets a value indicating whether the control should be clipped to its bounds.
///
@@ -523,11 +512,6 @@ namespace Avalonia
return CompositionVisual;
}
- internal void SetTransformedBounds(TransformedBounds? value)
- {
- SetAndRaise(TransformedBoundsProperty, ref _transformedBounds, value);
- }
-
///
/// Calls the method
/// for this control and all of its visual descendants.
diff --git a/src/Avalonia.Controls.ColorPicker/Themes/Fluent/ColorPicker.xaml b/src/Avalonia.Controls.ColorPicker/Themes/Fluent/ColorPicker.xaml
index 2ccf20d460..8f52b059f1 100644
--- a/src/Avalonia.Controls.ColorPicker/Themes/Fluent/ColorPicker.xaml
+++ b/src/Avalonia.Controls.ColorPicker/Themes/Fluent/ColorPicker.xaml
@@ -245,11 +245,12 @@
VerticalAlignment="Center" />
+
diff --git a/src/Avalonia.Controls.ColorPicker/Themes/Fluent/ColorView.xaml b/src/Avalonia.Controls.ColorPicker/Themes/Fluent/ColorView.xaml
index 286ca71bc2..fab5415bfb 100644
--- a/src/Avalonia.Controls.ColorPicker/Themes/Fluent/ColorView.xaml
+++ b/src/Avalonia.Controls.ColorPicker/Themes/Fluent/ColorView.xaml
@@ -494,11 +494,12 @@
VerticalAlignment="Center" />
+
diff --git a/src/Avalonia.Controls.ColorPicker/Themes/Simple/ColorPicker.xaml b/src/Avalonia.Controls.ColorPicker/Themes/Simple/ColorPicker.xaml
index b82d36a288..d44902dd70 100644
--- a/src/Avalonia.Controls.ColorPicker/Themes/Simple/ColorPicker.xaml
+++ b/src/Avalonia.Controls.ColorPicker/Themes/Simple/ColorPicker.xaml
@@ -245,11 +245,12 @@
VerticalAlignment="Center" />
+
diff --git a/src/Avalonia.Controls.ColorPicker/Themes/Simple/ColorView.xaml b/src/Avalonia.Controls.ColorPicker/Themes/Simple/ColorView.xaml
index 556099f56b..0ce6dd5e04 100644
--- a/src/Avalonia.Controls.ColorPicker/Themes/Simple/ColorView.xaml
+++ b/src/Avalonia.Controls.ColorPicker/Themes/Simple/ColorView.xaml
@@ -456,11 +456,12 @@
VerticalAlignment="Center" />
+
diff --git a/src/Avalonia.Controls.DataGrid/DataGridBoundColumn.cs b/src/Avalonia.Controls.DataGrid/DataGridBoundColumn.cs
index 8f532b9803..110590fef2 100644
--- a/src/Avalonia.Controls.DataGrid/DataGridBoundColumn.cs
+++ b/src/Avalonia.Controls.DataGrid/DataGridBoundColumn.cs
@@ -7,6 +7,7 @@ using Avalonia.Data;
using System;
using Avalonia.Controls.Utils;
using Avalonia.Markup.Xaml.MarkupExtensions;
+using Avalonia.Metadata;
using Avalonia.Reactive;
namespace Avalonia.Controls
@@ -24,6 +25,7 @@ namespace Avalonia.Controls
///
//TODO Binding
[AssignBinding]
+ [InheritDataTypeFromItems(nameof(DataGrid.Items), AncestorType = typeof(DataGrid))]
public virtual IBinding Binding
{
get
diff --git a/src/Avalonia.Controls.DataGrid/DataGridTemplateColumn.cs b/src/Avalonia.Controls.DataGrid/DataGridTemplateColumn.cs
index 516e9cf6c2..00318e2dd8 100644
--- a/src/Avalonia.Controls.DataGrid/DataGridTemplateColumn.cs
+++ b/src/Avalonia.Controls.DataGrid/DataGridTemplateColumn.cs
@@ -24,6 +24,7 @@ namespace Avalonia.Controls
(o, v) => o.CellTemplate = v);
[Content]
+ [InheritDataTypeFromItems(nameof(DataGrid.Items), AncestorType = typeof(DataGrid))]
public IDataTemplate CellTemplate
{
get { return _cellTemplate; }
@@ -50,6 +51,7 @@ namespace Avalonia.Controls
///
/// If this property is the column is read-only.
///
+ [InheritDataTypeFromItems(nameof(DataGrid.Items), AncestorType = typeof(DataGrid))]
public IDataTemplate CellEditingTemplate
{
get => _cellEditingCellTemplate;
diff --git a/src/Avalonia.Controls/Button.cs b/src/Avalonia.Controls/Button.cs
index 1ec6f8dabc..9627f200df 100644
--- a/src/Avalonia.Controls/Button.cs
+++ b/src/Avalonia.Controls/Button.cs
@@ -394,10 +394,10 @@ namespace Avalonia.Controls
if (e.GetCurrentPoint(this).Properties.IsLeftButtonPressed)
{
IsPressed = true;
- e.Handled = true;
if (ClickMode == ClickMode.Press)
{
+ e.Handled = true;
OnClick();
}
}
@@ -411,11 +411,11 @@ namespace Avalonia.Controls
if (IsPressed && e.InitialPressMouseButton == MouseButton.Left)
{
IsPressed = false;
- e.Handled = true;
if (ClickMode == ClickMode.Release &&
this.GetVisualsAt(e.GetPosition(this)).Any(c => this == c || this.IsVisualAncestorOf(c)))
{
+ e.Handled = true;
OnClick();
}
}
diff --git a/src/Avalonia.Controls/Embedding/Offscreen/OffscreenTopLevelImpl.cs b/src/Avalonia.Controls/Embedding/Offscreen/OffscreenTopLevelImpl.cs
index 64f96b6987..3a4ae80cf4 100644
--- a/src/Avalonia.Controls/Embedding/Offscreen/OffscreenTopLevelImpl.cs
+++ b/src/Avalonia.Controls/Embedding/Offscreen/OffscreenTopLevelImpl.cs
@@ -1,10 +1,13 @@
using System;
using System.Collections.Generic;
+using System.Diagnostics;
using Avalonia.Input;
using Avalonia.Input.Raw;
using Avalonia.Metadata;
using Avalonia.Platform;
using Avalonia.Rendering;
+using Avalonia.Rendering.Composition;
+using Avalonia.Threading;
namespace Avalonia.Controls.Embedding.Offscreen
{
@@ -13,7 +16,7 @@ namespace Avalonia.Controls.Embedding.Offscreen
{
private double _scaling = 1;
private Size _clientSize;
- private PlatformRenderInterfaceContextManager _renderInterface = new(null);
+ private ManualRenderTimer _manualRenderTimer = new();
public IInputRoot? InputRoot { get; private set; }
public bool IsDisposed { get; private set; }
@@ -23,10 +26,19 @@ namespace Avalonia.Controls.Embedding.Offscreen
IsDisposed = true;
}
+ class ManualRenderTimer : IRenderTimer
+ {
+ static Stopwatch St = Stopwatch.StartNew();
+ public event Action? Tick;
+ public bool RunsInBackground => false;
+ public void TriggerTick() => Tick?.Invoke(St.Elapsed);
+ }
+
+
public IRenderer CreateRenderer(IRenderRoot root) =>
- new ImmediateRenderer((Visual)root, () => _renderInterface.CreateRenderTarget(Surfaces), _renderInterface);
+ new CompositingRenderer(root, new Compositor(new RenderLoop(_manualRenderTimer, Dispatcher.UIThread), null),
+ () => Surfaces);
- public abstract void Invalidate(Rect rect);
public abstract IEnumerable
public bool UseDBusFilePicker { get; set; } = true;
-
- ///
- /// Deferred renderer would be used when set to true. Immediate renderer when set to false. The default value is true.
- ///
- ///
- /// Avalonia has two rendering modes: Immediate and Deferred rendering.
- /// Immediate re-renders the whole scene when some element is changed on the scene. Deferred re-renders only changed elements.
- ///
- public bool UseDeferredRendering { get; set; } = true;
-
- public bool UseCompositor { get; set; } = true;
-
+
///
/// Determines whether to use IME.
/// IME would be enabled by default if the current user input language is one of the following: Mandarin, Japanese, Vietnamese or Korean.
diff --git a/src/Avalonia.X11/X11Window.Xim.cs b/src/Avalonia.X11/X11Window.Xim.cs
index 10aece7f2e..8446d35cc6 100644
--- a/src/Avalonia.X11/X11Window.Xim.cs
+++ b/src/Avalonia.X11/X11Window.Xim.cs
@@ -13,7 +13,7 @@ namespace Avalonia.X11
class XimInputMethod : ITextInputMethodImpl, IX11InputMethodControl
{
private readonly X11Window _parent;
- private bool _controlActive, _windowActive, _imeActive;
+ private bool _windowActive, _imeActive;
private Rect? _queuedCursorRect;
private ITextInputMethodClient? _client;
@@ -70,7 +70,7 @@ namespace Avalonia.X11
private void UpdateActive()
{
- var active = _windowActive && _controlActive;
+ var active = _windowActive && IsActive;
if(_parent._xic == IntPtr.Zero)
return;
if (active != _imeActive)
diff --git a/src/Avalonia.X11/X11Window.cs b/src/Avalonia.X11/X11Window.cs
index 84331aa43a..c78ef2350e 100644
--- a/src/Avalonia.X11/X11Window.cs
+++ b/src/Avalonia.X11/X11Window.cs
@@ -27,11 +27,7 @@ using static Avalonia.X11.XLib;
// ReSharper disable StringLiteralTypo
namespace Avalonia.X11
{
- unsafe partial class X11Window : IWindowImpl, IPopupImpl, IXI2Client,
- ITopLevelImplWithNativeMenuExporter,
- ITopLevelImplWithNativeControlHost,
- ITopLevelImplWithTextInputMethod,
- ITopLevelImplWithStorageProvider
+ unsafe partial class X11Window : IWindowImpl, IPopupImpl, IXI2Client
{
private readonly AvaloniaX11Platform _platform;
private readonly bool _popup;
@@ -43,6 +39,9 @@ namespace Avalonia.X11
private readonly MouseDevice _mouse;
private readonly TouchDevice _touch;
private readonly IKeyboardDevice _keyboard;
+ private readonly ITopLevelNativeMenuExporter _nativeMenuExporter;
+ private readonly IStorageProvider _storageProvider;
+ private readonly X11NativeControlHost _nativeControlHost;
private PixelPoint? _position;
private PixelSize _realSize;
private IntPtr _handle;
@@ -199,8 +198,8 @@ namespace Avalonia.X11
if(_popup)
PopupPositioner = new ManagedPopupPositioner(new ManagedPopupPositionerPopupImplHelper(popupParent, MoveResize));
if (platform.Options.UseDBusMenu)
- NativeMenuExporter = DBusMenuExporter.TryCreateTopLevelNativeMenu(_handle);
- NativeControlHost = new X11NativeControlHost(_platform, this);
+ _nativeMenuExporter = DBusMenuExporter.TryCreateTopLevelNativeMenu(_handle);
+ _nativeControlHost = new X11NativeControlHost(_platform, this);
InitializeIme();
XChangeProperty(_x11.Display, _handle, _x11.Atoms.WM_PROTOCOLS, _x11.Atoms.XA_ATOM, 32,
@@ -213,7 +212,7 @@ namespace Avalonia.X11
_x11.Atoms.XA_CARDINAL, 32, PropertyMode.Replace, ref _xSyncCounter, 1);
}
- StorageProvider = new CompositeStorageProvider(new Func>[]
+ _storageProvider = new CompositeStorageProvider(new Func>[]
{
() => _platform.Options.UseDBusFilePicker ? DBusSystemDialog.TryCreate(Handle) : Task.FromResult(null),
() => GtkSystemDialog.TryCreate(this),
@@ -384,25 +383,8 @@ namespace Avalonia.X11
public Action PositionChanged { get; set; }
public Action LostFocus { get; set; }
- public IRenderer CreateRenderer(IRenderRoot root)
- {
- var loop = AvaloniaLocator.Current.GetService();
- var customRendererFactory = AvaloniaLocator.Current.GetService();
-
- if (customRendererFactory != null)
- return customRendererFactory.Create(root, loop);
-
- return _platform.Options.UseDeferredRendering
- ? _platform.Options.UseCompositor
- ? new CompositingRenderer(root, this._platform.Compositor, () => Surfaces)
- : new DeferredRenderer(root, loop, () => _platform.RenderInterface.CreateRenderTarget(Surfaces), _platform.RenderInterface)
- {
- RenderOnlyOnRenderThread = true
- }
- : new X11ImmediateRendererProxy((Visual)root, loop,
- () => _platform.RenderInterface.CreateRenderTarget(Surfaces),
- _platform.RenderInterface);
- }
+ public IRenderer CreateRenderer(IRenderRoot root) =>
+ new CompositingRenderer(root, _platform.Compositor, () => Surfaces);
void OnEvent(ref XEvent ev)
{
@@ -813,6 +795,31 @@ namespace Avalonia.X11
Cleanup();
}
+ public virtual object TryGetFeature(Type featureType)
+ {
+ if (featureType == typeof(ITopLevelNativeMenuExporter))
+ {
+ return _nativeMenuExporter;
+ }
+
+ if (featureType == typeof(IStorageProvider))
+ {
+ return _storageProvider;
+ }
+
+ if (featureType == typeof(ITextInputMethodImpl))
+ {
+ return _ime;
+ }
+
+ if (featureType == typeof(INativeControlHostImpl))
+ {
+ return _nativeControlHost;
+ }
+
+ return null;
+ }
+
void Cleanup()
{
if (_rawEventGrouper != null)
@@ -1212,9 +1219,6 @@ namespace Avalonia.X11
}
public IPopupPositioner PopupPositioner { get; }
- public ITopLevelNativeMenuExporter NativeMenuExporter { get; }
- public INativeControlHostImpl NativeControlHost { get; }
- public ITextInputMethodImpl TextInputMethod => _ime;
public void SetTransparencyLevelHint(WindowTransparencyLevel transparencyLevel) =>
_transparencyHelper?.SetTransparencyRequest(transparencyLevel);
@@ -1232,8 +1236,6 @@ namespace Avalonia.X11
public bool NeedsManagedDecorations => false;
- public IStorageProvider StorageProvider { get; }
-
public class SurfacePlatformHandle : IPlatformNativeSurfaceHandle
{
private readonly X11Window _owner;
diff --git a/src/Browser/Avalonia.Browser/BrowserSystemNavigationManager.cs b/src/Browser/Avalonia.Browser/BrowserSystemNavigationManager.cs
index cc7d8c3af1..bc38067f4a 100644
--- a/src/Browser/Avalonia.Browser/BrowserSystemNavigationManager.cs
+++ b/src/Browser/Avalonia.Browser/BrowserSystemNavigationManager.cs
@@ -5,11 +5,11 @@ using Avalonia.Platform;
namespace Avalonia.Browser
{
- internal class BrowserSystemNavigationManager : ISystemNavigationManager
+ internal class BrowserSystemNavigationManagerImpl : ISystemNavigationManagerImpl
{
public event EventHandler? BackRequested;
- public BrowserSystemNavigationManager()
+ public BrowserSystemNavigationManagerImpl()
{
NavigationHelper.AddBackHandler(() =>
{
diff --git a/src/Browser/Avalonia.Browser/BrowserTopLevelImpl.cs b/src/Browser/Avalonia.Browser/BrowserTopLevelImpl.cs
index 3affcfb4c2..f1cd441f45 100644
--- a/src/Browser/Avalonia.Browser/BrowserTopLevelImpl.cs
+++ b/src/Browser/Avalonia.Browser/BrowserTopLevelImpl.cs
@@ -19,8 +19,7 @@ using Avalonia.Rendering.Composition;
namespace Avalonia.Browser
{
- internal class BrowserTopLevelImpl : ITopLevelImplWithTextInputMethod, ITopLevelImplWithNativeControlHost, ITopLevelImplWithStorageProvider,
- ITopLevelWithSystemNavigationManager
+ internal class BrowserTopLevelImpl : ITopLevelImpl
{
private Size _clientSize;
private IInputRoot? _inputRoot;
@@ -29,6 +28,9 @@ namespace Avalonia.Browser
private readonly TouchDevice _touchDevice;
private readonly PenDevice _penDevice;
private string _currentCursor = CssCursor.Default;
+ private readonly INativeControlHostImpl _nativeControlHost;
+ private readonly IStorageProvider _storageProvider;
+ private readonly ISystemNavigationManagerImpl _systemNavigationManager;
public BrowserTopLevelImpl(AvaloniaView avaloniaView)
{
@@ -38,7 +40,9 @@ namespace Avalonia.Browser
AcrylicCompensationLevels = new AcrylicPlatformCompensationLevels(1, 1, 1);
_touchDevice = new TouchDevice();
_penDevice = new PenDevice();
- NativeControlHost = _avaloniaView.GetNativeControlHostImpl();
+ _nativeControlHost = _avaloniaView.GetNativeControlHostImpl();
+ _storageProvider = new BrowserStorageProvider();
+ _systemNavigationManager = new BrowserSystemNavigationManagerImpl();
}
public ulong Timestamp => (ulong)_sw.ElapsedMilliseconds;
@@ -236,11 +240,29 @@ namespace Avalonia.Browser
public AcrylicPlatformCompensationLevels AcrylicCompensationLevels { get; }
- public ITextInputMethodImpl TextInputMethod => _avaloniaView;
+ public object? TryGetFeature(Type featureType)
+ {
+ if (featureType == typeof(IStorageProvider))
+ {
+ return _storageProvider;
+ }
- public INativeControlHostImpl? NativeControlHost { get; }
- public IStorageProvider StorageProvider { get; } = new BrowserStorageProvider();
+ if (featureType == typeof(ITextInputMethodImpl))
+ {
+ return _avaloniaView;
+ }
- public ISystemNavigationManager SystemNavigationManager { get; } = new BrowserSystemNavigationManager();
+ if (featureType == typeof(ISystemNavigationManagerImpl))
+ {
+ return _systemNavigationManager;
+ }
+
+ if (featureType == typeof(INativeControlHostImpl))
+ {
+ return _nativeControlHost;
+ }
+
+ return null;
+ }
}
}
diff --git a/src/Browser/Avalonia.Browser/Interop/StorageHelper.cs b/src/Browser/Avalonia.Browser/Interop/StorageHelper.cs
index e50f8790ef..a4d7bcdb87 100644
--- a/src/Browser/Avalonia.Browser/Interop/StorageHelper.cs
+++ b/src/Browser/Avalonia.Browser/Interop/StorageHelper.cs
@@ -25,6 +25,9 @@ internal static partial class StorageHelper
public static partial Task SaveFileDialog(JSObject? startIn, string? suggestedName,
[JSMarshalAs>] object[]? types, bool excludeAcceptAllOption);
+ [JSImport("StorageItem.createWellKnownDirectory", AvaloniaModule.StorageModuleName)]
+ public static partial JSObject CreateWellKnownDirectory(string wellKnownDirectory);
+
[JSImport("StorageProvider.openBookmark", AvaloniaModule.StorageModuleName)]
public static partial Task OpenBookmark(string key);
diff --git a/src/Browser/Avalonia.Browser/Storage/BrowserStorageProvider.cs b/src/Browser/Avalonia.Browser/Storage/BrowserStorageProvider.cs
index 0bebcb9f63..7e8e2e0990 100644
--- a/src/Browser/Avalonia.Browser/Storage/BrowserStorageProvider.cs
+++ b/src/Browser/Avalonia.Browser/Storage/BrowserStorageProvider.cs
@@ -116,6 +116,33 @@ internal class BrowserStorageProvider : IStorageProvider
return item is not null ? new JSStorageFolder(item) : null;
}
+ public Task TryGetFileFromPath(Uri filePath)
+ {
+ return Task.FromResult(null);
+ }
+
+ public Task TryGetFolderFromPath(Uri folderPath)
+ {
+ return Task.FromResult(null);
+ }
+
+ public async Task TryGetWellKnownFolder(WellKnownFolder wellKnownFolder)
+ {
+ await _lazyModule.Value;
+ var directory = StorageHelper.CreateWellKnownDirectory(wellKnownFolder switch
+ {
+ WellKnownFolder.Desktop => "desktop",
+ WellKnownFolder.Documents => "documents",
+ WellKnownFolder.Downloads => "downloads",
+ WellKnownFolder.Music => "music",
+ WellKnownFolder.Pictures => "pictures",
+ WellKnownFolder.Videos => "videos",
+ _ => throw new ArgumentOutOfRangeException(nameof(wellKnownFolder), wellKnownFolder, null)
+ });
+
+ return new JSStorageFolder(directory);
+ }
+
private static (JSObject[]? types, bool excludeAllOption) ConvertFileTypes(IEnumerable? input)
{
var types = input?
@@ -145,12 +172,7 @@ internal abstract class JSStorageItem : IStorageBookmarkItem
internal JSObject FileHandle => _fileHandle ?? throw new ObjectDisposedException(nameof(JSStorageItem));
public string Name => FileHandle.GetPropertyAsString("name") ?? string.Empty;
-
- public bool TryGetUri([NotNullWhen(true)] out Uri? uri)
- {
- uri = new Uri(Name, UriKind.Relative);
- return false;
- }
+ public Uri Path => new Uri(Name, UriKind.Relative);
public async Task GetBasicPropertiesAsync()
{
diff --git a/src/Browser/Avalonia.Browser/webapp/modules/storage/storageItem.ts b/src/Browser/Avalonia.Browser/webapp/modules/storage/storageItem.ts
index b4cf3d278b..c6e5254329 100644
--- a/src/Browser/Avalonia.Browser/webapp/modules/storage/storageItem.ts
+++ b/src/Browser/Avalonia.Browser/webapp/modules/storage/storageItem.ts
@@ -1,14 +1,29 @@
import { avaloniaDb, fileBookmarksStore } from "./indexedDb";
export class StorageItem {
- constructor(public handle: FileSystemHandle, private readonly bookmarkId?: string) { }
+ constructor(
+ public handle?: FileSystemHandle,
+ private readonly bookmarkId?: string,
+ public wellKnownType?: WellKnownDirectory
+ ) {
+ }
public get name(): string {
- return this.handle.name;
+ if (this.handle) {
+ return this.handle.name;
+ }
+ return this.wellKnownType ?? "";
+ }
+
+ public get kind(): "file" | "directory" {
+ if (this.handle) {
+ return this.handle.kind;
+ }
+ return "directory";
}
- public get kind(): string {
- return this.handle.kind;
+ public static createWellKnownDirectory(type: WellKnownDirectory) {
+ return new StorageItem(undefined, undefined, type);
}
public static async openRead(item: StorageItem): Promise {
@@ -48,7 +63,7 @@ export class StorageItem {
}
public static async getItems(item: StorageItem): Promise {
- if (item.handle.kind !== "directory") {
+ if (item.kind !== "directory" || !item.handle) {
return new StorageItems([]);
}
@@ -60,6 +75,10 @@ export class StorageItem {
}
private async verityPermissions(mode: FileSystemPermissionMode): Promise {
+ if (!this.handle) {
+ return;
+ }
+
if (await this.handle.queryPermission({ mode }) === "granted") {
return;
}
@@ -69,11 +88,14 @@ export class StorageItem {
}
}
- public static async saveBookmark(item: StorageItem): Promise {
+ public static async saveBookmark(item: StorageItem): Promise {
// If file was previously bookmarked, just return old one.
if (item.bookmarkId) {
return item.bookmarkId;
}
+ if (!item.handle) {
+ return null;
+ }
const connection = await avaloniaDb.connect();
try {
diff --git a/src/Browser/Avalonia.Browser/webapp/modules/storage/storageProvider.ts b/src/Browser/Avalonia.Browser/webapp/modules/storage/storageProvider.ts
index 0198f92528..e621a1ed30 100644
--- a/src/Browser/Avalonia.Browser/webapp/modules/storage/storageProvider.ts
+++ b/src/Browser/Avalonia.Browser/webapp/modules/storage/storageProvider.ts
@@ -17,7 +17,7 @@ export class StorageProvider {
startIn: StorageItem | null): Promise {
// 'Picker' API doesn't accept "null" as a parameter, so it should be set to undefined.
const options: DirectoryPickerOptions = {
- startIn: (startIn?.handle ?? undefined)
+ startIn: (startIn?.wellKnownType ?? startIn?.handle ?? undefined)
};
const handle = await window.showDirectoryPicker(options);
@@ -28,7 +28,7 @@ export class StorageProvider {
startIn: StorageItem | null, multiple: boolean,
types: FilePickerAcceptType[] | null, excludeAcceptAllOption: boolean): Promise {
const options: OpenFilePickerOptions = {
- startIn: (startIn?.handle ?? undefined),
+ startIn: (startIn?.wellKnownType ?? startIn?.handle ?? undefined),
multiple,
excludeAcceptAllOption,
types: (types ?? undefined)
@@ -42,7 +42,7 @@ export class StorageProvider {
startIn: StorageItem | null, suggestedName: string | null,
types: FilePickerAcceptType[] | null, excludeAcceptAllOption: boolean): Promise {
const options: SaveFilePickerOptions = {
- startIn: (startIn?.handle ?? undefined),
+ startIn: (startIn?.wellKnownType ?? startIn?.handle ?? undefined),
suggestedName: (suggestedName ?? undefined),
excludeAcceptAllOption,
types: (types ?? undefined)
diff --git a/src/Linux/Avalonia.LinuxFramebuffer/FramebufferToplevelImpl.cs b/src/Linux/Avalonia.LinuxFramebuffer/FramebufferToplevelImpl.cs
index cb158097eb..9dc7f08064 100644
--- a/src/Linux/Avalonia.LinuxFramebuffer/FramebufferToplevelImpl.cs
+++ b/src/Linux/Avalonia.LinuxFramebuffer/FramebufferToplevelImpl.cs
@@ -24,16 +24,12 @@ namespace Avalonia.LinuxFramebuffer
_inputBackend = inputBackend;
Surfaces = new object[] { _outputBackend };
-
- Invalidate(default);
_inputBackend.Initialize(this, e => Input?.Invoke(e));
}
public IRenderer CreateRenderer(IRenderRoot root)
{
- var factory = AvaloniaLocator.Current.GetService();
- var renderLoop = AvaloniaLocator.Current.GetService();
- return factory?.Create(root, renderLoop) ?? new CompositingRenderer(root, LinuxFramebufferPlatform.Compositor, () => Surfaces);
+ return new CompositingRenderer(root, LinuxFramebufferPlatform.Compositor, () => Surfaces);
}
public void Dispose()
@@ -41,11 +37,6 @@ namespace Avalonia.LinuxFramebuffer
throw new NotSupportedException();
}
-
- public void Invalidate(Rect rect)
- {
- }
-
public void SetInputRoot(IInputRoot inputRoot)
{
InputRoot = inputRoot;
@@ -86,5 +77,6 @@ namespace Avalonia.LinuxFramebuffer
public void SetFrameThemeVariant(PlatformThemeVariant themeVariant) { }
public AcrylicPlatformCompensationLevels AcrylicCompensationLevels { get; } = new AcrylicPlatformCompensationLevels(1, 1, 1);
+ public object TryGetFeature(Type featureType) => null;
}
}
diff --git a/src/Linux/Avalonia.LinuxFramebuffer/LinuxFramebufferPlatform.cs b/src/Linux/Avalonia.LinuxFramebuffer/LinuxFramebufferPlatform.cs
index 4202ba821f..2dcce12df9 100644
--- a/src/Linux/Avalonia.LinuxFramebuffer/LinuxFramebufferPlatform.cs
+++ b/src/Linux/Avalonia.LinuxFramebuffer/LinuxFramebufferPlatform.cs
@@ -26,8 +26,8 @@ namespace Avalonia.LinuxFramebuffer
private static readonly Stopwatch St = Stopwatch.StartNew();
internal static uint Timestamp => (uint)St.ElapsedTicks;
public static InternalPlatformThreadingInterface? Threading;
-
- internal static Compositor? Compositor { get; private set; }
+
+ internal static Compositor Compositor { get; private set; } = null!;
LinuxFramebufferPlatform(IOutputBackend backend)
diff --git a/src/Linux/Avalonia.LinuxFramebuffer/Output/FbdevOutput.cs b/src/Linux/Avalonia.LinuxFramebuffer/Output/FbdevOutput.cs
index f3f9a12ac8..56c6bdb8c4 100644
--- a/src/Linux/Avalonia.LinuxFramebuffer/Output/FbdevOutput.cs
+++ b/src/Linux/Avalonia.LinuxFramebuffer/Output/FbdevOutput.cs
@@ -93,9 +93,8 @@ namespace Avalonia.LinuxFramebuffer
void SetBpp(PixelFormat format)
{
- switch (format)
+ if (format == PixelFormat.Rgba8888)
{
- case PixelFormat.Rgba8888:
_varInfo.bits_per_pixel = 32;
_varInfo.grayscale = 0;
_varInfo.red = _varInfo.blue = _varInfo.green = _varInfo.transp = new fb_bitfield
@@ -105,8 +104,9 @@ namespace Avalonia.LinuxFramebuffer
_varInfo.green.offset = 8;
_varInfo.blue.offset = 16;
_varInfo.transp.offset = 24;
- break;
- case PixelFormat.Bgra8888:
+ }
+ else if (format == PixelFormat.Bgra8888)
+ {
_varInfo.bits_per_pixel = 32;
_varInfo.grayscale = 0;
_varInfo.red = _varInfo.blue = _varInfo.green = _varInfo.transp = new fb_bitfield
@@ -116,8 +116,9 @@ namespace Avalonia.LinuxFramebuffer
_varInfo.green.offset = 8;
_varInfo.red.offset = 16;
_varInfo.transp.offset = 24;
- break;
- case PixelFormat.Rgb565:
+ }
+ else if (format == PixelFormat.Rgb565)
+ {
_varInfo.bits_per_pixel = 16;
_varInfo.grayscale = 0;
_varInfo.red = _varInfo.blue = _varInfo.green = _varInfo.transp = new fb_bitfield();
@@ -126,8 +127,8 @@ namespace Avalonia.LinuxFramebuffer
_varInfo.green.length = 6;
_varInfo.blue.offset = 11;
_varInfo.blue.length = 5;
- break;
}
+ else throw new NotSupportedException($"Pixel format {format} is not supported");
}
public string Id { get; private set; }
diff --git a/src/Markup/Avalonia.Markup.Xaml.Loader/CompilerExtensions/Transformers/AvaloniaXamlIlDataContextTypeTransformer.cs b/src/Markup/Avalonia.Markup.Xaml.Loader/CompilerExtensions/Transformers/AvaloniaXamlIlDataContextTypeTransformer.cs
index 574d46e737..a24d4eb6e9 100644
--- a/src/Markup/Avalonia.Markup.Xaml.Loader/CompilerExtensions/Transformers/AvaloniaXamlIlDataContextTypeTransformer.cs
+++ b/src/Markup/Avalonia.Markup.Xaml.Loader/CompilerExtensions/Transformers/AvaloniaXamlIlDataContextTypeTransformer.cs
@@ -68,29 +68,44 @@ namespace Avalonia.Markup.Xaml.XamlIl.CompilerExtensions.Transformers
// If there is no x:DataType directive,
// do more specialized inference
- if (directiveDataContextTypeNode is null)
+ if (directiveDataContextTypeNode is null && inferredDataContextTypeNode is null)
{
- if (context.GetAvaloniaTypes().IDataTemplate.IsAssignableFrom(on.Type.GetClrType())
- && inferredDataContextTypeNode is null)
+ // Infer data type from collection binding on a control that displays items.
+ var property = context.ParentNodes().OfType().FirstOrDefault();
+ var attributeType = context.GetAvaloniaTypes().InheritDataTypeFromItemsAttribute;
+ var attribute = property?.Property?.GetClrProperty().CustomAttributes
+ .FirstOrDefault(a => a.Type == attributeType);
+
+ if (attribute is not null)
{
- // Infer data type from collection binding on a control that displays items.
- var parentObject = context.ParentNodes().OfType().FirstOrDefault();
- if (parentObject != null)
+ var propertyName = (string)attribute.Parameters.First();
+ XamlAstConstructableObjectNode parentObject;
+ if (attribute.Properties.TryGetValue("AncestorType", out var type)
+ && type is IXamlType xamlType)
{
- var parentType = parentObject.Type.GetClrType();
-
- if (context.GetAvaloniaTypes().ItemsControl.IsDirectlyAssignableFrom(parentType)
- || context.GetAvaloniaTypes().ItemsRepeater.IsDirectlyAssignableFrom(parentType))
- {
- inferredDataContextTypeNode = InferDataContextOfPresentedItem(context, on, parentObject);
- }
+ parentObject = context.ParentNodes().OfType()
+ .FirstOrDefault(n => n.Type.GetClrType().FullName == xamlType.FullName);
}
-
- if (inferredDataContextTypeNode is null)
+ else
+ {
+ parentObject = context.ParentNodes().OfType().FirstOrDefault();
+ }
+
+ if (parentObject != null)
{
- inferredDataContextTypeNode = new AvaloniaXamlIlUninferrableDataContextMetadataNode(on);
+ inferredDataContextTypeNode = InferDataContextOfPresentedItem(context, on, parentObject, propertyName);
}
}
+
+ if (inferredDataContextTypeNode is null
+ // Only for IDataTemplate, as we want to notify user as early as possible,
+ // and IDataTemplate cannot inherit DataType from the parent implicitly.
+ && context.GetAvaloniaTypes().IDataTemplate.IsAssignableFrom(on.Type.GetClrType()))
+ {
+ // We can't infer the collection type and the currently calculated type is definitely wrong.
+ // Notify the user that we were unable to infer the data context type if they use a compiled binding.
+ inferredDataContextTypeNode = new AvaloniaXamlIlUninferrableDataContextMetadataNode(on);
+ }
}
return directiveDataContextTypeNode ?? inferredDataContextTypeNode ?? node;
@@ -98,18 +113,18 @@ namespace Avalonia.Markup.Xaml.XamlIl.CompilerExtensions.Transformers
return node;
}
-
- private static AvaloniaXamlIlDataContextTypeMetadataNode InferDataContextOfPresentedItem(AstTransformationContext context, XamlAstConstructableObjectNode on, XamlAstConstructableObjectNode parentObject)
+
+ private static AvaloniaXamlIlDataContextTypeMetadataNode InferDataContextOfPresentedItem(
+ AstTransformationContext context, XamlAstConstructableObjectNode on,
+ XamlAstConstructableObjectNode parentObject, string propertyName)
{
var parentItemsValue = parentObject
.Children.OfType()
- .FirstOrDefault(pa => pa.Property.Name == "Items")
+ .FirstOrDefault(pa => pa.Property.Name == propertyName)
?.Values[0];
if (parentItemsValue is null)
{
- // We can't infer the collection type and the currently calculated type is definitely wrong.
- // Notify the user that we were unable to infer the data context type if they use a compiled binding.
- return new AvaloniaXamlIlUninferrableDataContextMetadataNode(on);
+ return null;
}
IXamlType itemsCollectionType = null;
@@ -140,9 +155,8 @@ namespace Avalonia.Markup.Xaml.XamlIl.CompilerExtensions.Transformers
}
}
}
- // We can't infer the collection type and the currently calculated type is definitely wrong.
- // Notify the user that we were unable to infer the data context type if they use a compiled binding.
- return new AvaloniaXamlIlUninferrableDataContextMetadataNode(on);
+
+ return null;
}
private static AvaloniaXamlIlDataContextTypeMetadataNode ParseDataContext(AstTransformationContext context, XamlAstConstructableObjectNode on, XamlAstConstructableObjectNode obj)
@@ -208,6 +222,6 @@ namespace Avalonia.Markup.Xaml.XamlIl.CompilerExtensions.Transformers
{
}
- public override IXamlType DataContextType => throw new XamlTransformException("Unable to infer DataContext type for compiled bindings nested within this element.", Value);
+ public override IXamlType DataContextType => throw new XamlTransformException("Unable to infer DataContext type for compiled bindings nested within this element. Please set x:DataType on the Binding or parent.", Value);
}
}
diff --git a/src/Markup/Avalonia.Markup.Xaml.Loader/CompilerExtensions/Transformers/AvaloniaXamlIlWellKnownTypes.cs b/src/Markup/Avalonia.Markup.Xaml.Loader/CompilerExtensions/Transformers/AvaloniaXamlIlWellKnownTypes.cs
index 0b61316603..0aa3dda693 100644
--- a/src/Markup/Avalonia.Markup.Xaml.Loader/CompilerExtensions/Transformers/AvaloniaXamlIlWellKnownTypes.cs
+++ b/src/Markup/Avalonia.Markup.Xaml.Loader/CompilerExtensions/Transformers/AvaloniaXamlIlWellKnownTypes.cs
@@ -30,6 +30,7 @@ namespace Avalonia.Markup.Xaml.XamlIl.CompilerExtensions.Transformers
public IXamlType AssignBindingAttribute { get; }
public IXamlType DependsOnAttribute { get; }
public IXamlType DataTypeAttribute { get; }
+ public IXamlType InheritDataTypeFromItemsAttribute { get; }
public IXamlType MarkupExtensionOptionAttribute { get; }
public IXamlType MarkupExtensionDefaultOptionAttribute { get; }
public IXamlType OnExtensionType { get; }
@@ -135,6 +136,7 @@ namespace Avalonia.Markup.Xaml.XamlIl.CompilerExtensions.Transformers
AssignBindingAttribute = cfg.TypeSystem.GetType("Avalonia.Data.AssignBindingAttribute");
DependsOnAttribute = cfg.TypeSystem.GetType("Avalonia.Metadata.DependsOnAttribute");
DataTypeAttribute = cfg.TypeSystem.GetType("Avalonia.Metadata.DataTypeAttribute");
+ InheritDataTypeFromItemsAttribute = cfg.TypeSystem.GetType("Avalonia.Metadata.InheritDataTypeFromItemsAttribute");
MarkupExtensionOptionAttribute = cfg.TypeSystem.GetType("Avalonia.Metadata.MarkupExtensionOptionAttribute");
MarkupExtensionDefaultOptionAttribute = cfg.TypeSystem.GetType("Avalonia.Metadata.MarkupExtensionDefaultOptionAttribute");
OnExtensionType = cfg.TypeSystem.GetType("Avalonia.Markup.Xaml.MarkupExtensions.On");
diff --git a/src/Markup/Avalonia.Markup.Xaml.Loader/CompilerExtensions/XamlIlBindingPathHelper.cs b/src/Markup/Avalonia.Markup.Xaml.Loader/CompilerExtensions/XamlIlBindingPathHelper.cs
index ae29dcf9cb..9c7b6deb7c 100644
--- a/src/Markup/Avalonia.Markup.Xaml.Loader/CompilerExtensions/XamlIlBindingPathHelper.cs
+++ b/src/Markup/Avalonia.Markup.Xaml.Loader/CompilerExtensions/XamlIlBindingPathHelper.cs
@@ -37,6 +37,10 @@ namespace Avalonia.Markup.Xaml.XamlIl.CompilerExtensions
bindingResultType = transformed.BindingResultType;
binding.Arguments[0] = transformed;
}
+ else if (binding.Arguments.Count > 0 && binding.Arguments[0] is XamlIlBindingPathNode alreadyTransformed)
+ {
+ bindingResultType = alreadyTransformed.BindingResultType;
+ }
else
{
var bindingPathAssignment = binding.Children.OfType()
diff --git a/src/Markup/Avalonia.Markup/Data/TemplateBinding.cs b/src/Markup/Avalonia.Markup/Data/TemplateBinding.cs
index 4270063f87..ba321db144 100644
--- a/src/Markup/Avalonia.Markup/Data/TemplateBinding.cs
+++ b/src/Markup/Avalonia.Markup/Data/TemplateBinding.cs
@@ -137,9 +137,9 @@ namespace Avalonia.Data
templatedParent.GetValue(Property) :
_target.TemplatedParent;
- if (Converter is not null && _targetType is not null)
+ if (Converter is not null)
{
- value = Converter.Convert(value, _targetType, ConverterParameter, CultureInfo.CurrentCulture);
+ value = Converter.Convert(value, _targetType ?? typeof(object), ConverterParameter, CultureInfo.CurrentCulture);
}
PublishNext(value);
diff --git a/src/Skia/Avalonia.Skia/ImmutableBitmap.cs b/src/Skia/Avalonia.Skia/ImmutableBitmap.cs
index 802736119f..4ab873fd8d 100644
--- a/src/Skia/Avalonia.Skia/ImmutableBitmap.cs
+++ b/src/Skia/Avalonia.Skia/ImmutableBitmap.cs
@@ -10,9 +10,10 @@ namespace Avalonia.Skia
///
/// Immutable Skia bitmap.
///
- internal class ImmutableBitmap : IDrawableBitmapImpl
+ internal class ImmutableBitmap : IDrawableBitmapImpl, IReadableBitmapImpl
{
private readonly SKImage _image;
+ private readonly SKBitmap? _bitmap;
///
/// Create immutable bitmap from given stream.
@@ -23,12 +24,13 @@ namespace Avalonia.Skia
using (var skiaStream = new SKManagedStream(stream))
{
using (var data = SKData.Create(skiaStream))
- _image = SKImage.FromEncodedData(data);
-
- if (_image == null)
- {
+ _bitmap = SKBitmap.Decode(data);
+
+ if (_bitmap == null)
throw new ArgumentException("Unable to load bitmap from provided data");
- }
+
+ _bitmap.SetImmutable();
+ _image = SKImage.FromBitmap(_bitmap);
PixelSize = new PixelSize(_image.Width, _image.Height);
@@ -47,10 +49,10 @@ namespace Avalonia.Skia
public ImmutableBitmap(ImmutableBitmap src, PixelSize destinationSize, BitmapInterpolationMode interpolationMode)
{
SKImageInfo info = new SKImageInfo(destinationSize.Width, destinationSize.Height, SKColorType.Bgra8888);
- SKImage output = SKImage.Create(info);
- src._image.ScalePixels(output.PeekPixels(), interpolationMode.ToSKFilterQuality());
-
- _image = output;
+ _bitmap = new SKBitmap(info);
+ src._image.ScalePixels(_bitmap.PeekPixels(), interpolationMode.ToSKFilterQuality());
+ _bitmap.SetImmutable();
+ _image = SKImage.FromBitmap(_bitmap);
PixelSize = new PixelSize(_image.Width, _image.Height);
@@ -71,8 +73,11 @@ namespace Avalonia.Skia
// decode the bitmap at the nearest size
var nearest = new SKImageInfo(supportedScale.Width, supportedScale.Height);
- var bmp = SKBitmap.Decode(codec, nearest);
+ _bitmap = SKBitmap.Decode(codec, nearest);
+ if (_bitmap == null)
+ throw new ArgumentException("Unable to load bitmap from provided data");
+
// now scale that to the size that we want
var realScale = horizontal ? ((double)info.Height / info.Width) : ((double)info.Width / info.Height);
@@ -88,15 +93,16 @@ namespace Avalonia.Skia
desired = new SKImageInfo((int)(realScale * decodeSize), decodeSize);
}
- if (bmp.Width != desired.Width || bmp.Height != desired.Height)
+ if (_bitmap.Width != desired.Width || _bitmap.Height != desired.Height)
{
- var scaledBmp = bmp.Resize(desired, interpolationMode.ToSKFilterQuality());
- bmp.Dispose();
- bmp = scaledBmp;
+ var scaledBmp = _bitmap.Resize(desired, interpolationMode.ToSKFilterQuality());
+ _bitmap.Dispose();
+ _bitmap = scaledBmp;
}
+
+ _bitmap!.SetImmutable();
- _image = SKImage.FromBitmap(bmp);
- bmp.Dispose();
+ _image = SKImage.FromBitmap(_bitmap);
if (_image == null)
{
@@ -121,9 +127,15 @@ namespace Avalonia.Skia
/// Data pixels.
public ImmutableBitmap(PixelSize size, Vector dpi, int stride, PixelFormat format, AlphaFormat alphaFormat, IntPtr data)
{
- var imageInfo = new SKImageInfo(size.Width, size.Height, format.ToSkColorType(), alphaFormat.ToSkAlphaType());
-
- _image = SKImage.FromPixelCopy(imageInfo, data, stride);
+ using (var tmp = new SKBitmap())
+ {
+ tmp.InstallPixels(
+ new SKImageInfo(size.Width, size.Height, format.ToSkColorType(), alphaFormat.ToSkAlphaType()),
+ data);
+ _bitmap = tmp.Copy();
+ }
+ _bitmap!.SetImmutable();
+ _image = SKImage.FromBitmap(_bitmap);
if (_image == null)
{
@@ -143,6 +155,7 @@ namespace Avalonia.Skia
public void Dispose()
{
_image.Dispose();
+ _bitmap?.Dispose();
}
///
@@ -162,5 +175,14 @@ namespace Avalonia.Skia
{
context.Canvas.DrawImage(_image, sourceRect, destRect, paint);
}
+
+ public PixelFormat? Format => _bitmap?.ColorType.ToAvalonia();
+ public ILockedFramebuffer Lock()
+ {
+ if (_bitmap == null)
+ throw new NotSupportedException();
+ return new LockedFramebuffer(_bitmap.GetPixels(), PixelSize, _bitmap.RowBytes, Dpi,
+ _bitmap.ColorType.ToAvalonia().Value, null);
+ }
}
}
diff --git a/src/Skia/Avalonia.Skia/PlatformRenderInterface.cs b/src/Skia/Avalonia.Skia/PlatformRenderInterface.cs
index b4297a7c33..d12db39ad6 100644
--- a/src/Skia/Avalonia.Skia/PlatformRenderInterface.cs
+++ b/src/Skia/Avalonia.Skia/PlatformRenderInterface.cs
@@ -41,6 +41,11 @@ namespace Avalonia.Skia
public PixelFormat DefaultPixelFormat { get; }
+ public bool IsSupportedBitmapPixelFormat(PixelFormat format) =>
+ format == PixelFormats.Rgb565
+ || format == PixelFormats.Bgra8888
+ || format == PixelFormats.Rgba8888;
+
public IGeometryImpl CreateEllipseGeometry(Rect rect) => new EllipseGeometryImpl(rect);
public IGeometryImpl CreateLineGeometry(Point p1, Point p2) => new LineGeometryImpl(p1, p2);
diff --git a/src/Skia/Avalonia.Skia/SkiaSharpExtensions.cs b/src/Skia/Avalonia.Skia/SkiaSharpExtensions.cs
index d584216f17..20dde27e9a 100644
--- a/src/Skia/Avalonia.Skia/SkiaSharpExtensions.cs
+++ b/src/Skia/Avalonia.Skia/SkiaSharpExtensions.cs
@@ -127,6 +127,17 @@ namespace Avalonia.Skia
throw new ArgumentException("Unknown pixel format: " + fmt);
}
+ public static PixelFormat? ToAvalonia(this SKColorType colorType)
+ {
+ if (colorType == SKColorType.Rgb565)
+ return PixelFormats.Rgb565;
+ if (colorType == SKColorType.Bgra8888)
+ return PixelFormats.Bgra8888;
+ if (colorType == SKColorType.Rgba8888)
+ return PixelFormats.Rgba8888;
+ return null;
+ }
+
public static PixelFormat ToPixelFormat(this SKColorType fmt)
{
if (fmt == SKColorType.Rgb565)
diff --git a/src/Skia/Avalonia.Skia/WriteableBitmapImpl.cs b/src/Skia/Avalonia.Skia/WriteableBitmapImpl.cs
index 9864a14a9c..56e627f2d8 100644
--- a/src/Skia/Avalonia.Skia/WriteableBitmapImpl.cs
+++ b/src/Skia/Avalonia.Skia/WriteableBitmapImpl.cs
@@ -154,6 +154,8 @@ namespace Avalonia.Skia
}
}
+ public PixelFormat? Format => _bitmap.ColorType.ToAvalonia();
+
///
public ILockedFramebuffer Lock() => new BitmapFramebuffer(this, _bitmap);
diff --git a/src/Windows/Avalonia.Direct2D1/Direct2D1Platform.cs b/src/Windows/Avalonia.Direct2D1/Direct2D1Platform.cs
index fbf8097ece..eb3f9911df 100644
--- a/src/Windows/Avalonia.Direct2D1/Direct2D1Platform.cs
+++ b/src/Windows/Avalonia.Direct2D1/Direct2D1Platform.cs
@@ -339,5 +339,8 @@ namespace Avalonia.Direct2D1
public AlphaFormat DefaultAlphaFormat => AlphaFormat.Premul;
public PixelFormat DefaultPixelFormat => PixelFormat.Bgra8888;
+ public bool IsSupportedBitmapPixelFormat(PixelFormat format) =>
+ format == PixelFormats.Bgra8888
+ || format == PixelFormats.Rgba8888;
}
}
diff --git a/src/Windows/Avalonia.Direct2D1/Media/Imaging/WicBitmapImpl.cs b/src/Windows/Avalonia.Direct2D1/Media/Imaging/WicBitmapImpl.cs
index 051790ef03..72a48aca0c 100644
--- a/src/Windows/Avalonia.Direct2D1/Media/Imaging/WicBitmapImpl.cs
+++ b/src/Windows/Avalonia.Direct2D1/Media/Imaging/WicBitmapImpl.cs
@@ -1,11 +1,14 @@
using System;
using System.IO;
+using Avalonia.Direct2D1.Media.Imaging;
using Avalonia.Win32.Interop;
using SharpDX.WIC;
using APixelFormat = Avalonia.Platform.PixelFormat;
using AlphaFormat = Avalonia.Platform.AlphaFormat;
using D2DBitmap = SharpDX.Direct2D1.Bitmap;
using Avalonia.Metadata;
+using Avalonia.Platform;
+using PixelFormat = SharpDX.WIC.PixelFormat;
namespace Avalonia.Direct2D1.Media
{
@@ -13,7 +16,7 @@ namespace Avalonia.Direct2D1.Media
/// A WIC implementation of a .
///
[Unstable]
- public class WicBitmapImpl : BitmapImpl
+ public class WicBitmapImpl : BitmapImpl, IReadableBitmapImpl
{
private readonly BitmapDecoder _decoder;
@@ -197,5 +200,38 @@ namespace Avalonia.Direct2D1.Media
encoder.Commit();
}
}
+
+ class LockedBitmap : ILockedFramebuffer
+ {
+ private readonly WicBitmapImpl _parent;
+ private readonly BitmapLock _lock;
+ private readonly APixelFormat _format;
+
+ public LockedBitmap(WicBitmapImpl parent, BitmapLock l, APixelFormat format)
+ {
+ _parent = parent;
+ _lock = l;
+ _format = format;
+ }
+
+
+ public void Dispose()
+ {
+ _lock.Dispose();
+ _parent.Version++;
+ }
+
+ public IntPtr Address => _lock.Data.DataPointer;
+ public PixelSize Size => _lock.Size.ToAvalonia();
+ public int RowBytes => _lock.Stride;
+ public Vector Dpi => _parent.Dpi;
+ public APixelFormat Format => _format;
+
+ }
+
+ APixelFormat? IReadableBitmapImpl.Format => PixelFormat;
+
+ public ILockedFramebuffer Lock() =>
+ new LockedBitmap(this, WicImpl.Lock(BitmapLockFlags.Write), PixelFormat.Value);
}
}
diff --git a/src/Windows/Avalonia.Direct2D1/Media/Imaging/WriteableWicBitmapImpl.cs b/src/Windows/Avalonia.Direct2D1/Media/Imaging/WriteableWicBitmapImpl.cs
index 2e40bdd9d1..5f4c033cf7 100644
--- a/src/Windows/Avalonia.Direct2D1/Media/Imaging/WriteableWicBitmapImpl.cs
+++ b/src/Windows/Avalonia.Direct2D1/Media/Imaging/WriteableWicBitmapImpl.cs
@@ -29,35 +29,6 @@ namespace Avalonia.Direct2D1.Media.Imaging
{
}
- class LockedBitmap : ILockedFramebuffer
- {
- private readonly WriteableWicBitmapImpl _parent;
- private readonly BitmapLock _lock;
- private readonly PixelFormat _format;
-
- public LockedBitmap(WriteableWicBitmapImpl parent, BitmapLock l, PixelFormat format)
- {
- _parent = parent;
- _lock = l;
- _format = format;
- }
-
-
- public void Dispose()
- {
- _lock.Dispose();
- _parent.Version++;
- }
-
- public IntPtr Address => _lock.Data.DataPointer;
- public PixelSize Size => _lock.Size.ToAvalonia();
- public int RowBytes => _lock.Stride;
- public Vector Dpi => _parent.Dpi;
- public PixelFormat Format => _format;
-
- }
-
- public ILockedFramebuffer Lock() =>
- new LockedBitmap(this, WicImpl.Lock(BitmapLockFlags.Write), PixelFormat.Value);
+ public PixelFormat? Format => PixelFormat;
}
}
diff --git a/src/Windows/Avalonia.Win32.Interop/Wpf/WpfTopLevelImpl.cs b/src/Windows/Avalonia.Win32.Interop/Wpf/WpfTopLevelImpl.cs
index 76fc4fa21d..aff533c443 100644
--- a/src/Windows/Avalonia.Win32.Interop/Wpf/WpfTopLevelImpl.cs
+++ b/src/Windows/Avalonia.Win32.Interop/Wpf/WpfTopLevelImpl.cs
@@ -261,5 +261,7 @@ namespace Avalonia.Win32.Interop.Wpf
public void SetFrameThemeVariant(PlatformThemeVariant themeVariant) { }
public AcrylicPlatformCompensationLevels AcrylicCompensationLevels { get; } = new AcrylicPlatformCompensationLevels(1, 1, 1);
+
+ public object TryGetFeature(Type featureType) => null;
}
}
diff --git a/src/Windows/Avalonia.Win32/FramebufferManager.cs b/src/Windows/Avalonia.Win32/FramebufferManager.cs
index 7f4b1c976d..8feecab4dd 100644
--- a/src/Windows/Avalonia.Win32/FramebufferManager.cs
+++ b/src/Windows/Avalonia.Win32/FramebufferManager.cs
@@ -11,7 +11,7 @@ namespace Avalonia.Win32
internal class FramebufferManager : IFramebufferPlatformSurface, IDisposable
{
private const int _bytesPerPixel = 4;
- private const PixelFormat _format = PixelFormat.Bgra8888;
+ private static readonly PixelFormat s_format = PixelFormat.Bgra8888;
private readonly IntPtr _hwnd;
private readonly object _lock;
@@ -50,7 +50,7 @@ namespace Avalonia.Win32
return fb = new LockedFramebuffer(
framebufferData.Data.Address, framebufferData.Size, framebufferData.RowBytes,
- GetCurrentDpi(), _format, _onDisposeAction);
+ GetCurrentDpi(), s_format, _onDisposeAction);
}
finally
{
diff --git a/src/Windows/Avalonia.Win32/Win32Platform.cs b/src/Windows/Avalonia.Win32/Win32Platform.cs
index 7e1e00bded..f16a1ca8cf 100644
--- a/src/Windows/Avalonia.Win32/Win32Platform.cs
+++ b/src/Windows/Avalonia.Win32/Win32Platform.cs
@@ -42,17 +42,6 @@ namespace Avalonia
///
public class Win32PlatformOptions
{
- ///
- /// Deferred renderer would be used when set to true. Immediate renderer when set to false. The default value is true.
- ///
- ///
- /// Avalonia has two rendering modes: Immediate and Deferred rendering.
- /// Immediate re-renders the whole scene when some element is changed on the scene. Deferred re-renders only changed elements.
- ///
- public bool UseDeferredRendering { get; set; } = true;
-
- public bool UseCompositor { get; set; } = true;
-
///
/// Enables ANGLE for Windows. For every Windows version that is above Windows 7, the default is true otherwise it's false.
///
@@ -137,12 +126,10 @@ namespace Avalonia.Win32
///
public static Version WindowsVersion { get; } = RtlGetVersion();
- public static bool UseDeferredRendering => Options.UseDeferredRendering;
internal static bool UseOverlayPopups => Options.OverlayPopups;
public static Win32PlatformOptions Options { get; private set; }
internal static Compositor Compositor { get; private set; }
- internal static PlatformRenderInterfaceContextManager RenderInterface { get; private set; }
public static void Initialize()
{
@@ -181,11 +168,8 @@ namespace Avalonia.Win32
if (OleContext.Current != null)
AvaloniaLocator.CurrentMutable.Bind().ToSingleton();
-
- if (Options.UseCompositor)
- Compositor = new Compositor(AvaloniaLocator.Current.GetRequiredService(), platformGraphics);
- else
- RenderInterface = new PlatformRenderInterfaceContextManager(platformGraphics);
+
+ Compositor = new Compositor(AvaloniaLocator.Current.GetRequiredService(), platformGraphics);
}
public bool HasMessages()
diff --git a/src/Windows/Avalonia.Win32/Win32StorageProvider.cs b/src/Windows/Avalonia.Win32/Win32StorageProvider.cs
index 8cc0a380f9..2da642bd19 100644
--- a/src/Windows/Avalonia.Win32/Win32StorageProvider.cs
+++ b/src/Windows/Avalonia.Win32/Win32StorageProvider.cs
@@ -134,10 +134,10 @@ namespace Avalonia.Win32
}
}
- if (folder?.TryGetUri(out var folderPath) == true)
+ if (folder?.TryGetFullPath() is { } folderPath)
{
var riid = UnmanagedMethods.ShellIds.IShellItem;
- if (UnmanagedMethods.SHCreateItemFromParsingName(folderPath.LocalPath, IntPtr.Zero, ref riid, out var directoryShellItem)
+ if (UnmanagedMethods.SHCreateItemFromParsingName(folderPath, IntPtr.Zero, ref riid, out var directoryShellItem)
== (uint)UnmanagedMethods.HRESULT.S_OK)
{
var proxy = MicroComRuntime.CreateProxyFor(directoryShellItem, true);
diff --git a/src/Windows/Avalonia.Win32/WindowImpl.cs b/src/Windows/Avalonia.Win32/WindowImpl.cs
index a7741477e5..1e0d92d442 100644
--- a/src/Windows/Avalonia.Win32/WindowImpl.cs
+++ b/src/Windows/Avalonia.Win32/WindowImpl.cs
@@ -35,10 +35,7 @@ namespace Avalonia.Win32
/// Window implementation for Win32 platform.
///
[Unstable]
- public partial class WindowImpl : IWindowImpl, EglGlPlatformSurface.IEglWindowGlPlatformSurfaceInfo,
- ITopLevelImplWithNativeControlHost,
- ITopLevelImplWithTextInputMethod,
- ITopLevelImplWithStorageProvider
+ public partial class WindowImpl : IWindowImpl, EglGlPlatformSurface.IEglWindowGlPlatformSurfaceInfo
{
private static readonly List s_instances = new List();
@@ -83,6 +80,7 @@ namespace Avalonia.Win32
private readonly bool _wmPointerEnabled;
private Win32NativeControlHost _nativeControlHost;
+ private IStorageProvider _storageProvider;
private WndProc _wndProcDelegate;
private string _className;
private IntPtr _hwnd;
@@ -183,7 +181,7 @@ namespace Avalonia.Win32
}
Screen = new ScreenImpl();
- StorageProvider = new Win32StorageProvider(this);
+ _storageProvider = new Win32StorageProvider(this);
_nativeControlHost = new Win32NativeControlHost(this, _isUsingComposition);
s_instances.Add(this);
@@ -322,6 +320,26 @@ namespace Avalonia.Win32
private bool IsMouseInPointerEnabled => _wmPointerEnabled && IsMouseInPointerEnabled();
+ public object TryGetFeature(Type featureType)
+ {
+ if (featureType == typeof(ITextInputMethodImpl))
+ {
+ return Imm32InputMethod.Current;
+ }
+
+ if (featureType == typeof(INativeControlHostImpl))
+ {
+ return _nativeControlHost;
+ }
+
+ if (featureType == typeof(IStorageProvider))
+ {
+ return _storageProvider;
+ }
+
+ return null;
+ }
+
public void SetTransparencyLevelHint(WindowTransparencyLevel transparencyLevel)
{
TransparencyLevel = EnableBlur(transparencyLevel);
@@ -558,32 +576,8 @@ namespace Avalonia.Win32
_maxSize = maxSize;
}
- public IRenderer CreateRenderer(IRenderRoot root)
- {
- var loop = AvaloniaLocator.Current.GetService();
- var customRendererFactory = AvaloniaLocator.Current.GetService();
-
- if (customRendererFactory != null)
- return customRendererFactory.Create(root, loop);
-
- if (Win32Platform.Compositor != null)
- return new CompositingRenderer(root, Win32Platform.Compositor, () => Surfaces);
-
- return Win32Platform.UseDeferredRendering
- ? _isUsingComposition
- ? new DeferredRenderer(root, loop,
- () => Win32Platform.RenderInterface.CreateRenderTarget(Surfaces),
- Win32Platform.RenderInterface)
- {
- RenderOnlyOnRenderThread = true
- }
- : (IRenderer)new DeferredRenderer(root, loop, rendererLock: _rendererLock,
- renderTargetFactory: () => Win32Platform.RenderInterface.CreateRenderTarget(Surfaces),
- renderInterface: Win32Platform.RenderInterface)
- : new ImmediateRenderer((Visual)root,
- () => Win32Platform.RenderInterface.CreateRenderTarget(Surfaces),
- Win32Platform.RenderInterface);
- }
+ public IRenderer CreateRenderer(IRenderRoot root) =>
+ new CompositingRenderer(root, Win32Platform.Compositor, () => Surfaces);
public void Resize(Size value, PlatformResizeReason reason)
{
@@ -1489,10 +1483,6 @@ namespace Avalonia.Win32
public void Dispose() => _owner._resizeReason = _restore;
}
- public ITextInputMethodImpl TextInputMethod => Imm32InputMethod.Current;
-
- public IStorageProvider StorageProvider { get; }
-
private class WindowImplPlatformHandle : IPlatformNativeSurfaceHandle
{
private readonly WindowImpl _owner;
diff --git a/src/iOS/Avalonia.iOS/AvaloniaView.cs b/src/iOS/Avalonia.iOS/AvaloniaView.cs
index 7442861597..2d6b93f818 100644
--- a/src/iOS/Avalonia.iOS/AvaloniaView.cs
+++ b/src/iOS/Avalonia.iOS/AvaloniaView.cs
@@ -68,17 +68,18 @@ namespace Avalonia.iOS
settings?.TraitCollectionDidChange();
}
- internal class TopLevelImpl : ITopLevelImplWithTextInputMethod, ITopLevelImplWithNativeControlHost,
- ITopLevelImplWithStorageProvider
+ internal class TopLevelImpl : ITopLevelImpl
{
private readonly AvaloniaView _view;
+ private readonly INativeControlHostImpl _nativeControlHost;
+ private readonly IStorageProvider _storageProvider;
public AvaloniaView View => _view;
public TopLevelImpl(AvaloniaView view)
{
_view = view;
- NativeControlHost = new NativeControlHostImpl(_view);
- StorageProvider = new IOSStorageProvider(view);
+ _nativeControlHost = new NativeControlHostImpl(_view);
+ _storageProvider = new IOSStorageProvider(view);
}
public void Dispose()
@@ -157,9 +158,25 @@ namespace Avalonia.iOS
public AcrylicPlatformCompensationLevels AcrylicCompensationLevels { get; } =
new AcrylicPlatformCompensationLevels();
- public ITextInputMethodImpl? TextInputMethod => _view;
- public INativeControlHostImpl NativeControlHost { get; }
- public IStorageProvider StorageProvider { get; }
+ public object? TryGetFeature(Type featureType)
+ {
+ if (featureType == typeof(IStorageProvider))
+ {
+ return _storageProvider;
+ }
+
+ if (featureType == typeof(ITextInputMethodImpl))
+ {
+ return _view;
+ }
+
+ if (featureType == typeof(INativeControlHostImpl))
+ {
+ return _nativeControlHost;
+ }
+
+ return null;
+ }
}
[Export("layerClass")]
diff --git a/src/iOS/Avalonia.iOS/Storage/IOSStorageItem.cs b/src/iOS/Avalonia.iOS/Storage/IOSStorageItem.cs
index a801e83562..ef0e2467dc 100644
--- a/src/iOS/Avalonia.iOS/Storage/IOSStorageItem.cs
+++ b/src/iOS/Avalonia.iOS/Storage/IOSStorageItem.cs
@@ -25,7 +25,7 @@ internal abstract class IOSStorageItem : IStorageBookmarkItem
using (var doc = new UIDocument(url))
{
_filePath = doc.FileUrl?.Path ?? url.FilePathUrl.Path;
- Name = doc.LocalizedName ?? Path.GetFileName(_filePath) ?? url.FilePathUrl.LastPathComponent;
+ Name = doc.LocalizedName ?? System.IO.Path.GetFileName(_filePath) ?? url.FilePathUrl.LastPathComponent;
}
}
@@ -34,6 +34,7 @@ internal abstract class IOSStorageItem : IStorageBookmarkItem
public bool CanBookmark => true;
public string Name { get; }
+ public Uri Path => Url!;
public Task GetBasicPropertiesAsync()
{
@@ -83,12 +84,6 @@ internal abstract class IOSStorageItem : IStorageBookmarkItem
}
}
- public bool TryGetUri([NotNullWhen(true)] out Uri uri)
- {
- uri = Url;
- return uri is not null;
- }
-
public void Dispose()
{
}
@@ -121,18 +116,34 @@ internal sealed class IOSStorageFolder : IOSStorageItem, IStorageBookmarkFolder
{
}
- public Task> GetItemsAsync()
+ public async Task> GetItemsAsync()
{
- var content = NSFileManager.DefaultManager.GetDirectoryContent(Url, null, NSDirectoryEnumerationOptions.None, out var error);
+ var tcs = new TaskCompletionSource>();
+
+ new NSFileCoordinator().CoordinateRead(Url,
+ NSFileCoordinatorReadingOptions.WithoutChanges,
+ out var error,
+ uri =>
+ {
+ var content = NSFileManager.DefaultManager.GetDirectoryContent(uri, null, NSDirectoryEnumerationOptions.None, out var error);
+ if (error is not null)
+ {
+ tcs.TrySetException(new NSErrorException(error));
+ }
+ else
+ {
+ var items = content
+ .Select(u => u.HasDirectoryPath ? (IStorageItem)new IOSStorageFolder(u) : new IOSStorageFile(u))
+ .ToArray();
+ tcs.TrySetResult(items);
+ }
+ });
+
if (error is not null)
{
- return Task.FromException>(new NSErrorException(error));
+ throw new NSErrorException(error);
}
- var items = content
- .Select(u => u.HasDirectoryPath ? (IStorageItem)new IOSStorageFolder(u) : new IOSStorageFile(u))
- .ToArray();
-
- return Task.FromResult>(items);
+ return await tcs.Task;
}
}
diff --git a/src/iOS/Avalonia.iOS/Storage/IOSStorageProvider.cs b/src/iOS/Avalonia.iOS/Storage/IOSStorageProvider.cs
index 5fdb7ecce1..376e988e74 100644
--- a/src/iOS/Avalonia.iOS/Storage/IOSStorageProvider.cs
+++ b/src/iOS/Avalonia.iOS/Storage/IOSStorageProvider.cs
@@ -3,6 +3,7 @@ using System.Linq;
using System.Collections.Generic;
using System.Diagnostics;
using System.Threading.Tasks;
+using Avalonia.Controls;
using Avalonia.Logging;
using Avalonia.Platform.Storage;
using UIKit;
@@ -99,6 +100,40 @@ internal class IOSStorageProvider : IStorageProvider
? new IOSStorageFolder(url) : null);
}
+ public Task TryGetFileFromPath(Uri filePath)
+ {
+ // TODO: research if it's possible, maybe with additional permissions.
+ return Task.FromResult(null);
+ }
+
+ public Task TryGetFolderFromPath(Uri folderPath)
+ {
+ // TODO: research if it's possible, maybe with additional permissions.
+ return Task.FromResult(null);
+ }
+
+ public Task TryGetWellKnownFolder(WellKnownFolder wellKnownFolder)
+ {
+ var directoryType = wellKnownFolder switch
+ {
+ WellKnownFolder.Desktop => NSSearchPathDirectory.DesktopDirectory,
+ WellKnownFolder.Documents => NSSearchPathDirectory.DocumentDirectory,
+ WellKnownFolder.Downloads => NSSearchPathDirectory.DownloadsDirectory,
+ WellKnownFolder.Music => NSSearchPathDirectory.MusicDirectory,
+ WellKnownFolder.Pictures => NSSearchPathDirectory.PicturesDirectory,
+ WellKnownFolder.Videos => NSSearchPathDirectory.MoviesDirectory,
+ _ => throw new ArgumentOutOfRangeException(nameof(wellKnownFolder), wellKnownFolder, null)
+ };
+
+ var uri = NSFileManager.DefaultManager.GetUrl(directoryType, NSSearchPathDomain.Local, null, true, out var error);
+ if (error != null)
+ {
+ throw new NSErrorException(error);
+ }
+
+ return Task.FromResult(new IOSStorageFolder(uri));
+ }
+
public Task SaveFilePickerAsync(FilePickerSaveOptions options)
{
return Task.FromException(
@@ -127,17 +162,12 @@ internal class IOSStorageProvider : IStorageProvider
private static NSUrl? GetUrlFromFolder(IStorageFolder? folder)
{
- if (folder is IOSStorageFolder iosFolder)
+ return folder switch
{
- return iosFolder.Url;
- }
-
- if (folder?.TryGetUri(out var fullPath) == true)
- {
- return fullPath;
- }
-
- return null;
+ IOSStorageFolder iosFolder => iosFolder.Url,
+ null => null,
+ _ => folder.Path
+ };
}
private Task ShowPicker(UIDocumentPickerViewController documentPicker)
diff --git a/tests/Avalonia.Base.UnitTests/Composition/CompositionAnimationTests.cs b/tests/Avalonia.Base.UnitTests/Composition/CompositionAnimationTests.cs
index 9b38422dde..41c0eb3958 100644
--- a/tests/Avalonia.Base.UnitTests/Composition/CompositionAnimationTests.cs
+++ b/tests/Avalonia.Base.UnitTests/Composition/CompositionAnimationTests.cs
@@ -9,6 +9,7 @@ using Avalonia.Rendering.Composition;
using Avalonia.Rendering.Composition.Expressions;
using Avalonia.Rendering.Composition.Server;
using Avalonia.Threading;
+using Avalonia.UnitTests;
using Xunit;
using Xunit.Sdk;
@@ -67,7 +68,7 @@ public class CompositionAnimationTests
public void GenericCheck(AnimationData data)
{
var compositor =
- new Compositor(new RenderLoop(new CompositorTestsBase.ManualRenderTimer(), new Dispatcher(null)), null);
+ new Compositor(new RenderLoop(new CompositorTestServices.ManualRenderTimer(), new Dispatcher(null)), null);
var target = compositor.CreateSolidColorVisual();
var ani = new ScalarKeyFrameAnimation(null);
foreach (var frame in data.Frames)
diff --git a/tests/Avalonia.Base.UnitTests/Input/PointerTestsBase.cs b/tests/Avalonia.Base.UnitTests/Input/PointerTestsBase.cs
index 510adb37b5..2d45c699f1 100644
--- a/tests/Avalonia.Base.UnitTests/Input/PointerTestsBase.cs
+++ b/tests/Avalonia.Base.UnitTests/Input/PointerTestsBase.cs
@@ -35,6 +35,7 @@ public abstract class PointerTestsBase
impl.DefaultValue = DefaultValue.Mock;
impl.SetupAllProperties();
impl.SetupGet(r => r.RenderScaling).Returns(1);
+ impl.Setup(r => r.TryGetFeature(It.IsAny())).Returns(null);
impl.Setup(r => r.CreateRenderer(It.IsAny())).Returns(renderer);
impl.Setup(r => r.PointToScreen(It.IsAny())).Returns(p => new PixelPoint((int)p.X, (int)p.Y));
impl.Setup(r => r.PointToClient(It.IsAny())).Returns(p => new Point(p.X, p.Y));
diff --git a/tests/Avalonia.Base.UnitTests/Rendering/CompositorHitTestingTests.cs b/tests/Avalonia.Base.UnitTests/Rendering/CompositorHitTestingTests.cs
index 3d8369faeb..27bb0355e6 100644
--- a/tests/Avalonia.Base.UnitTests/Rendering/CompositorHitTestingTests.cs
+++ b/tests/Avalonia.Base.UnitTests/Rendering/CompositorHitTestingTests.cs
@@ -20,7 +20,7 @@ public class CompositorHitTestingTests : CompositorTestsBase
[Fact]
public void HitTest_Should_Find_Controls_At_Point()
{
- using (var s = new CompositorServices(new Size(200, 200)))
+ using (var s = new CompositorTestServices(new Size(200, 200)))
{
var border = new Border
{
@@ -40,7 +40,7 @@ public class CompositorHitTestingTests : CompositorTestsBase
[Fact]
public void HitTest_Should_Not_Find_Empty_Controls_At_Point()
{
- using (var s = new CompositorServices(new Size(200, 200)))
+ using (var s = new CompositorTestServices(new Size(200, 200)))
{
var border = new Border
{
@@ -59,7 +59,7 @@ public class CompositorHitTestingTests : CompositorTestsBase
[Fact]
public void HitTest_Should_Not_Find_Invisible_Controls_At_Point()
{
- using (var s = new CompositorServices(new Size(200, 200)))
+ using (var s = new CompositorTestServices(new Size(200, 200)))
{
Border visible, border;
s.TopLevel.Content = border = new Border
@@ -91,7 +91,7 @@ public class CompositorHitTestingTests : CompositorTestsBase
public void HitTest_Should_Find_Zero_Opacity_Controls_At_Point(bool parent, bool child)
{
- using (var s = new CompositorServices(new Size(200, 200)))
+ using (var s = new CompositorTestServices(new Size(200, 200)))
{
Border visible, border;
s.TopLevel.Content = border = new Border
@@ -118,7 +118,7 @@ public class CompositorHitTestingTests : CompositorTestsBase
[Fact]
public void HitTest_Should_Not_Find_Control_Outside_Point()
{
- using (var s = new CompositorServices(new Size(200, 200)))
+ using (var s = new CompositorTestServices(new Size(200, 200)))
{
var border = new Border
{
@@ -138,7 +138,7 @@ public class CompositorHitTestingTests : CompositorTestsBase
[Fact]
public void HitTest_Should_Return_Top_Controls_First()
{
- using (var s = new CompositorServices(new Size(200, 200)))
+ using (var s = new CompositorTestServices(new Size(200, 200)))
{
Panel container = new Panel
{
@@ -173,7 +173,7 @@ public class CompositorHitTestingTests : CompositorTestsBase
[Fact]
public void HitTest_Should_Return_Top_Controls_First_With_ZIndex()
{
- using (var s = new CompositorServices(new Size(200, 200)))
+ using (var s = new CompositorTestServices(new Size(200, 200)))
{
Panel container = new Panel
{
@@ -219,7 +219,7 @@ public class CompositorHitTestingTests : CompositorTestsBase
[Fact]
public void HitTest_Should_Find_Control_Translated_Outside_Parent_Bounds()
{
- using (var s = new CompositorServices(new Size(200, 200)))
+ using (var s = new CompositorTestServices(new Size(200, 200)))
{
Border target;
Panel container = new Panel
@@ -259,7 +259,7 @@ public class CompositorHitTestingTests : CompositorTestsBase
[Fact]
public void HitTest_Should_Not_Find_Control_Outside_Parent_Bounds_When_Clipped()
{
- using (var s = new CompositorServices(new Size(200, 200)))
+ using (var s = new CompositorTestServices(new Size(200, 200)))
{
Border target;
Panel container = new Panel
@@ -299,7 +299,7 @@ public class CompositorHitTestingTests : CompositorTestsBase
[Fact]
public void HitTest_Should_Not_Find_Control_Outside_Scroll_Viewport()
{
- using (var s = new CompositorServices(new Size(100, 200)))
+ using (var s = new CompositorTestServices(new Size(100, 200)))
{
Border target;
Border item1;
@@ -373,7 +373,7 @@ public class CompositorHitTestingTests : CompositorTestsBase
[Fact]
public void HitTest_Should_Not_Find_Path_When_Outside_Fill()
{
- using (var s = new CompositorServices(new Size(200, 200)))
+ using (var s = new CompositorTestServices(new Size(200, 200)))
{
Path path = new Path
{
@@ -392,7 +392,7 @@ public class CompositorHitTestingTests : CompositorTestsBase
[Fact]
public void HitTest_Should_Respect_Geometry_Clip()
{
- using (var s = new CompositorServices(new Size(400, 400)))
+ using (var s = new CompositorTestServices(new Size(400, 400)))
{
Canvas canvas;
Border border = new Border
@@ -422,7 +422,7 @@ public class CompositorHitTestingTests : CompositorTestsBase
[Fact]
public void HitTest_Should_Accommodate_ICustomHitTest()
{
- using (var s = new CompositorServices(new Size(300, 200)))
+ using (var s = new CompositorTestServices(new Size(300, 200)))
{
Border border = new CustomHitTestBorder
{
@@ -445,7 +445,7 @@ public class CompositorHitTestingTests : CompositorTestsBase
[Fact]
public void HitTest_Should_Not_Hit_Controls_Next_Pixel()
{
- using (var s = new CompositorServices(new Size(200, 200)))
+ using (var s = new CompositorTestServices(new Size(200, 200)))
{
Border targetRectangle;
@@ -470,7 +470,7 @@ public class CompositorHitTestingTests : CompositorTestsBase
[Fact]
public void HitTest_Filter_Should_Filter_Out_Children()
{
- using (var s = new CompositorServices(new Size(200, 200)))
+ using (var s = new CompositorTestServices(new Size(200, 200)))
{
Border child, parent;
s.TopLevel.Content = parent = new Border
diff --git a/tests/Avalonia.Base.UnitTests/Rendering/CompositorTestsBase.cs b/tests/Avalonia.Base.UnitTests/Rendering/CompositorTestsBase.cs
index 12db249f04..fed193e973 100644
--- a/tests/Avalonia.Base.UnitTests/Rendering/CompositorTestsBase.cs
+++ b/tests/Avalonia.Base.UnitTests/Rendering/CompositorTestsBase.cs
@@ -24,178 +24,7 @@ namespace Avalonia.Base.UnitTests.Rendering;
public class CompositorTestsBase
{
- public class DebugEvents : ICompositionTargetDebugEvents
- {
- public List Rects = new();
-
- public void RectInvalidated(Rect rc)
- {
- Rects.Add(rc);
- }
-
- public void Reset()
- {
- Rects.Clear();
- }
- }
-
- public class ManualRenderTimer : IRenderTimer
- {
- public event Action Tick;
- public bool RunsInBackground => false;
- public void TriggerTick() => Tick?.Invoke(TimeSpan.Zero);
- public Task TriggerBackgroundTick() => Task.Run(TriggerTick);
- }
-
- class TopLevelImpl : ITopLevelImpl
- {
- private readonly Compositor _compositor;
- public CompositingRenderer Renderer { get; private set; }
-
- public TopLevelImpl(Compositor compositor, Size clientSize)
- {
- ClientSize = clientSize;
- _compositor = compositor;
- }
-
- public void Dispose()
- {
-
- }
-
- public Size ClientSize { get; }
- public Size? FrameSize { get; }
- public double RenderScaling => 1;
- public IEnumerable