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

Unified Diff: base/gfx/img_resize_perftest.cc

Issue 12842: Move convolver, image_operations, and skia_utils from base/gfx to skia/ext.... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 12 years 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « base/gfx/image_operations_unittest.cc ('k') | base/gfx/native_theme.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/gfx/img_resize_perftest.cc
===================================================================
--- base/gfx/img_resize_perftest.cc (revision 6142)
+++ base/gfx/img_resize_perftest.cc (working copy)
@@ -1,70 +0,0 @@
-// Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include <stdlib.h>
-#include <time.h>
-
-#include "base/perftimer.h"
-#include "base/gfx/convolver.h"
-#include "base/gfx/image_operations.h"
-#include "base/gfx/image_resizer.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-namespace {
-
-void FillRandomData(char* dest, int byte_count) {
- srand(static_cast<unsigned>(time(NULL)));
- for (int i = 0; i < byte_count; i++)
- dest[i] = rand() * 255 / RAND_MAX;
-}
-
-} // namespace
-
-// Old code gives [1521, 1519]ms for this, 4000x4000 -> 2100x2100 lanczos8
-
-TEST(ImageResizePerf, BigFilter) {
- static const int kSrcWidth = 4000;
- static const int kSrcHeight = 4000;
- static const int kSrcByteSize = kSrcWidth * kSrcHeight * 4;
-
- SkBitmap src_bmp;
- src_bmp.setConfig(SkBitmap::kARGB_8888_Config, kSrcWidth, kSrcHeight);
- src_bmp.allocPixels();
- FillRandomData(reinterpret_cast<char*>(src_bmp.getAddr32(0, 0)),
- kSrcByteSize);
-
- // Make the dest size > 1/2 so the 50% optimization doesn't kick in.
- static const int kDestWidth = 1400;
- static const int kDestHeight = 1400;
-
- PerfTimeLogger resize_timer("resize");
- gfx::ImageResizer resizer(gfx::ImageResizer::LANCZOS3);
- SkBitmap dest = resizer.Resize(src_bmp, kDestWidth, kDestHeight);
-}
-
-// The original image filter we were using took 523ms for this test, while this
-// one takes 857ms.
-// TODO(brettw) make this at least 64% faster.
-TEST(ImageOperationPerf, BigFilter) {
- static const int kSrcWidth = 4000;
- static const int kSrcHeight = 4000;
- static const int kSrcByteSize = kSrcWidth * kSrcHeight * 4;
-
- SkBitmap src_bmp;
- src_bmp.setConfig(SkBitmap::kARGB_8888_Config, kSrcWidth, kSrcHeight);
- src_bmp.allocPixels();
- src_bmp.setIsOpaque(true);
- FillRandomData(reinterpret_cast<char*>(src_bmp.getAddr32(0, 0)),
- kSrcByteSize);
-
- // Make the dest size > 1/2 so the 50% optimization doesn't kick in.
- static const int kDestWidth = 1400;
- static const int kDestHeight = 1400;
-
- PerfTimeLogger resize_timer("resize");
- SkBitmap dest = gfx::ImageOperations::Resize(src_bmp,
- gfx::ImageOperations::RESIZE_LANCZOS3, (float)kDestWidth / (float)kSrcWidth,
- (float)kDestHeight / (float)kSrcHeight);
-}
-
« no previous file with comments | « base/gfx/image_operations_unittest.cc ('k') | base/gfx/native_theme.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698