|
|
Chromium Code Reviews|
Created:
7 years, 1 month ago by robertphillips Modified:
7 years, 1 month ago CC:
chromium-reviews, skia-team_google.com Base URL:
svn://svn.chromium.org/chrome/trunk/src Visibility:
Public. |
DescriptionRoll Skia DEPS to r12124
R=rmistry@google.com
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=232963
Patch Set 1 #Patch Set 2 : Suppress blur optimization #Messages
Total messages: 7 (0 generated)
This roll attempt is failing on the following 79 layout tests:
Regressions: Unexpected image-only failures (79)
css3/filters/blur-filter-page-scroll-parents.html [ ImageOnlyFailure ]
css3/filters/blur-filter-page-scroll-self.html [ ImageOnlyFailure ]
css3/filters/blur-filter-page-scroll.html [ ImageOnlyFailure ]
css3/filters/crash-filter-change.html [ ImageOnlyFailure ]
css3/filters/effect-blur.html [ ImageOnlyFailure ]
css3/filters/effect-brightness-clamping.html [ ImageOnlyFailure ]
css3/filters/effect-combined.html [ ImageOnlyFailure ]
css3/filters/effect-drop-shadow.html [ ImageOnlyFailure ]
css3/filters/effect-reference-colorspace.html [ ImageOnlyFailure ]
css3/filters/effect-reference-hidpi.html [ ImageOnlyFailure ]
css3/filters/effect-reference-ordering.html [ ImageOnlyFailure ]
css3/filters/effect-reference-subregion.html [ ImageOnlyFailure ]
css3/filters/effect-reference-zoom.html [ ImageOnlyFailure ]
css3/filters/effect-reference.html [ ImageOnlyFailure ]
css3/filters/filter-change-repaint.html [ ImageOnlyFailure ]
css3/filters/filter-repaint-blur.html [ ImageOnlyFailure ]
css3/filters/filter-repaint-child-layers.html [ ImageOnlyFailure ]
css3/filters/filter-repaint-shadow.html [ ImageOnlyFailure ]
css3/filters/filter-repaint.html [ ImageOnlyFailure ]
css3/filters/filtered-inline.html [ ImageOnlyFailure ]
css3/filters/regions-expanding.html [ ImageOnlyFailure ]
css3/masking/clip-path-circle-filter.html [ ImageOnlyFailure ]
svg/W3C-SVG-1.1/filters-example-01-b.svg [ ImageOnlyFailure ]
svg/W3C-SVG-1.1/filters-gauss-01-b.svg [ ImageOnlyFailure ]
svg/batik/text/smallFonts.svg [ ImageOnlyFailure ]
svg/batik/text/textEffect.svg [ ImageOnlyFailure ]
svg/batik/text/textEffect3.svg [ ImageOnlyFailure ]
svg/batik/text/textFeatures.svg [ ImageOnlyFailure ]
svg/batik/text/textProperties.svg [ ImageOnlyFailure ]
svg/css/path-gradient-stroke-shadow.svg [ ImageOnlyFailure ]
svg/css/rect-gradient-stroke-shadow.svg [ ImageOnlyFailure ]
svg/custom/image-with-transform-clip-filter.svg [ ImageOnlyFailure ]
svg/custom/recursive-filter.svg [ ImageOnlyFailure ]
svg/custom/resource-invalidate-on-target-update.svg [ ImageOnlyFailure ]
svg/custom/text-filter.svg [ ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDiffuseLightingElement-dom-diffuseConstant-attr.html
[ ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDiffuseLightingElement-dom-in-attr.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDiffuseLightingElement-dom-lighting-color-attr.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDiffuseLightingElement-dom-surfaceScale-attr.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDiffuseLightingElement-inherit-lighting-color-css-prop.html
[ ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDiffuseLightingElement-lighting-color-css-prop.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDiffuseLightingElement-svgdom-diffuseConstant-prop.html
[ ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDiffuseLightingElement-svgdom-in-prop.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDiffuseLightingElement-svgdom-surfaceScale-prop.html
[ ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDistantLightElement-dom-azimuth-attr.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDistantLightElement-dom-elevation-attr.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDistantLightElement-svgdom-azimuth-prop.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDistantLightElement-svgdom-elevation-prop.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDropShadowElement-dom-dx-attr.html [ ImageOnlyFailure
]
svg/dynamic-updates/SVGFEDropShadowElement-dom-dy-attr.html [ ImageOnlyFailure
]
svg/dynamic-updates/SVGFEDropShadowElement-dom-in-attr.html [ ImageOnlyFailure
]
svg/dynamic-updates/SVGFEDropShadowElement-dom-stdDeviation-attr.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDropShadowElement-svgdom-dx-prop.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDropShadowElement-svgdom-dy-prop.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDropShadowElement-svgdom-in-prop.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEDropShadowElement-svgdom-stdDeviation-prop.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEGaussianBlurElement-dom-in-attr.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEGaussianBlurElement-dom-stdDeviation-attr.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEGaussianBlurElement-dom-stdDeviation-call.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEGaussianBlurElement-svgdom-in-prop.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEPointLightElement-dom-x-attr.html [ ImageOnlyFailure
]
svg/dynamic-updates/SVGFEPointLightElement-dom-y-attr.html [ ImageOnlyFailure
]
svg/dynamic-updates/SVGFEPointLightElement-dom-z-attr.html [ ImageOnlyFailure
]
svg/dynamic-updates/SVGFEPointLightElement-svgdom-x-prop.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEPointLightElement-svgdom-y-prop.html [
ImageOnlyFailure ]
svg/dynamic-updates/SVGFEPointLightElement-svgdom-z-prop.html [
ImageOnlyFailure ]
svg/filters/feDropShadow-zero-deviation.svg [ ImageOnlyFailure ]
svg/filters/feDropShadow.svg [ ImageOnlyFailure ]
svg/filters/feGaussianBlur-zero-deviation.svg [ ImageOnlyFailure ]
svg/filters/feGaussianBlur.svg [ ImageOnlyFailure ]
svg/filters/filter-clip.svg [ ImageOnlyFailure ]
svg/filters/filter-on-filter-for-text.svg [ ImageOnlyFailure ]
svg/filters/filter-width-update.svg [ ImageOnlyFailure ]
svg/filters/filterRes.svg [ ImageOnlyFailure ]
svg/filters/invalidate-on-child-layout.svg [ ImageOnlyFailure ]
svg/filters/shadow-on-filter.svg [ ImageOnlyFailure ]
svg/repaint/filter-child-repaint.svg [ ImageOnlyFailure ]
virtual/softwarecompositing/culling/filter-occlusion-blur-large.html [
ImageOnlyFailure ]
virtual/softwarecompositing/culling/filter-occlusion-blur.html [
ImageOnlyFailure ]
Note that the following 2 layout tests are not failing on Mac:
virtual/softwarecompositing/culling/filter-occlusion-blur-large.html [
ImageOnlyFailure ]
virtual/softwarecompositing/culling/filter-occlusion-blur.html [
ImageOnlyFailure ]
The discrepancies between the roll and control are:
android_dbg
roll:
slave_steps
preamble
zip_build
control:
lost bot
linux
roll:
buildrunner_tests
preamble
check_deps
control:
gclient_revert failed
update r232882 failed
mac
roll:
buildrunner_tests
preamble
telemetry_perf_unittests failed 2
testCacheHandled
testCalcResultsTraceEvents
control:
still running?
win
roll:
buildrunner_tests
preamble
telemetry_perf_unittests failed 2
testCacheHandled
testCalcResultsTraceEvents
content_unittests failed 1
SingleGesture
content_browsertests did not complete failed 6
SerializeHTMLDOMWithNoMetaCharsetInOriginalDoc ***************
ResumeInterruptedDownloadBadPrecondition *********************
AccessibilityAOnclick ****************************************
TestGetUserMediaConstraints **********************************
LimitTotal
WebSocketSharedWorker
control:
buildrunner_tests
preamble
telemetry_perf_unittests failed 2
testCacheHandled
testCalcResultsTraceEvents
content_unittests failed 2
SingleGesture
TestValidInputRates
content_browsertests did not complete failed 5
GetAudioAndVideoStreamAndClone
GetAudioAndVideoStreamAndStop
GetVideoStreamAndStop
LimitTotal
WebSocketSharedWorker
linux_layout
roll:
webkit_tests 213 fixable (0 skipped) failed 79
...
control:
green
linux_layout_rel
roll:
webkit_tests 189 fixable (0 skipped) failed 79
...
control:
green
mac_layout
roll:
webkit_tests 763 fixable (0 skipped) failed 77
...
control:
green
mac_layout_rel
roll:
webkit_tests 749 fixable (0 skipped) failed 77
...
control:
green
win_layout
roll:
webkit_tests 287 fixable (0 skipped) failed 79
...
control:
green
win_layout_rel
roll:
webkit_tests 249 fixable (0 skipped) failed 79
...
control:
green
The majority of these changes are what you would expect from r12113 (Change SkBlurImageFilter to use fixed-point division https://codereview.chromium.org/57513002) i.e., imperceptible blur changes. The following four image differences are perceptible however: svg/dynamic-updates/SVGFEGaussianBlurElement-svgdom-in-prop.html svg/dynamic-updates/SVGFEGaussianBlurElement-dom-stdDeviation-call.html svg/dynamic-updates/SVGFEGaussianBlurElement-dom-stdDeviation-attr.html svg/dynamic-updates/SVGFEGaussianBlurElement-dom-in-attr.html and the blurred area has shrunk in all of them.
Added the suppression to skia_library.gypi & reran the bots. This looks ready to go.
LGTM
Message was sent while issue was closed.
Committed patchset #2 manually as r232963 (presubmit successful).
Message was sent while issue was closed.
Does the new impl of blur maintain opaqueness? i.e. if all the sample values are 0xFF, is the result 0xFF? Couldn't tell if we need to add anything in the code before shifting to ensure this. |
