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

Unified Diff: chrome/browser/ui/views/omnibox/omnibox_view_views.h

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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/omnibox/omnibox_view_views.h
diff --git a/chrome/browser/ui/views/omnibox/omnibox_view_views.h b/chrome/browser/ui/views/omnibox/omnibox_view_views.h
index 0d1c70e51e008b00df782950942493a5a9494328..d8b436b73c06deebffd5f39da21c706ca0934a94 100644
--- a/chrome/browser/ui/views/omnibox/omnibox_view_views.h
+++ b/chrome/browser/ui/views/omnibox/omnibox_view_views.h
@@ -13,7 +13,6 @@
#include "base/gtest_prod_util.h"
#include "base/macros.h"
-#include "base/memory/weak_ptr.h"
#include "build/build_config.h"
#include "components/omnibox/browser/omnibox_view.h"
#include "components/security_state/security_state_model.h"
@@ -125,14 +124,6 @@ class OmniboxViewViews
// Handle keyword hint tab-to-search and tabbing through dropdown results.
bool HandleEarlyTabActions(const ui::KeyEvent& event);
- // Handles a request to change the value of this text field from software
- // using an accessibility API (typically automation software, screen readers
- // don't normally use this). If |clear_first| is true, this replaces all text
- // with the |new_value|. Otherwise this inserts |new_value| at the cursor
- // position, replacing any selected text. The cursor is placed at the end of
- // |new_value|.
- void AccessibilitySetValue(const base::string16& new_value, bool clear_first);
-
// Updates |security_level_| based on the toolbar model's current value.
void UpdateSecurityLevel();
@@ -175,7 +166,8 @@ class OmniboxViewViews
void OnGestureEvent(ui::GestureEvent* event) override;
void AboutToRequestFocusFromTabTraversal(bool reverse) override;
bool SkipDefaultKeyEventProcessing(const ui::KeyEvent& event) override;
- void GetAccessibleState(ui::AXViewState* state) override;
+ void GetAccessibleNodeData(ui::AXNodeData* node_data) override;
+ bool HandleAccessibleAction(const ui::AXActionData& action_data) override;
void OnFocus() override;
void OnBlur() override;
bool IsCommandIdEnabled(int command_id) const override;
@@ -255,9 +247,6 @@ class OmniboxViewViews
// painted. Used to measure omnibox responsiveness with a histogram.
base::TimeTicks insert_char_time_;
- // Used to bind callback functions to this object.
- base::WeakPtrFactory<OmniboxViewViews> weak_ptr_factory_;
-
DISALLOW_COPY_AND_ASSIGN(OmniboxViewViews);
};
« no previous file with comments | « chrome/browser/ui/views/omnibox/omnibox_result_view.cc ('k') | chrome/browser/ui/views/omnibox/omnibox_view_views.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698