Implement initial support for nesting one ax tree in another.

content/browser/accessibility/browser_accessibility.h

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..72e46ee489b0e37690c86eb9888a51703f4dbff1 100644
--- a/content/browser/accessibility/browser_accessibility.h
+++ b/content/browser/accessibility/browser_accessibility.h
@@ -245,6 +245,9 @@ class CONTENT_EXPORT BrowserAccessibility {
   // Append the text from this node and its children.
   std::string GetTextRecursive() const;
 
+  // For an iframe element

[David Tseng, 2014/05/01 18:07:54]

nit: period and perhaps "if this element is an iframe"?

[dmazzoni, 2014/05/05 07:17:57]

Done.

+  void SetChildFrameId(int child_frame_id);

[David Tseng, 2014/05/01 18:07:54]

nit: perhaps ID?

Also, on a higher level, child frame ID to me means one child frame's id. What
about other child iframes? The naming can be confusing especially as this is the
first file I'm reviewing.

[dmazzoni, 2014/05/05 07:17:57]

Id is consistent with the rest of this file; I have no strong preference if you
want to switch to ID throughout all accessibility files.
I tried to clarify the comment a lot, see if it's more clear nw.

+
  protected:
   BrowserAccessibility();
 
@@ -262,6 +265,8 @@ class CONTENT_EXPORT BrowserAccessibility {
   std::string name_;
   std::string value_;
 
+  int child_frame_id_;

[David Tseng, 2014/05/01 18:07:54]

nit: did you want these sorted?

[dmazzoni, 2014/05/05 07:17:57]

I don't want to sort unrelated fields.

+
  private:

[David Tseng, 2014/05/01 18:07:54]

nit: I think the above private: specifier suffices?

[dmazzoni, 2014/05/05 07:17:57]

Done.

   DISALLOW_COPY_AND_ASSIGN(BrowserAccessibility);
 };