Commit 1ef0a40e authored by Jimmy Gong's avatar Jimmy Gong Committed by Commit Bot

Phonehub: fix notification timestamp

Was previously calculating notification timestamp from a Unix epoch
time source. This patch fixes this by properly converting the timestamp
to Windows FILETIME epoch.

Bug: 1106937
Fixed: 1141587
Change-Id: I6d085a2b6449f8a8424a8c3e203939659d8bb32c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2493429
Commit-Queue: Jimmy Gong <jimmyxgong@chromium.org>
Reviewed-by: default avatarKyle Horimoto <khorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#820521}
parent 88cdf3e9
...@@ -141,8 +141,7 @@ base::Optional<Notification> ProcessNotificationProto( ...@@ -141,8 +141,7 @@ base::Optional<Notification> ProcessNotificationProto(
base::UTF8ToUTF16(proto.origin_app().visible_name()), base::UTF8ToUTF16(proto.origin_app().visible_name()),
proto.origin_app().package_name(), proto.origin_app().package_name(),
CreateImageFromSerializedIcon(proto.origin_app().icon())), CreateImageFromSerializedIcon(proto.origin_app().icon())),
base::Time::FromDeltaSinceWindowsEpoch( base::Time::FromJsTime(proto.epoch_time_millis()),
base::TimeDelta::FromMilliseconds(proto.epoch_time_millis())),
GetNotificationImportanceFromProto(proto.importance()), actions_it->id(), GetNotificationImportanceFromProto(proto.importance()), actions_it->id(),
title, text_content, shared_image, contact_image); title, text_content, shared_image, contact_image);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment