Connect BackgroundFetch to the OfflineItemCollection

content/browser/background_fetch/background_fetch_registration_id.h

Index: content/browser/background_fetch/background_fetch_registration_id.h
diff --git a/content/browser/background_fetch/background_fetch_registration_id.h b/content/browser/background_fetch/background_fetch_registration_id.h
index 32fb5da8145ae94420d4c4c0c74085ddeeb09f5c..bf7990ee42144339874e886560a11a72cd52cd26 100644
--- a/content/browser/background_fetch/background_fetch_registration_id.h
+++ b/content/browser/background_fetch/background_fetch_registration_id.h
@@ -39,6 +39,9 @@ class CONTENT_EXPORT BackgroundFetchRegistrationId {
   // Returns whether this registration id refers to valid data.
   bool is_null() const;
 
+  void FromString(const std::string& str);
+  std::string ToString() const;

[Peter Beverloo, 2017/03/29 14:32:12]

Could we call them Serialize and Deserialize(), where the latter is static?
Especially since FromString() doesn't actually return anything, the caller would
not know if decoding fails w/o checking is_null().

(Since this data will be stored in SharedPreferences there's always a chance of
corruption or deliberate mocking with the values.)

[Peter Beverloo, 2017/03/29 14:32:13]

Also ++docs

[harkness, 2017/03/30 12:42:36]

Done.

[harkness, 2017/03/30 12:42:36]

Done.

+
   int64_t service_worker_registration_id() const {
     return service_worker_registration_id_;
   }