SkPDF: SkPDFDocument reorganized so that some objects can be serialized early.

src/pdf/SkPDFDocument.h

 /*
  * Copyright 2016 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 #ifndef SkPDFDocument_DEFINED
 #define SkPDFDocument_DEFINED
 
 #include "SkDocument.h"
+#include "SkPDFMetadata.h"
 

[tomhudson, 2016/03/21 15:55:12]

You also need to #include SkPDFTypes.h here to get SkPDFObjNumMap et al!?

[hal.canary, 2016/03/21 18:49:18]

Done.

 sk_sp<SkDocument> SkPDFMakeDocument(
         SkWStream* stream,
         void (*doneProc)(SkWStream*, bool),
         SkScalar rasterDpi,
         SkPixelSerializer* jpegEncoder);
 
+// Logically part of SkPDFDocument (like SkPDFCanon), but

[djsollen, 2016/03/16 12:41:49]

but what?

[hal.canary, 2016/03/21 18:49:18]

Done.

+struct SkPDFObjectSerializer : SkNoncopyable {
+    SkPDFObjNumMap fObjNumMap;
+    SkPDFSubstituteMap fSubstituteMap;
+    SkTDArray<int32_t> fOffsets;
+    sk_sp<SkPDFObject> fInfoDict;
+    size_t fBaseOffset;
+    int32_t fNextToBeSerialized;  // index in fObjNumMap
+
+    SkPDFObjectSerializer();
+    void addObjectRecursively(const sk_sp<SkPDFObject>&);
+    void serializeHeader(SkWStream*, const SkPDFMetadata&);
+    void serializeObjects(SkWStream*);
+    void serializeFooter(SkWStream*, const sk_sp<SkPDFObject>, sk_sp<SkPDFObject>);
+    int32_t offset(SkWStream*);
+};
+
+class SkPDFDocument : public SkDocument {

[tomhudson, 2016/03/21 15:55:12]

This is an implementation detail of SkDocument, buried in src/. Is there really
*nothing* that a developer needs to know to understand the relationship between
the two?

(From my reading the code the answer may very well be yes, but I want explicit
agreement before landing without doxygen.)

[hal.canary, 2016/03/21 19:40:08]

/** Concrete implementation of SkDocument that creates PDF files. This
    class does not produced linearized or optimized PDFs; instead it
    it attempts to use a minimum amount of RAM. */

+public:
+    SkPDFDocument(SkWStream*,
+                  void (*)(SkWStream*, bool),
+                  SkScalar,
+                  SkPixelSerializer*);
+    virtual ~SkPDFDocument();
+    SkCanvas* onBeginPage(SkScalar, SkScalar, const SkRect&) override;
+    void onEndPage() override;
+    bool onClose(SkWStream*) override;
+    void onAbort() override;
+    void setMetadata(const SkDocument::Attribute[],
+                     int,
+                     const SkTime::DateTime*,
+                     const SkTime::DateTime*) override;
+    void serialize(const sk_sp<SkPDFObject>&);
+    SkPDFCanon* canon() { return &fCanon; }
+
+private:
+    SkPDFObjectSerializer fObjectSerializer;
+    SkPDFCanon fCanon;
+    SkTArray<sk_sp<const SkPDFDevice>> fPageDevices;
+    sk_sp<SkCanvas> fCanvas;
+    SkScalar fRasterDpi;
+    SkPDFMetadata fMetadata;
+};
+
 #endif  // SkPDFDocument_DEFINED