chromium/src
0
Fork 0
Code Activity

Add explicit java_package to java proto source files

Browse Source 
All proto source files listed in a java_proto_library gn target soon
must explicitly specify a java package. Fix proto files in ToT before we
enable enforcement.

Bug: 1114836
Change-Id: I9453b9fbe94b3a65517708cdaef564daef4494c5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2427364
Reviewed-by: Mikel Astiz <mastiz@chromium.org>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: Lambros Lambrou <lambroslambrou@chromium.org>
Commit-Queue: Mohamed Heikal <mheikal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#810628}
This commit is contained in:
Mohamed Heikal
2020-09-25 15:26:41 +00:00
committed by Commit Bot
parent 19c19d8e78
commit dcc407ef40
9 changed files with 10 additions and 1 deletions

1
build/config/android/test/proto/absolute_dep/absolute_dep.proto

@ -5,5 +5,6 @@
syntax = "proto2";
package build.config.android.test;
option java_package = "build.config.android.test";
message AbsoluteDep {}

1
build/config/android/test/proto/relative_dep/relative_dep.proto

@ -5,5 +5,6 @@
syntax = "proto2";
package build.config.android.test;
option java_package = "build.config.android.test";
message RelativeDep {}

1
build/config/android/test/proto/root/absolute_child.proto

@ -5,5 +5,6 @@
syntax = "proto2";
package build.config.android.test;
option java_package = "build.config.android.test";
message AbsoluteChild {}

1
build/config/android/test/proto/root/absolute_root.proto

@ -5,6 +5,7 @@
syntax = "proto2";
package build.config.android.test;
option java_package = "build.config.android.test";
import "build/config/android/test/proto/root/absolute_child.proto";
import "build/config/android/test/proto/absolute_dep/absolute_dep.proto";

1
build/config/android/test/proto/root/relative_child.proto

@ -5,5 +5,6 @@
syntax = "proto2";
package build.config.android.test;
option java_package = "build.config.android.test";
message RelativeChild {}

1
build/config/android/test/proto/root/relative_root.proto

@ -5,6 +5,7 @@
syntax = "proto2";
package build.config.android.test;
option java_package = "build.config.android.test";
import "relative_child.proto";
import "build/config/android/test/proto/absolute_dep/absolute_dep.proto";

3
components/sync/protocol/local_trusted_vault.proto

@ -6,6 +6,7 @@ syntax = "proto2";
option optimize_for = LITE_RUNTIME;
option java_package = "sync_pb";
package sync_pb;
message LocalTrustedVaultKey {
@ -43,4 +44,4 @@ message LocalTrustedVaultPerUser {
message LocalTrustedVault {
repeated LocalTrustedVaultPerUser user = 1;
}
}

1
remoting/proto/remoting/v1/directory_messages.proto

@ -7,6 +7,7 @@ syntax = "proto2";
option java_multiple_files = true;
option optimize_for = LITE_RUNTIME;
option java_package = "remoting.apis.v1";
package remoting.apis.v1;
import "host_info.proto";

1
remoting/proto/remoting/v1/host_info.proto

@ -7,6 +7,7 @@ syntax = "proto2";
option java_multiple_files = true;
option optimize_for = LITE_RUNTIME;
option java_package = "remoting.apis.v1";
package remoting.apis.v1;
// Represents a Chrome Remote Desktop host instance. It includes static data