Commit 2cd83bce authored by Ahmed Mehfooz's avatar Ahmed Mehfooz Committed by Commit Bot

Correctly show notification timestamp

Adding workaround for keeping
the notification timestamp visible. Invalidating
the layout of the parent view of the timestamp any time
it's visibility is changed.

Bug: 991150
Change-Id: If8c750bfc8f34310a8e93ff2ea30e180e19a9e68
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739976Reviewed-by: default avatarTim Song <tengs@chromium.org>
Reviewed-by: default avatarRichard Knoll <knollr@chromium.org>
Commit-Queue: Ahmed Mehfooz <amehfooz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#685335}
parent 1fb12a84
...@@ -404,6 +404,9 @@ void NotificationHeaderView::UpdateSummaryTextVisibility() { ...@@ -404,6 +404,9 @@ void NotificationHeaderView::UpdateSummaryTextVisibility() {
const bool timestamp_visible = !has_progress_ && timestamp_; const bool timestamp_visible = !has_progress_ && timestamp_;
timestamp_divider_->SetVisible(timestamp_visible); timestamp_divider_->SetVisible(timestamp_visible);
timestamp_view_->SetVisible(timestamp_visible); timestamp_view_->SetVisible(timestamp_visible);
// TODO(crbug.com/991492): this should not be necessary.
detail_views_->InvalidateLayout();
} }
} // namespace message_center } // namespace message_center
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