Move wire_protocol.proto under protos/perfetto/ipc
This got missed in the last proto reshuffle.
Not a big deal, but makes the copybara rules easier to deal with.
Also keeps the codebaase more consistent, moving all .proto files
under protos/.
TBR: skyostil
Bug: 140126865
Change-Id: Id9680e2fa18dc4a8ac8f463bd01d7a1db9ca87aa
diff --git a/Android.bp b/Android.bp
index 36c42b6..a92e98e 100644
--- a/Android.bp
+++ b/Android.bp
@@ -73,6 +73,7 @@
":perfetto_protos_perfetto_config_sys_stats_zero_gen",
":perfetto_protos_perfetto_config_zero_gen",
":perfetto_protos_perfetto_ipc_ipc_gen",
+ ":perfetto_protos_perfetto_ipc_wire_protocol_gen",
":perfetto_protos_perfetto_trace_android_zero_gen",
":perfetto_protos_perfetto_trace_chrome_zero_gen",
":perfetto_protos_perfetto_trace_filesystem_zero_gen",
@@ -89,7 +90,6 @@
":perfetto_protos_perfetto_trace_sys_stats_zero_gen",
":perfetto_protos_perfetto_trace_track_event_zero_gen",
":perfetto_protos_perfetto_trace_trusted_lite_gen",
- ":perfetto_src_ipc_wire_protocol_gen",
"src/base/event_fd.cc",
"src/base/file_utils.cc",
"src/base/metatrace.cc",
@@ -197,6 +197,7 @@
"perfetto_protos_perfetto_config_sys_stats_zero_gen_headers",
"perfetto_protos_perfetto_config_zero_gen_headers",
"perfetto_protos_perfetto_ipc_ipc_gen_headers",
+ "perfetto_protos_perfetto_ipc_wire_protocol_gen_headers",
"perfetto_protos_perfetto_trace_android_zero_gen_headers",
"perfetto_protos_perfetto_trace_chrome_zero_gen_headers",
"perfetto_protos_perfetto_trace_filesystem_zero_gen_headers",
@@ -213,7 +214,6 @@
"perfetto_protos_perfetto_trace_sys_stats_zero_gen_headers",
"perfetto_protos_perfetto_trace_track_event_zero_gen_headers",
"perfetto_protos_perfetto_trace_trusted_lite_gen_headers",
- "perfetto_src_ipc_wire_protocol_gen_headers",
],
defaults: [
"perfetto_defaults",
@@ -301,6 +301,7 @@
":perfetto_protos_perfetto_config_sys_stats_zero_gen",
":perfetto_protos_perfetto_config_zero_gen",
":perfetto_protos_perfetto_ipc_ipc_gen",
+ ":perfetto_protos_perfetto_ipc_wire_protocol_gen",
":perfetto_protos_perfetto_trace_android_zero_gen",
":perfetto_protos_perfetto_trace_chrome_zero_gen",
":perfetto_protos_perfetto_trace_filesystem_zero_gen",
@@ -317,7 +318,6 @@
":perfetto_protos_perfetto_trace_sys_stats_zero_gen",
":perfetto_protos_perfetto_trace_track_event_zero_gen",
":perfetto_protos_perfetto_trace_trusted_lite_gen",
- ":perfetto_src_ipc_wire_protocol_gen",
"src/android_internal/lazy_library_loader.cc",
"src/base/event_fd.cc",
"src/base/file_utils.cc",
@@ -445,6 +445,7 @@
"perfetto_protos_perfetto_config_sys_stats_zero_gen_headers",
"perfetto_protos_perfetto_config_zero_gen_headers",
"perfetto_protos_perfetto_ipc_ipc_gen_headers",
+ "perfetto_protos_perfetto_ipc_wire_protocol_gen_headers",
"perfetto_protos_perfetto_trace_android_zero_gen_headers",
"perfetto_protos_perfetto_trace_chrome_zero_gen_headers",
"perfetto_protos_perfetto_trace_filesystem_zero_gen_headers",
@@ -461,7 +462,6 @@
"perfetto_protos_perfetto_trace_sys_stats_zero_gen_headers",
"perfetto_protos_perfetto_trace_track_event_zero_gen_headers",
"perfetto_protos_perfetto_trace_trusted_lite_gen_headers",
- "perfetto_src_ipc_wire_protocol_gen_headers",
],
defaults: [
"perfetto_defaults",
@@ -536,6 +536,7 @@
":perfetto_protos_perfetto_config_sys_stats_zero_gen",
":perfetto_protos_perfetto_config_zero_gen",
":perfetto_protos_perfetto_ipc_ipc_gen",
+ ":perfetto_protos_perfetto_ipc_wire_protocol_gen",
":perfetto_protos_perfetto_trace_android_zero_gen",
":perfetto_protos_perfetto_trace_chrome_zero_gen",
":perfetto_protos_perfetto_trace_filesystem_zero_gen",
@@ -552,7 +553,6 @@
":perfetto_protos_perfetto_trace_sys_stats_zero_gen",
":perfetto_protos_perfetto_trace_track_event_zero_gen",
":perfetto_protos_perfetto_trace_trusted_lite_gen",
- ":perfetto_src_ipc_wire_protocol_gen",
"src/base/event_fd.cc",
"src/base/file_utils.cc",
"src/base/metatrace.cc",
@@ -653,6 +653,7 @@
"perfetto_protos_perfetto_config_sys_stats_zero_gen_headers",
"perfetto_protos_perfetto_config_zero_gen_headers",
"perfetto_protos_perfetto_ipc_ipc_gen_headers",
+ "perfetto_protos_perfetto_ipc_wire_protocol_gen_headers",
"perfetto_protos_perfetto_trace_android_zero_gen_headers",
"perfetto_protos_perfetto_trace_chrome_zero_gen_headers",
"perfetto_protos_perfetto_trace_filesystem_zero_gen_headers",
@@ -669,7 +670,6 @@
"perfetto_protos_perfetto_trace_sys_stats_zero_gen_headers",
"perfetto_protos_perfetto_trace_track_event_zero_gen_headers",
"perfetto_protos_perfetto_trace_trusted_lite_gen_headers",
- "perfetto_src_ipc_wire_protocol_gen_headers",
],
export_generated_headers: [
"perfetto_protos_perfetto_common_lite_gen_headers",
@@ -693,6 +693,7 @@
"perfetto_protos_perfetto_config_sys_stats_zero_gen_headers",
"perfetto_protos_perfetto_config_zero_gen_headers",
"perfetto_protos_perfetto_ipc_ipc_gen_headers",
+ "perfetto_protos_perfetto_ipc_wire_protocol_gen_headers",
"perfetto_protos_perfetto_trace_android_zero_gen_headers",
"perfetto_protos_perfetto_trace_chrome_zero_gen_headers",
"perfetto_protos_perfetto_trace_filesystem_zero_gen_headers",
@@ -709,7 +710,6 @@
"perfetto_protos_perfetto_trace_sys_stats_zero_gen_headers",
"perfetto_protos_perfetto_trace_track_event_zero_gen_headers",
"perfetto_protos_perfetto_trace_trusted_lite_gen_headers",
- "perfetto_src_ipc_wire_protocol_gen_headers",
],
defaults: [
"perfetto_defaults",
@@ -745,6 +745,7 @@
":perfetto_protos_perfetto_config_sys_stats_zero_gen",
":perfetto_protos_perfetto_config_zero_gen",
":perfetto_protos_perfetto_ipc_ipc_gen",
+ ":perfetto_protos_perfetto_ipc_wire_protocol_gen",
":perfetto_protos_perfetto_trace_android_zero_gen",
":perfetto_protos_perfetto_trace_chrome_zero_gen",
":perfetto_protos_perfetto_trace_filesystem_zero_gen",
@@ -761,7 +762,6 @@
":perfetto_protos_perfetto_trace_sys_stats_zero_gen",
":perfetto_protos_perfetto_trace_track_event_zero_gen",
":perfetto_protos_perfetto_trace_trusted_lite_gen",
- ":perfetto_src_ipc_wire_protocol_gen",
":perfetto_src_perfetto_cmd_protos_gen",
"src/android_internal/lazy_library_loader.cc",
"src/base/event_fd.cc",
@@ -861,6 +861,7 @@
"perfetto_protos_perfetto_config_sys_stats_zero_gen_headers",
"perfetto_protos_perfetto_config_zero_gen_headers",
"perfetto_protos_perfetto_ipc_ipc_gen_headers",
+ "perfetto_protos_perfetto_ipc_wire_protocol_gen_headers",
"perfetto_protos_perfetto_trace_android_zero_gen_headers",
"perfetto_protos_perfetto_trace_chrome_zero_gen_headers",
"perfetto_protos_perfetto_trace_filesystem_zero_gen_headers",
@@ -877,7 +878,6 @@
"perfetto_protos_perfetto_trace_sys_stats_zero_gen_headers",
"perfetto_protos_perfetto_trace_track_event_zero_gen_headers",
"perfetto_protos_perfetto_trace_trusted_lite_gen_headers",
- "perfetto_src_ipc_wire_protocol_gen_headers",
"perfetto_src_perfetto_cmd_protos_gen_headers",
],
defaults: [
@@ -944,6 +944,7 @@
":perfetto_protos_perfetto_config_sys_stats_zero_gen",
":perfetto_protos_perfetto_config_zero_gen",
":perfetto_protos_perfetto_ipc_ipc_gen",
+ ":perfetto_protos_perfetto_ipc_wire_protocol_gen",
":perfetto_protos_perfetto_trace_android_lite_gen",
":perfetto_protos_perfetto_trace_android_zero_gen",
":perfetto_protos_perfetto_trace_chrome_lite_gen",
@@ -973,7 +974,6 @@
":perfetto_protos_perfetto_trace_track_event_lite_gen",
":perfetto_protos_perfetto_trace_track_event_zero_gen",
":perfetto_protos_perfetto_trace_trusted_lite_gen",
- ":perfetto_src_ipc_wire_protocol_gen",
"src/android_internal/lazy_library_loader.cc",
"src/base/event_fd.cc",
"src/base/file_utils.cc",
@@ -1135,6 +1135,7 @@
"perfetto_protos_perfetto_config_sys_stats_zero_gen_headers",
"perfetto_protos_perfetto_config_zero_gen_headers",
"perfetto_protos_perfetto_ipc_ipc_gen_headers",
+ "perfetto_protos_perfetto_ipc_wire_protocol_gen_headers",
"perfetto_protos_perfetto_trace_android_lite_gen_headers",
"perfetto_protos_perfetto_trace_android_zero_gen_headers",
"perfetto_protos_perfetto_trace_chrome_lite_gen_headers",
@@ -1164,7 +1165,6 @@
"perfetto_protos_perfetto_trace_track_event_lite_gen_headers",
"perfetto_protos_perfetto_trace_track_event_zero_gen_headers",
"perfetto_protos_perfetto_trace_trusted_lite_gen_headers",
- "perfetto_src_ipc_wire_protocol_gen_headers",
],
defaults: [
"perfetto_defaults",
@@ -2023,6 +2023,40 @@
],
}
+// GN target: //protos/perfetto/ipc:wire_protocol_gen
+genrule {
+ name: "perfetto_protos_perfetto_ipc_wire_protocol_gen",
+ srcs: [
+ "protos/perfetto/ipc/wire_protocol.proto",
+ ],
+ tools: [
+ "aprotoc",
+ ],
+ cmd: "mkdir -p $(genDir)/external/perfetto && $(location aprotoc) --cpp_out=$(genDir)/external/perfetto --proto_path=external/perfetto $(in)",
+ out: [
+ "external/perfetto/protos/perfetto/ipc/wire_protocol.pb.cc",
+ ],
+}
+
+// GN target: //protos/perfetto/ipc:wire_protocol_gen
+genrule {
+ name: "perfetto_protos_perfetto_ipc_wire_protocol_gen_headers",
+ srcs: [
+ "protos/perfetto/ipc/wire_protocol.proto",
+ ],
+ tools: [
+ "aprotoc",
+ ],
+ cmd: "mkdir -p $(genDir)/external/perfetto && $(location aprotoc) --cpp_out=$(genDir)/external/perfetto --proto_path=external/perfetto $(in)",
+ out: [
+ "external/perfetto/protos/perfetto/ipc/wire_protocol.pb.h",
+ ],
+ export_include_dirs: [
+ ".",
+ "protos",
+ ],
+}
+
// GN target: //protos/perfetto/metrics/android:zero_gen
genrule {
name: "perfetto_protos_perfetto_metrics_android_zero_gen",
@@ -3719,40 +3753,6 @@
],
}
-// GN target: //src/ipc:wire_protocol_gen
-genrule {
- name: "perfetto_src_ipc_wire_protocol_gen",
- srcs: [
- "src/ipc/wire_protocol.proto",
- ],
- tools: [
- "aprotoc",
- ],
- cmd: "mkdir -p $(genDir)/external/perfetto && $(location aprotoc) --cpp_out=$(genDir)/external/perfetto --proto_path=external/perfetto $(in)",
- out: [
- "external/perfetto/src/ipc/wire_protocol.pb.cc",
- ],
-}
-
-// GN target: //src/ipc:wire_protocol_gen
-genrule {
- name: "perfetto_src_ipc_wire_protocol_gen_headers",
- srcs: [
- "src/ipc/wire_protocol.proto",
- ],
- tools: [
- "aprotoc",
- ],
- cmd: "mkdir -p $(genDir)/external/perfetto && $(location aprotoc) --cpp_out=$(genDir)/external/perfetto --proto_path=external/perfetto $(in)",
- out: [
- "external/perfetto/src/ipc/wire_protocol.pb.h",
- ],
- export_include_dirs: [
- ".",
- "protos",
- ],
-}
-
// GN target: //src/perfetto_cmd:protos_gen
genrule {
name: "perfetto_src_perfetto_cmd_protos_gen",
@@ -4092,6 +4092,7 @@
":perfetto_protos_perfetto_config_sys_stats_zero_gen",
":perfetto_protos_perfetto_config_zero_gen",
":perfetto_protos_perfetto_ipc_ipc_gen",
+ ":perfetto_protos_perfetto_ipc_wire_protocol_gen",
":perfetto_protos_perfetto_metrics_android_zero_gen",
":perfetto_protos_perfetto_metrics_zero_gen",
":perfetto_protos_perfetto_trace_android_lite_gen",
@@ -4125,7 +4126,6 @@
":perfetto_protos_perfetto_trace_track_event_zero_gen",
":perfetto_protos_perfetto_trace_trusted_lite_gen",
":perfetto_src_ipc_test_messages_gen",
- ":perfetto_src_ipc_wire_protocol_gen",
":perfetto_src_perfetto_cmd_protos_gen",
":perfetto_src_protozero_testing_messages_lite_gen",
":perfetto_src_protozero_testing_messages_zero_gen",
@@ -4471,6 +4471,7 @@
"perfetto_protos_perfetto_config_sys_stats_zero_gen_headers",
"perfetto_protos_perfetto_config_zero_gen_headers",
"perfetto_protos_perfetto_ipc_ipc_gen_headers",
+ "perfetto_protos_perfetto_ipc_wire_protocol_gen_headers",
"perfetto_protos_perfetto_metrics_android_zero_gen_headers",
"perfetto_protos_perfetto_metrics_zero_gen_headers",
"perfetto_protos_perfetto_trace_android_lite_gen_headers",
@@ -4504,7 +4505,6 @@
"perfetto_protos_perfetto_trace_track_event_zero_gen_headers",
"perfetto_protos_perfetto_trace_trusted_lite_gen_headers",
"perfetto_src_ipc_test_messages_gen_headers",
- "perfetto_src_ipc_wire_protocol_gen_headers",
"perfetto_src_perfetto_cmd_protos_gen_headers",
"perfetto_src_protozero_testing_messages_lite_gen_headers",
"perfetto_src_protozero_testing_messages_zero_gen_headers",
@@ -5012,6 +5012,7 @@
":perfetto_protos_perfetto_config_sys_stats_zero_gen",
":perfetto_protos_perfetto_config_zero_gen",
":perfetto_protos_perfetto_ipc_ipc_gen",
+ ":perfetto_protos_perfetto_ipc_wire_protocol_gen",
":perfetto_protos_perfetto_trace_android_zero_gen",
":perfetto_protos_perfetto_trace_chrome_zero_gen",
":perfetto_protos_perfetto_trace_filesystem_zero_gen",
@@ -5028,7 +5029,6 @@
":perfetto_protos_perfetto_trace_sys_stats_zero_gen",
":perfetto_protos_perfetto_trace_track_event_zero_gen",
":perfetto_protos_perfetto_trace_trusted_lite_gen",
- ":perfetto_src_ipc_wire_protocol_gen",
":perfetto_src_perfetto_cmd_protos_gen",
"src/base/event_fd.cc",
"src/base/file_utils.cc",
@@ -5121,6 +5121,7 @@
"perfetto_protos_perfetto_config_sys_stats_zero_gen_headers",
"perfetto_protos_perfetto_config_zero_gen_headers",
"perfetto_protos_perfetto_ipc_ipc_gen_headers",
+ "perfetto_protos_perfetto_ipc_wire_protocol_gen_headers",
"perfetto_protos_perfetto_trace_android_zero_gen_headers",
"perfetto_protos_perfetto_trace_chrome_zero_gen_headers",
"perfetto_protos_perfetto_trace_filesystem_zero_gen_headers",
@@ -5137,7 +5138,6 @@
"perfetto_protos_perfetto_trace_sys_stats_zero_gen_headers",
"perfetto_protos_perfetto_trace_track_event_zero_gen_headers",
"perfetto_protos_perfetto_trace_trusted_lite_gen_headers",
- "perfetto_src_ipc_wire_protocol_gen_headers",
"perfetto_src_perfetto_cmd_protos_gen_headers",
],
defaults: [
diff --git a/docs/embedder-guide.md b/docs/embedder-guide.md
index 226e086..fbe3be3 100644
--- a/docs/embedder-guide.md
+++ b/docs/embedder-guide.md
@@ -20,7 +20,7 @@
- [service.h](/include/perfetto/tracing/core/tracing_service.h)
- Explain the ABI surface:
- [shared_memory_abi.h](/include/perfetto/tracing/core/shared_memory_abi.h)
- - IPC's [wire protocol](/src/ipc/wire_protocol.proto) (if used)
+ - IPC's [wire protocol](/protos/perfetto/ipc/wire_protocol.proto) (if used)
- The input [config protos](/protos/perfetto/config)
- The output [trace protos](/protos/perfetto/trace)
diff --git a/docs/ipc.md b/docs/ipc.md
index 33d3a5c..ac7c36a 100644
--- a/docs/ipc.md
+++ b/docs/ipc.md
@@ -18,7 +18,7 @@
- Allows to send file descriptors over the wire: for setting up shared memory
and passing the FD for the output trace from a consumer to the service.
- Service definition uses same protobuf rpc syntax of [gRPC](https://grpc.io)
-- Extremely simple [wire protocol](/src/ipc/wire_protocol.proto).
+- Extremely simple [wire protocol](/protos/perfetto/ipc/wire_protocol.proto).
- C++11 friendly, allows to bind `std::function` to each request.
- Leak (un)friendly: tries hard to guarantee that callbacks are left unresolved,
using C++11 move semantics.
@@ -31,7 +31,7 @@
- Debugging friendly: single-thread only, based on non-blocking socket I/O.
- Binary size friendly: generates one protobuf per message, doesn't have any
external dependency.
-- Hopefully safe:
+- Safe:
- The rx buffer has guard regions around.
- The wire protocol is based on protobuf.
- [Fuzzed](/src/ipc/buffered_frame_deserializer_fuzzer.cc)
diff --git a/protos/perfetto/ipc/BUILD.gn b/protos/perfetto/ipc/BUILD.gn
index a3e0bfd..48f54a1 100644
--- a/protos/perfetto/ipc/BUILD.gn
+++ b/protos/perfetto/ipc/BUILD.gn
@@ -14,6 +14,7 @@
import("../../../gn/ipc_library.gni")
import("../../../gn/perfetto.gni")
+import("../../../gn/proto_library.gni")
# IPC service definitions.
ipc_library("ipc") {
@@ -28,3 +29,10 @@
"producer_port.proto",
]
}
+
+perfetto_proto_library("wire_protocol") {
+ proto_generators = [ "lite" ]
+ sources = [
+ "wire_protocol.proto",
+ ]
+}
diff --git a/src/ipc/wire_protocol.proto b/protos/perfetto/ipc/wire_protocol.proto
similarity index 100%
rename from src/ipc/wire_protocol.proto
rename to protos/perfetto/ipc/wire_protocol.proto
diff --git a/src/ipc/BUILD.gn b/src/ipc/BUILD.gn
index e476e1c..12353de 100644
--- a/src/ipc/BUILD.gn
+++ b/src/ipc/BUILD.gn
@@ -29,8 +29,8 @@
"../base:unix_socket",
]
deps = [
- ":wire_protocol",
"../../gn:default_deps",
+ "../../protos/perfetto/ipc:wire_protocol",
"../base",
]
sources = [
@@ -52,8 +52,8 @@
]
deps = [
":ipc",
- ":wire_protocol",
"../../gn:default_deps",
+ "../../protos/perfetto/ipc:wire_protocol",
]
}
@@ -62,9 +62,9 @@
deps = [
":ipc",
":test_messages",
- ":wire_protocol",
"../../gn:default_deps",
"../../gn:gtest_and_gmock",
+ "../../protos/perfetto/ipc:wire_protocol",
"../base",
"../base:test_support",
]
@@ -77,14 +77,6 @@
]
}
-perfetto_proto_library("wire_protocol") {
- proto_generators = [ "lite" ]
- sources = [
- "wire_protocol.proto",
- ]
- proto_path = perfetto_root_path
-}
-
ipc_library("test_messages") {
sources = [
"test/client_unittest_messages.proto",
diff --git a/src/ipc/buffered_frame_deserializer.cc b/src/ipc/buffered_frame_deserializer.cc
index 290db65..9974654 100644
--- a/src/ipc/buffered_frame_deserializer.cc
+++ b/src/ipc/buffered_frame_deserializer.cc
@@ -26,7 +26,7 @@
#include "perfetto/base/logging.h"
#include "perfetto/ext/base/utils.h"
-#include "src/ipc/wire_protocol.pb.h"
+#include "protos/perfetto/ipc/wire_protocol.pb.h"
namespace perfetto {
namespace ipc {
diff --git a/src/ipc/buffered_frame_deserializer_fuzzer.cc b/src/ipc/buffered_frame_deserializer_fuzzer.cc
index c3b22bb..61a818d 100644
--- a/src/ipc/buffered_frame_deserializer_fuzzer.cc
+++ b/src/ipc/buffered_frame_deserializer_fuzzer.cc
@@ -19,7 +19,8 @@
#include "perfetto/ext/base/utils.h"
#include "src/ipc/buffered_frame_deserializer.h"
-#include "src/ipc/wire_protocol.pb.h"
+
+#include "protos/perfetto/ipc/wire_protocol.pb.h"
extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size);
diff --git a/src/ipc/buffered_frame_deserializer_unittest.cc b/src/ipc/buffered_frame_deserializer_unittest.cc
index 727ee7e..792365b 100644
--- a/src/ipc/buffered_frame_deserializer_unittest.cc
+++ b/src/ipc/buffered_frame_deserializer_unittest.cc
@@ -23,7 +23,7 @@
#include "perfetto/ext/base/utils.h"
#include "test/gtest_and_gmock.h"
-#include "src/ipc/wire_protocol.pb.h"
+#include "protos/perfetto/ipc/wire_protocol.pb.h"
namespace perfetto {
namespace ipc {
diff --git a/src/ipc/client_impl.h b/src/ipc/client_impl.h
index db65f02..71c211f 100644
--- a/src/ipc/client_impl.h
+++ b/src/ipc/client_impl.h
@@ -23,7 +23,7 @@
#include "perfetto/ext/ipc/client.h"
#include "src/ipc/buffered_frame_deserializer.h"
-#include "src/ipc/wire_protocol.pb.h"
+#include "protos/perfetto/ipc/wire_protocol.pb.h"
#include <list>
#include <map>
diff --git a/src/ipc/host_impl.cc b/src/ipc/host_impl.cc
index 75b1e2d..662e6e6 100644
--- a/src/ipc/host_impl.cc
+++ b/src/ipc/host_impl.cc
@@ -26,7 +26,7 @@
#include "perfetto/ext/ipc/service.h"
#include "perfetto/ext/ipc/service_descriptor.h"
-#include "src/ipc/wire_protocol.pb.h"
+#include "protos/perfetto/ipc/wire_protocol.pb.h"
// TODO(primiano): put limits on #connections/uid and req. queue (b/69093705).
diff --git a/src/ipc/host_impl_unittest.cc b/src/ipc/host_impl_unittest.cc
index 025d4c1..70215ff 100644
--- a/src/ipc/host_impl_unittest.cc
+++ b/src/ipc/host_impl_unittest.cc
@@ -30,8 +30,8 @@
#include "src/ipc/test/test_socket.h"
#include "test/gtest_and_gmock.h"
+#include "protos/perfetto/ipc/wire_protocol.pb.h"
#include "src/ipc/test/client_unittest_messages.pb.h"
-#include "src/ipc/wire_protocol.pb.h"
namespace perfetto {
namespace ipc {
