Bug 1415656 - Fix android serializing dateAdded as a string in sync bookmark records r?Grisha draft
authorThom Chiovoloni <tchiovoloni@mozilla.com>
Tue, 14 Nov 2017 15:14:08 -0500
changeset 697843 414197b17f0a8501e519dce550d63b97fca9c4a3
parent 697757 b0a6b4678b2f7dfb499328946b95366775f71edd
child 740228 cdd79f37b8c95144fe7f28b55e46c4a85da873ee
push id89118
push userbmo:tchiovoloni@mozilla.com
push dateTue, 14 Nov 2017 20:14:30 +0000
reviewersGrisha
bugs1415656
milestone59.0a1
Bug 1415656 - Fix android serializing dateAdded as a string in sync bookmark records r?Grisha MozReview-Commit-ID: 1u9DeVvYwjY
mobile/android/services/src/main/java/org/mozilla/gecko/sync/repositories/domain/BookmarkRecord.java
--- a/mobile/android/services/src/main/java/org/mozilla/gecko/sync/repositories/domain/BookmarkRecord.java
+++ b/mobile/android/services/src/main/java/org/mozilla/gecko/sync/repositories/domain/BookmarkRecord.java
@@ -280,17 +280,17 @@ public class BookmarkRecord extends Reco
 
     // This field is marked as nullable, so we're careful when unboxing it. However, we do not expect
     // it to be null at this point, since it was constructed from our DB cursor which will always have
     // a `created` timestamp.
     // However, very old bookmarks that didn't have a corresponding history item might not have a
     // creation date. `created` is not a NOT NULL field in the bookmarks table. Either way, it pays
     // to be cautious here.
     if (this.dateAdded != null) {
-      putPayload(payload, "dateAdded", Long.toString(this.dateAdded));
+      payload.put("dateAdded", this.dateAdded);
     }
 
     if (isFolder()) {
       payload.put("children", this.children);
       return;
     }
 
     // bookmark, microsummary, query.