Commit c983a015 authored by vitaliii's avatar vitaliii Committed by Commit Bot

[Sync::Cleanup] Remove stray semicolons after message definitions.

They are not needed and get autoremoved on the server side, which
impedes the reviews.

Change-Id: Ie5a56e2d57946eaf09718b601dc6ddc8e75036f1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015025
Commit-Queue: vitaliii <vitaliii@chromium.org>
Commit-Queue: Marc Treib <treib@chromium.org>
Auto-Submit: vitaliii <vitaliii@chromium.org>
Reviewed-by: default avatarMarc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#734424}
parent c1dab961
...@@ -19,7 +19,7 @@ package sync_pb; ...@@ -19,7 +19,7 @@ package sync_pb;
message CustomNudgeDelay { message CustomNudgeDelay {
optional int32 datatype_id = 1; // Datatype id. optional int32 datatype_id = 1; // Datatype id.
optional int32 delay_ms = 2; // Delay in milliseconds. optional int32 delay_ms = 2; // Delay in milliseconds.
}; }
message ClientCommand { message ClientCommand {
// Time to wait before sending any requests to the server. // Time to wait before sending any requests to the server.
...@@ -47,4 +47,4 @@ message ClientCommand { ...@@ -47,4 +47,4 @@ message ClientCommand {
// |sessions_commit_delay_seconds| // |sessions_commit_delay_seconds|
// New in M39. // New in M39.
repeated CustomNudgeDelay custom_nudge_delays = 8; repeated CustomNudgeDelay custom_nudge_delays = 8;
}; }
...@@ -31,4 +31,4 @@ package sync_pb; ...@@ -31,4 +31,4 @@ package sync_pb;
message EncryptedData { message EncryptedData {
optional string key_name = 1; optional string key_name = 1;
optional string blob = 2; optional string blob = 2;
}; }
...@@ -60,4 +60,4 @@ message GetUpdatesCallerInfo { ...@@ -60,4 +60,4 @@ message GetUpdatesCallerInfo {
// TODO(crbug.com/510165): Move this bool out of GetUpdatesCallerInfo so that // TODO(crbug.com/510165): Move this bool out of GetUpdatesCallerInfo so that
// GetUpdatesCallerInfo can eventually be removed. // GetUpdatesCallerInfo can eventually be removed.
optional bool notifications_enabled = 2; optional bool notifications_enabled = 2;
}; }
...@@ -119,7 +119,7 @@ message PasswordSpecificsData { ...@@ -119,7 +119,7 @@ message PasswordSpecificsData {
// Contains the password specifics metadata which simplifies its lookup. // Contains the password specifics metadata which simplifies its lookup.
message PasswordSpecificsMetadata { message PasswordSpecificsMetadata {
optional string url = 1; optional string url = 1;
}; }
// Properties of password sync objects. // Properties of password sync objects.
message PasswordSpecifics { message PasswordSpecifics {
......
...@@ -402,7 +402,7 @@ message SyncEntity { ...@@ -402,7 +402,7 @@ message SyncEntity {
// and the code has been removed as of M66. // and the code has been removed as of M66.
reserved 26; reserved 26;
reserved "attachment_id"; reserved "attachment_id";
}; }
// This message contains diagnostic information used to correlate // This message contains diagnostic information used to correlate
// commit-related traffic with extensions-related mutations to the // commit-related traffic with extensions-related mutations to the
...@@ -416,7 +416,7 @@ message ChromiumExtensionsActivity { ...@@ -416,7 +416,7 @@ message ChromiumExtensionsActivity {
// How many times the extension successfully invoked a write // How many times the extension successfully invoked a write
// operation through the bookmarks API since the last CommitMessage. // operation through the bookmarks API since the last CommitMessage.
optional uint32 bookmark_writes_since_last_commit = 2; optional uint32 bookmark_writes_since_last_commit = 2;
}; }
// Client specific configuration information. // Client specific configuration information.
message ClientConfigParams { message ClientConfigParams {
...@@ -432,7 +432,7 @@ message ClientConfigParams { ...@@ -432,7 +432,7 @@ message ClientConfigParams {
// chrome account. If multiple accounts are present in the cookie jar, a // chrome account. If multiple accounts are present in the cookie jar, a
// mismatch implies all of them are different from the chrome account. // mismatch implies all of them are different from the chrome account.
optional bool cookie_jar_mismatch = 3; optional bool cookie_jar_mismatch = 3;
}; }
message CommitMessage { message CommitMessage {
repeated SyncEntity entries = 1; repeated SyncEntity entries = 1;
...@@ -459,7 +459,7 @@ message CommitMessage { ...@@ -459,7 +459,7 @@ message CommitMessage {
// compression, this field should not be set. The server can ignore the // compression, this field should not be set. The server can ignore the
// padding. // padding.
optional string padding = 6; optional string padding = 6;
}; }
// This message communicates additional per-type information related to // This message communicates additional per-type information related to
// requests with origin GU_TRIGGER. This message is not relevant when any // requests with origin GU_TRIGGER. This message is not relevant when any
...@@ -687,19 +687,19 @@ message GetUpdatesMessage { ...@@ -687,19 +687,19 @@ message GetUpdatesMessage {
reserved 4; reserved 4;
reserved "requested_types"; reserved "requested_types";
}; }
// Message from a client asking the server to clear its data. // Message from a client asking the server to clear its data.
message ClearServerDataMessage { message ClearServerDataMessage {
// No arguments needed as the store birthday and user identifier are part of // No arguments needed as the store birthday and user identifier are part of
// an enclosing message. // an enclosing message.
}; }
// Response to a ClearServerData request. // Response to a ClearServerData request.
message ClearServerDataResponse { message ClearServerDataResponse {
// No result fields necessary. Success/failure is indicated in // No result fields necessary. Success/failure is indicated in
// ClientToServerResponse. // ClientToServerResponse.
}; }
// The client must preserve, store, and resend the chip bag with // The client must preserve, store, and resend the chip bag with
// every request. The server depends on the chip bag in order // every request. The server depends on the chip bag in order
...@@ -803,7 +803,7 @@ message ClientToServerMessage { ...@@ -803,7 +803,7 @@ message ClientToServerMessage {
// Identifies this ClientToServerMessage as a clear server data request. This // Identifies this ClientToServerMessage as a clear server data request. This
// field is present when message_contents is CLEAR_SERVER_DATA. // field is present when message_contents is CLEAR_SERVER_DATA.
optional ClearServerDataMessage clear_server_data = 15; optional ClearServerDataMessage clear_server_data = 15;
}; }
// This request allows the client to convert a specific crash identifier // This request allows the client to convert a specific crash identifier
// into more general information (e.g. hash of the crashing call stack) // into more general information (e.g. hash of the crashing call stack)
...@@ -873,7 +873,7 @@ message CommitResponse { ...@@ -873,7 +873,7 @@ message CommitResponse {
SharingMessageCommitError sharing_message_error = 11; SharingMessageCommitError sharing_message_error = 11;
} }
} }
}; }
message GetUpdatesResponse { message GetUpdatesResponse {
// New sync entries that the client should apply. // New sync entries that the client should apply.
...@@ -917,7 +917,7 @@ message GetUpdatesResponse { ...@@ -917,7 +917,7 @@ message GetUpdatesResponse {
// Set of optional datatype contexts server mutations. // Set of optional datatype contexts server mutations.
repeated DataTypeContext context_mutations = 7; repeated DataTypeContext context_mutations = 7;
}; }
message ClientToServerResponse { message ClientToServerResponse {
optional CommitResponse commit = 1; optional CommitResponse commit = 1;
...@@ -975,15 +975,15 @@ message ClientToServerResponse { ...@@ -975,15 +975,15 @@ message ClientToServerResponse {
// Present if this ClientToServerResponse is in response to a ClearServerData // Present if this ClientToServerResponse is in response to a ClearServerData
// request. // request.
optional ClearServerDataResponse clear_server_data = 15; optional ClearServerDataResponse clear_server_data = 15;
}; }
// A message to notify the server of certain sync events. Idempotent. Send these // A message to notify the server of certain sync events. Idempotent. Send these
// to the /event endpoint. // to the /event endpoint.
message EventRequest { message EventRequest {
optional SyncDisabledEvent sync_disabled = 1; optional SyncDisabledEvent sync_disabled = 1;
}; }
message EventResponse {}; message EventResponse {}
// A message indicating that the sync engine has been disabled on a client. // A message indicating that the sync engine has been disabled on a client.
message SyncDisabledEvent { message SyncDisabledEvent {
...@@ -992,4 +992,4 @@ message SyncDisabledEvent { ...@@ -992,4 +992,4 @@ message SyncDisabledEvent {
// The store birthday that the client was using before disabling sync. // The store birthday that the client was using before disabling sync.
optional string store_birthday = 2; optional string store_birthday = 2;
}; }
...@@ -42,7 +42,7 @@ message ThemeSpecifics { ...@@ -42,7 +42,7 @@ message ThemeSpecifics {
message AutogeneratedTheme { message AutogeneratedTheme {
// SkColor used to autogenerate theme. // SkColor used to autogenerate theme.
optional uint32 color = 1; optional uint32 color = 1;
}; }
// If set, autogenerated theme is used. // If set, autogenerated theme is used.
optional AutogeneratedTheme autogenerated_theme = 6; optional AutogeneratedTheme autogenerated_theme = 6;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment