Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(31)

Side by Side Diff: gm/gm_json.py

Issue 19112002: svndiff.py: add ability to compare before-and-after JSON files, not just raw images (Closed) Base URL: http://skia.googlecode.com/svn/trunk/
Patch Set: sync_to_r10112 Created 7 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | tools/jsondiff.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright (c) 2013 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """Schema of the JSON summary file written out by the GM tool. 6 """Schema of the JSON summary file written out by the GM tool.
7 7
8 This must be kept in sync with the kJsonKey_ constants in gm_expectations.cpp ! 8 This must be kept in sync with the kJsonKey_ constants in gm_expectations.cpp !
9 """ 9 """
10 10
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
42 # expectations, and the test will pass if any one of them is matched. 42 # expectations, and the test will pass if any one of them is matched.
43 JSONKEY_EXPECTEDRESULTS_ALLOWEDDIGESTS = 'allowed-digests' 43 JSONKEY_EXPECTEDRESULTS_ALLOWEDDIGESTS = 'allowed-digests'
44 # If IGNOREFAILURE is set to True, a failure of this test will be reported 44 # If IGNOREFAILURE is set to True, a failure of this test will be reported
45 # within the FAILUREIGNORED section (thus NOT causing the buildbots to go red) 45 # within the FAILUREIGNORED section (thus NOT causing the buildbots to go red)
46 # rather than the FAILED section (which WOULD cause the buildbots to go red). 46 # rather than the FAILED section (which WOULD cause the buildbots to go red).
47 JSONKEY_EXPECTEDRESULTS_IGNOREFAILURE = 'ignore-failure' 47 JSONKEY_EXPECTEDRESULTS_IGNOREFAILURE = 'ignore-failure'
48 48
49 # Allowed hash types for test expectations. 49 # Allowed hash types for test expectations.
50 JSONKEY_HASHTYPE_BITMAP_64BITMD5 = 'bitmap-64bitMD5' 50 JSONKEY_HASHTYPE_BITMAP_64BITMD5 = 'bitmap-64bitMD5'
51 51
52 # Root directory where the buildbots store their actually-generated images...
53 # as a publicly readable HTTP URL:
54 GM_ACTUALS_ROOT_HTTP_URL = (
55 'http://chromium-skia-gm.commondatastorage.googleapis.com/gm')
56 # as a GS URL that allows credential-protected write access:
57 GM_ACTUALS_ROOT_GS_URL = 'gs://chromium-skia-gm/gm'
58
59 # Pattern used to assemble each image's filename
60 IMAGE_FILENAME_PATTERN = '(\S+)_(\S+).png' # matches (testname, config)
61
62 def CreateGmActualUrl(test_name, hash_type, hash_digest,
63 gm_actuals_root_url=GM_ACTUALS_ROOT_HTTP_URL):
64 """Return the URL we can use to download a particular version of
65 the actually-generated image for this particular GM test.
66
67 test_name: name of the test, e.g. 'perlinnoise'
68 hash_type: string indicating the hash type used to generate hash_digest,
69 e.g. JSONKEY_HASHTYPE_BITMAP_64BITMD5
70 hash_digest: the hash digest of the image to retrieve
71 gm_actuals_root_url: root url where actual images are stored
72 """
73 return '%s/%s/%s/%s.png' % (gm_actuals_root_url, hash_type, test_name,
74 hash_digest)
75
52 def LoadFromString(file_contents): 76 def LoadFromString(file_contents):
53 """Loads the JSON summary written out by the GM tool. 77 """Loads the JSON summary written out by the GM tool.
54 Returns a dictionary keyed by the values listed as JSONKEY_ constants 78 Returns a dictionary keyed by the values listed as JSONKEY_ constants
55 above.""" 79 above."""
56 # TODO(epoger): we should add a version number to the JSON file to ensure 80 # TODO(epoger): we should add a version number to the JSON file to ensure
57 # that the writer and reader agree on the schema (raising an exception 81 # that the writer and reader agree on the schema (raising an exception
58 # otherwise). 82 # otherwise).
59 json_dict = json.loads(file_contents) 83 json_dict = json.loads(file_contents)
60 return json_dict 84 return json_dict
61 85
62 def LoadFromFile(file_path): 86 def LoadFromFile(file_path):
63 """Loads the JSON summary written out by the GM tool. 87 """Loads the JSON summary written out by the GM tool.
64 Returns a dictionary keyed by the values listed as JSONKEY_ constants 88 Returns a dictionary keyed by the values listed as JSONKEY_ constants
65 above.""" 89 above."""
66 file_contents = open(file_path, 'r').read() 90 file_contents = open(file_path, 'r').read()
67 return LoadFromString(file_contents) 91 return LoadFromString(file_contents)
68 92
69 def WriteToFile(json_dict, file_path): 93 def WriteToFile(json_dict, file_path):
70 """Writes the JSON summary in json_dict out to file_path.""" 94 """Writes the JSON summary in json_dict out to file_path."""
71 with open(file_path, 'w') as outfile: 95 with open(file_path, 'w') as outfile:
72 json.dump(json_dict, outfile, sort_keys=True, indent=2) 96 json.dump(json_dict, outfile, sort_keys=True, indent=2)
OLDNEW
« no previous file with comments | « no previous file | tools/jsondiff.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698