Bug 1415656 - Fix android serializing dateAdded as a string in sync bookmark records r=Grisha
authorThom Chiovoloni <tchiovoloni@mozilla.com>
Tue, 14 Nov 2017 15:14:08 -0500
changeset 392050 65f08f1b6fb2b3ed566d153f4d2aa1bad0c525a8
parent 391946 33f727c87866409ce1b1898b2acd1caeeca5c8e0
child 392051 5d62ecf6c1165a3c96861c3ab3c212bd7356f77d
push id32910
push userrgurzau@mozilla.com
push dateThu, 16 Nov 2017 10:02:59 +0000
treeherdermozilla-central@9941e68b5a53 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersGrisha
bugs1415656
milestone59.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
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.