DescriptionRevert of Upgrade Blink to milliseconds-based last modified filetimes, part 3. (patchset #4 id:60001 of https://codereview.chromium.org/884393002/)
Reason for revert:
Two linux32 failures coming through,
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#master=ChromiumWebkit&tests=http/tests/local/fileapi/file-last-modified.html,storage/indexeddb/structured-clone.html
Original issue's description:
> Upgrade Blink to milliseconds-based last modified filetimes, part 3.
>
> With WebFileInfo now recording modification time in milliseconds via
> modificationTimeMS, switch FileMetadata over to using it and keep a
> milliseconds-based modification time field also. Also have the fileapi's
> File object's snapshot times be milliseconds-based.
>
> To make it clearer what units the modification time values are over,
> append a MS suffix. This is temporary until the conversion is complete.
>
> This patch is the third one in the following series,
>
> 1: [blink] add WebFileInfo::modificationTimeMS [ https://codereview.chromium.org/873723004/ ]
> 2: [chromium] fill in modificationTimeMS [ https://codereview.chromium.org/884413002/ ]
> 3: [blink] *this one* [ https://codereview.chromium.org/884393002/ ]
> 4: [chromium] set modificationTime to something msec-based [ https://codereview.chromium.org/862203003/ ]
> 5: [blink] switch to using modificationTime instead of *MS [ https://codereview.chromium.org/882343002/ ]
> 6: [chromium] stop setting modificationTimeMS [ https://codereview.chromium.org/890523002/ ]
> 7: [blink] remove modificationTimeMS [ https://codereview.chromium.org/869613005/ ]
>
> R=mkwst,jsbell
> BUG=451747
>
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=189475
TBR=mkwst@chromium.org,jsbell@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=451747
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=189494
Patch Set 1 #
Created: 5 years, 10 months ago
(Patch set is too large to download)
Messages
Total messages: 4 (0 generated)
|