Chromium Code Reviews| Index: ui/app_list/views/folder_header_view.cc |
| diff --git a/ui/app_list/views/folder_header_view.cc b/ui/app_list/views/folder_header_view.cc |
| index 29f84d32b8e39200d2c1af55f1035c79b5223506..19c125fa1b015fb23e66323313d603a3291c0c67 100644 |
| --- a/ui/app_list/views/folder_header_view.cc |
| +++ b/ui/app_list/views/folder_header_view.cc |
| @@ -117,11 +117,19 @@ void FolderHeaderView::Update() { |
| folder_name_view_->SetVisible(folder_name_visible_); |
| if (folder_name_visible_) { |
| folder_name_view_->SetText(base::UTF8ToUTF16(folder_item_->name())); |
| + UpdateFolderNameAccessibleName(); |
| } |
| Layout(); |
| } |
| +void FolderHeaderView::UpdateFolderNameAccessibleName() { |
| + base::string16 access_name = folder_name_view_->text().empty() |
| + ? folder_name_placeholder_text_ |
| + : base::string16(); |
|
xiyuan
2014/04/28 21:43:27
Why use empty string when folder_name_view_->text(
jennyz
2014/04/28 22:23:01
Added a comment to explain.
|
| + folder_name_view_->SetAccessibleName(access_name); |
| +} |
| + |
| const base::string16& FolderHeaderView::GetFolderNameForTest() { |
| return folder_name_view_->text(); |
| } |
| @@ -198,6 +206,8 @@ void FolderHeaderView::ContentsChanged(views::Textfield* sender, |
| delegate_->SetItemName(folder_item_, name); |
| folder_item_->AddObserver(this); |
| + UpdateFolderNameAccessibleName(); |
| + |
| Layout(); |
| } |