Tweak --gtest_filter_file switch to support blank lines and comment lines.

Tweak --gtest_filter_file switch to support blank lines and comment lines.

BUG=654589
TEST=Manually tried running out/.../bin/run_content_browsertests --gtest-filter-file=$HOME/scratch/file-with-comments-and-blank-lines.txt

Committed: https://crrev.com/705aa265b8d39f3b121086030e00986782748e23
Cr-Commit-Position: refs/heads/master@{#424437}

[Łukasz Anforowicz, 2016-10-10 23:41:54]

Description was changed from

==========
Tweak --gtest_filter_file switch to support blank lines and comment lines.

BUG=654589
==========

to

==========
Tweak --gtest_filter_file switch to support blank lines and comment lines.

BUG=654589
TEST=Manually tried running out/.../bin/run_content_browsertests
--gtest-filter-file=$HOME/scratch/file-with-comments-and-blank-lines.txt
==========

[Łukasz Anforowicz, 2016-10-10 23:42:48]

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

[commit-bot: I haz the power, 2016-10-10 23:43:14]

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

[Łukasz Anforowicz, 2016-10-10 23:57:47]

lukasza@chromium.org changed reviewers:
+ klundberg@chromium.org

[Łukasz Anforowicz, 2016-10-10 23:57:48]

klundberg@, could you please take a look?  (I would have asked jbudorick@, since
he added --gtest_filter_file switch in r314254, but it seems that he will be out
of office for the next 1-2 weeks;  OTOH, I see that you were the reviewer for
https://crrev.com/892273002 :-).

[mikecase (-- gone --), 2016-10-11 00:09:18]

mikecase@chromium.org changed reviewers:
+ mikecase@chromium.org

[mikecase (-- gone --), 2016-10-11 00:09:18]

I can review CLs while John is out. Don't think klundberg@ has looked at this
code for a bit.

[mikecase (-- gone --), 2016-10-11 00:26:34]

lgtm

[commit-bot: I haz the power, 2016-10-11 00:47:14]

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

[commit-bot: I haz the power, 2016-10-11 00:47:15]

Dry run: This issue passed the CQ dry run.

[Łukasz Anforowicz, 2016-10-11 15:38:17]

The CQ bit was checked by lukasza@chromium.org

[commit-bot: I haz the power, 2016-10-11 15:38:39]

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-10-11 15:44:36]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-11 15:46:00]

Description was changed from

==========
Tweak --gtest_filter_file switch to support blank lines and comment lines.

BUG=654589
TEST=Manually tried running out/.../bin/run_content_browsertests
--gtest-filter-file=$HOME/scratch/file-with-comments-and-blank-lines.txt
==========

to

==========
Tweak --gtest_filter_file switch to support blank lines and comment lines.

BUG=654589
TEST=Manually tried running out/.../bin/run_content_browsertests
--gtest-filter-file=$HOME/scratch/file-with-comments-and-blank-lines.txt

Committed: https://crrev.com/705aa265b8d39f3b121086030e00986782748e23
Cr-Commit-Position: refs/heads/master@{#424437}
==========

[commit-bot: I haz the power, 2016-10-11 15:46:01]

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

[jbudorick, 2016-10-18 16:48:10]

On 2016/10/11 00:09:18, mikecase wrote:
> I can review CLs while John is out. Don't think klundberg@ has looked at this
> code for a bit.

Thanks for reviewing this :)