diff --git a/Tests/LibGfx/TestGfxBitmap.cpp b/Tests/LibGfx/TestGfxBitmap.cpp index 271b7e7f8a..bd214b9ebf 100644 --- a/Tests/LibGfx/TestGfxBitmap.cpp +++ b/Tests/LibGfx/TestGfxBitmap.cpp @@ -126,3 +126,12 @@ TEST_CASE(0008_bitmap_downscaling_height1) } } } + +TEST_CASE(0009_serialize_and_deserialize_roundtrip) +{ + auto original_bitmap = MUST(Gfx::Bitmap::create(Gfx::BitmapFormat::BGRx8888, Gfx::IntSize { 10, 10 })); + original_bitmap->fill(Gfx::Color::Red); + auto bytes = MUST(original_bitmap->serialize_to_byte_buffer()); + auto bitmap = MUST(Gfx::Bitmap::create_from_serialized_bytes(bytes)); + EXPECT(bitmap->visually_equals(original_bitmap)); +} diff --git a/Userland/Libraries/LibGfx/Bitmap.cpp b/Userland/Libraries/LibGfx/Bitmap.cpp index d76dead824..9d3499ea15 100644 --- a/Userland/Libraries/LibGfx/Bitmap.cpp +++ b/Userland/Libraries/LibGfx/Bitmap.cpp @@ -237,7 +237,7 @@ ErrorOr> Bitmap::create_from_serialized_bytes(ReadonlyByte ErrorOr Bitmap::serialize_to_byte_buffer() const { - auto buffer = TRY(ByteBuffer::create_uninitialized(sizeof(size_t) + 4 * sizeof(unsigned) + sizeof(BitmapFormat) + size_in_bytes())); + auto buffer = TRY(ByteBuffer::create_uninitialized(sizeof(size_t) + 3 * sizeof(unsigned) + sizeof(BitmapFormat) + size_in_bytes())); FixedMemoryStream stream { buffer.span() }; TRY(stream.write_value(size_in_bytes()));