Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(647)

Side by Side Diff: chrome/browser/ui/views/toolbar/toolbar_action_view.cc

Issue 2477463003: Replace ui::AXViewState with AXNodeData and AXActionData (Closed)
Patch Set: Fix test Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/views/toolbar/toolbar_action_view.h" 5 #include "chrome/browser/ui/views/toolbar/toolbar_action_view.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/auto_reset.h" 9 #include "base/auto_reset.h"
10 #include "base/bind.h" 10 #include "base/bind.h"
11 #include "chrome/browser/chrome_notification_types.h" 11 #include "chrome/browser/chrome_notification_types.h"
12 #include "chrome/browser/sessions/session_tab_helper.h" 12 #include "chrome/browser/sessions/session_tab_helper.h"
13 #include "chrome/browser/themes/theme_properties.h" 13 #include "chrome/browser/themes/theme_properties.h"
14 #include "chrome/browser/ui/toolbar/toolbar_action_view_controller.h" 14 #include "chrome/browser/ui/toolbar/toolbar_action_view_controller.h"
15 #include "chrome/browser/ui/toolbar/toolbar_actions_bar.h" 15 #include "chrome/browser/ui/toolbar/toolbar_actions_bar.h"
16 #include "chrome/browser/ui/view_ids.h" 16 #include "chrome/browser/ui/view_ids.h"
17 #include "content/public/browser/notification_source.h" 17 #include "content/public/browser/notification_source.h"
18 #include "ui/accessibility/ax_view_state.h" 18 #include "ui/accessibility/ax_node_data.h"
19 #include "ui/base/resource/resource_bundle.h" 19 #include "ui/base/resource/resource_bundle.h"
20 #include "ui/base/theme_provider.h" 20 #include "ui/base/theme_provider.h"
21 #include "ui/compositor/paint_recorder.h" 21 #include "ui/compositor/paint_recorder.h"
22 #include "ui/events/event.h" 22 #include "ui/events/event.h"
23 #include "ui/gfx/image/image_skia.h" 23 #include "ui/gfx/image/image_skia.h"
24 #include "ui/gfx/image/image_skia_operations.h" 24 #include "ui/gfx/image/image_skia_operations.h"
25 #include "ui/gfx/image/image_skia_source.h" 25 #include "ui/gfx/image/image_skia_source.h"
26 #include "ui/views/controls/button/label_button_border.h" 26 #include "ui/views/controls/button/label_button_border.h"
27 #include "ui/views/controls/menu/menu_controller.h" 27 #include "ui/views/controls/menu/menu_controller.h"
28 #include "ui/views/controls/menu/menu_model_adapter.h" 28 #include "ui/views/controls/menu/menu_model_adapter.h"
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
68 if (delegate_->ShownInsideMenu()) 68 if (delegate_->ShownInsideMenu())
69 SetFocusBehavior(FocusBehavior::ALWAYS); 69 SetFocusBehavior(FocusBehavior::ALWAYS);
70 70
71 UpdateState(); 71 UpdateState();
72 } 72 }
73 73
74 ToolbarActionView::~ToolbarActionView() { 74 ToolbarActionView::~ToolbarActionView() {
75 view_controller_->SetDelegate(nullptr); 75 view_controller_->SetDelegate(nullptr);
76 } 76 }
77 77
78 void ToolbarActionView::GetAccessibleState(ui::AXViewState* state) { 78 void ToolbarActionView::GetAccessibleNodeData(ui::AXNodeData* node_data) {
79 views::MenuButton::GetAccessibleState(state); 79 views::MenuButton::GetAccessibleNodeData(node_data);
80 state->role = ui::AX_ROLE_BUTTON; 80 node_data->role = ui::AX_ROLE_BUTTON;
81 } 81 }
82 82
83 std::unique_ptr<LabelButtonBorder> ToolbarActionView::CreateDefaultBorder() 83 std::unique_ptr<LabelButtonBorder> ToolbarActionView::CreateDefaultBorder()
84 const { 84 const {
85 std::unique_ptr<LabelButtonBorder> border = 85 std::unique_ptr<LabelButtonBorder> border =
86 LabelButton::CreateDefaultBorder(); 86 LabelButton::CreateDefaultBorder();
87 border->set_insets(gfx::Insets(kBorderInset, kBorderInset, 87 border->set_insets(gfx::Insets(kBorderInset, kBorderInset,
88 kBorderInset, kBorderInset)); 88 kBorderInset, kBorderInset));
89 return border; 89 return border;
90 } 90 }
(...skipping 217 matching lines...) Expand 10 before | Expand all | Expand 10 after
308 if (menu_controller->in_nested_run()) { 308 if (menu_controller->in_nested_run()) {
309 // There is another menu showing. Close the outermost menu (since we are 309 // There is another menu showing. Close the outermost menu (since we are
310 // shown in the same menu, we don't want to close the whole thing). 310 // shown in the same menu, we don't want to close the whole thing).
311 menu_controller->Cancel(views::MenuController::EXIT_OUTERMOST); 311 menu_controller->Cancel(views::MenuController::EXIT_OUTERMOST);
312 return true; 312 return true;
313 } 313 }
314 } 314 }
315 315
316 return false; 316 return false;
317 } 317 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/toolbar/toolbar_action_view.h ('k') | chrome/browser/ui/views/toolbar/toolbar_action_view_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698