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

Side by Side Diff: components/sync/protocol/extension_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
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 extensions. 5 // Sync protocol datatype extension for extensions.
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 extension sync objects. 16 // Properties of extension sync objects.
18 // 17 //
19 // Merge policy: the settings for the higher version number win; in 18 // Merge policy: the settings for the higher version number win; in
20 // the case of a tie, server wins. 19 // the case of a tie, server wins.
21 message ExtensionSpecifics { 20 message ExtensionSpecifics {
22 // Globally unique id for this extension. 21 // Globally unique id for this extension.
23 optional string id = 1; 22 optional string id = 1;
(...skipping 23 matching lines...) Expand all
47 // This is a tri-state boolean, so, unlike most fields here, it really *is* 46 // This is a tri-state boolean, so, unlike most fields here, it really *is*
48 // optional and may be absent. We need this for the time being because we need 47 // optional and may be absent. We need this for the time being because we need
49 // to know if a user has not set an explicit preference. 48 // to know if a user has not set an explicit preference.
50 optional bool all_urls_enabled = 9; 49 optional bool all_urls_enabled = 9;
51 50
52 // Bitmask of the set of reasons why the extension is disabled (see 51 // Bitmask of the set of reasons why the extension is disabled (see
53 // Extension::DisableReason). Only relevant when enabled == false. Note that 52 // Extension::DisableReason). Only relevant when enabled == false. Note that
54 // old clients (<M45) won't set this, even when enabled is false. 53 // old clients (<M45) won't set this, even when enabled is false.
55 optional int32 disable_reasons = 10; 54 optional int32 disable_reasons = 10;
56 } 55 }
OLDNEW
« no previous file with comments | « components/sync/protocol/extension_setting_specifics.proto ('k') | components/sync/protocol/favicon_image_specifics.proto » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698