Allow sending frame tree updates for frames being added, even while loading.

Allow sending frame tree updates for frames being added, even while loading.

We used to prevent frame tree updates during navigation from one page to another to avoid spurious updates. The goal was to not broadcast updates when we know we will be removing all the subframes when leaving a page. Allowing subframe additions to send updates allows us to support cross process calls without having to wait for all subframes to finish loading.


BUG=145014
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=158813

[nasko, 2012-09-20 15:31:12]

Hey Charlie,
Can you review this simplistic change? I should help us allow  cross process JS
calls to succeed earlier in a page load than the current state.
Thanks,
Nasko

[Charlie Reis, 2012-09-20 18:32:36]

LGTM.  Can you update the CL description, though?  It's missing the title (not
sure how that happened), and it initially sounds like the opposite of what this
change is doing.  (Maybe "We used to prevent... The goal was...")

[nasko, 2012-09-20 20:40:27]

Fixed the description.

[commit-bot: I haz the power, 2012-09-25 18:58:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nasko@chromium.org/10965011/1

[commit-bot: I haz the power, 2012-09-25 19:05:05]

Sorry for I got bad news for ya.
Compile failed with a clobber build.
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2012-09-25 19:08:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nasko@chromium.org/10965011/1

[commit-bot: I haz the power, 2012-09-25 19:11:29]

Sorry for I got bad news for ya.
Compile failed with a clobber build.
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2012-09-26 15:23:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nasko@chromium.org/10965011/17001

[commit-bot: I haz the power, 2012-09-26 16:39:34]

Change committed as 158813