presubmit: unblacklist third_party/WebKit

presubmit: unblacklist third_party/WebKit

The current DEFAULT_BLACKLIST in presubmit_support blacklists by
default third_party/.* in every repo. This requires an adjustement
after the blink merge, as it causing all the blink presubmits to
become noops.

BUG=539768
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=297030

[Primiano Tucci (use gerrit), 2015-10-06 08:25:16]

primiano@chromium.org changed reviewers:
+ tandrii@chromium.org

[tandrii_google, 2015-10-06 09:22:11]

tandrii@google.com changed reviewers:
+ tandrii@google.com

[tandrii_google, 2015-10-06 09:22:11]

lgtm

though, is this list still supposed to be here? Should we move most of this into
PRESUBMIT.py of chromium/src itself?

[Primiano Tucci (use gerrit), 2015-10-06 09:30:50]

On 2015/10/06 09:22:11, tandrii_google wrote:
> lgtm
> 
> though, is this list still supposed to be here? Should we move most of this
into
> PRESUBMIT.py of chromium/src itself?

I think these are general rules, not specific to chromium. And I bet that there
are N non-chromium projects out there that would break if we remove this (one I
can think about: breakpad).

[Primiano Tucci (use gerrit), 2015-10-06 09:30:52]

The CQ bit was checked by primiano@chromium.org

[commit-bot: I haz the power, 2015-10-06 09:30:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1382993004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1382993004/20001

[commit-bot: I haz the power, 2015-10-06 09:35:14]

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

[commit-bot: I haz the power, 2015-10-06 09:35:15]

Try jobs failed on following builders:
  depot_tools_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/depot_tools_pre...)

[tandrii(chromium), 2015-10-06 10:46:56]

On 2015/10/06 09:30:50, Primiano Tucci wrote:
> On 2015/10/06 09:22:11, tandrii_google wrote:
> > lgtm
> > 
> > though, is this list still supposed to be here? Should we move most of this
> into
> > PRESUBMIT.py of chromium/src itself?
> 
> I think these are general rules, not specific to chromium. And I bet that
there
> are N non-chromium projects out there that would break if we remove this (one
I
> can think about: breakpad).

True.

[tandrii(chromium), 2015-10-06 10:48:03]

Ah, i gave lgtm with wrong account before. But this one is the owner :)

[tandrii(chromium), 2015-10-06 10:48:06]

The CQ bit was checked by tandrii@chromium.org

[commit-bot: I haz the power, 2015-10-06 10:48:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1382993004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1382993004/20001

[commit-bot: I haz the power, 2015-10-06 10:50:14]

Committed patchset #2 (id:20001) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=297030