New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 721273 link

Starred by 1 user
Status: WontFix
Owner:
olivierrobin@chromium.org
Closed: Aug 2017
Cc:
gambard@chromium.org
█ jif@chromium.org
ios-bugs@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 3
Type: Task



Sign in to add a comment

Convert all time in Reading List to base::Time

Project Member Reported by gambard@chromium.org, May 11 2017 
In ReadingListEntry, the timestamps are stored as int_64.
They are also exposed as int_64 in the public API. They should be exposed as base::Time.

Comment 1 by olivierrobin@chromium.org, Jul 18 2017 

This has been a design choice documented as in the header of ReadingListEntry.
We only use time for sorting and exposing base::Time would require conversion on every operation.
If this is really a problem, we an change that, or reneme the field to "orderKey".

Comment 2 by olivierrobin@chromium.org, Aug 11 2017

Status: WontFix (was: Assigned)
►

Sign in to add a comment