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

Unified Diff: content/browser/accessibility/browser_accessibility_manager.h

Issue 252253002: Implement initial support for nesting one ax tree in another. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 8 months 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: content/browser/accessibility/browser_accessibility_manager.h
diff --git a/content/browser/accessibility/browser_accessibility_manager.h b/content/browser/accessibility/browser_accessibility_manager.h
index a5926c7b70f577fb93c07c76dbc7c17c5c151ce1..06c597bf2e1abf54a3dd4536f4734626f50100f7 100644
--- a/content/browser/accessibility/browser_accessibility_manager.h
+++ b/content/browser/accessibility/browser_accessibility_manager.h
@@ -69,7 +69,8 @@ class CONTENT_EXPORT BrowserAccessibilityFactory {
};
// Manages a tree of BrowserAccessibility objects.
-class CONTENT_EXPORT BrowserAccessibilityManager : public ui::AXTreeDelegate {
+class CONTENT_EXPORT BrowserAccessibilityManager
+ : public ui::AXTreeDelegate {
David Tseng 2014/05/01 18:07:54 nit: why was this changed? I think it still fits o
dmazzoni 2014/05/05 07:17:57 Accident. Thanks.
public:
// Creates the platform-specific BrowserAccessibilityManager, but
// with no parent window pointer. Only useful for unit tests.
@@ -178,6 +179,12 @@ class CONTENT_EXPORT BrowserAccessibilityManager : public ui::AXTreeDelegate {
virtual void OnNodeChangeFinished(ui::AXNode* node) OVERRIDE;
virtual void OnRootChanged(ui::AXNode* new_root) OVERRIDE {}
+ BrowserAccessibilityDelegate* delegate() const { return delegate_; }
+
+ // Frame tree support.
+ void SetParentFrameIds(int parent_frame_frame_id, int parent_frame_node_id);
+ BrowserAccessibility* GetCrossFrameParent();
+
protected:
BrowserAccessibilityManager(
BrowserAccessibilityDelegate* delegate,
@@ -242,6 +249,12 @@ class CONTENT_EXPORT BrowserAccessibilityManager : public ui::AXTreeDelegate {
// A mapping from a node id to its wrapper of type BrowserAccessibility.
base::hash_map<int32, BrowserAccessibility*> id_wrapper_map_;
+ // The id of the parent frame and the hosting node within the parent frame.
+ // The parent frame id can be mapped to another BrowserAccessibilityManager
+ // using FrameTreeAccessibility.
+ int32 parent_frame_frame_id_;
David Tseng 2014/05/01 18:07:54 I would split this comment into separate ones for
dmazzoni 2014/05/05 07:17:57 Done.
+ int32 parent_frame_node_id_;
+
// The on-screen keyboard state.
OnScreenKeyboardState osk_state_;

Powered by Google App Engine
This is Rietveld 408576698