[ios] Protect JS messaging system from overriding window.webkit object.

[ios] Protect JS messaging system from overriding window.webkit object.

Any webpage can override window.webkit object (intentionally or by
accident). Since window.webkit is used for JS->Native messaging it should
be protected from overriding.

Before sending a message, delete |window.webkit| to restore the original
implementation.

BUG=611023
Review-Url: https://codereview.chromium.org/1973683002
Cr-Commit-Position: refs/heads/master@{#393351}
(cherry picked from commit 3eccc98ae84d9f47151d329b13fe2527f3f649ae)
NOTRY=true
NOPRESUBMIT=true

[Eugene But (OOO till 7-30), 2016-05-16 15:23:31]

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

[Eugene But (OOO till 7-30), 2016-05-16 15:23:32]

Cherry picked only fix. Tests depend on other CL which I don't think we should
cherry-pick.

[Andy Caruso, 2016-05-16 19:49:09]

On 2016/05/16 15:23:32, Eugene But wrote:
> Cherry picked only fix. Tests depend on other CL which I don't think we should
> cherry-pick.

lgtm

[Eugene But (OOO till 7-30), 2016-05-16 19:53:35]

The CQ bit was checked by eugenebut@chromium.org

[commit-bot: I haz the power, 2016-05-16 19:54:19]

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

[commit-bot: I haz the power, 2016-05-16 19:54:20]

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

[commit-bot: I haz the power, 2016-05-16 19:54:21]

CLs for remote refs other than refs/pending/heads/master must contain NOTRY=true
and NOPRESUBMIT=true in order for the CQ to process them

[Eugene But (OOO till 7-30), 2016-05-16 19:58:47]

Description was changed from

==========
[ios] Protect JS messaging system from overriding window.webkit object.

Any webpage can override window.webkit object (intentionally or by
accident). Since window.webkit is used for JS->Native messaging it should
be protected from overriding.

Before sending a message, delete |window.webkit| to restore the original
implementation.

BUG=611023

Review-Url: https://codereview.chromium.org/1973683002
Cr-Commit-Position: refs/heads/master@{#393351}
(cherry picked from commit 3eccc98ae84d9f47151d329b13fe2527f3f649ae)
==========

to

==========
[ios] Protect JS messaging system from overriding window.webkit object.

Any webpage can override window.webkit object (intentionally or by
accident). Since window.webkit is used for JS->Native messaging it should
be protected from overriding.

Before sending a message, delete |window.webkit| to restore the original
implementation.

BUG=611023

Review-Url: https://codereview.chromium.org/1973683002
Cr-Commit-Position: refs/heads/master@{#393351}
(cherry picked from commit 3eccc98ae84d9f47151d329b13fe2527f3f649ae)
NOTRY=true
NOPRESUBMIT=true
==========

[Eugene But (OOO till 7-30), 2016-05-16 19:58:52]

The CQ bit was checked by eugenebut@chromium.org

[commit-bot: I haz the power, 2016-05-16 19:59:17]

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

[commit-bot: I haz the power, 2016-05-16 21:03:55]

Description was changed from

==========
[ios] Protect JS messaging system from overriding window.webkit object.

Any webpage can override window.webkit object (intentionally or by
accident). Since window.webkit is used for JS->Native messaging it should
be protected from overriding.

Before sending a message, delete |window.webkit| to restore the original
implementation.

BUG=611023

Review-Url: https://codereview.chromium.org/1973683002
Cr-Commit-Position: refs/heads/master@{#393351}
(cherry picked from commit 3eccc98ae84d9f47151d329b13fe2527f3f649ae)
NOTRY=true
NOPRESUBMIT=true
==========

to

==========
[ios] Protect JS messaging system from overriding window.webkit object.

Any webpage can override window.webkit object (intentionally or by
accident). Since window.webkit is used for JS->Native messaging it should
be protected from overriding.

Before sending a message, delete |window.webkit| to restore the original
implementation.

BUG=611023

Review-Url: https://codereview.chromium.org/1973683002
Cr-Commit-Position: refs/heads/master@{#393351}
(cherry picked from commit 3eccc98ae84d9f47151d329b13fe2527f3f649ae)
NOTRY=true
NOPRESUBMIT=true
==========

[commit-bot: I haz the power, 2016-05-16 21:03:56]

Committed patchset #1 (id:1)