• Elly Fong-Jones's avatar
    cbui: simplify ConfirmBubbleViews a bit · c477438a
    Elly Fong-Jones authored
    This change:
    1) Removes ConfirmBubbleModel::GetButtons(), because every user had both
       buttons. Design-wise this makes sense - a ConfirmBubbleModel represents a
       yes/no choice, so having one or both of the buttons absent doesn't make much
       sense.
    2) Removes ConfirmBubbleModel::BubbleButton, which duplicates ui::DialogButton
    3) Removes ConfirmBubbleViews::IsDialogButtonEnabled() - because of point (1),
       this always returned true.
    
    Bug: 1011446
    Change-Id: I04bf65b7145ce00d70887feef1d0a6d218255697
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2028217
    Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org>
    Reviewed-by: default avatarAvi Drissman <avi@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#736878}
    c477438a
test_confirm_bubble_model.cc 1.64 KB