Index: tools/telemetry/telemetry/util/cloud_storage.py |
diff --git a/tools/telemetry/telemetry/util/cloud_storage.py b/tools/telemetry/telemetry/util/cloud_storage.py |
index 1e9fc4f9831455c249b8a9b3588d5a632b71843d..ed97ccbb4837bcbab573b4fab844fc0898e9c1f1 100644 |
--- a/tools/telemetry/telemetry/util/cloud_storage.py |
+++ b/tools/telemetry/telemetry/util/cloud_storage.py |
@@ -15,7 +15,6 @@ |
import tarfile |
import urllib2 |
-from telemetry import decorators |
from telemetry.core import util |
from telemetry.util import path |
@@ -221,8 +220,10 @@ |
def GetIfChanged(file_path, bucket): |
- """Gets the file at file_path if it has a hash file that doesn't match or |
- if there is no local copy of file_path, but there is a hash file for it. |
+ """Gets the file at file_path if it has a hash file that doesn't match. |
+ |
+ If the file is not in Cloud Storage, log a warning instead of raising an |
+ exception. We assume that the user just hasn't uploaded the file yet. |
Returns: |
True if the binary was changed. |
@@ -243,25 +244,6 @@ |
Get(bucket, expected_hash, file_path) |
return True |
-# TODO(aiolos): remove @decorators.Cache for http://crbug.com/459787 |
-@decorators.Cache |
-def GetFilesInDirectoryIfChanged(directory, bucket): |
- """ Scan the directory for .sha1 files, and download them from the given |
- bucket in cloud storage if the local and remote hash don't match or |
- there is no local copy. |
- """ |
- if not os.path.isdir(directory): |
- raise ValueError('Must provide a valid directory.') |
- # Don't allow the root directory to be a serving_dir. |
- if directory == os.path.abspath(os.sep): |
- raise ValueError('Trying to serve root directory from HTTP server.') |
- for dirpath, _, filenames in os.walk(directory): |
- for filename in filenames: |
- path_name, extension = os.path.splitext( |
- os.path.join(dirpath, filename)) |
- if extension != '.sha1': |
- continue |
- GetIfChanged(path_name, bucket) |
def CalculateHash(file_path): |
"""Calculates and returns the hash of the file at file_path.""" |