[ios] Cleaned up UIWebView workarounds from window_id.js.

[ios] Cleaned up UIWebView workarounds from window_id.js.

1.) Delaying queues flushing is unnecessary because reentrace can't
happen with asynchronous JS API.
2.) Check for message object presence is unnecessary because it is
injected together with __crWeb object.

BUG=402682, 628832
Committed: https://crrev.com/cfa709c1f383195a9e134e4d86fd7664272a2783
Cr-Commit-Position: refs/heads/master@{#408250}

[Eugene But (OOO till 7-30), 2016-07-27 16:15:35]

Description was changed from

==========
[ios] Cleaned up UIWebView workarounds from window_id.js.

1.) Delaying queues flushing is unnecessary because reentrace can't
happen with asynchronous JS API.
2.) Check for message object presence is unnecessary because it is
injected together with __crWeb object.

BUG=402682
==========

to

==========
[ios] Cleaned up UIWebView workarounds from window_id.js.

1.) Delaying queues flushing is unnecessary because reentrace can't
happen with asynchronous JS API.
2.) Check for message object presence is unnecessary because it is
injected together with __crWeb object.

BUG=402682,628832
==========

[Eugene But (OOO till 7-30), 2016-07-27 16:15:41]

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

[commit-bot: I haz the power, 2016-07-27 16:15:52]

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, 2016-07-27 16:38:20]

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

[commit-bot: I haz the power, 2016-07-27 16:38:21]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Eugene But (OOO till 7-30), 2016-07-27 17:44:33]

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

[commit-bot: I haz the power, 2016-07-27 17:45:01]

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, 2016-07-27 18:17:21]

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

[commit-bot: I haz the power, 2016-07-27 18:17:22]

Dry run: This issue passed the CQ dry run.

[Eugene But (OOO till 7-30), 2016-07-27 18:49:03]

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

[kkhorimoto, 2016-07-27 20:44:49]

lgtm with nit

https://codereview.chromium.org/2192503002/diff/20001/ios/web/web_state/js/re...
File ios/web/web_state/js/resources/window_id.js (right):

https://codereview.chromium.org/2192503002/diff/20001/ios/web/web_state/js/re...
ios/web/web_state/js/resources/window_id.js:15: // CRWJSWindowIdManager replaces
$(WINDOW_ID) with appropriate string upon
s/CRWJSWindowIdManager/CRWJSWindowIDManager

[Eugene But (OOO till 7-30), 2016-07-27 20:51:36]

Thanks!

https://codereview.chromium.org/2192503002/diff/20001/ios/web/web_state/js/re...
File ios/web/web_state/js/resources/window_id.js (right):

https://codereview.chromium.org/2192503002/diff/20001/ios/web/web_state/js/re...
ios/web/web_state/js/resources/window_id.js:15: // CRWJSWindowIdManager replaces
$(WINDOW_ID) with appropriate string upon
On 2016/07/27 20:44:48, kkhorimoto_ wrote:
> s/CRWJSWindowIdManager/CRWJSWindowIDManager

Done.

[Eugene But (OOO till 7-30), 2016-07-27 20:51:39]

The CQ bit was checked by eugenebut@chromium.org

[Eugene But (OOO till 7-30), 2016-07-27 20:51:39]

The patchset sent to the CQ was uploaded after l-g-t-m from
kkhorimoto@chromium.org
Link to the patchset: https://codereview.chromium.org/2192503002/#ps40001
(title: "s/CRWJSWindowIdManager/CRWJSWindowIDManager")

[commit-bot: I haz the power, 2016-07-27 20:51:53]

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-07-27 21:57:10]

Description was changed from

==========
[ios] Cleaned up UIWebView workarounds from window_id.js.

1.) Delaying queues flushing is unnecessary because reentrace can't
happen with asynchronous JS API.
2.) Check for message object presence is unnecessary because it is
injected together with __crWeb object.

BUG=402682,628832
==========

to

==========
[ios] Cleaned up UIWebView workarounds from window_id.js.

1.) Delaying queues flushing is unnecessary because reentrace can't
happen with asynchronous JS API.
2.) Check for message object presence is unnecessary because it is
injected together with __crWeb object.

BUG=402682,628832
==========

[commit-bot: I haz the power, 2016-07-27 21:57:11]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-27 22:00:09]

Description was changed from

==========
[ios] Cleaned up UIWebView workarounds from window_id.js.

1.) Delaying queues flushing is unnecessary because reentrace can't
happen with asynchronous JS API.
2.) Check for message object presence is unnecessary because it is
injected together with __crWeb object.

BUG=402682,628832
==========

to

==========
[ios] Cleaned up UIWebView workarounds from window_id.js.

1.) Delaying queues flushing is unnecessary because reentrace can't
happen with asynchronous JS API.
2.) Check for message object presence is unnecessary because it is
injected together with __crWeb object.

BUG=402682,628832

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

[commit-bot: I haz the power, 2016-07-27 22:00:10]

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