Closed Bug 1272431 Opened 8 years ago Closed 8 years ago

Make TelemetryPingStore.getAllPings return pings in order – earliest to latest

Categories

(Firefox for Android Graveyard :: General, defect)

All
Android
defect
Not set
normal

Tracking

(firefox49 fixed)

RESOLVED FIXED
Firefox 49
Tracking Status
firefox49 --- fixed

People

(Reporter: mcomella, Assigned: mcomella)

References

Details

Attachments

(3 files)

via bug 1270213 comment 24. This lets us try to offload our data before it gets deleted if we run out of space to store pings.
Comment on attachment 8753144 [details]
MozReview Request: Bug 1272431 - Correct order errors in maybePrunePings tests. r=ahunt

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/53036/diff/1-2/
Attachment #8753144 - Attachment description: MozReview Request: Bug 1272431 - Correct order errors in maybePrunePings tests. r=sebastian → MozReview Request: Bug 1272431 - Correct order errors in maybePrunePings tests. r=ahunt
Attachment #8753145 - Attachment description: MozReview Request: Bug 1272431 - Return pings from getAllPings in guaranteed order. r=sebastian → MozReview Request: Bug 1272431 - Return pings from getAllPings in guaranteed order. r=ahunt
Attachment #8753146 - Attachment description: MozReview Request: Bug 1272431 - Do not use API 19+ method for FileLastModifiedComparator. r=sebastian → MozReview Request: Bug 1272431 - Do not use API 19+ method for FileLastModifiedComparator. r=ahunt
Attachment #8753144 - Flags: review?(s.kaspari) → review?(ahunt)
Attachment #8753145 - Flags: review?(s.kaspari) → review?(ahunt)
Attachment #8753146 - Flags: review?(s.kaspari) → review?(ahunt)
Comment on attachment 8753145 [details]
MozReview Request: Bug 1272431 - Return pings from getAllPings in guaranteed order. r=ahunt

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/53038/diff/1-2/
Comment on attachment 8753146 [details]
MozReview Request: Bug 1272431 - Do not use API 19+ method for FileLastModifiedComparator. r=ahunt

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/53040/diff/1-2/
Comment on attachment 8753144 [details]
MozReview Request: Bug 1272431 - Correct order errors in maybePrunePings tests. r=ahunt

https://reviewboard.mozilla.org/r/53036/#review50382
Attachment #8753144 - Flags: review?(ahunt) → review+
Comment on attachment 8753145 [details]
MozReview Request: Bug 1272431 - Return pings from getAllPings in guaranteed order. r=ahunt

https://reviewboard.mozilla.org/r/53038/#review50386
Attachment #8753145 - Flags: review?(ahunt) → review+
Attachment #8753146 - Flags: review?(ahunt) → review+
Comment on attachment 8753146 [details]
MozReview Request: Bug 1272431 - Do not use API 19+ method for FileLastModifiedComparator. r=ahunt

https://reviewboard.mozilla.org/r/53040/#review50406
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: