Stop logging the start of a run for individual layout tests.

Stop logging the start of a run for individual layout tests.

NOTRY=true
NOPRESUBMIT=true
NOTREECHECKS=true
BUG=673207
Committed: https://crrev.com/67aaccae4c80bb66c0a623a6e3e56b8151d6e3eb
Cr-Commit-Position: refs/heads/master@{#437924}

[qyearsley, 2016-12-12 18:34:46]

qyearsley@chromium.org changed reviewers:
+ dpranke@chromium.org

[Dirk Pranke, 2016-12-12 18:54:56]

lgtm, though I would've left the line in but commented out.

[qyearsley, 2016-12-12 18:55:53]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2016-12-12 18:56:18]

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

[qyearsley, 2016-12-12 18:59:05]

On 2016/12/12 at 18:54:56, dpranke wrote:
> lgtm, though I would've left the line in but commented out.

I was hesitating whether to do that -- I've heard advice before to never have
commented-out code checked in to source control, since the original line can
always be seen in the source control history.

I think at this point we don't know whether we want to add this line back in the
future, right?

[Dirk Pranke, 2016-12-12 19:01:41]

On 2016/12/12 18:59:05, qyearsley wrote:
> On 2016/12/12 at 18:54:56, dpranke wrote:
> > lgtm, though I would've left the line in but commented out.
> 
> I was hesitating whether to do that -- I've heard advice before to never have
> commented-out code checked in to source control, since the original line can
> always be seen in the source control history.
> 
> I think at this point we don't know whether we want to add this line back in
the
> future, right?

Actually in this case I'd probably like to add the line back in, but your
concern is
a good one so what you did wasn't particularly wrong :).

[qyearsley, 2016-12-12 20:55:14]

The CQ bit was unchecked by qyearsley@chromium.org

[qyearsley, 2016-12-12 20:55:17]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2016-12-12 20:55:44]

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

[Dirk Pranke, 2016-12-12 21:49:01]

The CQ bit was unchecked by dpranke@chromium.org

[Dirk Pranke, 2016-12-12 21:49:17]

Description was changed from

==========
Stop logging the start of a run for individual layout tests.

BUG=673207
==========

to

==========
Stop logging the start of a run for individual layout tests.

NOTRY=true
NOPRESUBMIT=true
NOTREECHECKS=true
BUG=673207
==========

[Dirk Pranke, 2016-12-12 21:49:22]

The CQ bit was checked by dpranke@chromium.org

[Dirk Pranke, 2016-12-12 21:49:40]

skipping the tryjobs for this since the tryserver is busted ...

[commit-bot: I haz the power, 2016-12-12 21:50:23]

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-12-12 21:51:17]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481579362166930,
"parent_rev": "57adccfc6e7d537e437e5a52ab4ada2fc4b44282", "commit_rev":
"c80ee3f777a777a422379ad20f95929866a89b9e"}

[commit-bot: I haz the power, 2016-12-12 21:51:49]

Description was changed from

==========
Stop logging the start of a run for individual layout tests.

NOTRY=true
NOPRESUBMIT=true
NOTREECHECKS=true
BUG=673207
==========

to

==========
Stop logging the start of a run for individual layout tests.

NOTRY=true
NOPRESUBMIT=true
NOTREECHECKS=true
BUG=673207

Review-Url: https://codereview.chromium.org/2564343003
==========

[commit-bot: I haz the power, 2016-12-12 21:51:50]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-12 21:54:29]

Description was changed from

==========
Stop logging the start of a run for individual layout tests.

NOTRY=true
NOPRESUBMIT=true
NOTREECHECKS=true
BUG=673207

Review-Url: https://codereview.chromium.org/2564343003
==========

to

==========
Stop logging the start of a run for individual layout tests.

NOTRY=true
NOPRESUBMIT=true
NOTREECHECKS=true
BUG=673207

Committed: https://crrev.com/67aaccae4c80bb66c0a623a6e3e56b8151d6e3eb
Cr-Commit-Position: refs/heads/master@{#437924}
==========

[commit-bot: I haz the power, 2016-12-12 21:54:29]

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