[ios] Removed UIWebView JS evaluation code.

[ios] Removed UIWebView JS evaluation code.

BUG=579697
Committed: https://crrev.com/281c2845e0dbcbee2aad4227c40556192ff2872a
Cr-Commit-Position: refs/heads/master@{#381743}

[Eugene But (OOO till 7-30), 2016-03-17 06:00:59]

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

[commit-bot: I haz the power, 2016-03-17 06:01:13]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1808993002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1808993002/1

[commit-bot: I haz the power, 2016-03-17 06:33:30]

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

[commit-bot: I haz the power, 2016-03-17 06:33:31]

Dry run: This issue passed the CQ dry run.

[Eugene But (OOO till 7-30), 2016-03-17 13:36:38]

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

[Jackie Quinn, 2016-03-17 17:06:43]

lgtm, with question about future work.

https://codereview.chromium.org/1808993002/diff/1/ios/web/web_state/ui/web_vi...
File ios/web/web_state/ui/web_view_js_utils.mm (right):

https://codereview.chromium.org/1808993002/diff/1/ios/web/web_state/ui/web_vi...
ios/web/web_state/ui/web_view_js_utils.mm:16: NSString*
StringResultFromWKResult(id result) {
Are we planning on making it so that we don't convert to string? If so, maybe
file a bug and add a TODO.

[Eugene But (OOO till 7-30), 2016-03-17 17:19:14]

Thanks!

https://codereview.chromium.org/1808993002/diff/1/ios/web/web_state/ui/web_vi...
File ios/web/web_state/ui/web_view_js_utils.mm (right):

https://codereview.chromium.org/1808993002/diff/1/ios/web/web_state/ui/web_vi...
ios/web/web_state/ui/web_view_js_utils.mm:16: NSString*
StringResultFromWKResult(id result) {
On 2016/03/17 17:06:43, Jackie Quinn wrote:
> Are we planning on making it so that we don't convert to string? If so, maybe
> file a bug and add a TODO.
Web Controller will not be used as public API. WebState currently does not have
API to evaluate JS with callback, so if we add it, then maybe we'll make it
return |id| or even |base::Value|. I added TODO in JSInjectionEvaluator.

[Eugene But (OOO till 7-30), 2016-03-17 17:19:18]

The CQ bit was checked by eugenebut@chromium.org

[Eugene But (OOO till 7-30), 2016-03-17 17:19:18]

The patchset sent to the CQ was uploaded after l-g-t-m from jyquinn@chromium.org
Link to the patchset: https://codereview.chromium.org/1808993002/#ps20001
(title: "Added TODO")

[commit-bot: I haz the power, 2016-03-17 17:19:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1808993002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1808993002/20001

[commit-bot: I haz the power, 2016-03-17 18:02:50]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-17 18:04:11]

Description was changed from

==========
[ios] Removed UIWebView JS evaluation code.

BUG=579697
==========

to

==========
[ios] Removed UIWebView JS evaluation code.

BUG=579697

Committed: https://crrev.com/281c2845e0dbcbee2aad4227c40556192ff2872a
Cr-Commit-Position: refs/heads/master@{#381743}
==========

[commit-bot: I haz the power, 2016-03-17 18:04:12]

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