Combine TimeView with BaseDateTimeView |
||
Issue descriptionAs DateView is removed, TimeView is the only subclass of BaseDateTimeView. https://cs.chromium.org/chromium/src/ash/system/date/date_view.h
,
Jan 18
(5 days ago)
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b20f0034bae4ae8eaabd659ff17da0fbce00b893 commit b20f0034bae4ae8eaabd659ff17da0fbce00b893 Author: Tetsui Ohkubo <tetsui@chromium.org> Date: Fri Jan 18 03:30:11 2019 Unified: Merge TimeView with BaseDateTimeView. This CL merges TimewView with its parent class, BaseDateTimeView. Previously, there was another subclass DateView, but now TimeView is the sole subclass of BaseDateTimewView. TEST=trybot pass BUG= 901712 Change-Id: I16296bd7ba973e8f17cdbdf0bf3fac78663a0b3f Reviewed-on: https://chromium-review.googlesource.com/c/1415059 Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org> Reviewed-by: James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#623990} [modify] https://crrev.com/b20f0034bae4ae8eaabd659ff17da0fbce00b893/ash/system/date/date_view.cc [modify] https://crrev.com/b20f0034bae4ae8eaabd659ff17da0fbce00b893/ash/system/date/date_view.h
,
Jan 18
(5 days ago)
|
||
►
Sign in to add a comment |
||
Comment 1 by tetsui@chromium.org
, Jan 17 (6 days ago)