[ios] Use args destructuring in Mojo test promise.

[ios] Use args destructuring in Mojo test promise.

BUG=None

Committed: https://crrev.com/a62168c59e1595a10507027ea0a34e474fa514a3
Cr-Commit-Position: refs/heads/master@{#419256}

[Eugene But (OOO till 7-30), 2016-08-27 18:54:20]

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

[Eugene But (OOO till 7-30), 2016-08-27 18:54:21]

Took EcmaScript6 class and liked args destructuring :)

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

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

[commit-bot: I haz the power, 2016-08-27 20:54:15]

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-08-27 21:36:21]

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

[commit-bot: I haz the power, 2016-08-27 21:36:21]

Dry run: This issue passed the CQ dry run.

[Eugene But (OOO till 7-30), 2016-09-16 17:57:41]

Demetrios?

[dpapad, 2016-09-16 18:01:36]

LGTM. Sorry for the delay.

@dbeam: Is it OK to use such assignments in WebUI's code (assuming that the JS
compiler understands them)?

[Dan Beam, 2016-09-16 18:46:40]

On 2016/09/16 18:01:36, dpapad wrote:
> LGTM. Sorry for the delay.
> 
> @dbeam: Is it OK to use such assignments in WebUI's code (assuming that the JS
> compiler understands them)?

supposedly destructuring parameters work on mobile Safari 8.  eugenebut@: I
assume you tested this, yes?

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/D...
http://kangax.github.io/compat-table/es6/#test-destructuring,_parameters_with...

the answer is ultimately: if you're willing to maintain this code, the tooling
required, or OK with getting reverted, do whatever you want ;)

[Eugene But (OOO till 7-30), 2016-09-16 18:56:00]

On 2016/09/16 18:46:40, Dan Beam wrote:
> On 2016/09/16 18:01:36, dpapad wrote:
> > LGTM. Sorry for the delay.
> > 
> > @dbeam: Is it OK to use such assignments in WebUI's code (assuming that the
JS
> > compiler understands them)?
> 
> supposedly destructuring parameters work on mobile Safari 8.  eugenebut@: I
> assume you tested this, yes?
> 
>
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/D...
>
http://kangax.github.io/compat-table/es6/#test-destructuring,_parameters_with...
> 
> the answer is ultimately: if you're willing to maintain this code, the tooling
> required, or OK with getting reverted, do whatever you want ;)
Thanks! Yes, this is tested (actually the code is used only in automated tests).

[Eugene But (OOO till 7-30), 2016-09-16 18:56:10]

The CQ bit was checked by eugenebut@chromium.org

[commit-bot: I haz the power, 2016-09-16 18:56: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-09-16 19:48:45]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-16 19:51:43]

Description was changed from

==========
[ios] Use args destructuring in Mojo test promise.

BUG=None
==========

to

==========
[ios] Use args destructuring in Mojo test promise.

BUG=None

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

[commit-bot: I haz the power, 2016-09-16 19:51:44]

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