diff options
author | miket@chromium.org <miket@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-12-06 19:27:30 +0000 |
---|---|---|
committer | miket@chromium.org <miket@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-12-06 19:27:30 +0000 |
commit | 3b94628f7d3157770e58974a5ef03aba589179a4 (patch) | |
tree | daad848c83942b63f03fea0883566c057bacf93d /cc/test | |
parent | 1f30b2c9d395d78ddf8f8f9de9cfd636b8d7689c (diff) | |
download | chromium_src-3b94628f7d3157770e58974a5ef03aba589179a4.zip chromium_src-3b94628f7d3157770e58974a5ef03aba589179a4.tar.gz chromium_src-3b94628f7d3157770e58974a5ef03aba589179a4.tar.bz2 |
Revert 171544
> Basic pixel tests for cc
>
> BUG=
>
>
> Review URL: https://chromiumcodereview.appspot.com/11412289
TBR=jamesr@chromium.org
Review URL: https://codereview.chromium.org/11470003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171557 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'cc/test')
-rw-r--r-- | cc/test/DEPS | 5 | ||||
-rw-r--r-- | cc/test/data/green.png | 0 | ||||
-rw-r--r-- | cc/test/paths.cc | 44 | ||||
-rw-r--r-- | cc/test/paths.h | 26 | ||||
-rw-r--r-- | cc/test/pixel_test_output_surface.cc | 36 | ||||
-rw-r--r-- | cc/test/pixel_test_output_surface.h | 37 | ||||
-rw-r--r-- | cc/test/pixel_test_utils.cc | 81 | ||||
-rw-r--r-- | cc/test/pixel_test_utils.h | 25 | ||||
-rw-r--r-- | cc/test/run_all_unittests.cc | 2 |
9 files changed, 0 insertions, 256 deletions
diff --git a/cc/test/DEPS b/cc/test/DEPS deleted file mode 100644 index 76f1349..0000000 --- a/cc/test/DEPS +++ /dev/null @@ -1,5 +0,0 @@ -include_rules = [ - "+ui/gl/", -# TODO(jamesr): Remove once cc depends on GLES2Interface instead of WGC3D - "+webkit/gpu", -] diff --git a/cc/test/data/green.png b/cc/test/data/green.png deleted file mode 100644 index e69de29..0000000 --- a/cc/test/data/green.png +++ /dev/null diff --git a/cc/test/paths.cc b/cc/test/paths.cc deleted file mode 100644 index 3551afb..0000000 --- a/cc/test/paths.cc +++ /dev/null @@ -1,44 +0,0 @@ -// Copyright (c) 2012 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 "cc/test/paths.h" - -#include "base/file_path.h" -#include "base/file_util.h" -#include "base/path_service.h" - -namespace cc { -namespace test { - -bool PathProvider(int key, FilePath* result) { - FilePath cur; - switch (key) { - // The following are only valid in the development environment, and - // will fail if executed from an installed executable (because the - // generated path won't exist). - case DIR_TEST_DATA: - if (!PathService::Get(base::DIR_SOURCE_ROOT, &cur)) - return false; - cur = cur.Append(FILE_PATH_LITERAL("cc")); - cur = cur.Append(FILE_PATH_LITERAL("test")); - cur = cur.Append(FILE_PATH_LITERAL("data")); - if (!file_util::PathExists(cur)) // we don't want to create this - return false; - break; - default: - return false; - } - - *result = cur; - return true; -} - -// This cannot be done as a static initializer sadly since Visual Studio will -// eliminate this object file if there is no direct entry point into it. -void RegisterPathProvider() { - PathService::RegisterProvider(PathProvider, PATH_START, PATH_END); -} - -} // namespace test -} // namespace cc diff --git a/cc/test/paths.h b/cc/test/paths.h deleted file mode 100644 index bc3f4e0..0000000 --- a/cc/test/paths.h +++ /dev/null @@ -1,26 +0,0 @@ -// Copyright (c) 2012 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. - -#ifndef CC_TEST_PATHS_H_ -#define CC_TEST_PATHS_H_ - -namespace cc { -namespace test { - -enum { - PATH_START = 5000, - - // Valid only in development and testing environments. - DIR_TEST_DATA, - - PATH_END -}; - -// Call once to register the provider for the path keys defined above. -void RegisterPathProvider(); - -} // namespace test -} // namespace cc - -#endif // CC_TEST_PATHS_H_ diff --git a/cc/test/pixel_test_output_surface.cc b/cc/test/pixel_test_output_surface.cc deleted file mode 100644 index 46985b7..0000000 --- a/cc/test/pixel_test_output_surface.cc +++ /dev/null @@ -1,36 +0,0 @@ -// Copyright 2012 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 "cc/test/pixel_test_output_surface.h" - -#include "webkit/gpu/webgraphicscontext3d_in_process_command_buffer_impl.h" - -namespace cc { - -PixelTestOutputSurface::PixelTestOutputSurface() { - scoped_ptr<webkit::gpu::WebGraphicsContext3DInProcessCommandBufferImpl> - context(new webkit::gpu::WebGraphicsContext3DInProcessCommandBufferImpl); - context->Initialize(WebKit::WebGraphicsContext3D::Attributes(), NULL); - context_ = context.PassAs<WebKit::WebGraphicsContext3D>(); -} - -PixelTestOutputSurface::~PixelTestOutputSurface() { -} - -bool PixelTestOutputSurface::bindToClient( - WebKit::WebCompositorOutputSurfaceClient*) { - return context_->makeContextCurrent(); -} - -const WebKit::WebCompositorOutputSurface::Capabilities& - PixelTestOutputSurface::capabilities() const { - static WebKit::WebCompositorOutputSurface::Capabilities capabilities; - return capabilities; -} - -WebKit::WebGraphicsContext3D* PixelTestOutputSurface::context3D() const { - return context_.get(); -} - -} // namespace cc diff --git a/cc/test/pixel_test_output_surface.h b/cc/test/pixel_test_output_surface.h deleted file mode 100644 index f59e63c..0000000 --- a/cc/test/pixel_test_output_surface.h +++ /dev/null @@ -1,37 +0,0 @@ -// Copyright 2012 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. - -#ifndef CC_TEST_PIXEL_TEST_GRAPHICS_CONTEXT_H_ -#define CC_TEST_PIXEL_TEST_GRAPHICS_CONTEXT_H_ - -#include "base/memory/scoped_ptr.h" -#include "cc/output_surface.h" - -namespace cc { - -class PixelTestOutputSurface : public OutputSurface { - public: - static scoped_ptr<PixelTestOutputSurface> create() { - return make_scoped_ptr(new PixelTestOutputSurface); - } - - virtual ~PixelTestOutputSurface(); - - // OutputSurface overrides. - virtual bool bindToClient(WebKit::WebCompositorOutputSurfaceClient*) OVERRIDE; - virtual const WebKit::WebCompositorOutputSurface::Capabilities& capabilities() const OVERRIDE; - virtual WebKit::WebGraphicsContext3D* context3D() const OVERRIDE; - virtual void sendFrameToParentCompositor( - const WebKit::WebCompositorFrame&) OVERRIDE { } - - private: - PixelTestOutputSurface(); - - scoped_ptr<WebKit::WebGraphicsContext3D> context_; -}; - -} // namespace cc - - -#endif // CC_TEST_PIXEL_TEST_GRAPHICS_CONTEXT_H_ diff --git a/cc/test/pixel_test_utils.cc b/cc/test/pixel_test_utils.cc deleted file mode 100644 index ad455c0..0000000 --- a/cc/test/pixel_test_utils.cc +++ /dev/null @@ -1,81 +0,0 @@ -// Copyright (c) 2012 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 "cc/test/pixel_test_utils.h" - -#include "base/file_util.h" -#include "base/logging.h" -#include "third_party/skia/include/core/SkBitmap.h" -#include "ui/gfx/codec/png_codec.h" - -namespace cc { -namespace test { - -bool WritePNGFile(const SkBitmap& bitmap, const FilePath& file_path) { - std::vector<unsigned char> png_data; - const bool discard_transparency = true; - if (gfx::PNGCodec::EncodeBGRASkBitmap(bitmap, - discard_transparency, - &png_data) && - file_util::CreateDirectory(file_path.DirName())) { - char* data = reinterpret_cast<char*>(&png_data[0]); - int size = static_cast<int>(png_data.size()); - return file_util::WriteFile(file_path, data, size) == size; - } - return false; -} - -bool ReadPNGFile(const FilePath& file_path, SkBitmap* bitmap) { - DCHECK(bitmap); - std::string png_data; - return file_util::ReadFileToString(file_path, &png_data) && - gfx::PNGCodec::Decode(reinterpret_cast<unsigned char*>(&png_data[0]), - png_data.length(), - bitmap); -} - -bool IsSameAsPNGFile(const SkBitmap& gen_bmp, FilePath ref_img_path) { - SkBitmap ref_bmp; - if (!ReadPNGFile(ref_img_path, &ref_bmp)) { - LOG(ERROR) << "Cannot read reference image: " << ref_img_path.value(); - return false; - } - - if (ref_bmp.width() != gen_bmp.width() || - ref_bmp.height() != gen_bmp.height()) { - LOG(ERROR) - << "Dimensions do not match (Expected) vs (Actual):" - << "(" << ref_bmp.width() << "x" << ref_bmp.height() - << ") vs. " - << "(" << gen_bmp.width() << "x" << gen_bmp.height() << ")"; - return false; - } - - // Compare pixels and create a simple diff image. - int diff_pixels_count = 0; - SkAutoLockPixels lock_bmp(gen_bmp); - SkAutoLockPixels lock_ref_bmp(ref_bmp); - // The reference images were saved with no alpha channel. Use the mask to - // set alpha to 0. - uint32_t kAlphaMask = 0x00FFFFFF; - for (int x = 0; x < gen_bmp.width(); ++x) { - for (int y = 0; y < gen_bmp.height(); ++y) { - if ((*gen_bmp.getAddr32(x, y) & kAlphaMask) != - (*ref_bmp.getAddr32(x, y) & kAlphaMask)) { - ++diff_pixels_count; - } - } - } - - if (diff_pixels_count != 0) { - LOG(ERROR) << "Images differ by pixel count: " << diff_pixels_count; - return false; - } - - return true; -} - -} // namespace test -} // namespace cc - diff --git a/cc/test/pixel_test_utils.h b/cc/test/pixel_test_utils.h deleted file mode 100644 index 93367d8..0000000 --- a/cc/test/pixel_test_utils.h +++ /dev/null @@ -1,25 +0,0 @@ -// Copyright (c) 2012 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 "base/file_path.h" - -class SkBitmap; - -namespace cc { -namespace test { - -// Encodes a bitmap into a PNG and write to disk. Returns true on success. The -// parent directory does not have to exist. -bool WritePNGFile(const SkBitmap& bitmap, const FilePath& file_path); - -// Reads and decodes a PNG image to a bitmap. Returns true on success. The PNG -// should have been encoded using |gfx::PNGCodec::Encode|. -bool ReadPNGFile(const FilePath& file_path, SkBitmap* bitmap); - -// Compares with a PNG file on disk, and returns true if it is the same as -// the given image. |ref_img_path| is absolute. -bool IsSameAsPNGFile(const SkBitmap& gen_bmp, FilePath ref_img_path); - -} // namespace test -} // namespace cc diff --git a/cc/test/run_all_unittests.cc b/cc/test/run_all_unittests.cc index e1092c9..4ea8697 100644 --- a/cc/test/run_all_unittests.cc +++ b/cc/test/run_all_unittests.cc @@ -4,11 +4,9 @@ #include "base/message_loop.h" #include "base/test/test_suite.h" -#include "cc/test/paths.h" #include "testing/gmock/include/gmock/gmock.h" int main(int argc, char** argv) { - cc::test::RegisterPathProvider(); ::testing::InitGoogleMock(&argc, argv); TestSuite test_suite(argc, argv); MessageLoop message_loop; |