Distinguish common and unique names for skiaperf.com.
Turns out we tack on the size post-facto in ResultsWriter::bench(), so the only
place we need getUniqueName() to differ from getName() is SKPBench.
BUG=skia:
R=jcgregorio@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/552303004
diff --git a/bench/SKPBench.cpp b/bench/SKPBench.cpp
index 308cea6..9d822d9 100644
--- a/bench/SKPBench.cpp
+++ b/bench/SKPBench.cpp
@@ -10,14 +10,19 @@
SKPBench::SKPBench(const char* name, const SkPicture* pic, const SkIRect& clip, SkScalar scale)
: fPic(SkRef(pic))
, fClip(clip)
- , fScale(scale) {
- fName.printf("%s_%.2g", name, scale);
+ , fScale(scale)
+ , fName(name) {
+ fUniqueName.printf("%s_%.2g", name, scale); // Scale makes this unqiue for skiaperf.com traces.
}
const char* SKPBench::onGetName() {
return fName.c_str();
}
+const char* SKPBench::onGetUniqueName() {
+ return fUniqueName.c_str();
+}
+
bool SKPBench::isSuitableFor(Backend backend) {
return backend != kNonRendering_Backend;
}