Fix window.history.go() with no parameter.

Fix window.history.go() with no parameter.

BUG=673022
Committed: https://crrev.com/b7a4d89f56a89ebf6c65cbe5f568143f86cde70e
Cr-Commit-Position: refs/heads/master@{#437982}

[kkhorimoto, 2016-12-09 22:36:55]

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

[Eugene But (OOO till 7-30), 2016-12-09 22:58:20]

https://codereview.chromium.org/2565673003/diff/1/ios/web/web_state/ui/crw_we...
File ios/web/web_state/ui/crw_web_controller.mm (right):

https://codereview.chromium.org/2565673003/diff/1/ios/web/web_state/ui/crw_we...
ios/web/web_state/ui/crw_web_controller.mm:3089: // window.history.go() with no
input parameter should behave as
Should we fix this in JS as follows?:

|invokeOnHost_({'command': 'window.history.go', 'value': delta | 0});|

[kkhorimoto, 2016-12-13 00:30:56]

https://codereview.chromium.org/2565673003/diff/1/ios/web/web_state/ui/crw_we...
File ios/web/web_state/ui/crw_web_controller.mm (right):

https://codereview.chromium.org/2565673003/diff/1/ios/web/web_state/ui/crw_we...
ios/web/web_state/ui/crw_web_controller.mm:3089: // window.history.go() with no
input parameter should behave as
On 2016/12/09 22:58:19, Eugene But wrote:
> Should we fix this in JS as follows?:
> 
> |invokeOnHost_({'command': 'window.history.go', 'value': delta | 0});|

Done.

[Eugene But (OOO till 7-30), 2016-12-13 01:33:11]

lgtm!

[kkhorimoto, 2016-12-13 01:41:44]

The CQ bit was checked by kkhorimoto@chromium.org

[commit-bot: I haz the power, 2016-12-13 01:42:42]

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, 2016-12-13 01:54:15]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481593304847900,
"parent_rev": "d6528762ef6e0e97857b11f51548735997401f79", "commit_rev":
"886dbba3134996d94cfb0b7ae0e5d797a13b72ee"}

[commit-bot: I haz the power, 2016-12-13 01:55:00]

Description was changed from

==========
Fix window.history.go() with no parameter.

BUG=673022
==========

to

==========
Fix window.history.go() with no parameter.

BUG=673022

Review-Url: https://codereview.chromium.org/2565673003
==========

[commit-bot: I haz the power, 2016-12-13 01:55:01]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-13 01:57:09]

Description was changed from

==========
Fix window.history.go() with no parameter.

BUG=673022

Review-Url: https://codereview.chromium.org/2565673003
==========

to

==========
Fix window.history.go() with no parameter.

BUG=673022

Committed: https://crrev.com/b7a4d89f56a89ebf6c65cbe5f568143f86cde70e
Cr-Commit-Position: refs/heads/master@{#437982}
==========

[commit-bot: I haz the power, 2016-12-13 01:57:10]

Patchset 2 (id:??) landed as
https://crrev.com/b7a4d89f56a89ebf6c65cbe5f568143f86cde70e
Cr-Commit-Position: refs/heads/master@{#437982}