Fixed isCurrentNavigationItemPOST for user initiated POSTs.

Fixed isCurrentNavigationItemPOST for user initiated POSTs.

CRWWebController allows post requests in loadWithParams:. Those requests
will not create _pendingNavigationInfo but will set HasPostData to
navigation item. Checking both _pendingNavigationInfo and HasPostData
fixes the problem.

Tests will be in a separate CL, which implements
NavigaitonContext::IsPost():
https://codereview.chromium.org/2903883002/patch/20001/30004

BUG=725241
Review-Url: https://codereview.chromium.org/2899193003
Cr-Commit-Position: refs/heads/master@{#474453}
Committed: https://chromium.googlesource.com/chromium/src/+/517153c7d9b005cc6f7495d759a6e26f9f29ca87

[Eugene But (OOO till 7-30), 2017-05-24 18:17:55]

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

[commit-bot: I haz the power, 2017-05-24 18:18:28]

Dry run: 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-24 18:40:28]

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

[commit-bot: I haz the power, 2017-05-24 18:40:28]

Dry run: This issue passed the CQ dry run.

[Eugene But (OOO till 7-30), 2017-05-24 20:45:32]

eugenebut@chromium.org changed reviewers:
+ kkhorimoto@chromium.org

[kkhorimoto, 2017-05-24 21:12:04]

lgtm

[Eugene But (OOO till 7-30), 2017-05-24 22:30:30]

The CQ bit was checked by eugenebut@chromium.org

[commit-bot: I haz the power, 2017-05-24 22:31:28]

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-24 22:36:21]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495665030615600, "parent_rev":
"c114d280bcd00241e9981a9ab854c998363175be", "commit_rev":
"517153c7d9b005cc6f7495d759a6e26f9f29ca87"}

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

Description was changed from

==========
Fixed isCurrentNavigationItemPOST for user initiated POSTs.

CRWWebController allows post requests in loadWithParams:. Those requests
will not create _pendingNavigationInfo but will set HasPostData to
navigation item. Checking both _pendingNavigationInfo and HasPostData
fixes the problem.

Tests will be in a separate CL, which implements
NavigaitonContext::IsPost():
https://codereview.chromium.org/2903883002/patch/20001/30004

BUG=725241
==========

to

==========
Fixed isCurrentNavigationItemPOST for user initiated POSTs.

CRWWebController allows post requests in loadWithParams:. Those requests
will not create _pendingNavigationInfo but will set HasPostData to
navigation item. Checking both _pendingNavigationInfo and HasPostData
fixes the problem.

Tests will be in a separate CL, which implements
NavigaitonContext::IsPost():
https://codereview.chromium.org/2903883002/patch/20001/30004

BUG=725241

Review-Url: https://codereview.chromium.org/2899193003
Cr-Commit-Position: refs/heads/master@{#474453}
Committed:
https://chromium.googlesource.com/chromium/src/+/517153c7d9b005cc6f7495d759a6...
==========

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

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