[NTP] Cleanup: add brackets in components/ntp_snippets. [1/2]

[NTP] Cleanup: add brackets in components/ntp_snippets. [1/2]

This CL adds curly brackets to |if|, |for| and |while| in
components/ntp_snippets. The search was done using
"((if)|(for)|(while)) \([^{]*?\n" regexp. Note that it matches
multiline statements and this was intended, since there were
not many such cases and I wanted to confirm that we did not
miss brackets there.

BUG=667202
Committed: https://crrev.com/b1a1155c1d237ddcdeba4828c4daccb5db105f72
Cr-Commit-Position: refs/heads/master@{#433505}

[vitaliii, 2016-11-21 08:22:58]

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

[commit-bot: I haz the power, 2016-11-21 08:23:06]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[vitaliii, 2016-11-21 08:23:32]

Description was changed from

==========
[NTP] Cleanup: add brackets in components/ntp_snippets [1/2]

This CL adds curly brackets to |if|, |for| and |while| in
components/ntp_snippets. The search was done using
((if)|(for)|(while)) \([^{]*?\n regexp.

BUG=667202
==========

to

==========
[NTP] Cleanup: add brackets in components/ntp_snippets [1/2]

This CL adds curly brackets to |if|, |for| and |while| in
components/ntp_snippets. The search was done using
"((if)|(for)|(while)) \([^{]*?\n" regexp.

BUG=667202
==========

[vitaliii, 2016-11-21 08:23:56]

vitaliii@chromium.org changed reviewers:
+ treib@chromium.org

[vitaliii, 2016-11-21 08:23:56]

Hi Marc,

could you have a look?

[vitaliii, 2016-11-21 08:45:53]

Description was changed from

==========
[NTP] Cleanup: add brackets in components/ntp_snippets [1/2]

This CL adds curly brackets to |if|, |for| and |while| in
components/ntp_snippets. The search was done using
"((if)|(for)|(while)) \([^{]*?\n" regexp.

BUG=667202
==========

to

==========
[NTP] Cleanup: add brackets in components/ntp_snippets [1/2]

This CL adds curly brackets to |if|, |for| and |while| in
components/ntp_snippets. The search was done using
"((if)|(for)|(while)) \([^{]*?\n" regexp. Note that it matches
multiline statements and this was intended, since there were
not many such cases and I wanted to confirm that we did not
miss brackets there.

BUG=667202
==========

[vitaliii, 2016-11-21 08:56:53]

Description was changed from

==========
[NTP] Cleanup: add brackets in components/ntp_snippets [1/2]

This CL adds curly brackets to |if|, |for| and |while| in
components/ntp_snippets. The search was done using
"((if)|(for)|(while)) \([^{]*?\n" regexp. Note that it matches
multiline statements and this was intended, since there were
not many such cases and I wanted to confirm that we did not
miss brackets there.

BUG=667202
==========

to

==========
[NTP] Cleanup: add brackets in components/ntp_snippets. [1/2]

This CL adds curly brackets to |if|, |for| and |while| in
components/ntp_snippets. The search was done using
"((if)|(for)|(while)) \([^{]*?\n" regexp. Note that it matches
multiline statements and this was intended, since there were
not many such cases and I wanted to confirm that we did not
miss brackets there.

BUG=667202
==========

[Marc Treib, 2016-11-21 09:37:33]

lgtm

[vitaliii, 2016-11-21 09:38:10]

The CQ bit was unchecked by vitaliii@chromium.org

[vitaliii, 2016-11-21 09:38:42]

The CQ bit was checked by vitaliii@chromium.org

[commit-bot: I haz the power, 2016-11-21 09:38:58]

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-11-21 10:50:36]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479721122734650, "parent_rev":
["ef09fab6e4edb48b453820a84a2b4d949e4cf3ad", null], "commit_rev":
["7529ccf893a18e291f6b05a48fa502609813062d", null]}

[commit-bot: I haz the power, 2016-11-21 10:50:58]

Description was changed from

==========
[NTP] Cleanup: add brackets in components/ntp_snippets. [1/2]

This CL adds curly brackets to |if|, |for| and |while| in
components/ntp_snippets. The search was done using
"((if)|(for)|(while)) \([^{]*?\n" regexp. Note that it matches
multiline statements and this was intended, since there were
not many such cases and I wanted to confirm that we did not
miss brackets there.

BUG=667202
==========

to

==========
[NTP] Cleanup: add brackets in components/ntp_snippets. [1/2]

This CL adds curly brackets to |if|, |for| and |while| in
components/ntp_snippets. The search was done using
"((if)|(for)|(while)) \([^{]*?\n" regexp. Note that it matches
multiline statements and this was intended, since there were
not many such cases and I wanted to confirm that we did not
miss brackets there.

BUG=667202
==========

[commit-bot: I haz the power, 2016-11-21 10:50:59]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-21 10:54:44]

Description was changed from

==========
[NTP] Cleanup: add brackets in components/ntp_snippets. [1/2]

This CL adds curly brackets to |if|, |for| and |while| in
components/ntp_snippets. The search was done using
"((if)|(for)|(while)) \([^{]*?\n" regexp. Note that it matches
multiline statements and this was intended, since there were
not many such cases and I wanted to confirm that we did not
miss brackets there.

BUG=667202
==========

to

==========
[NTP] Cleanup: add brackets in components/ntp_snippets. [1/2]

This CL adds curly brackets to |if|, |for| and |while| in
components/ntp_snippets. The search was done using
"((if)|(for)|(while)) \([^{]*?\n" regexp. Note that it matches
multiline statements and this was intended, since there were
not many such cases and I wanted to confirm that we did not
miss brackets there.

BUG=667202

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

[commit-bot: I haz the power, 2016-11-21 10:54:45]

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