Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Commit 95feed1

Browse files
authored
[Impeller] Delete the archivist framework. (#50474)
It is unused and can be brought back from Git if needed. This unnecessarily adds to binary size estimates on standalone Impeller investigations.
1 parent 6de2ac1 commit 95feed1

25 files changed

+0
-1923
lines changed

ci/licenses_golden/excluded_files

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,6 @@
132132
../../../flutter/impeller/aiks/canvas_unittests.cc
133133
../../../flutter/impeller/aiks/testing
134134
../../../flutter/impeller/aiks/trace_serializer_unittests.cc
135-
../../../flutter/impeller/archivist/archivist_unittests.cc
136135
../../../flutter/impeller/base/README.md
137136
../../../flutter/impeller/base/base_unittests.cc
138137
../../../flutter/impeller/compiler/README.md

ci/licenses_golden/licenses_flutter

Lines changed: 0 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -4961,24 +4961,6 @@ ORIGIN: ../../../flutter/impeller/aiks/picture_recorder.cc + ../../../flutter/LI
49614961
ORIGIN: ../../../flutter/impeller/aiks/picture_recorder.h + ../../../flutter/LICENSE
49624962
ORIGIN: ../../../flutter/impeller/aiks/trace_serializer.cc + ../../../flutter/LICENSE
49634963
ORIGIN: ../../../flutter/impeller/aiks/trace_serializer.h + ../../../flutter/LICENSE
4964-
ORIGIN: ../../../flutter/impeller/archivist/archivable.cc + ../../../flutter/LICENSE
4965-
ORIGIN: ../../../flutter/impeller/archivist/archivable.h + ../../../flutter/LICENSE
4966-
ORIGIN: ../../../flutter/impeller/archivist/archive.cc + ../../../flutter/LICENSE
4967-
ORIGIN: ../../../flutter/impeller/archivist/archive.h + ../../../flutter/LICENSE
4968-
ORIGIN: ../../../flutter/impeller/archivist/archive_class_registration.cc + ../../../flutter/LICENSE
4969-
ORIGIN: ../../../flutter/impeller/archivist/archive_class_registration.h + ../../../flutter/LICENSE
4970-
ORIGIN: ../../../flutter/impeller/archivist/archive_database.cc + ../../../flutter/LICENSE
4971-
ORIGIN: ../../../flutter/impeller/archivist/archive_database.h + ../../../flutter/LICENSE
4972-
ORIGIN: ../../../flutter/impeller/archivist/archive_location.cc + ../../../flutter/LICENSE
4973-
ORIGIN: ../../../flutter/impeller/archivist/archive_location.h + ../../../flutter/LICENSE
4974-
ORIGIN: ../../../flutter/impeller/archivist/archive_statement.cc + ../../../flutter/LICENSE
4975-
ORIGIN: ../../../flutter/impeller/archivist/archive_statement.h + ../../../flutter/LICENSE
4976-
ORIGIN: ../../../flutter/impeller/archivist/archive_transaction.cc + ../../../flutter/LICENSE
4977-
ORIGIN: ../../../flutter/impeller/archivist/archive_transaction.h + ../../../flutter/LICENSE
4978-
ORIGIN: ../../../flutter/impeller/archivist/archive_vector.cc + ../../../flutter/LICENSE
4979-
ORIGIN: ../../../flutter/impeller/archivist/archive_vector.h + ../../../flutter/LICENSE
4980-
ORIGIN: ../../../flutter/impeller/archivist/archivist_fixture.cc + ../../../flutter/LICENSE
4981-
ORIGIN: ../../../flutter/impeller/archivist/archivist_fixture.h + ../../../flutter/LICENSE
49824964
ORIGIN: ../../../flutter/impeller/base/allocation.cc + ../../../flutter/LICENSE
49834965
ORIGIN: ../../../flutter/impeller/base/allocation.h + ../../../flutter/LICENSE
49844966
ORIGIN: ../../../flutter/impeller/base/backend_cast.h + ../../../flutter/LICENSE
@@ -7818,24 +7800,6 @@ FILE: ../../../flutter/impeller/aiks/picture_recorder.cc
78187800
FILE: ../../../flutter/impeller/aiks/picture_recorder.h
78197801
FILE: ../../../flutter/impeller/aiks/trace_serializer.cc
78207802
FILE: ../../../flutter/impeller/aiks/trace_serializer.h
7821-
FILE: ../../../flutter/impeller/archivist/archivable.cc
7822-
FILE: ../../../flutter/impeller/archivist/archivable.h
7823-
FILE: ../../../flutter/impeller/archivist/archive.cc
7824-
FILE: ../../../flutter/impeller/archivist/archive.h
7825-
FILE: ../../../flutter/impeller/archivist/archive_class_registration.cc
7826-
FILE: ../../../flutter/impeller/archivist/archive_class_registration.h
7827-
FILE: ../../../flutter/impeller/archivist/archive_database.cc
7828-
FILE: ../../../flutter/impeller/archivist/archive_database.h
7829-
FILE: ../../../flutter/impeller/archivist/archive_location.cc
7830-
FILE: ../../../flutter/impeller/archivist/archive_location.h
7831-
FILE: ../../../flutter/impeller/archivist/archive_statement.cc
7832-
FILE: ../../../flutter/impeller/archivist/archive_statement.h
7833-
FILE: ../../../flutter/impeller/archivist/archive_transaction.cc
7834-
FILE: ../../../flutter/impeller/archivist/archive_transaction.h
7835-
FILE: ../../../flutter/impeller/archivist/archive_vector.cc
7836-
FILE: ../../../flutter/impeller/archivist/archive_vector.h
7837-
FILE: ../../../flutter/impeller/archivist/archivist_fixture.cc
7838-
FILE: ../../../flutter/impeller/archivist/archivist_fixture.h
78397803
FILE: ../../../flutter/impeller/base/allocation.cc
78407804
FILE: ../../../flutter/impeller/base/allocation.h
78417805
FILE: ../../../flutter/impeller/base/backend_cast.h

impeller/BUILD.gn

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,6 @@ config("impeller_public_config") {
5656

5757
group("impeller") {
5858
public_deps = [
59-
"archivist",
6059
"base",
6160
"geometry",
6261
"tessellator",
@@ -81,7 +80,6 @@ impeller_component("impeller_unittests") {
8180
testonly = true
8281

8382
deps = [
84-
"archivist:archivist_unittests",
8583
"base:base_unittests",
8684
"compiler:compiler_unittests",
8785
"core:allocator_unittests",

impeller/README.md

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -60,12 +60,6 @@ states of completion:
6060
for a specific client rendering API. The interfaces in these targets are
6161
meant to be private for non-WSI user targets. No Impeller sub-frameworks
6262
may depend on these targets.
63-
* **`//impeller/archivist`**: Allows persisting objects to disk as performantly
64-
as possible (usually on a background thread). The framework is meant to be
65-
used for storing frame meta-data and related profiling/instrumentation
66-
information. Collection of information should succeed despite process crashes
67-
and retrieval of traces must not use inordinate amounts of time or memory
68-
(which usually leads to crashes).
6963
* **`//impeller/geometry`**: All (or, most of) the math! This C++ mathematics
7064
library is used extensively by Impeller and its clients. The reasonably
7165
interesting bit about this library is that all types can be used

impeller/archivist/BUILD.gn

Lines changed: 0 additions & 51 deletions
This file was deleted.

impeller/archivist/archivable.cc

Lines changed: 0 additions & 11 deletions
This file was deleted.

impeller/archivist/archivable.h

Lines changed: 0 additions & 41 deletions
This file was deleted.

impeller/archivist/archive.cc

Lines changed: 0 additions & 172 deletions
This file was deleted.

0 commit comments

Comments
 (0)