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

Unified Diff: sync/protocol/synced_notification_specifics.proto

Issue 11441005: Create a fresh sync datatype for Synced Notifications (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: few last CR comments, fix sync_invalidation_tests Created 8 years 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « sync/protocol/sync_proto.gyp ('k') | sync/syncable/model_type.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sync/protocol/synced_notification_specifics.proto
diff --git a/sync/protocol/preference_specifics.proto b/sync/protocol/synced_notification_specifics.proto
similarity index 67%
copy from sync/protocol/preference_specifics.proto
copy to sync/protocol/synced_notification_specifics.proto
index 4827c5dc20ceba8b127ce45bb52d0d7ee6f9c0d5..b3a0380100fe50a17dc37801d55991b541b047d3 100644
--- a/sync/protocol/preference_specifics.proto
+++ b/sync/protocol/synced_notification_specifics.proto
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
//
-// Sync protocol datatype extension for preferences.
+// Sync protocol datatype extension for push notifications..
akalin 2012/12/21 20:27:55 push -> synced
akalin 2012/12/21 20:28:12 also remove extra dot
// Update proto_value_conversions{.h,.cc,_unittest.cc} if you change
// any fields in this file.
@@ -14,9 +14,9 @@ option retain_unknown_fields = true;
package sync_pb;
-// Properties of preference sync objects.
-message PreferenceSpecifics {
+// Properties of SyncedNotificationSpecifics objects.
+message SyncedNotificationSpecifics {
+ // TODO(petewil): this is placeholder data, make a real one
optional string name = 1;
optional string value = 2;
}
-
« no previous file with comments | « sync/protocol/sync_proto.gyp ('k') | sync/syncable/model_type.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698