Add clipping support and some small fixes.

Add clipping support and some small fixes.

Reorganize how the PDF graphic state stack is managed (fixing several bugs incidentally).
Style: fix variables with underscores.
Bug: fix image matrix application order, which enabled a small refactor.

Committed: http://code.google.com/p/skia/source/detail?r=622

[agl, 2010-10-28 22:23:03]

On Thu, Oct 28, 2010 at 6:21 PM,  <vandebo@chromium.org> wrote:
> Description:
> Add clipping support and some small fixes.

On vacation until Tues or Wed next week.


AGL

[agl, 2010-11-02 17:08:05]

LGTM

http://codereview.appspot.com/2771042/diff/1/src/pdf/SkPDFDevice.cpp
File src/pdf/SkPDFDevice.cpp (right):

http://codereview.appspot.com/2771042/diff/1/src/pdf/SkPDFDevice.cpp#newcode92
src/pdf/SkPDFDevice.cpp:92: SkPath::FillType clipFill;
No need to declare this before assigning it, below.

[Steve VanDeBogart, 2010-11-03 23:55:11]

http://codereview.appspot.com/2771042/diff/1/src/pdf/SkPDFDevice.cpp
File src/pdf/SkPDFDevice.cpp (right):

http://codereview.appspot.com/2771042/diff/1/src/pdf/SkPDFDevice.cpp#newcode92
src/pdf/SkPDFDevice.cpp:92: SkPath::FillType clipFill;
On 2010/11/02 17:08:05, agl wrote:
> No need to declare this before assigning it, below.

Done.