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

Side by Side Diff: chrome/browser/sync/protocol/app_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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 apps. 5 // Sync protocol datatype extension for apps.
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 import "extension_specifics.proto"; 17 import "extension_specifics.proto";
19 18
20 // Settings related to push notifications for apps. 19 // Settings related to push notifications for apps.
21 message AppNotificationSettings { 20 message AppNotificationSettings {
22 // DEPRECATED: Use oauth_client_id below. 21 // DEPRECATED: Use oauth_client_id below.
23 // Whether or not the user has setup notifications at least once. 22 // Whether or not the user has setup notifications at least once.
24 // The value for this field will start out false and will be set 23 // The value for this field will start out false and will be set
25 // to true when the user accepts receiving notifications for the 24 // to true when the user accepts receiving notifications for the
26 // first time and then it will always remain true. 25 // first time and then it will always remain true.
27 optional bool initial_setup_done = 1; 26 optional bool initial_setup_done = 1;
(...skipping 24 matching lines...) Expand all
52 51
53 // This controls where on a page this application icon will appear. 52 // This controls where on a page this application icon will appear.
54 optional string app_launch_ordinal = 3; 53 optional string app_launch_ordinal = 3;
55 54
56 // This specifics which page the application icon will appear on in the NTP. 55 // This specifics which page the application icon will appear on in the NTP.
57 // This values only provide the order within the application pages, not within 56 // This values only provide the order within the application pages, not within
58 // all of the panels in the NTP. 57 // all of the panels in the NTP.
59 optional string page_ordinal = 4; 58 optional string page_ordinal = 4;
60 } 59 }
61 60
62 extend EntitySpecifics {
63 optional AppSpecifics app = 48364;
64 }
OLDNEW
« no previous file with comments | « chrome/browser/sync/protocol/app_setting_specifics.proto ('k') | chrome/browser/sync/protocol/autofill_specifics.proto » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698