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

Side by Side Diff: components/sync/protocol/theme_specifics.proto

Issue 2291053002: Cleanup: remove non-standard retain_unknown_fields from .proto files (Closed)
Patch Set: Created 4 years, 3 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 | « components/sync/protocol/test.proto ('k') | components/sync/protocol/typed_url_specifics.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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 // Sync protocol datatype extension for themes. 5 // Sync protocol datatype extension for themes.
6 6
7 // Update proto_value_conversions{.h,.cc,_unittest.cc} if you change 7 // Update proto_value_conversions{.h,.cc,_unittest.cc} if you change
8 // any fields in this file. 8 // any fields in this file.
9 9
10 syntax = "proto2"; 10 syntax = "proto2";
11 11
12 option optimize_for = LITE_RUNTIME; 12 option optimize_for = LITE_RUNTIME;
13 option retain_unknown_fields = true;
14 13
15 package sync_pb; 14 package sync_pb;
16 15
17 // Properties of theme sync objects. 16 // Properties of theme sync objects.
18 message ThemeSpecifics { 17 message ThemeSpecifics {
19 // If set, we're using a custom theme and all custom_* fields should be 18 // If set, we're using a custom theme and all custom_* fields should be
20 // present. If not set, we use the default or system theme (see below) 19 // present. If not set, we use the default or system theme (see below)
21 // and all custom_* fields should be omitted. 20 // and all custom_* fields should be omitted.
22 optional bool use_custom_theme = 1; 21 optional bool use_custom_theme = 1;
23 // This field is only set (i.e., not cleared) on platforms that have 22 // This field is only set (i.e., not cleared) on platforms that have
24 // a distinction between the system theme and the default theme, but 23 // a distinction between the system theme and the default theme, but
25 // other platforms must be careful to pass through the set state (not 24 // other platforms must be careful to pass through the set state (not
26 // just the value) of this flag. 25 // just the value) of this flag.
27 // 26 //
28 // If true, we use the system theme by default (i.e., when we don't use 27 // If true, we use the system theme by default (i.e., when we don't use
29 // a custom theme) for platforms that make a distinction between the 28 // a custom theme) for platforms that make a distinction between the
30 // default theme and the system theme. Has no effect if use_custom_theme 29 // default theme and the system theme. Has no effect if use_custom_theme
31 // is set. 30 // is set.
32 optional bool use_system_theme_by_default = 2; 31 optional bool use_system_theme_by_default = 2;
33 32
34 // Custom-theme-specific fields. 33 // Custom-theme-specific fields.
35 optional string custom_theme_name = 3; 34 optional string custom_theme_name = 3;
36 optional string custom_theme_id = 4; 35 optional string custom_theme_id = 4;
37 optional string custom_theme_update_url = 5; 36 optional string custom_theme_update_url = 5;
38 } 37 }
OLDNEW
« no previous file with comments | « components/sync/protocol/test.proto ('k') | components/sync/protocol/typed_url_specifics.proto » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698