Simplify SkPdfTokenLooper behavior.
authorscroggo@google.com <scroggo@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81>
Mon, 2 Dec 2013 22:34:45 +0000 (22:34 +0000)
committerscroggo@google.com <scroggo@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81>
Mon, 2 Dec 2013 22:34:45 +0000 (22:34 +0000)
commit36026de64400cbe91552c549cf9a906a0926fef3
tree0b1dc703a4bdd524ae09b8d45ea0243a51ce5dde
parent845da77bc04de38e3c410ee1e26d9df70ab7d0ee
Simplify SkPdfTokenLooper behavior.

Instead of passing a pointer to a pointer to a NULL looper, which can
then be set to point to a new looper (which then must be looped and
deleted), pass a pointer to the current looper. Each function can then
create a looper based on the parent (on the stack) and call loop()
itself.

Remove setUp(). Now that there is a pointer to the parent at creation
time, there is no need for this function.

Modify the constructors to only provide ones that are needed.

Add documentation.

Remove PdfInlineImageLooper::done(), which is never used.

R=mtklein@google.com

Review URL: https://codereview.chromium.org/83183004

git-svn-id: http://skia.googlecode.com/svn/trunk@12447 2bbb7eff-a529-9590-31e7-b0007b416f81
experimental/PdfViewer/inc/SkPdfTokenLooper.h
experimental/PdfViewer/src/SkPdfContext.cpp
experimental/PdfViewer/src/SkPdfOps.h
experimental/PdfViewer/src/SkPdfRenderer.cpp