diff options
| author |  Mike Klein <mtklein@chromium.org> | 2018-02-07 12:54:27 -0500 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-02-07 18:18:52 +0000 |
| commit | e083f7c9f2ed6640866e2d974aabb98e6f3d0712 (patch) | |
| tree | d80ce32276699b7401ff70537387fe97d441272d /tools/debugger | |
| parent | 4043702b9ac251a8191f12a5f6cbf5e4bc95be01 (diff) | |
Reland "remove SkCanvas::kIsOpaque_SaveLayerFlag"
This reverts the revert 9ff8c8c073ba553dc8b22bb9aacc23949292f72f.
Original:
This is a performance-only hint that no one but fuzzers
is using. It's even explicitly filtered out in Android.
The fuzzers have noticed they can trick us into allocating
uninitialized memory and treating it as opaque, blending
uninitialized pixels, etc.
Since no one's using this, we can just kill the bit.
Bug: skia:7566, chromium:808830
Docs-Preview: https://skia.org/?cl=105282
Change-Id: I4326c663f777aa373ff7ec9f319519da9729350d
Reviewed-on: https://skia-review.googlesource.com/105282
Reviewed-by: Mike Klein <mtklein@chromium.org>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'tools/debugger')
| -rw-r--r-- | tools/debugger/SkObjectParser.cpp | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/tools/debugger/SkObjectParser.cpp b/tools/debugger/SkObjectParser.cpp index 352a01da23..dae2cde92e 100644 --- a/tools/debugger/SkObjectParser.cpp +++ b/tools/debugger/SkObjectParser.cpp @@ -349,9 +349,6 @@ SkString* SkObjectParser::RegionToString(const SkRegion& region) { SkString* SkObjectParser::SaveLayerFlagsToString(SkCanvas::SaveLayerFlags saveLayerFlags) { SkString* mFlags = new SkString("SkCanvas::SaveFlags: "); - if (saveLayerFlags & SkCanvas::kIsOpaque_SaveLayerFlag) { - mFlags->append("kIsOpaque_SaveLayerFlag "); - } if (saveLayerFlags & SkCanvas::kPreserveLCDText_SaveLayerFlag) { mFlags->append("kPreserveLCDText_SaveLayerFlag "); } |