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

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

Issue 268543008: Cross-process iframe accessibility. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address feedback Created 6 years, 4 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.h
diff --git a/content/browser/accessibility/browser_accessibility.h b/content/browser/accessibility/browser_accessibility.h
index 17f70caf675fe7aa1a983d4e057d90e7d1f26f6e..090586037cc46eac1ae39f472364e7134ee6a77c 100644
--- a/content/browser/accessibility/browser_accessibility.h
+++ b/content/browser/accessibility/browser_accessibility.h
@@ -31,12 +31,13 @@ class BrowserAccessibilityWin;
//
// BrowserAccessibility
//
-// Class implementing the cross platform interface for the Browser-Renderer
-// communication of accessibility information, providing accessibility
-// to be used by screen readers and other assistive technology (AT).
+// A BrowserAccessibility object represents one node in the accessibility
+// tree on the browser side. It exactly corresponds to one WebAXObject from
+// Blink. It's owned by a BrowserAccessibilityManager.
//
-// An implementation for each platform handles platform specific accessibility
-// APIs.
+// There are subclasses of BrowserAccessibility for each platform where
+// we implement native accessibility APIs. This base class is used occasionally
+// for tests.
//
////////////////////////////////////////////////////////////////////////////////
class CONTENT_EXPORT BrowserAccessibility {
@@ -245,6 +246,15 @@ class CONTENT_EXPORT BrowserAccessibility {
// Append the text from this node and its children.
std::string GetTextRecursive() const;
+ // Identifies the given frame tree node id as the only child of this node,
+ // so any call to PlatformChildCount/PlatformGetChild will use
+ // BrowserAccessibilityDelegate::AccessibilityGetChildFrame to retrieve
+ // the BrowserAccessibilityManager of the child frame and return its root
+ // node as this node's child.
+ void SetChildFrameTreeNodeId(int64 child_frame_tree_node_id);
+
+ int64 child_frame_tree_node_id() const { return child_frame_tree_node_id_; }
+
protected:
BrowserAccessibility();
@@ -262,7 +272,9 @@ class CONTENT_EXPORT BrowserAccessibility {
std::string name_;
std::string value_;
- private:
+ // If nonzero, the frame tree node id of the child frame of this node.
+ int64 child_frame_tree_node_id_;
+
DISALLOW_COPY_AND_ASSIGN(BrowserAccessibility);
};

Powered by Google App Engine
This is Rietveld 408576698