Fix mobile user agent string not being used when PlzNavigate is enabled.

Fix mobile user agent string not being used when PlzNavigate is enabled.

The user agent override string should only have been used when the NavigationEntry has the flag set.

BUG=705218
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2777813003
Cr-Commit-Position: refs/heads/master@{#459895}
Committed: https://chromium.googlesource.com/chromium/src/+/d83b035e1cb466ff77c52973fe633fe780833325

[jam, 2017-03-27 18:07:02]

Description was changed from

==========
Fix mobile user agent string not being used when PlzNavigate is enabled.

BUG=705218
==========

to

==========
Fix mobile user agent string not being used when PlzNavigate is enabled.

BUG=705218
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[jam, 2017-03-27 18:07:06]

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

[commit-bot: I haz the power, 2017-03-27 18:07:43]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jam, 2017-03-27 18:26:51]

Description was changed from

==========
Fix mobile user agent string not being used when PlzNavigate is enabled.

BUG=705218
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Fix mobile user agent string not being used when PlzNavigate is enabled.

The user agent override string should only have been used when the
NavigationEntry has the flag set.

BUG=705218
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[jam, 2017-03-27 18:26:51]

jam@chromium.org changed reviewers:
+ nasko@chromium.org

[jam, 2017-03-27 18:56:04]

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

[commit-bot: I haz the power, 2017-03-27 18:56:56]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[nasko, 2017-03-27 19:52:31]

LGTM with one note.

https://codereview.chromium.org/2777813003/diff/20001/content/browser/web_con...
File content/browser/web_contents/web_contents_impl_browsertest.cc (right):

https://codereview.chromium.org/2777813003/diff/20001/content/browser/web_con...
content/browser/web_contents/web_contents_impl_browsertest.cc:1468:
.GetActiveEntry()
GetLastCommittedEntry(), GetActiveEntry is deprecated and should not be used.

[jam, 2017-03-27 19:59:01]

The CQ bit was unchecked by jam@chromium.org

[jam, 2017-03-27 20:00:14]

https://codereview.chromium.org/2777813003/diff/20001/content/browser/web_con...
File content/browser/web_contents/web_contents_impl_browsertest.cc (right):

https://codereview.chromium.org/2777813003/diff/20001/content/browser/web_con...
content/browser/web_contents/web_contents_impl_browsertest.cc:1468:
.GetActiveEntry()
On 2017/03/27 19:52:31, nasko wrote:
> GetLastCommittedEntry(), GetActiveEntry is deprecated and should not be used.

Done.

[jam, 2017-03-27 20:00:46]

The CQ bit was checked by jam@chromium.org

[jam, 2017-03-27 20:00:47]

The patchset sent to the CQ was uploaded after l-g-t-m from nasko@chromium.org
Link to the patchset: https://codereview.chromium.org/2777813003/#ps40001
(title: "review comment")

[commit-bot: I haz the power, 2017-03-27 20:01:18]

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-03-27 21:57:46]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1490644846803790,
"parent_rev": "82843780df15988017b386494f1e76c333410b1d", "commit_rev":
"d83b035e1cb466ff77c52973fe633fe780833325"}

[commit-bot: I haz the power, 2017-03-27 21:58:55]

Description was changed from

==========
Fix mobile user agent string not being used when PlzNavigate is enabled.

The user agent override string should only have been used when the
NavigationEntry has the flag set.

BUG=705218
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Fix mobile user agent string not being used when PlzNavigate is enabled.

The user agent override string should only have been used when the
NavigationEntry has the flag set.

BUG=705218
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

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

[commit-bot: I haz the power, 2017-03-27 21:58:56]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/d83b035e1cb466ff77c52973fe63...