[Android WebView] Make 3rd party licenses checker to turn bot red

build/android/buildbot/buildbot_functions.sh

 #!/bin/bash
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 #
 # Bash functions used by buildbot annotator scripts for the android
 # build of chromium.  Executing this script should not perform actions
 # other than setting variables and defining of functions.
 
 # Number of jobs on the compile line; e.g.  make -j"${JOBS}"
@@ skipping 242 matching lines @@
   local extract_exit_code=$?
   if (( $extract_exit_code > 1 )); then
     echo "@@@STEP_WARNINGS@@@"
     return
   fi
   return $extract_exit_code
   )
 }
 
 # Runs the license checker for the WebView build.
+# License checker may return error code 1 meaning that
+# there are non-fatal problems (warnings). Everything
+# above 1 is considered to be a show-stopper.
 function bb_check_webview_licenses {
   echo "@@@BUILD_STEP Check licenses for WebView@@@"
   (
   set +e
   cd "${SRC_ROOT}"
   python android_webview/tools/webview_licenses.py scan
-  if [[ $? -ne 0 ]]; then
+  local licenses_exit_code=$?
+  if [[ $licenses_exit_code -eq 1 ]]; then
     echo "@@@STEP_WARNINGS@@@"
+    return 0
   fi
-  return 0
+  return $licenses_exit_code

[Isaac (away), 2013/02/07 12:08:47]

This syntax will not work - the bot while halt if this step goes red instead of
running the other tests.  You could use bb_run_step instead.

However unless you have a plan to put this on the main waterfall, it should not
turn the fyi bots red.  And the last time we discussed this I thought it was
going to be converted to a presubmit check?

[mnaganov (inactive), 2013/02/07 13:48:52]

Thanks for pointing this out, Isaac! I think I can't use bb_run_step directly
because it can only report a step failure, not a warning condition. But I
changed the code to avoid halting the build.
 
I remember about that goal, this change is a step towards it. We have improved
the copyright scanner to have substantially less false positives, so it should
only become red on real issues, not when somebody uses 'copyright' as a variable
name or in a string.

The plan is to evaluate the new version on our FYI bot first, then promote it
into a presubmit check.

   )
 }
 
 # Retrieve a packed json property using python
 function bb_get_json_prop {
   local JSON="$1"
   local PROP="$2"
 
   python -c "import json; print json.loads('$JSON').get('$PROP', '')"
 }