1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586 |
- From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
- From: deepak1556 <[email protected]>
- Date: Thu, 21 May 2020 13:58:01 -0700
- Subject: Allow setting secondary label via SimpleMenuModel
- Builds on https://chromium-review.googlesource.com/c/chromium/src/+/2208976
- diff --git a/ui/base/models/simple_menu_model.cc b/ui/base/models/simple_menu_model.cc
- index ffafdacb3e0171f81d22fa1736012b92e9df2ba9..dad79e82221c15b887cfde9a35ff7447989da62a 100644
- --- a/ui/base/models/simple_menu_model.cc
- +++ b/ui/base/models/simple_menu_model.cc
- @@ -53,6 +53,11 @@ std::u16string SimpleMenuModel::Delegate::GetLabelForCommandId(
- return std::u16string();
- }
-
- +std::u16string SimpleMenuModel::Delegate::GetSecondaryLabelForCommandId(
- + int command_id) const {
- + return std::u16string();
- +}
- +
- ImageModel SimpleMenuModel::Delegate::GetIconForCommandId(
- int command_id) const {
- return ImageModel();
- @@ -338,6 +343,11 @@ void SimpleMenuModel::SetLabel(size_t index, const std::u16string& label) {
- MenuItemsChanged();
- }
-
- +void SimpleMenuModel::SetSecondaryLabel(size_t index, const std::u16string& secondary_label) {
- + items_[ValidateItemIndex(index)].secondary_label = secondary_label;
- + MenuItemsChanged();
- +}
- +
- void SimpleMenuModel::SetMinorText(size_t index,
- const std::u16string& minor_text) {
- items_[ValidateItemIndex(index)].minor_text = minor_text;
- @@ -428,6 +438,12 @@ std::u16string SimpleMenuModel::GetLabelAt(size_t index) const {
- return items_[ValidateItemIndex(index)].label;
- }
-
- +std::u16string SimpleMenuModel::GetSecondaryLabelAt(size_t index) const {
- + if (IsItemDynamicAt(index))
- + return delegate_->GetSecondaryLabelForCommandId(GetCommandIdAt(index));
- + return items_[ValidateItemIndex(index)].secondary_label;
- +}
- +
- std::u16string SimpleMenuModel::GetMinorTextAt(size_t index) const {
- return items_[ValidateItemIndex(index)].minor_text;
- }
- diff --git a/ui/base/models/simple_menu_model.h b/ui/base/models/simple_menu_model.h
- index 1539464b1942db2a5e493022109a1c8dc247dc5a..b3979cbccf8eb2f0145e050e11ecbb6303c38d46 100644
- --- a/ui/base/models/simple_menu_model.h
- +++ b/ui/base/models/simple_menu_model.h
- @@ -55,6 +55,7 @@ class COMPONENT_EXPORT(UI_BASE) SimpleMenuModel : public MenuModel {
- // Some command ids have labels and icons that change over time.
- virtual bool IsItemForCommandIdDynamic(int command_id) const;
- virtual std::u16string GetLabelForCommandId(int command_id) const;
- + virtual std::u16string GetSecondaryLabelForCommandId(int command_id) const;
- // Gets the icon for the item with the specified id.
- virtual ImageModel GetIconForCommandId(int command_id) const;
-
- @@ -174,6 +175,9 @@ class COMPONENT_EXPORT(UI_BASE) SimpleMenuModel : public MenuModel {
- // Sets the label for the item at |index|.
- void SetLabel(size_t index, const std::u16string& label);
-
- + // Sets the secondary_label for the item at |index|.
- + void SetSecondaryLabel(size_t index, const std::u16string& secondary_label);
- +
- // Sets the minor text for the item at |index|.
- void SetMinorText(size_t index, const std::u16string& minor_text);
-
- @@ -216,6 +220,7 @@ class COMPONENT_EXPORT(UI_BASE) SimpleMenuModel : public MenuModel {
- ui::MenuSeparatorType GetSeparatorTypeAt(size_t index) const override;
- int GetCommandIdAt(size_t index) const override;
- std::u16string GetLabelAt(size_t index) const override;
- + std::u16string GetSecondaryLabelAt(size_t index) const override;
- std::u16string GetMinorTextAt(size_t index) const override;
- ImageModel GetMinorIconAt(size_t index) const override;
- bool IsItemDynamicAt(size_t index) const override;
- @@ -255,6 +260,7 @@ class COMPONENT_EXPORT(UI_BASE) SimpleMenuModel : public MenuModel {
- int command_id = 0;
- ItemType type = TYPE_COMMAND;
- std::u16string label;
- + std::u16string secondary_label;
- std::u16string minor_text;
- ImageModel minor_icon;
- ImageModel icon;
|