History log of /external/skia/src/core/SkRecordPattern.h
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
6b20a559968e1ed0d3aa4ecbd6ddda9e681b4f86 07-Feb-2017 Hal Canary <halcanary@google.com> Make header files self-sufficient

Change-Id: Ice7d761b1023da77e50e5d6aa597964f7d9aa1d8
Reviewed-on: https://skia-review.googlesource.com/8302
Commit-Queue: Hal Canary <halcanary@google.com>
Reviewed-by: Mike Reed <reed@google.com>
/external/skia/src/core/SkRecordPattern.h
343a63d082bda969d7e8a4e09ba850e931185269 22-Mar-2016 mtklein <mtklein@chromium.org> SkRecord: infer return type for visit() and mutate().

Review URL: https://codereview.chromium.org/1824983003
/external/skia/src/core/SkRecordPattern.h
24e7db8b2ea663f8fe4d7bbfa1d686fd643990bb 19-Nov-2015 mtklein <mtklein@chromium.org> Modernize SkRecordPattern.h

- Fold Or, Or3, Or4 into one flexible Or.
- Fold Pattern1...Pattern7 into one flexible Pattern.
- Rename Star Greedy

Still fighting with a flexible get<N>() method instead of first, second, third, etc.

BUG=skia:

Review URL: https://codereview.chromium.org/1465443002
/external/skia/src/core/SkRecordPattern.h
449d9b7e2d1b2e20963f18639c6e541ef953f069 28-Sep-2015 mtklein <mtklein@chromium.org> simplify code in SkRecords.h

- use C++11 features ({} init, move constructors) to eliminate the need
for explicit constructors
- collapse RECORD0...RECORD8 into just one RECORD macro
- explicitly tag record types instead of using member detectors.

Removing member detectors makes this code significantly less fragile.

This exposes a few places where we didn't really think through what to do
with SkDrawable. I've marked them TODO for now.

BUG=skia:

Review URL: https://codereview.chromium.org/1360943003
/external/skia/src/core/SkRecordPattern.h
96fcdcc219d2a0d3579719b84b28bede76efba64 27-Aug-2015 halcanary <halcanary@google.com> Style Change: NULL->nullptr
DOCS_PREVIEW= https://skia.org/?cl=1316233002

Review URL: https://codereview.chromium.org/1316233002
/external/skia/src/core/SkRecordPattern.h
c6ad06acefa096716f8dabed5342f9b89dc43dfe 19-Aug-2015 mtklein <mtklein@chromium.org> unsigned -> int for counts and indices in picture-related code

also, (C)

BUG=skia:

Review URL: https://codereview.chromium.org/1300163002
/external/skia/src/core/SkRecordPattern.h
dc0f408a961b6ce4e7631e06031bbfe1d8d7bcec 26-Jan-2015 kkinnunen <kkinnunen@nvidia.com> Fold alpha to the inner savelayer in savelayer-savelayer-restore patterns

Fold alpha to the inner savelayer in savelayer-savelayer-restore
patterns such as this:

SaveLayer (non-opaque)
Save
ClipRect
SaveLayer
Restore
Restore
Restore

Current blink generates these for example for SVG content such as this:

<path style="opacity:0.5 filter:url(#blur_filter)"/>

The outer save layer is due to the opacity and the inner one is due to
blur filter being implemented with picture image filter.

Reduces layers in desk_carsvg.skp testcase from 115 to 78.

BUG=skia:3119

Review URL: https://codereview.chromium.org/835973005
/external/skia/src/core/SkRecordPattern.h
99d6a9ee8b3516de892d118c71aa5e6e5c865efd 11-Sep-2014 mtklein <mtklein@chromium.org> Fix a bug in Save-Restore no-op optimization.

We optimize
Save
SaveLayer
Restore
Restore
into
NoOp
NoOp
NoOp
Restore

I'm considering skipping the call to SkRecordOptimize again just to eliminate
this extra variable from landing SkRecord. Thoughts?

BUG=skia:
R=robertphillips@google.com, mtklein@google.com

Author: mtklein@chromium.org

Review URL: https://codereview.chromium.org/560163002
/external/skia/src/core/SkRecordPattern.h
887f3979f0c717e69a8b7d169169bc27eb46d3b5 17-Jun-2014 mtklein <mtklein@chromium.org> Add EXPERIMENTAL_beginRecording() for SkRecord-based recording.

The interesting stuff is in SkPictureRecorder.{h,cpp}. The rest is mostly moving SkRecord from its own directories into core to avoid circular dependencies in GYP.

After plumbing SkRecord all the way through in Picture, I'll delete its old entry point include/record/SkRecording.h. For now it and record.gypi need to stay where they are to keep Chrome building.

BUG=skia:
R=reed@google.com, mtklein@google.com

Author: mtklein@chromium.org

Review URL: https://codereview.chromium.org/331573004
/external/skia/src/core/SkRecordPattern.h