Relanding http://codereview.chromium.org/8416034

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 9ae0dc9a2efca4974e20abade9496bed904fcc9f..7183caa79f1262752e004b78d4722ada842e09b1 100644
--- a/content/browser/accessibility/browser_accessibility.h
+++ b/content/browser/accessibility/browser_accessibility.h
@@ -55,7 +55,14 @@ class CONTENT_EXPORT BrowserAccessibility {
   // Perform platform specific initialization. This can be called multiple times
   // during the lifetime of this instance after the members of this base object
   // have been reset with new values from the renderer process.
-  virtual void Initialize();
+  // Perform child independent initialization in this method.
+  virtual void PreInitialize();

[dmazzoni, 2011/12/02 23:33:42]

I think you can make this one protected now, right? The idea is for the Windows
implementation to extend this behavior without needing to take all of the
arguments.

[David Tseng, 2011/12/03 00:17:13]

Done.
On 2011/12/02 23:33:42, Dominic Mazzoni wrote:

+
+  // Perform platform specific initialization. This can be called multiple times
+  // during the lifetime of this instance after the members of this base object
+  // have been reset with new values from the renderer process.
+  // Child dependent initialization can be done here.
+  virtual void PostInitialize();

[dmazzoni, 2011/12/02 23:33:42]

You don't need a non-argument version and a version that takes arguments now,
just one PostInitialize.

[David Tseng, 2011/12/03 00:17:13]

First went for the symetry with PreInitialize. Kept this one and removed the
other.
On 2011/12/02 23:33:42, Dominic Mazzoni wrote:

 
   // Optionally send events triggered simply by the fact that this node
   // has been created or modified (and has been attached to the tree).
@@ -65,11 +72,19 @@ class CONTENT_EXPORT BrowserAccessibility {
 
   // Initialize this object, reading attributes from |src|. Does not
   // recurse into children of |src| and build the whole subtree.
-  void Initialize(BrowserAccessibilityManager* manager,
-                  BrowserAccessibility* parent,
-                  int32 child_id,
-                  int32 index_in_parent,
-                  const WebAccessibility& src);
+  void PreInitialize(BrowserAccessibilityManager* manager,
+      BrowserAccessibility* parent,
+      int32 child_id,
+      int32 index_in_parent,
+      const WebAccessibility& src);
+
+  // Initialize this object, reading attributes from |src|. Assumes that
+  // children of |this| have been populated.
+  void PostInitialize(BrowserAccessibilityManager* manager,

[dmazzoni, 2011/12/02 23:33:42]

This shouldn't take any arguments.

[David Tseng, 2011/12/03 00:17:13]

Done; (still needs to be virtual though) so keeping the  other).

On 2011/12/02 23:33:42, Dominic Mazzoni wrote:

+      BrowserAccessibility* parent,
+      int32 child_id,
+      int32 index_in_parent,
+      const WebAccessibility& src);
 
   // Add a child of this object.
   void AddChild(BrowserAccessibility* child);