[BackgroundSync] Initial land of the BackgroundSyncManager

content/browser/background_sync/background_sync.proto

+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+syntax = "proto2";
+
+option optimize_for = LITE_RUNTIME;
+
+package content;
+
+message BackgroundSyncRegistrationProto {
+  required int64 id = 1;
+  required string name = 2;
+  optional int64 min_period = 3;

[davidben, 2015/03/25 16:01:09]

Nit: Since you're storing it as 0 in-memory and having all these has_min_period
checks, maybe just make it required and call 0 the minimum period?

[jkarlin, 2015/03/25 19:24:17]

Acknowledged.

+}
+
+message BackgroundSyncRegistrationsProto {
+  repeated BackgroundSyncRegistrationProto registration = 1;
+  required int64 registration_id_counter = 2;

[davidben, 2015/03/25 16:01:09]

Nit: next_registration_id might be clearer. (I feel I've seen more next_foo_id
type names. Also makes it clear whether that was the last ID or the next one.)

[jkarlin, 2015/03/25 19:24:17]

Done.

+}