From aade6db347f4965fbd62f95934fe5616d9e80f8b Mon Sep 17 00:00:00 2001 From: "brettw@chromium.org" Date: Wed, 20 May 2009 16:49:47 +0000 Subject: Histogram the time we spend resampling images. We suspect this may be causing performance problems, so it would be nice to see what average users are seeing. Review URL: http://codereview.chromium.org/113604 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@16486 0039d316-1c4b-4281-b951-d872f2087c98 --- skia/ext/image_operations.cc | 8 ++++++++ 1 file changed, 8 insertions(+) (limited to 'skia') diff --git a/skia/ext/image_operations.cc b/skia/ext/image_operations.cc index fa5194b..561bd4e 100644 --- a/skia/ext/image_operations.cc +++ b/skia/ext/image_operations.cc @@ -11,8 +11,10 @@ #include "base/gfx/rect.h" #include "base/gfx/size.h" +#include "base/histogram.h" #include "base/logging.h" #include "base/stack_container.h" +#include "base/time.h" #include "third_party/skia/include/core/SkBitmap.h" #include "third_party/skia/include/core/SkColorPriv.h" #include "skia/ext/convolver.h" @@ -260,6 +262,9 @@ SkBitmap ImageOperations::Resize(const SkBitmap& source, ResizeMethod method, int dest_width, int dest_height, const gfx::Rect& dest_subset) { + // Time how long this takes to see if it's a problem for users. + base::TimeTicks resize_start = base::TimeTicks::Now(); + DCHECK(gfx::Rect(dest_width, dest_height).Contains(dest_subset)) << "The supplied subset does not fall within the destination image."; @@ -292,6 +297,9 @@ SkBitmap ImageOperations::Resize(const SkBitmap& source, // Preserve the "opaque" flag for use as an optimization later. result.setIsOpaque(source.isOpaque()); + base::TimeDelta delta = base::TimeTicks::Now() - resize_start; + UMA_HISTOGRAM_TIMES("Image.ResampleMS", delta); + return result; } -- cgit v1.1