Add and support notification types for Assistant

Desktop / Chromium - David Black [google.com] - 23 February 2019 02:34 EST

Previously all Assistant notifications were Message Center notifications. Now, we'll support notifications of type:

- kInAssistant- kPreferInAssistant- kSystem

The |kInAssistant| and |kPreferInAssistant| types are currently only used to support timer notification UX requirements.

Still TODO: Entry/exit animations for in-Assistant notifications.

Bug: b:118654460 Change-Id: I815789d89008ff55318d7b6e8c9492b07849f2ca Reviewed-on: https://chromium-review.googlesource.com/c/1480549

f45008617892 Add and support notification types for Assistant.
ash/assistant/assistant_alarm_timer_controller.cc | 9 +++
ash/assistant/assistant_notification_controller.cc | 89 +++++++++++++++++++++-
ash/assistant/assistant_notification_controller.h | 17 +++++
.../model/assistant_notification_model.cc | 11 +++
ash/assistant/model/assistant_notification_model.h | 7 ++
ash/assistant/ui/assistant_notification_overlay.cc | 26 ++++++-
ash/assistant/ui/assistant_notification_overlay.h | 9 +++
ash/assistant/ui/assistant_notification_view.cc | 12 +++
.../assistant/public/mojom/assistant.mojom | 29 +++++++
9 files changed, 206 insertions(+), 3 deletions(-)

Upstream: git.chromium.org


  • Share