Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(227)

Side by Side Diff: blimp/common/proto/protocol_control.proto

Issue 1933053003: Used oneof in blimp_message.proto (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Nits Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « blimp/common/proto/blimp_message.proto ('k') | blimp/common/proto/tab_control.proto » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 syntax = "proto2"; 5 syntax = "proto2";
6 6
7 option optimize_for = LITE_RUNTIME; 7 option optimize_for = LITE_RUNTIME;
8 8
9 package blimp; 9 package blimp;
10 10
11 // Indicates the message-Id that the peer has most-recently processed. 11 // Indicates the message-Id that the peer has most-recently processed.
12 message CheckpointAckMessage { 12 message CheckpointAckMessage {
13 optional int64 checkpoint_id = 1; 13 optional int64 checkpoint_id = 1;
14 } 14 }
15 15
16 // Contains Client->Engine information to establish an authenticated connection. 16 // Contains Client->Engine information to establish an authenticated connection.
17 message StartConnectionMessage { 17 message StartConnectionMessage {
18 optional string client_token = 1; 18 optional string client_token = 1;
19 optional int32 protocol_version = 2; 19 optional int32 protocol_version = 2;
20 } 20 }
21 21
22 message ProtocolControlMessage { 22 message ProtocolControlMessage {
23 enum Type { 23 oneof connection_message {
24 UNKNOWN = 0; 24 // Client => Server types.
25 StartConnectionMessage start_connection = 41;
25 26
26 // Client => Server types. 27 // Server => Client types.
27 START_CONNECTION = 1; 28 CheckpointAckMessage checkpoint_ack = 42;
28
29 // Server => Client types will start from 100.
30
31 // Bi-directional types.
32 CHECKPOINT_ACK = 200;
33 } 29 }
34
35 optional Type type = 1;
36
37 optional StartConnectionMessage start_connection = 1001;
38 optional CheckpointAckMessage checkpoint_ack = 1200;
39 } 30 }
OLDNEW
« no previous file with comments | « blimp/common/proto/blimp_message.proto ('k') | blimp/common/proto/tab_control.proto » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698