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

Side by Side Diff: chrome/browser/sync/protocol/theme_specifics.proto

Issue 9460047: sync: remove use of protobuf extensions in protocol to reduce static init overhead. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fred's review Created 8 years, 9 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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; 13 option retain_unknown_fields = true;
14 14
15 package sync_pb; 15 package sync_pb;
16 16
17 import "sync.proto";
18
19 // Properties of theme sync objects. 17 // Properties of theme sync objects.
20 message ThemeSpecifics { 18 message ThemeSpecifics {
21 // If set, we're using a custom theme and all custom_* fields should be 19 // If set, we're using a custom theme and all custom_* fields should be
22 // present. If not set, we use the default or system theme (see below) 20 // present. If not set, we use the default or system theme (see below)
23 // and all custom_* fields should be omitted. 21 // and all custom_* fields should be omitted.
24 optional bool use_custom_theme = 1; 22 optional bool use_custom_theme = 1;
25 // This field is only set (i.e., not cleared) on platforms that have 23 // This field is only set (i.e., not cleared) on platforms that have
26 // a distinction between the system theme and the default theme, but 24 // a distinction between the system theme and the default theme, but
27 // other platforms must be careful to pass through the set state (not 25 // other platforms must be careful to pass through the set state (not
28 // just the value) of this flag. 26 // just the value) of this flag.
29 // 27 //
30 // If true, we use the system theme by default (i.e., when we don't use 28 // If true, we use the system theme by default (i.e., when we don't use
31 // a custom theme) for platforms that make a distinction between the 29 // a custom theme) for platforms that make a distinction between the
32 // default theme and the system theme. Has no effect if use_custom_theme 30 // default theme and the system theme. Has no effect if use_custom_theme
33 // is set. 31 // is set.
34 optional bool use_system_theme_by_default = 2; 32 optional bool use_system_theme_by_default = 2;
35 33
36 // Custom-theme-specific fields. 34 // Custom-theme-specific fields.
37 optional string custom_theme_name = 3; 35 optional string custom_theme_name = 3;
38 optional string custom_theme_id = 4; 36 optional string custom_theme_id = 4;
39 optional string custom_theme_update_url = 5; 37 optional string custom_theme_update_url = 5;
40 } 38 }
41 39
42 extend EntitySpecifics {
43 optional ThemeSpecifics theme = 41210;
44 }
OLDNEW
« no previous file with comments | « chrome/browser/sync/protocol/sync.proto ('k') | chrome/browser/sync/protocol/typed_url_specifics.proto » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698