tree c3bddd3d8f11d1ac1daa332215a79792448699e8
parent 08c6651ec45b8cf01e6b6bbe8f57b1052554de94
author Dan Sinclair <dsinclair@chromium.org> 1499710565 -0400
committer Chromium commit bot <commit-bot@chromium.org> 1499711828 +0000

Reland "typeface double delete"

This reverts commit 75e1ab05e731d99b9acdc86de47640720e848415.

Reason for revert: Using the custom skia smart pointer is the correct
approach over CFX_UnownedPtr.

Original change's description:
> Revert "typeface double delete"
> 
> This reverts commit ddf2418ba8e5d925909d7955ac22b33f37ccce44.
> 
> Reason for revert: not right
> 
> Original change's description:
> > typeface double delete
> > 
> > SkTypeface was doubly deleted at pdfium teardown
> > SkTypeface has two pointers but no owners.
> > Making the font cache an owner fixes the bug but
> > violates checkdeps rules. Let me know what to
> > do about that.
> > 
> > R=​dsinclair@chromium.org,npm@chromium.org
> > Bug: 736133
> > Change-Id: I756a41258a5ac86e70139d7a587c5da9bb7a707b
> > Reviewed-on: https://pdfium-review.googlesource.com/7270
> > Reviewed-by: Nicolás Peña <npm@chromium.org>
> > Commit-Queue: Cary Clark <caryclark@google.com>
> 
> TBR=dsinclair@chromium.org,caryclark@google.com,npm@chromium.org,caryclark@skia.org
> 
> Change-Id: I255f50acf2cbaecc85b8e5ac3893c1fbc77b6492
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 736133
> Reviewed-on: https://pdfium-review.googlesource.com/7310
> Reviewed-by: Cary Clark <caryclark@google.com>
> Commit-Queue: Cary Clark <caryclark@google.com>

TBR=dsinclair@chromium.org,caryclark@google.com,npm@chromium.org,caryclark@skia.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 736133
Change-Id: I0ab949e9da2fdab9559e7ffe0ceed24d8eb9b66a
Reviewed-on: https://pdfium-review.googlesource.com/7510
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>