Do not call check-webkit-style with empty affected file list as it tries to check all edited files …

Do not call check-webkit-style with empty affected file list as it tries to check all edited files including the skipped ones

This patch changes _CheckStyle to prevent calling check-webkit-style with empty arguments list if all files got filtered out.

BUG=635619
TEST=third_party/WebKit/PRESUBMIT_test.py

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

[Gleb Lanbin, 2016-08-11 14:57:43]

Patchset #1 (id:1) has been deleted

[Gleb Lanbin, 2016-08-11 14:57:50]

Patchset #1 (id:20001) has been deleted

[Gleb Lanbin, 2016-08-11 14:58:35]

glebl@chromium.org changed reviewers:
+ eae@chromium.org, ikilpatrick@chromium.org

[Gleb Lanbin, 2016-08-11 14:59:49]

glebl@chromium.org changed reviewers:
+ cbiesinger@chromium.org

[eae, 2016-08-11 14:59:58]

eae@chromium.org changed reviewers:
- cbiesinger@chromium.org

[eae, 2016-08-11 14:59:58]

Looks reasonable to me.

[Gleb Lanbin, 2016-08-11 15:02:14]

glebl@chromium.org changed reviewers:
+ dpranke@chromium.org
- eae@chromium.org, ikilpatrick@chromium.org

[Gleb Lanbin, 2016-08-11 15:10:27]

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

[commit-bot: I haz the power, 2016-08-11 15:10:43]

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-11 16:17:12]

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

[commit-bot: I haz the power, 2016-08-11 16:17:12]

Dry run: This issue passed the CQ dry run.

[eae, 2016-08-12 15:47:57]

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

[eae, 2016-08-12 15:47:57]

LGTM

[cbiesinger, 2016-08-22 23:00:02]

Ping?

[Dirk Pranke, 2016-08-22 23:06:46]

lgtm. Sorry for the delay, this must've fallen off my radar.

[cbiesinger, 2016-08-25 16:17:42]

The CQ bit was checked by cbiesinger@chromium.org

[commit-bot: I haz the power, 2016-08-25 16:18:27]

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-25 18:08:53]

Committed patchset #1 (id:40001)

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

Description was changed from

==========
Do not call check-webkit-style with empty affected file list as it tries to
check all edited files including the skipped ones

This patch changes _CheckStyle to prevent calling check-webkit-style with empty
arguments list if all files got filtered out.

BUG=635619
TEST=third_party/WebKit/PRESUBMIT_test.py
==========

to

==========
Do not call check-webkit-style with empty affected file list as it tries to
check all edited files including the skipped ones

This patch changes _CheckStyle to prevent calling check-webkit-style with empty
arguments list if all files got filtered out.

BUG=635619
TEST=third_party/WebKit/PRESUBMIT_test.py

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

[commit-bot: I haz the power, 2016-08-25 18:12:27]

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