Bug 1153358 - Client mitigation: don't upload stored_on. r=nalexander, a=sledru
authorRichard Newman <rnewman@mozilla.com>
Mon, 13 Apr 2015 12:46:25 -0700
changeset 258476 1412c445ff0d
parent 258475 4cc36a9a958b
child 258477 91df81e2edac
push id4676
push userryanvm@gmail.com
push date2015-04-15 02:06 +0000
treeherdermozilla-beta@91df81e2edac [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersnalexander, sledru
bugs1153358
milestone38.0
Bug 1153358 - Client mitigation: don't upload stored_on. r=nalexander, a=sledru
mobile/android/base/reading/ReadingListClientRecordFactory.java
--- a/mobile/android/base/reading/ReadingListClientRecordFactory.java
+++ b/mobile/android/base/reading/ReadingListClientRecordFactory.java
@@ -193,16 +193,20 @@ public class ReadingListClientRecordFact
     final ClientMetadata cm = new ClientMetadata(localID, clientLastModified, isDeleted, isArchived);
 
     // Remove things that aren't part of the spec.
     object.remove("last_modified");
     object.remove("guid");
     object.remove("client_last_modified");
     object.remove("is_deleted");
 
+    // We never want to upload stored_on; for new items it'll be null (and cause Bug 1153358),
+    // and for existing items it should never change.
+    object.remove("stored_on");
+
     object.remove(ReadingListItems.CONTENT_STATUS);
     object.remove(ReadingListItems.SYNC_STATUS);
     object.remove(ReadingListItems.SYNC_CHANGE_FLAGS);
     object.remove(ReadingListItems.CLIENT_LAST_MODIFIED);
 
     return new ClientReadingListRecord(sm, cm, object);
   }