diff options
| author | wez@chromium.org <wez@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-11 02:27:09 +0000 |
|---|---|---|
| committer | wez@chromium.org <wez@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-11 02:27:09 +0000 |
| commit | 6387b4de56f0ac62bae8ab2c356e617db9cb9c55 (patch) | |
| tree | db0c2f91281a6bb150b32c3bee1eb436c0047585 /remoting/proto/video.proto | |
| parent | 4a404e5356b27578de71b9b41de686768d2ecd42 (diff) | |
| download | chromium_src-6387b4de56f0ac62bae8ab2c356e617db9cb9c55.zip chromium_src-6387b4de56f0ac62bae8ab2c356e617db9cb9c55.tar.gz chromium_src-6387b4de56f0ac62bae8ab2c356e617db9cb9c55.tar.bz2 | |
This CL removes the unused trace.proto definition and build target, and makes all protobuf fields "optional".
Fields marked required cause protobuf messages to be silently ignored if those fields are not present. By moving the fields to be optional we have more flexibility to remove fields in future.
All of the previously-required fields were in client-to-host messages; clients will need to continue to provide those fields at least until all hosts are updated not to require them - this is primarily relevant for the KeyEvent "keycode" field.
Review URL: http://codereview.chromium.org/10024003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@131698 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'remoting/proto/video.proto')
| -rw-r--r-- | remoting/proto/video.proto | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/remoting/proto/video.proto b/remoting/proto/video.proto index 049351e..14415af 100644 --- a/remoting/proto/video.proto +++ b/remoting/proto/video.proto @@ -1,4 +1,4 @@ -// Copyright (c) 2011 The Chromium Authors. All rights reserved. +// Copyright (c) 2012 The Chromium Authors. All rights reserved. // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. @@ -10,7 +10,6 @@ option optimize_for = LITE_RUNTIME; package remoting; -// TODO(ajwong): Determine if these fields should be optional or required. message VideoPacketFormat { // Identifies how the image was encoded. enum Encoding { |