PDF: Now threadsafe!

src/pdf/SkPDFCanon.h

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 #ifndef SkPDFCanon_DEFINED
 #define SkPDFCanon_DEFINED
 
 #include "SkPDFShader.h"
-#include "SkThread.h"
 #include "SkTDArray.h"
 
-struct SkIRect;
 class SkBitmap;
-class SkMatrix;
 class SkPDFFont;
 class SkPDFGraphicState;
 class SkPDFBitmap;
 class SkPaint;
 
-// This class's fields and methods will eventually become part of
-// SkPDFDocument/SkDocument_PDF.  For now, it exists as a singleton to
-// preflight that transition.  This replaces three global arrays in
-// SkPDFFont, SkPDFShader, and SkPDFGraphicsContext.
-//
-// IF YOU ARE LOOKING AT THIS API PLEASE DO NOT WRITE THE CHANGE
-// YOU ARE ABOUT TO WRITE WITHOUT TALKING TO HALCANARY@.
-//
-// Note that this class does not create, delete, reference or
-// dereference the SkPDFObject objects that it indexes.  It is up to
-// the caller to manage the lifetime of these objects.
+/**
+ *  The SkPDFCanon canonicalizes objects across PDF pages(SkPDFDevices).
+ *
+ *  The PDF backend works correctly if:
+ *  -  There is no more than one SkPDFCanon for each thread.
+ *  -  Every SkPDFDevice is given a pointer to a SkPDFCanon on creation.
+ *  -  All SkPDFDevices in a document share the same SkPDFCanon.
+ *  The SkDocument_PDF class makes this happen by owning a single
+ *  SkPDFCanon.
+ *
+ *  Note that this class does not create, delete, reference or
+ *  dereference the SkPDFObject objects that it indexes.  It is up to
+ *  the caller to manage the lifetime of these objects.
+ */
 class SkPDFCanon : SkNoncopyable {
 public:
     SkPDFCanon();
     ~SkPDFCanon();
 
-    static SkPDFCanon& GetCanon();
-
-    // This mutexes will be removed once this class is subsumed into
-    // SkPDFDocument.
-    static SkBaseMutex& GetFontMutex();
-    static SkBaseMutex& GetShaderMutex();
-    static SkBaseMutex& GetPaintMutex();
-    static SkBaseMutex& GetBitmapMutex();
-
     // Returns exact match if there is one.  If not, it returns NULL.
     // If there is no exact match, but there is a related font, we
     // still return NULL, but also set *relatedFont.
     SkPDFFont* findFont(uint32_t fontID,
                         uint16_t glyphID,
                         SkPDFFont** relatedFont) const;
     void addFont(SkPDFFont* font, uint32_t fontID, uint16_t fGlyphID);
     void removeFont(SkPDFFont*);
 
     SkPDFFunctionShader* findFunctionShader(const SkPDFShader::State&) const;
@@ skipping 37 matching lines @@
 
     SkTDArray<SkPDFAlphaFunctionShader*> fAlphaShaderRecords;
 
     SkTDArray<SkPDFImageShader*> fImageShaderRecords;
 
     SkTDArray<SkPDFGraphicState*> fGraphicStateRecords;
 
     SkTDArray<SkPDFBitmap*> fBitmapRecords;
 };
 #endif  // SkPDFCanon_DEFINED