Exclude vulcanize-generated HTML/JS from PRESUBMIT.py checks

Exclude vulcanize-generated HTML/JS from PRESUBMIT.py checks

These files are automatically generated for the PDF extension. The long term
solution is not to check in vulcanize-generated files. These files may not
be needed once HTMLImports are fully implemented in chrome.

BUG=303491
R=arv@chromium.org, maruel@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=251890

[M-A Ruel, 2014-02-15 01:46:57]

lgtm

https://codereview.chromium.org/154223003/diff/1/chrome/browser/resources/PRE...
File chrome/browser/resources/PRESUBMIT.py (right):

https://codereview.chromium.org/154223003/diff/1/chrome/browser/resources/PRE...
chrome/browser/resources/PRESUBMIT.py:53: return (not(maybe_resource.LocalPath()
in WHITELIST) and
then it is a blacklist. Rename the variable, and use:

return (maybe_resource.LocalPath() not in BLACKLIST and
          _html_css_js_resource(maybe_resource.AbsoluteLocalPath()))

[raymes, 2014-02-16 23:14:30]

https://codereview.chromium.org/154223003/diff/1/chrome/browser/resources/PRE...
File chrome/browser/resources/PRESUBMIT.py (right):

https://codereview.chromium.org/154223003/diff/1/chrome/browser/resources/PRE...
chrome/browser/resources/PRESUBMIT.py:53: return (not(maybe_resource.LocalPath()
in WHITELIST) and
On 2014/02/15 01:46:57, M-A Ruel wrote:
> then it is a blacklist. Rename the variable, and use:
> 
> return (maybe_resource.LocalPath() not in BLACKLIST and
>           _html_css_js_resource(maybe_resource.AbsoluteLocalPath()))

Done.

[raymes, 2014-02-16 23:14:53]

The CQ bit was checked by raymes@chromium.org

[commit-bot: I haz the power, 2014-02-16 23:15:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/raymes@chromium.org/154223003/60001

[commit-bot: I haz the power, 2014-02-16 23:19:12]

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

[commit-bot: I haz the power, 2014-02-16 23:19:12]

Try jobs failed on following builders: linux_chromium_rel

[raymes, 2014-02-17 23:16:36]

+arv for chrome/browser/resources/PRESUBMIT.py

[arv (Not doing code reviews), 2014-02-18 17:58:10]

LGTM

[raymes, 2014-02-18 22:55:55]

The CQ bit was checked by raymes@chromium.org

[commit-bot: I haz the power, 2014-02-18 22:57:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/raymes@chromium.org/154223003/60001

[commit-bot: I haz the power, 2014-02-18 22:59:12]

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

[commit-bot: I haz the power, 2014-02-18 22:59:13]

Try jobs failed on following builders: linux_chromium_rel

[raymes, 2014-02-19 00:09:29]

Committed patchset #2 manually as r251890 (presubmit successful).