commit 5949c661b51295148706d53a03b41df2eb840ae3
author John Reck <jreck@google.com> Fri May 27 09:52:29 2011 -0700
committer John Reck <jreck@google.com> Fri May 27 09:52:29 2011 -0700
tree ddd816e87c94282d51225e6b7fd3d327accd872e
parent b9b2a8290874e447444c7791647cbade915bc47d

Fix set parent race condition

 Alter the openTab method to make sure that the parent is
 always set before trying to load a url. This is done because
 some web settings (useragent) are now inherited from the parent,
 and we need to make sure those are applied before the UA is sent
 to the server.

Change-Id: Ib33b75ac4648e8ba1ede9bb92406ea175aabb19d

src/com/android/browser/Tab.java

diff --git a/src/com/android/browser/Tab.java b/src/com/android/browser/Tab.java
index 8962475..4097334 100644
--- a/src/com/android/browser/Tab.java
+++ b/src/com/android/browser/Tab.java
@@ -855,11 +855,7 @@
                 transport.setWebView(mSubView);
             } else {
                 final Tab newTab = mWebViewController.openTab(null,
-                        Tab.this.isPrivateBrowsingEnabled(),
-                        true, true);
-                if (newTab != Tab.this) {
-                    Tab.this.addChildTab(newTab);
-                }
+                        Tab.this, true, true);
                 transport.setWebView(newTab.getWebView());
             }
             msg.sendToTarget();