Prevent bindings escalation on an existing NavigationEntry (attempt 3).

Prevent bindings escalation on an existing NavigationEntry (attempt 3).

BUG=173672
TEST=See bug for repro steps.


Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=184264

[Charlie Reis, 2013-02-22 22:15:48]

Hey Darin-- can you approve this again?  It's identical to
https://codereview.chromium.org/12210048.  I had to revert due to crashes, but I
believe I've resolved the crashes in https://codereview.chromium.org/12340035/.

I'm hoping to land this one again once the crash fix lands.

[darin (slow to review), 2013-02-22 23:29:08]

LGTM

On Fri, Feb 22, 2013 at 2:15 PM, <creis@chromium.org> wrote:

> Reviewers: darin,
>
> Message:
> Hey Darin-- can you approve this again?  It's identical to
>
https://codereview.chromium.**org/12210048<https://codereview.chromium.org/12....
>  I had to revert due to crashes, but I
> believe I've resolved the crashes in https://codereview.chromium.**
> org/12340035/ <https://codereview.chromium.org/12340035/>.
>
> I'm hoping to land this one again once the crash fix lands.
>
> Description:
> Prevent bindings escalation on an existing NavigationEntry (attempt 3).
>
> BUG=173672
> TEST=See bug for repro steps.
>
>
> Please review this at
https://codereview.chromium.**org/12313067/<https://codereview.chromium.org/1...
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/src<http://svn.chromium.org/chrome/trunk/src>
>
> Affected files:
>   M chrome/browser/sessions/**session_restore_browsertest.cc
>   M content/browser/web_contents/**navigation_controller_impl.cc
>   M content/browser/web_contents/**navigation_controller_impl_**
> unittest.cc
>   M content/browser/web_contents/**navigation_entry_impl.h
>   M content/browser/web_contents/**navigation_entry_impl.cc
>   M content/browser/web_contents/**render_view_host_manager.h
>   M content/browser/web_contents/**render_view_host_manager.cc
>
>
>

[commit-bot: I haz the power, 2013-02-23 00:02:37]

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

[commit-bot: I haz the power, 2013-02-23 02:33:51]

Change committed as 184264