Page Info: Add brackets in to one line if and for statements

Page Info: Add brackets in to one line if and for statements

BUG=699840
Review-Url: https://codereview.chromium.org/2946323003
Cr-Commit-Position: refs/heads/master@{#481409}
Committed: https://chromium.googlesource.com/chromium/src/+/1e5352e7505feac546757f8bcdd7217b09a136a0

[sperigo, 2017-06-22 00:11:02]

sperigo@chromium.org changed reviewers:
+ lgarron@chromium.org

[sperigo, 2017-06-22 00:11:49]

I went ahead and tackled bug #699840 which involved addressing a code style
issue.

[lgarron, 2017-06-22 00:30:31]

Thanks, this will make me happier to work on this code! :-D

LGTM

[lgarron, 2017-06-22 00:30:39]

The CQ bit was checked by lgarron@chromium.org

[commit-bot: I haz the power, 2017-06-22 00:30:50]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[lgarron, 2017-06-22 00:34:18]

The CQ bit was unchecked by lgarron@chromium.org

[lgarron, 2017-06-22 00:35:07]

I originally sent this to the commit queue, but first one nit: Could you add
something like "Page Info: " to the beginning of the commit description to make
the scope of the change clear?

[lgarron, 2017-06-22 00:35:12]

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

[commit-bot: I haz the power, 2017-06-22 00:35:30]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sperigo, 2017-06-22 00:36:51]

Description was changed from

==========
Add brackets in to one line if and for statements

BUG=699840
==========

to

==========
Page Info: Add brackets in to one line if and for statements

BUG=699840
==========

[sperigo, 2017-06-22 00:41:53]

On 2017/06/22 00:35:07, lgarron wrote:
> I originally sent this to the commit queue, but first one nit: Could you add
> something like "Page Info: " to the beginning of the commit description to
make
> the scope of the change clear?

Yes! Fixed.

[lgarron, 2017-06-22 00:42:30]

On 2017/06/22 at 00:41:53, sperigo wrote:
> On 2017/06/22 00:35:07, lgarron wrote:
> > I originally sent this to the commit queue, but first one nit: Could you add
> > something like "Page Info: " to the beginning of the commit description to
make
> > the scope of the change clear?
> 
> Yes! Fixed.

Thanks!

[lgarron, 2017-06-22 00:42:33]

The CQ bit was checked by lgarron@chromium.org

[lgarron, 2017-06-22 00:42:34]

The patchset sent to the CQ was uploaded after l-g-t-m from lgarron@chromium.org
Link to the patchset: https://codereview.chromium.org/2946323003/#ps20001
(title: "Added "Page Info:", clarifying the scope of the change, into the commit
message")

[commit-bot: I haz the power, 2017-06-22 00:42:46]

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, 2017-06-22 02:28:50]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1498092153438750,
"parent_rev": "6bc4b5accdea133c4b5de50e5ae7185ef6f483e4", "commit_rev":
"1e5352e7505feac546757f8bcdd7217b09a136a0"}

[commit-bot: I haz the power, 2017-06-22 02:29:00]

Description was changed from

==========
Page Info: Add brackets in to one line if and for statements

BUG=699840
==========

to

==========
Page Info: Add brackets in to one line if and for statements

BUG=699840

Review-Url: https://codereview.chromium.org/2946323003
Cr-Commit-Position: refs/heads/master@{#481409}
Committed:
https://chromium.googlesource.com/chromium/src/+/1e5352e7505feac546757f8bcdd7...
==========

[commit-bot: I haz the power, 2017-06-22 02:29:01]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/1e5352e7505feac546757f8bcdd7...