Revert of Testing

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 45 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))
+  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 44 matching lines @@
       requires_owner_check = True
 
   if not requires_owner_check:
     return results
 
   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)
+    # TODO(rmistry): Check the description here and determine if it is a
+    # one-click revert CL. Just look at the subject actually and then determine this!
+    print 'HERE HERE'
+    print issue_properties['subject']
+    print 'HERE HERE'
     if issue_properties['owner_email'] in PUBLIC_API_OWNERS:
       # An owner created the CL that is an automatic LGTM.
       lgtm_from_owner = True
 
     messages = issue_properties.get('messages')
     if messages:
       for message in messages:
         if (message['sender'] in PUBLIC_API_OWNERS and
             'lgtm' in message['text'].lower()):
           # Found an lgtm in a message from an owner.
           lgtm_from_owner = True
           break;
+  else:
+    print 'NO ISSUE AVAILABLE!!!!!!!!!!!!!!!!!!!!!'
        
   if not lgtm_from_owner:
     results.append(
         output_api.PresubmitError(
             'Since the CL is editing public API, you must have an LGTM from '
             'one of: %s' % str(PUBLIC_API_OWNERS)))
   return results
 
 
 def CheckChangeOnCommit(input_api, output_api):
   """Presubmit checks for the change on commit.
 
   The following are the presubmit checks:
   * Check change has one and only one EOL.
   * Ensures that the Skia tree is open in
     http://skia-tree-status.appspot.com/. Shows a warning if it is in 'Caution'
     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))
   return results