diff options
| author | gayane <gayane@chromium.org> | 2014-12-12 19:48:09 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2014-12-13 03:49:29 +0000 |
| commit | e17026679329c19ce4c2b988ee0c33c9cfeaa7fc (patch) | |
| tree | 7eacc8c2299477d77f39d4ad1f0779ce6b069301 /chrome/browser/resources/PRESUBMIT.py | |
| parent | 80295948bf0bc43ebb640afd81fa152e9679fd36 (diff) | |
| download | chromium_src-e17026679329c19ce4c2b988ee0c33c9cfeaa7fc.zip chromium_src-e17026679329c19ce4c2b988ee0c33c9cfeaa7fc.tar.gz chromium_src-e17026679329c19ce4c2b988ee0c33c9cfeaa7fc.tar.bz2 | |
Looking into all changed files to find actions.xml.
In case actions.xml is included in the changelist the chrome/browser/resources/PRESUBMIT.py shouldn't run because the presubmit for actions.xml will do more complete job.
Fixing the bug to look into all changed files in the CL where previously it was only looking at subdirectories of current presubmit script to find actions.xml
BUG=439978
Review URL: https://codereview.chromium.org/760163004
Cr-Commit-Position: refs/heads/master@{#308244}
Diffstat (limited to 'chrome/browser/resources/PRESUBMIT.py')
| -rw-r--r-- | chrome/browser/resources/PRESUBMIT.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chrome/browser/resources/PRESUBMIT.py b/chrome/browser/resources/PRESUBMIT.py index e22e2a0..d603cc0 100644 --- a/chrome/browser/resources/PRESUBMIT.py +++ b/chrome/browser/resources/PRESUBMIT.py @@ -12,8 +12,8 @@ ACTION_XML_PATH = '../../../tools/metrics/actions/actions.xml' def CheckUserActionUpdate(input_api, output_api, action_xml_path): """Checks if any new user action has been added.""" - if any('actions.xml' == input_api.os_path.basename(f.LocalPath()) for f in - input_api.AffectedFiles()): + if any('actions.xml' == input_api.os_path.basename(f) for f in + input_api.change.LocalPaths()): # If actions.xml is already included in the changelist, the PRESUBMIT # for actions.xml will do a more complete presubmit check. return [] |