mirror of http://codereview.chromium.org/10825089/

roll skia to 4889
Review URL: https://chromiumcodereview.appspot.com/10827122

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149468 0039d316-1c4b-4281-b951-d872f2087c98

DEPS

@@ -29,7 +29,7 @@ vars = {
   "ffmpeg_hash": "4287c0271f399b98eb6851e6e650098f26b8c3c2",
 
   "sfntly_revision": "134",
-  "skia_revision": "4834",
+  "skia_revision": "4889",
   # Three lines of non-changing comments so that
   # the commit queue can handle CLs rolling Skia
   # and V8 without interference from each other.

skia/skia.gyp

@@ -254,6 +254,8 @@
         '../third_party/skia/src/core/SkMallocPixelRef.cpp',
         '../third_party/skia/src/core/SkMask.cpp',
         '../third_party/skia/src/core/SkMaskFilter.cpp',
+        '../third_party/skia/src/core/SkMaskGamma.cpp',
+        '../third_party/skia/src/core/SkMaskGamma.h',
         '../third_party/skia/src/core/SkMath.cpp',
         '../third_party/skia/src/core/SkMatrix.cpp',
         '../third_party/skia/src/core/SkMetaData.cpp',
@@ -548,8 +550,6 @@
         '../third_party/skia/src/ports/SkFontHost_android.cpp',
         #'../third_party/skia/src/ports/SkFontHost_ascender.cpp',
         '../third_party/skia/src/ports/SkFontHost_tables.cpp',
-        '../third_party/skia/src/ports/SkFontHost_gamma.cpp',
-        '../third_party/skia/src/ports/SkFontHost_gamma_none.cpp',
         #'../third_party/skia/src/ports/SkFontHost_linux.cpp',
         '../third_party/skia/src/ports/SkFontHost_mac.cpp',
         #'../third_party/skia/src/ports/SkFontHost_none.cpp',
@@ -571,7 +571,6 @@
         #'../third_party/skia/src/ports/SkXMLParser_expat.cpp',
         #'../third_party/skia/src/ports/SkXMLParser_tinyxml.cpp',
         #'../third_party/skia/src/ports/SkXMLPullParser_expat.cpp',
-        '../third_party/skia/src/ports/sk_predefined_gamma.h',
 
         '../third_party/skia/src/sfnt/SkOTUtils.cpp',
         '../third_party/skia/src/sfnt/SkOTUtils.h',
@@ -822,6 +821,8 @@
       ],
       'msvs_disabled_warnings': [4244, 4267, 4341, 4345, 4390, 4554, 4800],
       'defines': [
+        'SK_GAMMA_SRGB',
+        #'SK_GAMMA_APPLY_TO_A8',
         'SK_BUILD_NO_IMAGE_ENCODE',
         'GR_GL_CUSTOM_SETUP_HEADER="GrGLConfig_chrome.h"',
         'GR_STATIC_RECT_VB=1',
@@ -870,8 +871,6 @@
         [ 'OS != "android"', {
           'sources/': [
             ['exclude', '_android\\.(cc|cpp)$'],
-            # Below files are only used by Android
-            ['exclude', '../third_party/skia/src/ports/SkFontHost_gamma\\.cpp$'],
           ],
           'defines': [
             'SK_DEFAULT_FONT_CACHE_LIMIT=(20*1024*1024)',
@@ -925,8 +924,6 @@
           'sources/': [ ['exclude', '_linux\\.(cc|cpp)$'] ],
           'sources!': [
             '../third_party/skia/src/ports/SkFontHost_FreeType.cpp',
-            '../third_party/skia/src/ports/SkFontHost_TryeType_Tables.cpp',
-            '../third_party/skia/src/ports/SkFontHost_gamma_none.cpp',
           ],
         }],
         [ 'use_aura == 1 and use_canvas_skia == 1', {
@@ -984,7 +981,6 @@
               'sources!': [
                 'ext/vector_platform_device_skia.cc',
                 '../third_party/skia/src/pdf/SkPDFFont.cpp',
-                '../third_party/skia/src/ports/SkFontHost_gamma_none.cpp',
               ],
               'export_dependent_settings': [
                 '../third_party/harfbuzz/harfbuzz.gyp:harfbuzz',
@@ -1094,6 +1090,10 @@
       'direct_dependent_settings': {
         'include_dirs': [
           'config',
+
+          #temporary until we can hide SkFontHost
+          '../third_party/skia/src/core',
+
           '../third_party/skia/include/config',
           '../third_party/skia/include/core',
           '../third_party/skia/include/effects',
@@ -1258,6 +1258,9 @@
           'sources': [
             '../third_party/skia/src/opts/memset16_neon.S',
             '../third_party/skia/src/opts/memset32_neon.S',
+            '../src/opts/SkBitmapProcState_matrixProcs_neon.cpp',
+            '../src/opts/SkBitmapProcState_matrix_clamp_neon.h',
+            '../src/opts/SkBitmapProcState_matrix_repeat_neon.h',
         ],
         }],
         [ 'target_arch == "arm" and armv7 != 1', {

skia/skia_test_expectations.txt

@@ -48,4 +48,28 @@
 //
 // START OVERRIDES HERE
 
+//Skia roll to r4843
+BUGCR139863 WIN : fast/css/shadow-multiple.html = IMAGE
+BUGCR139863 WIN : fast/multicol/shadow-breaking.html = IMAGE
+BUGCR139863 WIN : fast/repaint/shadow-multiple-vertical.html = IMAGE
+BUGCR139863 WIN : fast/transforms/shadows.html = IMAGE
+BUGCR139863 WIN : fast/text/shadow-translucent-fill.html = IMAGE
+BUGCR139863 WIN : fast/text/stroking-decorations.html = IMAGE
+BUGCR139863 WIN : fast/text/stroking.html = IMAGE
+BUGCR139863 WIN : fast/writing-mode/english-lr-text.html = IMAGE
+BUGCR139863 WIN : ietestcenter/css3/text/textshadow-002.htm = IMAGE
+BUGCR139863 WIN : ietestcenter/css3/text/textshadow-010.htm = IMAGE
+BUGCR139863 WIN : svg/css/text-gradient-shadow.svg = IMAGE
+BUGCR139863 WIN : svg/css/text-shadow-multiple.xhtml = IMAGE
+BUGCR139863 WIN : compositing/shadows/shadow-drawing.html = IMAGE
+//The following two do image comparison in javascript.
+BUGCR139863 WIN : fast/canvas/fillText-shadow.html = TEXT
+BUGCR139863 WIN : platform/chromium/virtual/gpu/fast/canvas/fillText-shadow.html = TEXT
+BUGCR139863 WIN : svg/css/composite-shadow-text.svg = IMAGE
+BUGCR139863 WIN : svg/css/group-with-shadow.svg = IMAGE
+BUGCR139863 WIN : svg/css/shadow-changes.svg = IMAGE
+BUGCR139863 WIN : transitions/svg-text-shadow-transition.html = IMAGE
+BUGCR139863 LINUX : platform/chromium-linux/fast/text/chromium-linux-fontconfig-renderstyle.html = IMAGE
+BUGCR139863 MAC : fast/repaint/shadow-multiple-horizontal.html = IMAGE
+
 // END OVERRIDES HERE (this line ensures that the file is newline-terminated)