[Culprit-Finder] Seperate gae related part in cache_decorator and gitile repository to gae_libs/

appengine/findit/waterfall/flake/step_mapper.py

 # Copyright 2016 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.
 
 import json
 import os
 
 import logging
 
 from common.http_client_appengine import HttpClientAppengine as HttpClient
+from gae_libs import caches
 from lib import cache_decorator
 from waterfall import buildbot
 from waterfall import swarming_util
 
 
 @cache_decorator.Cached(
-    namespace='trybots', cacher=cache_decorator.CompressedMemCacher())
+    namespace='trybots', cache=caches.CompressedMemCache())
 def _LoadTrybots():  # pragma: no cover.
   """Returns the mapping of Commit Queue trybots to Waterfall buildbots."""
   with open(os.path.join(os.path.dirname(__file__), 'trybots.json'), 'r') as f:
     return json.load(f)
 
 
 def _GetMatchingBuildbots(cq_master_name, cq_builder_name):  # pragma: no cover.
   """Returns a list of matching builder/tester buildbots on Waterfall."""
   trybot_map = _LoadTrybots()
   builders = trybot_map.get(cq_master_name, {}).get('builders', {})
@@ skipping 132 matching lines @@
     # Retrieve a sample output from Isolate.
     output = swarming_util.RetrieveShardedTestResultsFromIsolatedServer(
         step_isolated_data[:1], http_client)
     if output:
       # Guess from the format.
       build_step.supported = (
           isinstance(output, dict) and
           isinstance(output.get('all_tests'), list) and
           isinstance(output.get('per_iteration_data'), list) and
           all(isinstance(i, dict) for i in output.get('per_iteration_data'))
-      )
+      )

[wrengr, 2016/12/06 21:52:57]

not sure what Rietveld is trying to say the change is here. I assume it's
removing trailing space, rather than adding some

[Sharu Jiang, 2016/12/06 23:58:17]

it's removing the newline at the end.