blob: 5add0df515c23d0e7737c59a3f0d34b8977044b7 [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) {
halcanary385fe4d2015-08-26 13:07:48 -070031 delete[] fDiffers;
zachr@google.com945708a2013-07-02 19:55:32 +000032 }
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) {
halcanary385fe4d2015-08-26 13:07:48 -070060 delete[] fDiffers;
zachr@google.com945708a2013-07-02 19:55:32 +000061 fDiffers = NULL;
62 fDifferCount = 0;
63 }
64
65 // Copy over the new differs
66 fDifferCount = differs.count();
halcanary385fe4d2015-08-26 13:07:48 -070067 fDiffers = new SkImageDiffer* [fDifferCount];
zachr@google.com945708a2013-07-02 19:55:32 +000068 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
zachr@google.com945708a2013-07-02 19:55:32 +0000212void SkDiffContext::diffDirectories(const char baselinePath[], const char testPath[]) {
213 // Get the files in the baseline, we will then look for those inside the test path
214 SkTArray<SkString> baselineEntries;
215 if (!get_directory(baselinePath, &baselineEntries)) {
216 SkDebugf("Unable to open path \"%s\"\n", baselinePath);
217 return;
218 }
219
mtklein00b621c2015-06-17 15:26:15 -0700220 sk_parallel_for(baselineEntries.count(), [&](int i) {
221 const char* baseFilename = baselineEntries[i].c_str();
zachr@google.com945708a2013-07-02 19:55:32 +0000222
223 // Find the real location of each file to compare
tfarinaa8e2e152014-07-28 19:26:58 -0700224 SkString baselineFile = SkOSPath::Join(baselinePath, baseFilename);
225 SkString testFile = SkOSPath::Join(testPath, baseFilename);
zachr@google.com945708a2013-07-02 19:55:32 +0000226
227 // Check that the test file exists and is a file
228 if (sk_exists(testFile.c_str()) && !sk_isdir(testFile.c_str())) {
mtklein00b621c2015-06-17 15:26:15 -0700229 this->addDiff(baselineFile.c_str(), testFile.c_str());
zachr@google.com945708a2013-07-02 19:55:32 +0000230 } else {
231 SkDebugf("Baseline file \"%s\" has no corresponding test file\n", baselineFile.c_str());
232 }
mtklein00b621c2015-06-17 15:26:15 -0700233 });
zachr@google.com945708a2013-07-02 19:55:32 +0000234}
235
236
237void SkDiffContext::diffPatterns(const char baselinePattern[], const char testPattern[]) {
238 // Get the files in the baseline and test patterns. Because they are in sorted order, it's easy
239 // to find corresponding images by matching entry indices.
240
241 SkTArray<SkString> baselineEntries;
242 if (!glob_files(baselinePattern, &baselineEntries)) {
243 SkDebugf("Unable to get pattern \"%s\"\n", baselinePattern);
244 return;
245 }
246
247 SkTArray<SkString> testEntries;
248 if (!glob_files(testPattern, &testEntries)) {
249 SkDebugf("Unable to get pattern \"%s\"\n", testPattern);
250 return;
251 }
252
253 if (baselineEntries.count() != testEntries.count()) {
254 SkDebugf("Baseline and test patterns do not yield corresponding number of files\n");
255 return;
256 }
257
mtklein00b621c2015-06-17 15:26:15 -0700258 sk_parallel_for(baselineEntries.count(), [&](int i) {
259 this->addDiff(baselineEntries[i].c_str(), testEntries[i].c_str());
260 });
zachr@google.com945708a2013-07-02 19:55:32 +0000261}
262
zachr@google.coma95959c2013-07-08 15:04:45 +0000263void SkDiffContext::outputRecords(SkWStream& stream, bool useJSONP) {
djsollen@google.comefc51b72013-11-12 18:29:17 +0000264 SkTLList<DiffRecord>::Iter iter(fRecords, SkTLList<DiffRecord>::Iter::kHead_IterStart);
265 DiffRecord* currentRecord = iter.get();
266
zachr@google.coma95959c2013-07-08 15:04:45 +0000267 if (useJSONP) {
268 stream.writeText("var SkPDiffRecords = {\n");
zachr@google.com55173f22013-07-25 17:22:58 +0000269 } else {
zachr@google.coma95959c2013-07-08 15:04:45 +0000270 stream.writeText("{\n");
271 }
epoger54f1ad82014-07-02 07:43:04 -0700272
273 // TODO(djsollen): Would it be better to use the jsoncpp library to write out the JSON?
274 // This manual approach is probably more efficient, but it sure is ugly.
275 // See http://skbug.com/2713 ('make skpdiff use jsoncpp library to write out
276 // JSON output, instead of manual writeText() calls?')
zachr@google.com945708a2013-07-02 19:55:32 +0000277 stream.writeText(" \"records\": [\n");
bsalomon49f085d2014-09-05 13:34:00 -0700278 while (currentRecord) {
zachr@google.com945708a2013-07-02 19:55:32 +0000279 stream.writeText(" {\n");
280
zachr@google.coma479aa12013-08-02 15:54:30 +0000281 SkString baselineAbsPath = get_absolute_path(currentRecord->fBaselinePath);
282 SkString testAbsPath = get_absolute_path(currentRecord->fTestPath);
283
djsollen@google.com1e391b52013-10-16 15:00:11 +0000284 stream.writeText(" \"commonName\": \"");
djsollen@google.com513a7bf2013-11-07 19:24:06 +0000285 stream.writeText(currentRecord->fCommonName.c_str());
djsollen@google.com1e391b52013-10-16 15:00:11 +0000286 stream.writeText("\",\n");
287
djsollen@google.comcbbf1ca2013-10-16 18:36:49 +0000288 stream.writeText(" \"differencePath\": \"");
epoger54f1ad82014-07-02 07:43:04 -0700289 stream.writeText(get_absolute_path(currentRecord->fAlphaMaskPath).c_str());
290 stream.writeText("\",\n");
291
292 stream.writeText(" \"rgbDiffPath\": \"");
293 stream.writeText(get_absolute_path(currentRecord->fRgbDiffPath).c_str());
294 stream.writeText("\",\n");
295
296 stream.writeText(" \"whiteDiffPath\": \"");
297 stream.writeText(get_absolute_path(currentRecord->fWhiteDiffPath).c_str());
djsollen@google.comcbbf1ca2013-10-16 18:36:49 +0000298 stream.writeText("\",\n");
299
zachr@google.com945708a2013-07-02 19:55:32 +0000300 stream.writeText(" \"baselinePath\": \"");
zachr@google.coma479aa12013-08-02 15:54:30 +0000301 stream.writeText(baselineAbsPath.c_str());
zachr@google.com945708a2013-07-02 19:55:32 +0000302 stream.writeText("\",\n");
303
304 stream.writeText(" \"testPath\": \"");
zachr@google.coma479aa12013-08-02 15:54:30 +0000305 stream.writeText(testAbsPath.c_str());
zachr@google.com945708a2013-07-02 19:55:32 +0000306 stream.writeText("\",\n");
307
epoger54f1ad82014-07-02 07:43:04 -0700308 stream.writeText(" \"width\": ");
309 stream.writeDecAsText(currentRecord->fSize.width());
310 stream.writeText(",\n");
311 stream.writeText(" \"height\": ");
312 stream.writeDecAsText(currentRecord->fSize.height());
313 stream.writeText(",\n");
314
315 stream.writeText(" \"maxRedDiff\": ");
316 stream.writeDecAsText(currentRecord->fMaxRedDiff);
317 stream.writeText(",\n");
318 stream.writeText(" \"maxGreenDiff\": ");
319 stream.writeDecAsText(currentRecord->fMaxGreenDiff);
320 stream.writeText(",\n");
321 stream.writeText(" \"maxBlueDiff\": ");
322 stream.writeDecAsText(currentRecord->fMaxBlueDiff);
323 stream.writeText(",\n");
324
zachr@google.com945708a2013-07-02 19:55:32 +0000325 stream.writeText(" \"diffs\": [\n");
326 for (int diffIndex = 0; diffIndex < currentRecord->fDiffs.count(); diffIndex++) {
327 DiffData& data = currentRecord->fDiffs[diffIndex];
328 stream.writeText(" {\n");
329
330 stream.writeText(" \"differName\": \"");
331 stream.writeText(data.fDiffName);
332 stream.writeText("\",\n");
333
334 stream.writeText(" \"result\": ");
djsollen@google.comefc51b72013-11-12 18:29:17 +0000335 stream.writeScalarAsText((SkScalar)data.fResult.result);
zachr@google.com945708a2013-07-02 19:55:32 +0000336 stream.writeText(",\n");
337
djsollen@google.comefc51b72013-11-12 18:29:17 +0000338 stream.writeText(" \"pointsOfInterest\": ");
339 stream.writeDecAsText(data.fResult.poiCount);
340 stream.writeText("\n");
zachr@google.com945708a2013-07-02 19:55:32 +0000341
zachr@google.com945708a2013-07-02 19:55:32 +0000342 stream.writeText(" }");
343
344 // JSON does not allow trailing commas
zachr@google.com55173f22013-07-25 17:22:58 +0000345 if (diffIndex + 1 < currentRecord->fDiffs.count()) {
zachr@google.com945708a2013-07-02 19:55:32 +0000346 stream.writeText(",");
347 }
348 stream.writeText(" \n");
349 }
350 stream.writeText(" ]\n");
351
352 stream.writeText(" }");
353
djsollen@google.comefc51b72013-11-12 18:29:17 +0000354 currentRecord = iter.next();
355
zachr@google.com945708a2013-07-02 19:55:32 +0000356 // JSON does not allow trailing commas
bsalomon49f085d2014-09-05 13:34:00 -0700357 if (currentRecord) {
zachr@google.com945708a2013-07-02 19:55:32 +0000358 stream.writeText(",");
359 }
360 stream.writeText("\n");
zachr@google.com945708a2013-07-02 19:55:32 +0000361 }
362 stream.writeText(" ]\n");
zachr@google.coma95959c2013-07-08 15:04:45 +0000363 if (useJSONP) {
364 stream.writeText("};\n");
zachr@google.com55173f22013-07-25 17:22:58 +0000365 } else {
zachr@google.coma95959c2013-07-08 15:04:45 +0000366 stream.writeText("}\n");
367 }
zachr@google.com945708a2013-07-02 19:55:32 +0000368}
edisonn@google.comc93c8ac2013-07-22 15:24:26 +0000369
370void SkDiffContext::outputCsv(SkWStream& stream) {
371 SkTDict<int> columns(2);
372 int cntColumns = 0;
373
374 stream.writeText("key");
375
djsollen@google.comefc51b72013-11-12 18:29:17 +0000376 SkTLList<DiffRecord>::Iter iter(fRecords, SkTLList<DiffRecord>::Iter::kHead_IterStart);
377 DiffRecord* currentRecord = iter.get();
edisonn@google.comc93c8ac2013-07-22 15:24:26 +0000378
379 // Write CSV header and create a dictionary of all columns.
bsalomon49f085d2014-09-05 13:34:00 -0700380 while (currentRecord) {
edisonn@google.comc93c8ac2013-07-22 15:24:26 +0000381 for (int diffIndex = 0; diffIndex < currentRecord->fDiffs.count(); diffIndex++) {
382 DiffData& data = currentRecord->fDiffs[diffIndex];
383 if (!columns.find(data.fDiffName)) {
384 columns.set(data.fDiffName, cntColumns);
385 stream.writeText(", ");
386 stream.writeText(data.fDiffName);
387 cntColumns++;
388 }
389 }
djsollen@google.comefc51b72013-11-12 18:29:17 +0000390 currentRecord = iter.next();
edisonn@google.comc93c8ac2013-07-22 15:24:26 +0000391 }
392 stream.writeText("\n");
393
394 double values[100];
395 SkASSERT(cntColumns < 100); // Make the array larger, if we ever have so many diff types.
396
djsollen@google.comefc51b72013-11-12 18:29:17 +0000397 SkTLList<DiffRecord>::Iter iter2(fRecords, SkTLList<DiffRecord>::Iter::kHead_IterStart);
398 currentRecord = iter2.get();
bsalomon49f085d2014-09-05 13:34:00 -0700399 while (currentRecord) {
edisonn@google.comc93c8ac2013-07-22 15:24:26 +0000400 for (int i = 0; i < cntColumns; i++) {
401 values[i] = -1;
402 }
403
404 for (int diffIndex = 0; diffIndex < currentRecord->fDiffs.count(); diffIndex++) {
405 DiffData& data = currentRecord->fDiffs[diffIndex];
406 int index = -1;
407 SkAssertResult(columns.find(data.fDiffName, &index));
408 SkASSERT(index >= 0 && index < cntColumns);
djsollen@google.comefc51b72013-11-12 18:29:17 +0000409 values[index] = data.fResult.result;
edisonn@google.comc93c8ac2013-07-22 15:24:26 +0000410 }
411
412 const char* filename = currentRecord->fBaselinePath.c_str() +
413 strlen(currentRecord->fBaselinePath.c_str()) - 1;
414 while (filename > currentRecord->fBaselinePath.c_str() && *(filename - 1) != '/') {
415 filename--;
416 }
417
418 stream.writeText(filename);
419
420 for (int i = 0; i < cntColumns; i++) {
421 SkString str;
422 str.printf(", %f", values[i]);
423 stream.writeText(str.c_str());
424 }
425 stream.writeText("\n");
426
djsollen@google.comefc51b72013-11-12 18:29:17 +0000427 currentRecord = iter2.next();
edisonn@google.comc93c8ac2013-07-22 15:24:26 +0000428 }
429}