Use has_committed_real_load for main frames in addition to subframes.

Use has_committed_real_load for main frames in addition to subframes.

This fixes a bug where a remote-to-local main frame navigation
overwrote the previous history entry.

BUG=725716
Review-Url: https://codereview.chromium.org/2901093006
Cr-Commit-Position: refs/heads/master@{#474536}
Committed: https://chromium.googlesource.com/chromium/src/+/c5e91fafd0be6f67afc2dd41d33c36492c5ffb43

[alexmos, 2017-05-24 17:28:33]

The CQ bit was checked by alexmos@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-24 17:29:21]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[alexmos, 2017-05-24 18:53:22]

alexmos@chromium.org changed reviewers:
+ creis@chromium.org

[alexmos, 2017-05-24 18:53:23]

Charlie, when you have a chance, can you please take a look if this fix makes
sense?  I posted an analysis of why this happens in the bug.

[commit-bot: I haz the power, 2017-05-24 22:23:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-24 22:23:36]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Charlie Reis, 2017-05-25 00:56:07]

Thanks!  I agree.  LGTM.

[Charlie Reis, 2017-05-25 00:56:11]

The CQ bit was checked by creis@chromium.org

[commit-bot: I haz the power, 2017-05-25 00:57:05]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-25 02:55:05]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495673771935520, "parent_rev":
"5ff4eeaefbc41f51d02887e0c42bb6e72ba292a0", "commit_rev":
"c5e91fafd0be6f67afc2dd41d33c36492c5ffb43"}

[commit-bot: I haz the power, 2017-05-25 02:55:16]

Description was changed from

==========
Use has_committed_real_load for main frames in addition to subframes.

This fixes a bug where a remote-to-local main frame navigation
overwrote the previous history entry.

BUG=725716
==========

to

==========
Use has_committed_real_load for main frames in addition to subframes.

This fixes a bug where a remote-to-local main frame navigation
overwrote the previous history entry.

BUG=725716

Review-Url: https://codereview.chromium.org/2901093006
Cr-Commit-Position: refs/heads/master@{#474536}
Committed:
https://chromium.googlesource.com/chromium/src/+/c5e91fafd0be6f67afc2dd41d33c...
==========

[commit-bot: I haz the power, 2017-05-25 02:55:19]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/c5e91fafd0be6f67afc2dd41d33c...