Revert of Ignore public API checks if COMMIT=false is in the description

PRESUBMIT.py

 # Copyright (c) 2013 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.
 
 
 """Top-level presubmit script for Skia.
 
 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
 for more details about the presubmit API built into gcl.
 """
@@ skipping 125 matching lines @@
 
 
 def CheckChangeOnUpload(input_api, output_api):
   """Presubmit checks for the change on upload.
 
   The following are the presubmit checks:
   * Check change has one and only one EOL.
   """
   results = []
   results.extend(_CommonChecks(input_api, output_api))
-  # TODO(rmistry): Remove the below it is only for testing!!!
-  results.extend(_CheckLGTMsForPublicAPI(input_api, output_api))
   return results
 
 
 def _CheckTreeStatus(input_api, output_api, json_url):
   """Check whether to allow commit.
 
   Args:
     input_api: input related apis.
     output_api: output related apis.
     json_url: url to download json style status.
@@ skipping 91 matching lines @@
 
   lgtm_from_owner = False
   issue = input_api.change.issue
   if issue and input_api.rietveld:
     issue_properties = input_api.rietveld.get_issue_properties(
         issue=int(issue), messages=True)
     if re.match(REVERT_CL_SUBJECT_PREFIX, issue_properties['subject'], re.I):
       # It is a revert CL, ignore the public api owners check.
       return results
 
-    if re.search(r'^COMMIT=false$', issue_properties['description'], re.M):
-      # Ignore public api owners check for COMMIT=false CLs since they are not
-      # going to be committed.
-      return results
-
     match = re.search(r'^TBR=(.*)$', issue_properties['description'], re.M)
     if match:
       tbr_entries = match.group(1).strip().split(',')
       for owner in PUBLIC_API_OWNERS:
         if owner in tbr_entries or owner.split('@')[0] in tbr_entries:
           # If an owner is specified in the TBR= line then ignore the public
           # api owners check.
           return results
 
     if issue_properties['owner_email'] in PUBLIC_API_OWNERS:
@@ skipping 108 matching lines @@
     state and an error if it is in 'Closed' state.
   """
   results = []
   results.extend(_CommonChecks(input_api, output_api))
   results.extend(
       _CheckTreeStatus(input_api, output_api, json_url=(
           SKIA_TREE_STATUS_URL + '/banner-status?format=json')))
   results.extend(_CheckLGTMsForPublicAPI(input_api, output_api))
   results.extend(_CheckOwnerIsInAuthorsFile(input_api, output_api))
   return results