Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Submit feedback
  • Sign in / Register
T
tangled
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Labels
    • Milestones
  • Merge Requests 0
    • Merge Requests 0
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Analytics
    • Analytics
    • CI / CD
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • eriksson monteiro
  • tangled
  • Repository

Switch branch/tag
  • tangled
  • third_party
  • WebKit
  • LayoutTests
  • css3
  • blending
  • effect-background-blend-mode-tiled-ex...
Find file HistoryPermalink
  • Xianzhu Wang's avatar
    [SPv175] Output bounds for SaveLayerOp · b7e8c6eb
    Xianzhu Wang authored Mar 20, 2018
    This is mainly for performance. Without bounds, skia may execute some
    code to find a finite bounds, and the bounds may be too big for the
    SaveLayerOp which may lead other cost.
    
    Cluster telemetry showed that this CL can improve raster_time by 8%.
    https://ct.skia.org/results/cluster-telemetry/tasks/chromium_perf_runs/wangxianzhu-20180318183406/html/index.html
    
    Pin-point showed 20% improvement for raster_time in
    rasterize_and_record_micro.top_25.
    https://pinpoint-dot-chromeperf.appspot.com/results2/14f80e29440000
    
    Another purpose is for some skia bug causing crbug.com/816854. In the
    bug, a small chunk with opacity without bounds in SaveLayerOp may
    cause ghost rasterization far from the target area.
    
    Bug: 803867,816854
    Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
    Change-Id: Ife282e44b09fb2978e850e4c8cd6ca4e0150c857
    Reviewed-on: https://chromium-review.googlesource.com/966699
    Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
    Reviewed-by: default avatarvmpstr <vmpstr@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#544561}
    b7e8c6eb
effect-background-blend-mode-tiled-expected.png 214 KB

Download (214 KB)

effect-background-blend-mode-tiled-expected.png

Replace effect-background-blend-mode-tiled-expected.png

Attach a file by drag & drop or click to upload


Cancel
A new branch will be created in your fork and a new merge request will be started.