• David Black's avatar
    Refactor AnimatedContainerView for response processing v2. · 50dc3f49
    David Black authored
    This CL modifies AnimatedContainerView to:
    - Split HandleResponse() into HandleSuggestion() and HandleUiElement().
    - In this CL, these new functions are invoked at the same time as
      HandleResponse(), but a follow up CL will add additional invocations
      in response processing v2 in reaction to AssistantResponseObserver
      events.
    - Modifies existing paradigm of derivative classes explicitly having to
      call AddElementAnimator() to instead have HandleSuggestion() and
      HandleUiElement() return any new animators that should be owned and
      acted upon. This will be convenient in response processing v2 when
      animations are most often *not* performed in unison.
    
    Other changes in this CL are trivial and should be straightforward to
    follow :)
    
    Note that this CL has *no effect* on existing behavior.
    
    
    Bug: b:129411551
    Change-Id: Ibbcd7654c5a2b6dfe7ca488dc559738531085b09
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2067473
    Commit-Queue: David Black <dmblack@google.com>
    Reviewed-by: default avatarXiaohui Chen <xiaohuic@chromium.org>
    Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
    Reviewed-by: default avatarJeroen Dhollander <jeroendh@google.com>
    Cr-Commit-Position: refs/heads/master@{#745742}
    50dc3f49
assistant_response_container_view.cc 2.56 KB