2015-03-25 17:22:41 +00:00
|
|
|
# Copyright 2015 Google Inc.
|
|
|
|
#
|
|
|
|
# Use of this source code is governed by a BSD-style license that can be
|
|
|
|
# found in the LICENSE file.
|
2014-12-17 21:43:13 +00:00
|
|
|
# GYP file to build various tools.
|
2011-06-02 14:38:23 +00:00
|
|
|
#
|
|
|
|
# To build on Linux:
|
|
|
|
# ./gyp_skia tools.gyp && make tools
|
|
|
|
#
|
|
|
|
{
|
|
|
|
'includes': [
|
|
|
|
'apptype_console.gypi',
|
|
|
|
],
|
|
|
|
'targets': [
|
|
|
|
{
|
|
|
|
# Build all executable targets defined below.
|
|
|
|
'target_name': 'tools',
|
|
|
|
'type': 'none',
|
|
|
|
'dependencies': [
|
2015-05-12 15:36:48 +00:00
|
|
|
'chrome_fuzz',
|
2015-11-18 19:06:37 +00:00
|
|
|
'dump_record',
|
2014-04-28 13:19:34 +00:00
|
|
|
'gpuveto',
|
2015-10-19 13:39:17 +00:00
|
|
|
'imgblur',
|
|
|
|
'imgconv',
|
|
|
|
'imgslice',
|
2014-01-02 18:51:38 +00:00
|
|
|
'lua_app',
|
2014-01-02 19:42:15 +00:00
|
|
|
'lua_pictures',
|
2013-01-18 18:06:22 +00:00
|
|
|
'pinspect',
|
2011-06-02 14:38:23 +00:00
|
|
|
'skdiff',
|
2014-01-02 18:51:38 +00:00
|
|
|
'skhello',
|
2014-01-02 19:42:15 +00:00
|
|
|
'skpdiff',
|
2014-03-03 19:18:39 +00:00
|
|
|
'skpinfo',
|
2014-01-02 19:42:15 +00:00
|
|
|
'skpmaker',
|
Make image decoding more fault resistant, less verbose.
This change address what happens when a jpeg is partially downloaded
before failing. Many browsers will render it anyway: we want Skia to
do the same. The JpegTest takes a perfectly cromulent jpeg file and
only passes into the ImageDecoder the first half of the image. We
then verify that the image decoder returns a valid bitmap of the
correct dimensions.
We also fixed some png library errors, including issue 1691.
Also, suppressed the majority of warnings from using libpng and
libjpeg. By default, most warnings are *not* suppressed in debug mode.
If you have a debug binary and wish to suppress warnings, set the
following environment variables to true
skia_images_png_suppressDecoderWarnings
skia_images_jpeg_suppressDecoderWarnings
or from within a program that links to Skia:
#if defined(SK_DEBUG)
#include "SkRTConf.h"
SK_CONF_SET("images.jpeg.suppressDecoderWarnings", true);
SK_CONF_SET("images.png.suppressDecoderWarnings", true);
#endif
I tested this, before (control) and after these changes (test), on
364,295 skps from the cluster telemetry.
- number of errors+warnings in control = 2804
- number of errors+warnings fixed = 2283
- number of PNG verbosity fixed = 2152
- number of PNG error fixed = 4
- number of PNG segfault fixed = 3
- number of PNG errors changed to warnings = 62
- number of JPG verbosity fixed = 26
- number of JPG error fixed = 91
Not all errors and warning have been fixed.
These numbers were generated using the find_bad_images_in_skps.py
program. This program may be useful going forward for testing
image-decoding libraries on skp files from the cluster telemetry.
find_bad_images_in_skps.py depends on the test_image_decoder program,
which simply executes the SkImageDecoder::DecodeFile function and uses
its exit status to report success or failure.
BUG=skia:1649
BUG=skia:1691
BUG=skia:1680
R=scroggo@google.com
Review URL: https://codereview.chromium.org/24449003
git-svn-id: http://skia.googlecode.com/svn/trunk@11597 2bbb7eff-a529-9590-31e7-b0007b416f81
2013-10-04 12:46:45 +00:00
|
|
|
'test_image_decoder',
|
2014-11-13 20:52:35 +00:00
|
|
|
'test_public_includes',
|
2015-08-31 16:22:38 +00:00
|
|
|
'whitelist_typefaces',
|
2011-06-02 14:38:23 +00:00
|
|
|
],
|
2013-06-20 17:15:05 +00:00
|
|
|
'conditions': [
|
|
|
|
['skia_shared_lib',
|
|
|
|
{
|
|
|
|
'dependencies': [
|
|
|
|
'sklua', # This can only be built if skia is built as a shared library
|
|
|
|
],
|
|
|
|
},
|
|
|
|
],
|
|
|
|
],
|
2011-06-02 14:38:23 +00:00
|
|
|
},
|
2015-11-18 19:06:37 +00:00
|
|
|
{
|
|
|
|
'target_name': 'dump_record',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
|
|
|
'../tools/dump_record.cpp',
|
|
|
|
'../tools/DumpRecord.cpp',
|
|
|
|
],
|
|
|
|
'include_dirs': [
|
|
|
|
'../include/private',
|
|
|
|
'../src/core',
|
|
|
|
],
|
|
|
|
'dependencies': [
|
|
|
|
'flags.gyp:flags',
|
|
|
|
'lazy_decode_bitmap',
|
|
|
|
'skia_lib.gyp:skia_lib',
|
|
|
|
],
|
|
|
|
},
|
2015-05-12 15:36:48 +00:00
|
|
|
{
|
|
|
|
'target_name': 'chrome_fuzz',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
|
|
|
'../tools/chrome_fuzz.cpp',
|
|
|
|
],
|
|
|
|
'dependencies': [
|
|
|
|
'skia_lib.gyp:skia_lib',
|
|
|
|
],
|
|
|
|
},
|
2014-06-20 18:29:20 +00:00
|
|
|
{
|
|
|
|
'target_name': 'crash_handler',
|
|
|
|
'type': 'static_library',
|
|
|
|
'sources': [ '../tools/CrashHandler.cpp' ],
|
|
|
|
'dependencies': [ 'skia_lib.gyp:skia_lib' ],
|
|
|
|
'direct_dependent_settings': {
|
|
|
|
'include_dirs': [ '../tools' ],
|
|
|
|
},
|
2015-02-10 16:13:26 +00:00
|
|
|
'conditions': [
|
|
|
|
[ 'skia_is_bot', {
|
|
|
|
'defines': [ 'SK_CRASH_HANDLER' ],
|
|
|
|
}],
|
|
|
|
],
|
|
|
|
|
2014-06-20 18:29:20 +00:00
|
|
|
'all_dependent_settings': {
|
|
|
|
'msvs_settings': {
|
|
|
|
'VCLinkerTool': {
|
|
|
|
'AdditionalDependencies': [ 'Dbghelp.lib' ],
|
|
|
|
}
|
|
|
|
},
|
|
|
|
}
|
|
|
|
},
|
|
|
|
{
|
|
|
|
'target_name': 'resources',
|
|
|
|
'type': 'static_library',
|
|
|
|
'sources': [ '../tools/Resources.cpp' ],
|
|
|
|
'dependencies': [
|
|
|
|
'flags.gyp:flags',
|
|
|
|
'skia_lib.gyp:skia_lib',
|
|
|
|
],
|
|
|
|
'direct_dependent_settings': {
|
2014-06-21 17:54:17 +00:00
|
|
|
'include_dirs': [ '../tools', ],
|
|
|
|
},
|
|
|
|
},
|
|
|
|
{
|
|
|
|
'target_name': 'sk_tool_utils',
|
|
|
|
'type': 'static_library',
|
2014-06-23 18:25:00 +00:00
|
|
|
'sources': [
|
|
|
|
'../tools/sk_tool_utils.cpp',
|
|
|
|
'../tools/sk_tool_utils_font.cpp',
|
|
|
|
],
|
2014-07-31 12:58:44 +00:00
|
|
|
'include_dirs': [
|
2015-09-29 18:47:45 +00:00
|
|
|
'../include/private',
|
2014-07-31 12:58:44 +00:00
|
|
|
'../src/fonts',
|
2015-08-20 12:15:06 +00:00
|
|
|
'../src/core',
|
2014-07-31 12:58:44 +00:00
|
|
|
],
|
2014-06-21 17:54:17 +00:00
|
|
|
'dependencies': [
|
2014-07-31 15:31:56 +00:00
|
|
|
'resources',
|
2014-07-31 12:58:44 +00:00
|
|
|
'flags.gyp:flags',
|
2014-06-21 17:54:17 +00:00
|
|
|
'skia_lib.gyp:skia_lib',
|
|
|
|
],
|
|
|
|
'direct_dependent_settings': {
|
|
|
|
'include_dirs': [ '../tools', ],
|
2014-06-20 18:29:20 +00:00
|
|
|
},
|
|
|
|
},
|
|
|
|
{
|
|
|
|
'target_name' : 'timer',
|
|
|
|
'type': 'static_library',
|
2015-10-23 14:02:06 +00:00
|
|
|
'sources': [ '../tools/timer/Timer.cpp' ],
|
2014-06-20 18:29:20 +00:00
|
|
|
'direct_dependent_settings': {
|
|
|
|
'include_dirs': ['../tools/timer'],
|
|
|
|
},
|
2015-10-23 14:02:06 +00:00
|
|
|
'dependencies': [ 'skia_lib.gyp:skia_lib' ],
|
2014-06-20 18:29:20 +00:00
|
|
|
},
|
2011-06-02 14:38:23 +00:00
|
|
|
{
|
|
|
|
'target_name': 'skdiff',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
2012-12-05 20:13:12 +00:00
|
|
|
'../tools/skdiff.cpp',
|
|
|
|
'../tools/skdiff.h',
|
|
|
|
'../tools/skdiff_html.cpp',
|
|
|
|
'../tools/skdiff_html.h',
|
2011-06-02 14:38:23 +00:00
|
|
|
'../tools/skdiff_main.cpp',
|
2012-12-05 20:13:12 +00:00
|
|
|
'../tools/skdiff_utils.cpp',
|
|
|
|
'../tools/skdiff_utils.h',
|
|
|
|
],
|
|
|
|
'dependencies': [
|
2013-06-03 12:10:19 +00:00
|
|
|
'skia_lib.gyp:skia_lib',
|
2013-07-22 18:46:45 +00:00
|
|
|
],
|
|
|
|
},
|
|
|
|
{
|
|
|
|
'target_name': 'skpdiff',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
|
|
|
'../tools/skpdiff/skpdiff_main.cpp',
|
|
|
|
'../tools/skpdiff/SkDiffContext.cpp',
|
|
|
|
'../tools/skpdiff/SkImageDiffer.cpp',
|
|
|
|
'../tools/skpdiff/SkPMetric.cpp',
|
|
|
|
'../tools/skpdiff/skpdiff_util.cpp',
|
|
|
|
],
|
|
|
|
'include_dirs': [
|
2015-07-28 16:54:52 +00:00
|
|
|
'../include/private',
|
2013-11-12 18:29:17 +00:00
|
|
|
'../src/core/', # needed for SkTLList.h
|
2014-08-13 17:36:06 +00:00
|
|
|
'../tools/', # needed for picture_utils::replace_char
|
2013-07-22 18:46:45 +00:00
|
|
|
],
|
|
|
|
'dependencies': [
|
2014-06-22 23:13:00 +00:00
|
|
|
'flags.gyp:flags',
|
2013-07-22 18:46:45 +00:00
|
|
|
'skia_lib.gyp:skia_lib',
|
2014-08-13 17:36:06 +00:00
|
|
|
'tools.gyp:picture_utils',
|
2013-07-22 18:46:45 +00:00
|
|
|
],
|
|
|
|
'cflags': [
|
|
|
|
'-O3',
|
|
|
|
],
|
|
|
|
'conditions': [
|
|
|
|
[ 'skia_os in ["linux", "freebsd", "openbsd", "solaris", "chromeos"]', {
|
|
|
|
'link_settings': {
|
|
|
|
'libraries': [
|
|
|
|
'-lrt',
|
2015-06-25 15:05:19 +00:00
|
|
|
'-pthread',
|
2013-07-22 18:46:45 +00:00
|
|
|
],
|
|
|
|
},
|
|
|
|
}],
|
|
|
|
['skia_opencl', {
|
|
|
|
'sources': [
|
|
|
|
'../tools/skpdiff/SkCLImageDiffer.cpp',
|
|
|
|
'../tools/skpdiff/SkDifferentPixelsMetric_opencl.cpp',
|
|
|
|
],
|
|
|
|
'conditions': [
|
|
|
|
[ 'skia_os == "mac"', {
|
|
|
|
'link_settings': {
|
|
|
|
'libraries': [
|
|
|
|
'$(SDKROOT)/System/Library/Frameworks/OpenCL.framework',
|
|
|
|
]
|
|
|
|
}
|
|
|
|
}, {
|
|
|
|
'link_settings': {
|
|
|
|
'libraries': [
|
|
|
|
'-lOpenCL',
|
|
|
|
],
|
|
|
|
},
|
|
|
|
}],
|
|
|
|
],
|
|
|
|
}, { # !skia_opencl
|
|
|
|
'sources': [
|
|
|
|
'../tools/skpdiff/SkDifferentPixelsMetric_cpu.cpp',
|
|
|
|
],
|
|
|
|
}],
|
2012-12-05 20:13:12 +00:00
|
|
|
],
|
|
|
|
},
|
2014-01-02 19:42:15 +00:00
|
|
|
{
|
|
|
|
'target_name': 'skpmaker',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
|
|
|
'../tools/skpmaker.cpp',
|
|
|
|
],
|
|
|
|
'include_dirs': [
|
2015-07-28 16:54:52 +00:00
|
|
|
'../include/private',
|
2014-01-02 19:42:15 +00:00
|
|
|
'../src/core',
|
|
|
|
],
|
|
|
|
'dependencies': [
|
|
|
|
'flags.gyp:flags',
|
|
|
|
'skia_lib.gyp:skia_lib',
|
|
|
|
],
|
|
|
|
},
|
2012-12-05 20:13:12 +00:00
|
|
|
{
|
|
|
|
'target_name': 'skimagediff',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
|
|
|
'../tools/skdiff.cpp',
|
|
|
|
'../tools/skdiff.h',
|
|
|
|
'../tools/skdiff_html.cpp',
|
|
|
|
'../tools/skdiff_html.h',
|
|
|
|
'../tools/skdiff_image.cpp',
|
|
|
|
'../tools/skdiff_utils.cpp',
|
|
|
|
'../tools/skdiff_utils.h',
|
2011-06-02 14:38:23 +00:00
|
|
|
],
|
|
|
|
'dependencies': [
|
2013-06-03 12:10:19 +00:00
|
|
|
'skia_lib.gyp:skia_lib',
|
2011-06-02 14:38:23 +00:00
|
|
|
],
|
|
|
|
},
|
|
|
|
{
|
|
|
|
'target_name': 'skhello',
|
|
|
|
'type': 'executable',
|
|
|
|
'dependencies': [
|
2015-04-02 19:16:36 +00:00
|
|
|
'flags.gyp:flags',
|
|
|
|
'pdf.gyp:pdf',
|
2013-06-03 12:10:19 +00:00
|
|
|
'skia_lib.gyp:skia_lib',
|
2013-06-17 15:39:43 +00:00
|
|
|
],
|
2015-04-02 19:16:36 +00:00
|
|
|
'sources': [
|
|
|
|
'../tools/skhello.cpp',
|
2011-06-02 14:38:23 +00:00
|
|
|
],
|
|
|
|
},
|
2014-03-03 19:18:39 +00:00
|
|
|
{
|
|
|
|
'target_name': 'skpinfo',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
|
|
|
'../tools/skpinfo.cpp',
|
|
|
|
],
|
|
|
|
'include_dirs': [
|
2015-07-28 16:54:52 +00:00
|
|
|
'../include/private',
|
2014-03-03 19:18:39 +00:00
|
|
|
'../src/core/',
|
|
|
|
],
|
|
|
|
'dependencies': [
|
|
|
|
'flags.gyp:flags',
|
|
|
|
'skia_lib.gyp:skia_lib',
|
|
|
|
],
|
|
|
|
},
|
2015-10-19 13:39:17 +00:00
|
|
|
{
|
|
|
|
'target_name': 'imgblur',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
|
|
|
'../tools/imgblur.cpp',
|
|
|
|
],
|
|
|
|
'include_dirs': [
|
|
|
|
'../include/core',
|
|
|
|
],
|
|
|
|
'dependencies': [
|
|
|
|
'flags.gyp:flags',
|
|
|
|
'flags.gyp:flags_common',
|
|
|
|
'skia_lib.gyp:skia_lib',
|
|
|
|
'tools.gyp:sk_tool_utils',
|
|
|
|
],
|
|
|
|
},
|
2015-09-18 16:04:43 +00:00
|
|
|
{
|
|
|
|
'target_name': 'imgslice',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
|
|
|
'../tools/imgslice.cpp',
|
|
|
|
],
|
|
|
|
'include_dirs': [
|
|
|
|
'../include/core',
|
|
|
|
],
|
|
|
|
'dependencies': [
|
|
|
|
'flags.gyp:flags',
|
|
|
|
'skia_lib.gyp:skia_lib',
|
|
|
|
],
|
|
|
|
},
|
2015-07-08 17:06:01 +00:00
|
|
|
{
|
2015-10-20 21:29:10 +00:00
|
|
|
'target_name': 'lazy_decode_bitmap',
|
|
|
|
'type': 'static_library',
|
|
|
|
'sources': [ '../tools/LazyDecodeBitmap.cpp' ],
|
|
|
|
'include_dirs': [
|
|
|
|
'../include/private',
|
|
|
|
'../src/core',
|
|
|
|
'../src/lazy',
|
|
|
|
],
|
|
|
|
'dependencies': [
|
|
|
|
'flags.gyp:flags',
|
|
|
|
'skia_lib.gyp:skia_lib'
|
|
|
|
],
|
2015-01-30 17:03:29 +00:00
|
|
|
},
|
2014-04-28 13:19:34 +00:00
|
|
|
{
|
|
|
|
'target_name': 'gpuveto',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
|
|
|
'../tools/gpuveto.cpp',
|
|
|
|
],
|
|
|
|
'include_dirs': [
|
2015-07-28 16:54:52 +00:00
|
|
|
'../include/private',
|
2014-04-28 13:19:34 +00:00
|
|
|
'../src/core/',
|
|
|
|
'../src/images',
|
|
|
|
],
|
|
|
|
'dependencies': [
|
2015-10-20 21:29:10 +00:00
|
|
|
'lazy_decode_bitmap',
|
2014-04-28 13:19:34 +00:00
|
|
|
'flags.gyp:flags',
|
|
|
|
'skia_lib.gyp:skia_lib',
|
|
|
|
],
|
|
|
|
},
|
2013-06-09 01:54:56 +00:00
|
|
|
{
|
|
|
|
'target_name': 'lua_app',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
|
|
|
'../tools/lua/lua_app.cpp',
|
|
|
|
'../src/utils/SkLua.cpp',
|
|
|
|
],
|
2014-02-26 20:22:32 +00:00
|
|
|
'include_dirs': [
|
2015-07-28 16:54:52 +00:00
|
|
|
'../include/private',
|
2014-02-26 20:22:32 +00:00
|
|
|
# Lua exposes GrReduceClip which in turn requires src/core for SkTLList
|
|
|
|
'../src/gpu/',
|
|
|
|
'../src/core/',
|
|
|
|
],
|
2013-06-09 01:54:56 +00:00
|
|
|
'dependencies': [
|
|
|
|
'effects.gyp:effects',
|
|
|
|
'images.gyp:images',
|
2013-12-13 20:52:36 +00:00
|
|
|
'lua.gyp:lua',
|
2013-06-09 01:54:56 +00:00
|
|
|
'pdf.gyp:pdf',
|
|
|
|
'ports.gyp:ports',
|
2013-12-13 20:52:36 +00:00
|
|
|
'skia_lib.gyp:skia_lib',
|
2013-06-09 01:54:56 +00:00
|
|
|
],
|
|
|
|
},
|
2013-05-15 19:34:20 +00:00
|
|
|
{
|
|
|
|
'target_name': 'lua_pictures',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
|
|
|
'../tools/lua/lua_pictures.cpp',
|
|
|
|
'../src/utils/SkLuaCanvas.cpp',
|
2013-05-22 15:13:18 +00:00
|
|
|
'../src/utils/SkLua.cpp',
|
2013-05-15 19:34:20 +00:00
|
|
|
],
|
2014-02-26 20:22:32 +00:00
|
|
|
'include_dirs': [
|
2015-07-28 16:54:52 +00:00
|
|
|
'../include/private',
|
2014-02-26 20:22:32 +00:00
|
|
|
# Lua exposes GrReduceClip which in turn requires src/core for SkTLList
|
|
|
|
'../src/gpu/',
|
|
|
|
'../src/core/',
|
|
|
|
],
|
2013-05-15 19:34:20 +00:00
|
|
|
'dependencies': [
|
2015-10-20 21:29:10 +00:00
|
|
|
'lazy_decode_bitmap',
|
2013-05-15 19:34:20 +00:00
|
|
|
'effects.gyp:effects',
|
2013-12-13 20:52:36 +00:00
|
|
|
'flags.gyp:flags',
|
2013-05-15 19:34:20 +00:00
|
|
|
'images.gyp:images',
|
2013-12-13 20:52:36 +00:00
|
|
|
'lua.gyp:lua',
|
2013-05-15 19:34:20 +00:00
|
|
|
'tools.gyp:picture_utils',
|
2013-06-08 16:39:44 +00:00
|
|
|
'pdf.gyp:pdf',
|
2013-05-15 19:34:20 +00:00
|
|
|
'ports.gyp:ports',
|
2013-12-13 20:52:36 +00:00
|
|
|
'skia_lib.gyp:skia_lib',
|
2013-05-15 19:34:20 +00:00
|
|
|
],
|
|
|
|
},
|
2012-06-22 18:24:56 +00:00
|
|
|
{
|
|
|
|
'target_name': 'picture_utils',
|
|
|
|
'type': 'static_library',
|
|
|
|
'sources': [
|
|
|
|
'../tools/picture_utils.cpp',
|
2012-09-14 14:34:28 +00:00
|
|
|
'../tools/picture_utils.h',
|
2012-06-22 18:24:56 +00:00
|
|
|
],
|
|
|
|
'dependencies': [
|
2013-06-03 12:10:19 +00:00
|
|
|
'skia_lib.gyp:skia_lib',
|
2012-06-12 14:56:36 +00:00
|
|
|
],
|
2013-05-15 20:01:07 +00:00
|
|
|
'direct_dependent_settings': {
|
|
|
|
'include_dirs': [
|
2014-06-18 21:32:48 +00:00
|
|
|
'../tools/',
|
2013-05-15 20:01:07 +00:00
|
|
|
],
|
|
|
|
},
|
2012-06-12 14:56:36 +00:00
|
|
|
},
|
2012-06-14 18:58:40 +00:00
|
|
|
{
|
|
|
|
'target_name': 'pinspect',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
|
|
|
'../tools/pinspect.cpp',
|
|
|
|
],
|
|
|
|
'dependencies': [
|
2015-10-20 21:29:10 +00:00
|
|
|
'lazy_decode_bitmap',
|
2013-12-13 20:52:36 +00:00
|
|
|
'flags.gyp:flags',
|
2013-06-03 12:10:19 +00:00
|
|
|
'skia_lib.gyp:skia_lib',
|
2012-06-14 18:58:40 +00:00
|
|
|
],
|
|
|
|
},
|
2014-12-17 21:08:17 +00:00
|
|
|
{
|
|
|
|
'target_name': 'imgconv',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
|
|
|
'../tools/imgconv.cpp',
|
|
|
|
],
|
|
|
|
'dependencies': [
|
|
|
|
'flags.gyp:flags',
|
|
|
|
'skia_lib.gyp:skia_lib',
|
|
|
|
],
|
|
|
|
},
|
Make image decoding more fault resistant, less verbose.
This change address what happens when a jpeg is partially downloaded
before failing. Many browsers will render it anyway: we want Skia to
do the same. The JpegTest takes a perfectly cromulent jpeg file and
only passes into the ImageDecoder the first half of the image. We
then verify that the image decoder returns a valid bitmap of the
correct dimensions.
We also fixed some png library errors, including issue 1691.
Also, suppressed the majority of warnings from using libpng and
libjpeg. By default, most warnings are *not* suppressed in debug mode.
If you have a debug binary and wish to suppress warnings, set the
following environment variables to true
skia_images_png_suppressDecoderWarnings
skia_images_jpeg_suppressDecoderWarnings
or from within a program that links to Skia:
#if defined(SK_DEBUG)
#include "SkRTConf.h"
SK_CONF_SET("images.jpeg.suppressDecoderWarnings", true);
SK_CONF_SET("images.png.suppressDecoderWarnings", true);
#endif
I tested this, before (control) and after these changes (test), on
364,295 skps from the cluster telemetry.
- number of errors+warnings in control = 2804
- number of errors+warnings fixed = 2283
- number of PNG verbosity fixed = 2152
- number of PNG error fixed = 4
- number of PNG segfault fixed = 3
- number of PNG errors changed to warnings = 62
- number of JPG verbosity fixed = 26
- number of JPG error fixed = 91
Not all errors and warning have been fixed.
These numbers were generated using the find_bad_images_in_skps.py
program. This program may be useful going forward for testing
image-decoding libraries on skp files from the cluster telemetry.
find_bad_images_in_skps.py depends on the test_image_decoder program,
which simply executes the SkImageDecoder::DecodeFile function and uses
its exit status to report success or failure.
BUG=skia:1649
BUG=skia:1691
BUG=skia:1680
R=scroggo@google.com
Review URL: https://codereview.chromium.org/24449003
git-svn-id: http://skia.googlecode.com/svn/trunk@11597 2bbb7eff-a529-9590-31e7-b0007b416f81
2013-10-04 12:46:45 +00:00
|
|
|
{
|
|
|
|
'target_name': 'test_image_decoder',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
|
|
|
'../tools/test_image_decoder.cpp',
|
|
|
|
],
|
|
|
|
'dependencies': [
|
|
|
|
'skia_lib.gyp:skia_lib',
|
|
|
|
],
|
|
|
|
},
|
2014-08-11 18:33:51 +00:00
|
|
|
{
|
2014-08-19 22:55:55 +00:00
|
|
|
'target_name': 'proc_stats',
|
2014-08-11 18:33:51 +00:00
|
|
|
'type': 'static_library',
|
|
|
|
'sources': [
|
|
|
|
'../tools/ProcStats.h',
|
|
|
|
'../tools/ProcStats.cpp',
|
|
|
|
],
|
|
|
|
'direct_dependent_settings': {
|
|
|
|
'include_dirs': [ '../tools', ],
|
|
|
|
},
|
|
|
|
},
|
2015-08-31 16:22:38 +00:00
|
|
|
{
|
|
|
|
'target_name': 'whitelist_typefaces',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
|
|
|
'../tools/whitelist_typefaces.cpp',
|
|
|
|
],
|
|
|
|
'dependencies': [
|
|
|
|
'skia_lib.gyp:skia_lib',
|
|
|
|
],
|
|
|
|
},
|
2014-11-13 20:52:35 +00:00
|
|
|
{
|
|
|
|
'target_name': 'test_public_includes',
|
|
|
|
'type': 'static_library',
|
2014-11-14 19:11:46 +00:00
|
|
|
# Ensure that our public headers don't have unused params so that clients
|
|
|
|
# (e.g. Android) that include us can build with these warnings enabled
|
|
|
|
'cflags!': [ '-Wno-unused-parameter' ],
|
2014-11-13 20:52:35 +00:00
|
|
|
'variables': {
|
|
|
|
'includes_to_test': [
|
2015-11-13 14:11:09 +00:00
|
|
|
'<(skia_include_path)/android',
|
2014-11-13 20:52:35 +00:00
|
|
|
'<(skia_include_path)/animator',
|
|
|
|
'<(skia_include_path)/c',
|
2015-11-11 20:43:07 +00:00
|
|
|
'<(skia_include_path)/codec',
|
2014-11-13 20:52:35 +00:00
|
|
|
'<(skia_include_path)/config',
|
|
|
|
'<(skia_include_path)/core',
|
|
|
|
'<(skia_include_path)/effects',
|
|
|
|
'<(skia_include_path)/gpu',
|
|
|
|
'<(skia_include_path)/images',
|
|
|
|
'<(skia_include_path)/pathops',
|
|
|
|
'<(skia_include_path)/pipe',
|
|
|
|
'<(skia_include_path)/ports',
|
2015-02-06 16:43:51 +00:00
|
|
|
'<(skia_include_path)/svg/parser',
|
2014-11-13 20:52:35 +00:00
|
|
|
'<(skia_include_path)/utils',
|
|
|
|
'<(skia_include_path)/views',
|
|
|
|
'<(skia_include_path)/xml',
|
|
|
|
],
|
|
|
|
'paths_to_ignore': [
|
|
|
|
'<(skia_include_path)/gpu/gl/GrGLConfig_chrome.h',
|
2015-06-17 14:55:59 +00:00
|
|
|
'<(skia_include_path)/ports/SkFontMgr_fontconfig.h',
|
2014-11-13 20:52:35 +00:00
|
|
|
'<(skia_include_path)/ports/SkTypeface_mac.h',
|
|
|
|
'<(skia_include_path)/ports/SkTypeface_win.h',
|
|
|
|
'<(skia_include_path)/utils/ios',
|
|
|
|
'<(skia_include_path)/utils/mac',
|
|
|
|
'<(skia_include_path)/utils/win',
|
|
|
|
'<(skia_include_path)/utils/SkDebugUtils.h',
|
|
|
|
'<(skia_include_path)/utils/SkJSONCPP.h',
|
|
|
|
'<(skia_include_path)/views/animated',
|
|
|
|
'<(skia_include_path)/views/SkOSWindow_Android.h',
|
|
|
|
'<(skia_include_path)/views/SkOSWindow_iOS.h',
|
|
|
|
'<(skia_include_path)/views/SkOSWindow_Mac.h',
|
|
|
|
'<(skia_include_path)/views/SkOSWindow_SDL.h',
|
|
|
|
'<(skia_include_path)/views/SkOSWindow_Unix.h',
|
|
|
|
'<(skia_include_path)/views/SkOSWindow_Win.h',
|
|
|
|
'<(skia_include_path)/views/SkWindow.h',
|
|
|
|
],
|
|
|
|
},
|
|
|
|
'include_dirs': [
|
|
|
|
'<@(includes_to_test)',
|
|
|
|
],
|
|
|
|
'sources': [
|
|
|
|
# unused_param_test.cpp is generated by the action below.
|
|
|
|
'<(INTERMEDIATE_DIR)/test_public_includes.cpp',
|
|
|
|
],
|
|
|
|
'actions': [
|
|
|
|
{
|
|
|
|
'action_name': 'generate_includes_cpp',
|
|
|
|
'inputs': [
|
|
|
|
'../tools/generate_includes_cpp.py',
|
|
|
|
'<@(includes_to_test)',
|
|
|
|
# This causes the gyp generator on mac to fail
|
|
|
|
#'<@(paths_to_ignore)',
|
|
|
|
],
|
|
|
|
'outputs': [
|
|
|
|
'<(INTERMEDIATE_DIR)/test_public_includes.cpp',
|
|
|
|
],
|
|
|
|
'action': ['python', '../tools/generate_includes_cpp.py',
|
|
|
|
'--ignore', '<(paths_to_ignore)',
|
|
|
|
'<@(_outputs)', '<@(includes_to_test)'],
|
|
|
|
},
|
|
|
|
],
|
|
|
|
},
|
2013-03-08 18:43:35 +00:00
|
|
|
],
|
|
|
|
'conditions': [
|
2013-06-20 17:15:05 +00:00
|
|
|
['skia_shared_lib',
|
|
|
|
{
|
|
|
|
'targets': [
|
|
|
|
{
|
|
|
|
'target_name': 'sklua',
|
|
|
|
'product_name': 'skia',
|
|
|
|
'product_prefix': '',
|
|
|
|
'product_dir': '<(PRODUCT_DIR)/',
|
|
|
|
'type': 'shared_library',
|
|
|
|
'sources': [
|
|
|
|
'../src/utils/SkLuaCanvas.cpp',
|
|
|
|
'../src/utils/SkLua.cpp',
|
|
|
|
],
|
|
|
|
'include_dirs': [
|
2015-07-28 16:54:52 +00:00
|
|
|
'../include/private',
|
2014-02-26 20:22:32 +00:00
|
|
|
# Lua exposes GrReduceClip which in turn requires src/core for SkTLList
|
|
|
|
'../src/gpu/',
|
|
|
|
'../src/core/',
|
2013-06-20 17:15:05 +00:00
|
|
|
'../third_party/lua/src/',
|
|
|
|
],
|
|
|
|
'dependencies': [
|
|
|
|
'lua.gyp:lua',
|
|
|
|
'pdf.gyp:pdf',
|
|
|
|
'skia_lib.gyp:skia_lib',
|
|
|
|
],
|
|
|
|
'conditions': [
|
|
|
|
['skia_os != "win"',
|
|
|
|
{
|
|
|
|
'ldflags': [
|
|
|
|
'-Wl,-rpath,\$$ORIGIN,--enable-new-dtags',
|
|
|
|
],
|
|
|
|
},
|
|
|
|
],
|
|
|
|
],
|
|
|
|
},
|
|
|
|
],
|
|
|
|
},
|
|
|
|
],
|
2013-03-08 23:13:33 +00:00
|
|
|
['skia_win_debuggers_path and skia_os == "win"',
|
2013-03-08 18:43:35 +00:00
|
|
|
{
|
|
|
|
'targets': [
|
|
|
|
{
|
|
|
|
'target_name': 'win_dbghelp',
|
|
|
|
'type': 'static_library',
|
|
|
|
'defines': [
|
|
|
|
'SK_CDB_PATH="<(skia_win_debuggers_path)"',
|
|
|
|
],
|
|
|
|
'sources': [
|
|
|
|
'../tools/win_dbghelp.h',
|
|
|
|
'../tools/win_dbghelp.cpp',
|
|
|
|
],
|
|
|
|
},
|
|
|
|
],
|
|
|
|
},
|
|
|
|
],
|
2013-04-02 14:12:38 +00:00
|
|
|
['skia_os == "win"',
|
|
|
|
{
|
|
|
|
'targets': [
|
|
|
|
{
|
|
|
|
'target_name': 'win_lcid',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
|
|
|
'../tools/win_lcid.cpp',
|
|
|
|
],
|
|
|
|
},
|
|
|
|
],
|
|
|
|
},
|
|
|
|
],
|
2014-07-31 12:58:44 +00:00
|
|
|
['skia_os == "mac"',
|
|
|
|
{
|
|
|
|
'targets': [
|
|
|
|
{
|
|
|
|
'target_name': 'create_test_font',
|
|
|
|
'type': 'executable',
|
|
|
|
'sources': [
|
|
|
|
'../tools/create_test_font.cpp',
|
|
|
|
],
|
|
|
|
'include_dirs': [
|
2015-07-28 16:54:52 +00:00
|
|
|
'../include/private',
|
2014-07-31 12:58:44 +00:00
|
|
|
'../src/core',
|
|
|
|
],
|
|
|
|
'dependencies': [
|
|
|
|
'flags.gyp:flags',
|
|
|
|
'skia_lib.gyp:skia_lib',
|
|
|
|
'resources',
|
|
|
|
],
|
|
|
|
},
|
|
|
|
],
|
|
|
|
},
|
|
|
|
],
|
2011-06-02 14:38:23 +00:00
|
|
|
],
|
|
|
|
}
|