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

Issue 11779016: Roll Skia DEPS to r7044 & add SK_IGNORE_TRANS_CLAMP_FIX and SK_IGNORE_FAST_SCALEMATRIX_INVERT (Closed)

Created:
7 years, 11 months ago by robertphillips
Modified:
7 years, 11 months ago
Reviewers:
schenney, reed1
CC:
chromium-reviews, skia-dev_google.com
Visibility:
Public.

Description

Roll Skia DEPS to r7044 & add SK_IGNORE_TRANS_CLAMP_FIX and SK_IGNORE_FAST_SCALEMATRIX_INVERT control: https://codereview.chromium.org/11791014/ Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=175328

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download
M skia/skia.gyp View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
robertphillips
I believe this is 'good'. The discrepancies between the roll and control are: win roll: ...
7 years, 11 months ago (2013-01-07 13:06:19 UTC) #1
bsalomon
On 2013/01/07 13:06:19, robertphillips wrote: > I believe this is 'good'. The discrepancies between the ...
7 years, 11 months ago (2013-01-07 13:39:06 UTC) #2
schenney
Which of those gyp flags need to go into WebKit's skia gyp? That is, which ...
7 years, 11 months ago (2013-01-07 13:41:57 UTC) #3
robertphillips
7 years, 11 months ago (2013-01-07 13:46:23 UTC) #4
Message was sent while issue was closed.
Here are the layout tests impacted by these two new flags:

SK_IGNORE_FAST_SCALEMATRIX_INVERT
https://codereview.chromium.org/11797003/

  fast/borders/border-image-massive-scale.html [ ImageOnlyFailure ]
  fast/borders/border-image-outset-in-shorthand.html [ ImageOnlyFailure ]
  fast/borders/border-image-outset.html [ ImageOnlyFailure ]
  fast/borders/border-image-scaled.html [ ImageOnlyFailure ]    - mac-only?
  fast/canvas/canvas-incremental-repaint.html [ ImageOnlyFailure ]
  fast/speech/input-appearance-searchandspeech.html [ ImageOnlyFailure ] -
linux-only?
 
platform/chromium/virtual/softwarecompositing/visibility/visibility-image-layers.html
[ ImageOnlyFailure ] - mac-only?
  svg/W3C-SVG-1.1-SE/pservers-grad-17-b.svg [ ImageOnlyFailure ]
  svg/W3C-SVG-1.1-SE/pservers-grad-20-b.svg [ ImageOnlyFailure ]
  svg/W3C-SVG-1.1/pservers-grad-17-b.svg [ ImageOnlyFailure ]
  svg/custom/non-scaling-stroke.svg [ ImageOnlyFailure ]
  tables/mozilla/bugs/bug137388-2.html [ ImageOnlyFailure ]
  tables/mozilla/bugs/bug23235.html [ ImageOnlyFailure ] - linux-only?
  tables/mozilla/bugs/bug4093.html [ ImageOnlyFailure ]

SK_IGNORE_TRANS_CLAMP_FIX
https://codereview.chromium.org/11801004/

  fast/borders/border-image-outset-split-inline-vertical-lr.html [
ImageOnlyFailure ]
  fast/borders/border-image-outset-split-inline.html [ ImageOnlyFailure ]
  fast/borders/border-image-slice-constrained.html [ ImageOnlyFailure ]
  fast/writing-mode/border-image-horizontal-bt.html [ ImageOnlyFailure ] -
mac-only?
  fast/writing-mode/border-image-vertical-lr.html [ ImageOnlyFailure ] -
mac-only?
  fast/writing-mode/border-image-vertical-rl.html [ ImageOnlyFailure ] -
mac-only?
  svg/W3C-SVG-1.1/coords-units-01-b.svg [ ImageOnlyFailure ]
  svg/batik/filters/feTile.svg [ ImageOnlyFailure ] - linux-only?
  svg/filters/feTile.svg [ ImageOnlyFailure ]

Powered by Google App Engine
This is Rietveld 408576698