blob: c072c27955cc3b9381fa06d93b29ce24cc19b07b [file] [log] [blame]
zachr@google.com945708a2013-07-02 19:55:32 +00001/*
2 * Copyright 2013 Google Inc.
3 *
4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file.
6 */
7
8#include "SkBitmap.h"
9#include "SkImageDecoder.h"
10#include "SkOSFile.h"
djsollen@google.comcbbf1ca2013-10-16 18:36:49 +000011#include "SkRunnable.h"
epoger54f1ad82014-07-02 07:43:04 -070012#include "SkSize.h"
zachr@google.com945708a2013-07-02 19:55:32 +000013#include "SkStream.h"
edisonn@google.comc93c8ac2013-07-22 15:24:26 +000014#include "SkTDict.h"
mtklein406654b2014-09-03 15:34:37 -070015#include "SkTaskGroup.h"
zachr@google.com945708a2013-07-02 19:55:32 +000016
stephana21b342d2014-08-13 10:36:06 -070017// from the tools directory for replace_char(...)
18#include "picture_utils.h"
19
zachr@google.com945708a2013-07-02 19:55:32 +000020#include "SkDiffContext.h"
epoger54f1ad82014-07-02 07:43:04 -070021#include "SkImageDiffer.h"
zachr@google.com945708a2013-07-02 19:55:32 +000022#include "skpdiff_util.h"
23
24SkDiffContext::SkDiffContext() {
zachr@google.com945708a2013-07-02 19:55:32 +000025 fDiffers = NULL;
26 fDifferCount = 0;
27}
28
29SkDiffContext::~SkDiffContext() {
bsalomon49f085d2014-09-05 13:34:00 -070030 if (fDiffers) {
zachr@google.com945708a2013-07-02 19:55:32 +000031 SkDELETE_ARRAY(fDiffers);
32 }
33}
34
epoger54f1ad82014-07-02 07:43:04 -070035void SkDiffContext::setAlphaMaskDir(const SkString& path) {
djsollen@google.com513a7bf2013-11-07 19:24:06 +000036 if (!path.isEmpty() && sk_mkdir(path.c_str())) {
epoger54f1ad82014-07-02 07:43:04 -070037 fAlphaMaskDir = path;
38 }
39}
40
41void SkDiffContext::setRgbDiffDir(const SkString& path) {
42 if (!path.isEmpty() && sk_mkdir(path.c_str())) {
43 fRgbDiffDir = path;
44 }
45}
46
47void SkDiffContext::setWhiteDiffDir(const SkString& path) {
48 if (!path.isEmpty() && sk_mkdir(path.c_str())) {
49 fWhiteDiffDir = path;
djsollen@google.com513a7bf2013-11-07 19:24:06 +000050 }
51}
52
stephana21b342d2014-08-13 10:36:06 -070053void SkDiffContext::setLongNames(const bool useLongNames) {
54 longNames = useLongNames;
55}
56
zachr@google.com945708a2013-07-02 19:55:32 +000057void SkDiffContext::setDiffers(const SkTDArray<SkImageDiffer*>& differs) {
58 // Delete whatever the last array of differs was
bsalomon49f085d2014-09-05 13:34:00 -070059 if (fDiffers) {
zachr@google.com945708a2013-07-02 19:55:32 +000060 SkDELETE_ARRAY(fDiffers);
61 fDiffers = NULL;
62 fDifferCount = 0;
63 }
64
65 // Copy over the new differs
66 fDifferCount = differs.count();
67 fDiffers = SkNEW_ARRAY(SkImageDiffer*, fDifferCount);
68 differs.copy(fDiffers);
69}
70
djsollen@google.com513a7bf2013-11-07 19:24:06 +000071static SkString get_common_prefix(const SkString& a, const SkString& b) {
72 const size_t maxPrefixLength = SkTMin(a.size(), b.size());
73 SkASSERT(maxPrefixLength > 0);
74 for (size_t x = 0; x < maxPrefixLength; ++x) {
75 if (a[x] != b[x]) {
76 SkString result;
77 result.set(a.c_str(), x);
78 return result;
79 }
80 }
81 if (a.size() > b.size()) {
82 return b;
83 } else {
84 return a;
85 }
86}
87
stephana21b342d2014-08-13 10:36:06 -070088static SkString get_combined_name(const SkString& a, const SkString& b) {
mtklein406654b2014-09-03 15:34:37 -070089 // Note (stephana): We must keep this function in sync with
stephana21b342d2014-08-13 10:36:06 -070090 // getImageDiffRelativeUrl() in static/loader.js (under rebaseline_server).
91 SkString result = a;
92 result.append("-vs-");
93 result.append(b);
94 sk_tools::replace_char(&result, '.', '_');
95 return result;
96}
97
zachr@google.com945708a2013-07-02 19:55:32 +000098void SkDiffContext::addDiff(const char* baselinePath, const char* testPath) {
99 // Load the images at the paths
100 SkBitmap baselineBitmap;
101 SkBitmap testBitmap;
102 if (!SkImageDecoder::DecodeFile(baselinePath, &baselineBitmap)) {
103 SkDebugf("Failed to load bitmap \"%s\"\n", baselinePath);
104 return;
105 }
106 if (!SkImageDecoder::DecodeFile(testPath, &testBitmap)) {
107 SkDebugf("Failed to load bitmap \"%s\"\n", testPath);
108 return;
109 }
110
111 // Setup a record for this diff
djsollen@google.comefc51b72013-11-12 18:29:17 +0000112 fRecordMutex.acquire();
113 DiffRecord* newRecord = fRecords.addToHead(DiffRecord());
114 fRecordMutex.release();
zachr@google.com945708a2013-07-02 19:55:32 +0000115
djsollen@google.com513a7bf2013-11-07 19:24:06 +0000116 // compute the common name
tfarinaa8e2e152014-07-28 19:26:58 -0700117 SkString baseName = SkOSPath::Basename(baselinePath);
118 SkString testName = SkOSPath::Basename(testPath);
stephana21b342d2014-08-13 10:36:06 -0700119
120 if (longNames) {
121 newRecord->fCommonName = get_combined_name(baseName, testName);
122 } else {
123 newRecord->fCommonName = get_common_prefix(baseName, testName);
124 }
125 newRecord->fCommonName.append(".png");
djsollen@google.com513a7bf2013-11-07 19:24:06 +0000126
djsollen@google.comefc51b72013-11-12 18:29:17 +0000127 newRecord->fBaselinePath = baselinePath;
128 newRecord->fTestPath = testPath;
epoger54f1ad82014-07-02 07:43:04 -0700129 newRecord->fSize = SkISize::Make(baselineBitmap.width(), baselineBitmap.height());
djsollen@google.comefc51b72013-11-12 18:29:17 +0000130
epoger54f1ad82014-07-02 07:43:04 -0700131 // only generate diff images if we have a place to store them
132 SkImageDiffer::BitmapsToCreate bitmapsToCreate;
133 bitmapsToCreate.alphaMask = !fAlphaMaskDir.isEmpty();
134 bitmapsToCreate.rgbDiff = !fRgbDiffDir.isEmpty();
135 bitmapsToCreate.whiteDiff = !fWhiteDiffDir.isEmpty();
djsollen@google.com513a7bf2013-11-07 19:24:06 +0000136
zachr@google.com945708a2013-07-02 19:55:32 +0000137 // Perform each diff
138 for (int differIndex = 0; differIndex < fDifferCount; differIndex++) {
139 SkImageDiffer* differ = fDiffers[differIndex];
djsollen@google.comefc51b72013-11-12 18:29:17 +0000140
141 // Copy the results into data for this record
142 DiffData& diffData = newRecord->fDiffs.push_back();
143 diffData.fDiffName = differ->getName();
144
epoger54f1ad82014-07-02 07:43:04 -0700145 if (!differ->diff(&baselineBitmap, &testBitmap, bitmapsToCreate, &diffData.fResult)) {
146 // if the diff failed, record -1 as the result
147 // TODO(djsollen): Record more detailed information about exactly what failed.
148 // (Image dimension mismatch? etc.) See http://skbug.com/2710 ('make skpdiff
149 // report more detail when it fails to compare two images')
djsollen@google.com27d7ede2014-02-11 16:29:39 +0000150 diffData.fResult.result = -1;
djsollen@google.comefc51b72013-11-12 18:29:17 +0000151 continue;
djsollen@google.com513a7bf2013-11-07 19:24:06 +0000152 }
zachr@google.com945708a2013-07-02 19:55:32 +0000153
epoger54f1ad82014-07-02 07:43:04 -0700154 if (bitmapsToCreate.alphaMask
djsollen@google.comefc51b72013-11-12 18:29:17 +0000155 && SkImageDiffer::RESULT_CORRECT != diffData.fResult.result
156 && !diffData.fResult.poiAlphaMask.empty()
157 && !newRecord->fCommonName.isEmpty()) {
zachr@google.com945708a2013-07-02 19:55:32 +0000158
tfarinaa8e2e152014-07-28 19:26:58 -0700159 newRecord->fAlphaMaskPath = SkOSPath::Join(fAlphaMaskDir.c_str(),
160 newRecord->fCommonName.c_str());
zachr@google.com945708a2013-07-02 19:55:32 +0000161
djsollen@google.comefc51b72013-11-12 18:29:17 +0000162 // compute the image diff and output it
163 SkBitmap copy;
commit-bot@chromium.org28fcae22014-04-11 17:15:40 +0000164 diffData.fResult.poiAlphaMask.copyTo(&copy, kN32_SkColorType);
epoger54f1ad82014-07-02 07:43:04 -0700165 SkImageEncoder::EncodeFile(newRecord->fAlphaMaskPath.c_str(), copy,
djsollen@google.comefc51b72013-11-12 18:29:17 +0000166 SkImageEncoder::kPNG_Type, 100);
zachr@google.com945708a2013-07-02 19:55:32 +0000167
djsollen@google.comefc51b72013-11-12 18:29:17 +0000168 // cleanup the existing bitmap to free up resources;
169 diffData.fResult.poiAlphaMask.reset();
djsollen@google.com513a7bf2013-11-07 19:24:06 +0000170
epoger54f1ad82014-07-02 07:43:04 -0700171 bitmapsToCreate.alphaMask = false;
172 }
173
174 if (bitmapsToCreate.rgbDiff
175 && SkImageDiffer::RESULT_CORRECT != diffData.fResult.result
176 && !diffData.fResult.rgbDiffBitmap.empty()
177 && !newRecord->fCommonName.isEmpty()) {
178 // TODO(djsollen): Rather than taking the max r/g/b diffs that come back from
179 // a particular differ and storing them as toplevel fields within
180 // newRecord, we should extend outputRecords() to report optional
181 // fields for each differ (not just "result" and "pointsOfInterest").
182 // See http://skbug.com/2712 ('allow skpdiff to report different sets
183 // of result fields for different comparison algorithms')
184 newRecord->fMaxRedDiff = diffData.fResult.maxRedDiff;
185 newRecord->fMaxGreenDiff = diffData.fResult.maxGreenDiff;
186 newRecord->fMaxBlueDiff = diffData.fResult.maxBlueDiff;
187
tfarinaa8e2e152014-07-28 19:26:58 -0700188 newRecord->fRgbDiffPath = SkOSPath::Join(fRgbDiffDir.c_str(),
189 newRecord->fCommonName.c_str());
epoger54f1ad82014-07-02 07:43:04 -0700190 SkImageEncoder::EncodeFile(newRecord->fRgbDiffPath.c_str(),
191 diffData.fResult.rgbDiffBitmap,
192 SkImageEncoder::kPNG_Type, 100);
193 diffData.fResult.rgbDiffBitmap.reset();
194 bitmapsToCreate.rgbDiff = false;
195 }
196
197 if (bitmapsToCreate.whiteDiff
198 && SkImageDiffer::RESULT_CORRECT != diffData.fResult.result
199 && !diffData.fResult.whiteDiffBitmap.empty()
200 && !newRecord->fCommonName.isEmpty()) {
tfarinaa8e2e152014-07-28 19:26:58 -0700201 newRecord->fWhiteDiffPath = SkOSPath::Join(fWhiteDiffDir.c_str(),
202 newRecord->fCommonName.c_str());
epoger54f1ad82014-07-02 07:43:04 -0700203 SkImageEncoder::EncodeFile(newRecord->fWhiteDiffPath.c_str(),
204 diffData.fResult.whiteDiffBitmap,
205 SkImageEncoder::kPNG_Type, 100);
206 diffData.fResult.whiteDiffBitmap.reset();
207 bitmapsToCreate.whiteDiff = false;
zachr@google.com945708a2013-07-02 19:55:32 +0000208 }
209 }
210}
211
djsollen@google.comcbbf1ca2013-10-16 18:36:49 +0000212class SkThreadedDiff : public SkRunnable {
213public:
214 SkThreadedDiff() : fDiffContext(NULL) { }
215
216 void setup(SkDiffContext* diffContext, const SkString& baselinePath, const SkString& testPath) {
217 fDiffContext = diffContext;
218 fBaselinePath = baselinePath;
219 fTestPath = testPath;
220 }
221
mtklein72c9faa2015-01-09 10:06:39 -0800222 void run() SK_OVERRIDE {
djsollen@google.comcbbf1ca2013-10-16 18:36:49 +0000223 fDiffContext->addDiff(fBaselinePath.c_str(), fTestPath.c_str());
224 }
225
226private:
227 SkDiffContext* fDiffContext;
228 SkString fBaselinePath;
229 SkString fTestPath;
230};
zachr@google.com945708a2013-07-02 19:55:32 +0000231
232void SkDiffContext::diffDirectories(const char baselinePath[], const char testPath[]) {
233 // Get the files in the baseline, we will then look for those inside the test path
234 SkTArray<SkString> baselineEntries;
235 if (!get_directory(baselinePath, &baselineEntries)) {
236 SkDebugf("Unable to open path \"%s\"\n", baselinePath);
237 return;
238 }
239
mtklein406654b2014-09-03 15:34:37 -0700240 SkTaskGroup tg;
djsollen@google.comcbbf1ca2013-10-16 18:36:49 +0000241 SkTArray<SkThreadedDiff> runnableDiffs;
242 runnableDiffs.reset(baselineEntries.count());
243
244 for (int x = 0; x < baselineEntries.count(); x++) {
245 const char* baseFilename = baselineEntries[x].c_str();
zachr@google.com945708a2013-07-02 19:55:32 +0000246
247 // Find the real location of each file to compare
tfarinaa8e2e152014-07-28 19:26:58 -0700248 SkString baselineFile = SkOSPath::Join(baselinePath, baseFilename);
249 SkString testFile = SkOSPath::Join(testPath, baseFilename);
zachr@google.com945708a2013-07-02 19:55:32 +0000250
251 // Check that the test file exists and is a file
252 if (sk_exists(testFile.c_str()) && !sk_isdir(testFile.c_str())) {
253 // Queue up the comparison with the differ
djsollen@google.comcbbf1ca2013-10-16 18:36:49 +0000254 runnableDiffs[x].setup(this, baselineFile, testFile);
mtklein406654b2014-09-03 15:34:37 -0700255 tg.add(&runnableDiffs[x]);
zachr@google.com945708a2013-07-02 19:55:32 +0000256 } else {
257 SkDebugf("Baseline file \"%s\" has no corresponding test file\n", baselineFile.c_str());
258 }
259 }
260}
261
262
263void SkDiffContext::diffPatterns(const char baselinePattern[], const char testPattern[]) {
264 // Get the files in the baseline and test patterns. Because they are in sorted order, it's easy
265 // to find corresponding images by matching entry indices.
266
267 SkTArray<SkString> baselineEntries;
268 if (!glob_files(baselinePattern, &baselineEntries)) {
269 SkDebugf("Unable to get pattern \"%s\"\n", baselinePattern);
270 return;
271 }
272
273 SkTArray<SkString> testEntries;
274 if (!glob_files(testPattern, &testEntries)) {
275 SkDebugf("Unable to get pattern \"%s\"\n", testPattern);
276 return;
277 }
278
279 if (baselineEntries.count() != testEntries.count()) {
280 SkDebugf("Baseline and test patterns do not yield corresponding number of files\n");
281 return;
282 }
283
mtklein406654b2014-09-03 15:34:37 -0700284 SkTaskGroup tg;
djsollen@google.comcbbf1ca2013-10-16 18:36:49 +0000285 SkTArray<SkThreadedDiff> runnableDiffs;
286 runnableDiffs.reset(baselineEntries.count());
zachr@google.com945708a2013-07-02 19:55:32 +0000287
djsollen@google.comcbbf1ca2013-10-16 18:36:49 +0000288 for (int x = 0; x < baselineEntries.count(); x++) {
289 runnableDiffs[x].setup(this, baselineEntries[x], testEntries[x]);
mtklein406654b2014-09-03 15:34:37 -0700290 tg.add(&runnableDiffs[x]);
zachr@google.com945708a2013-07-02 19:55:32 +0000291 }
stephana7260d722014-09-05 13:21:42 -0700292 tg.wait();
zachr@google.com945708a2013-07-02 19:55:32 +0000293}
294
zachr@google.coma95959c2013-07-08 15:04:45 +0000295void SkDiffContext::outputRecords(SkWStream& stream, bool useJSONP) {
djsollen@google.comefc51b72013-11-12 18:29:17 +0000296 SkTLList<DiffRecord>::Iter iter(fRecords, SkTLList<DiffRecord>::Iter::kHead_IterStart);
297 DiffRecord* currentRecord = iter.get();
298
zachr@google.coma95959c2013-07-08 15:04:45 +0000299 if (useJSONP) {
300 stream.writeText("var SkPDiffRecords = {\n");
zachr@google.com55173f22013-07-25 17:22:58 +0000301 } else {
zachr@google.coma95959c2013-07-08 15:04:45 +0000302 stream.writeText("{\n");
303 }
epoger54f1ad82014-07-02 07:43:04 -0700304
305 // TODO(djsollen): Would it be better to use the jsoncpp library to write out the JSON?
306 // This manual approach is probably more efficient, but it sure is ugly.
307 // See http://skbug.com/2713 ('make skpdiff use jsoncpp library to write out
308 // JSON output, instead of manual writeText() calls?')
zachr@google.com945708a2013-07-02 19:55:32 +0000309 stream.writeText(" \"records\": [\n");
bsalomon49f085d2014-09-05 13:34:00 -0700310 while (currentRecord) {
zachr@google.com945708a2013-07-02 19:55:32 +0000311 stream.writeText(" {\n");
312
zachr@google.coma479aa12013-08-02 15:54:30 +0000313 SkString baselineAbsPath = get_absolute_path(currentRecord->fBaselinePath);
314 SkString testAbsPath = get_absolute_path(currentRecord->fTestPath);
315
djsollen@google.com1e391b52013-10-16 15:00:11 +0000316 stream.writeText(" \"commonName\": \"");
djsollen@google.com513a7bf2013-11-07 19:24:06 +0000317 stream.writeText(currentRecord->fCommonName.c_str());
djsollen@google.com1e391b52013-10-16 15:00:11 +0000318 stream.writeText("\",\n");
319
djsollen@google.comcbbf1ca2013-10-16 18:36:49 +0000320 stream.writeText(" \"differencePath\": \"");
epoger54f1ad82014-07-02 07:43:04 -0700321 stream.writeText(get_absolute_path(currentRecord->fAlphaMaskPath).c_str());
322 stream.writeText("\",\n");
323
324 stream.writeText(" \"rgbDiffPath\": \"");
325 stream.writeText(get_absolute_path(currentRecord->fRgbDiffPath).c_str());
326 stream.writeText("\",\n");
327
328 stream.writeText(" \"whiteDiffPath\": \"");
329 stream.writeText(get_absolute_path(currentRecord->fWhiteDiffPath).c_str());
djsollen@google.comcbbf1ca2013-10-16 18:36:49 +0000330 stream.writeText("\",\n");
331
zachr@google.com945708a2013-07-02 19:55:32 +0000332 stream.writeText(" \"baselinePath\": \"");
zachr@google.coma479aa12013-08-02 15:54:30 +0000333 stream.writeText(baselineAbsPath.c_str());
zachr@google.com945708a2013-07-02 19:55:32 +0000334 stream.writeText("\",\n");
335
336 stream.writeText(" \"testPath\": \"");
zachr@google.coma479aa12013-08-02 15:54:30 +0000337 stream.writeText(testAbsPath.c_str());
zachr@google.com945708a2013-07-02 19:55:32 +0000338 stream.writeText("\",\n");
339
epoger54f1ad82014-07-02 07:43:04 -0700340 stream.writeText(" \"width\": ");
341 stream.writeDecAsText(currentRecord->fSize.width());
342 stream.writeText(",\n");
343 stream.writeText(" \"height\": ");
344 stream.writeDecAsText(currentRecord->fSize.height());
345 stream.writeText(",\n");
346
347 stream.writeText(" \"maxRedDiff\": ");
348 stream.writeDecAsText(currentRecord->fMaxRedDiff);
349 stream.writeText(",\n");
350 stream.writeText(" \"maxGreenDiff\": ");
351 stream.writeDecAsText(currentRecord->fMaxGreenDiff);
352 stream.writeText(",\n");
353 stream.writeText(" \"maxBlueDiff\": ");
354 stream.writeDecAsText(currentRecord->fMaxBlueDiff);
355 stream.writeText(",\n");
356
zachr@google.com945708a2013-07-02 19:55:32 +0000357 stream.writeText(" \"diffs\": [\n");
358 for (int diffIndex = 0; diffIndex < currentRecord->fDiffs.count(); diffIndex++) {
359 DiffData& data = currentRecord->fDiffs[diffIndex];
360 stream.writeText(" {\n");
361
362 stream.writeText(" \"differName\": \"");
363 stream.writeText(data.fDiffName);
364 stream.writeText("\",\n");
365
366 stream.writeText(" \"result\": ");
djsollen@google.comefc51b72013-11-12 18:29:17 +0000367 stream.writeScalarAsText((SkScalar)data.fResult.result);
zachr@google.com945708a2013-07-02 19:55:32 +0000368 stream.writeText(",\n");
369
djsollen@google.comefc51b72013-11-12 18:29:17 +0000370 stream.writeText(" \"pointsOfInterest\": ");
371 stream.writeDecAsText(data.fResult.poiCount);
372 stream.writeText("\n");
zachr@google.com945708a2013-07-02 19:55:32 +0000373
zachr@google.com945708a2013-07-02 19:55:32 +0000374 stream.writeText(" }");
375
376 // JSON does not allow trailing commas
zachr@google.com55173f22013-07-25 17:22:58 +0000377 if (diffIndex + 1 < currentRecord->fDiffs.count()) {
zachr@google.com945708a2013-07-02 19:55:32 +0000378 stream.writeText(",");
379 }
380 stream.writeText(" \n");
381 }
382 stream.writeText(" ]\n");
383
384 stream.writeText(" }");
385
djsollen@google.comefc51b72013-11-12 18:29:17 +0000386 currentRecord = iter.next();
387
zachr@google.com945708a2013-07-02 19:55:32 +0000388 // JSON does not allow trailing commas
bsalomon49f085d2014-09-05 13:34:00 -0700389 if (currentRecord) {
zachr@google.com945708a2013-07-02 19:55:32 +0000390 stream.writeText(",");
391 }
392 stream.writeText("\n");
zachr@google.com945708a2013-07-02 19:55:32 +0000393 }
394 stream.writeText(" ]\n");
zachr@google.coma95959c2013-07-08 15:04:45 +0000395 if (useJSONP) {
396 stream.writeText("};\n");
zachr@google.com55173f22013-07-25 17:22:58 +0000397 } else {
zachr@google.coma95959c2013-07-08 15:04:45 +0000398 stream.writeText("}\n");
399 }
zachr@google.com945708a2013-07-02 19:55:32 +0000400}
edisonn@google.comc93c8ac2013-07-22 15:24:26 +0000401
402void SkDiffContext::outputCsv(SkWStream& stream) {
403 SkTDict<int> columns(2);
404 int cntColumns = 0;
405
406 stream.writeText("key");
407
djsollen@google.comefc51b72013-11-12 18:29:17 +0000408 SkTLList<DiffRecord>::Iter iter(fRecords, SkTLList<DiffRecord>::Iter::kHead_IterStart);
409 DiffRecord* currentRecord = iter.get();
edisonn@google.comc93c8ac2013-07-22 15:24:26 +0000410
411 // Write CSV header and create a dictionary of all columns.
bsalomon49f085d2014-09-05 13:34:00 -0700412 while (currentRecord) {
edisonn@google.comc93c8ac2013-07-22 15:24:26 +0000413 for (int diffIndex = 0; diffIndex < currentRecord->fDiffs.count(); diffIndex++) {
414 DiffData& data = currentRecord->fDiffs[diffIndex];
415 if (!columns.find(data.fDiffName)) {
416 columns.set(data.fDiffName, cntColumns);
417 stream.writeText(", ");
418 stream.writeText(data.fDiffName);
419 cntColumns++;
420 }
421 }
djsollen@google.comefc51b72013-11-12 18:29:17 +0000422 currentRecord = iter.next();
edisonn@google.comc93c8ac2013-07-22 15:24:26 +0000423 }
424 stream.writeText("\n");
425
426 double values[100];
427 SkASSERT(cntColumns < 100); // Make the array larger, if we ever have so many diff types.
428
djsollen@google.comefc51b72013-11-12 18:29:17 +0000429 SkTLList<DiffRecord>::Iter iter2(fRecords, SkTLList<DiffRecord>::Iter::kHead_IterStart);
430 currentRecord = iter2.get();
bsalomon49f085d2014-09-05 13:34:00 -0700431 while (currentRecord) {
edisonn@google.comc93c8ac2013-07-22 15:24:26 +0000432 for (int i = 0; i < cntColumns; i++) {
433 values[i] = -1;
434 }
435
436 for (int diffIndex = 0; diffIndex < currentRecord->fDiffs.count(); diffIndex++) {
437 DiffData& data = currentRecord->fDiffs[diffIndex];
438 int index = -1;
439 SkAssertResult(columns.find(data.fDiffName, &index));
440 SkASSERT(index >= 0 && index < cntColumns);
djsollen@google.comefc51b72013-11-12 18:29:17 +0000441 values[index] = data.fResult.result;
edisonn@google.comc93c8ac2013-07-22 15:24:26 +0000442 }
443
444 const char* filename = currentRecord->fBaselinePath.c_str() +
445 strlen(currentRecord->fBaselinePath.c_str()) - 1;
446 while (filename > currentRecord->fBaselinePath.c_str() && *(filename - 1) != '/') {
447 filename--;
448 }
449
450 stream.writeText(filename);
451
452 for (int i = 0; i < cntColumns; i++) {
453 SkString str;
454 str.printf(", %f", values[i]);
455 stream.writeText(str.c_str());
456 }
457 stream.writeText("\n");
458
djsollen@google.comefc51b72013-11-12 18:29:17 +0000459 currentRecord = iter2.next();
edisonn@google.comc93c8ac2013-07-22 15:24:26 +0000460 }
461}