SkPDF: Fix Type3 ToUnicode table.

SkPDF: Fix Type3 ToUnicode table.

This seems to fix text extraction on Adobe Reader

  - Registry/Ordering is now set to Skia/SkiaOrdering.
  - Type3 fonts now get a FontDescriptor (force symbolic font).
  - CMapName is now Skia-Identity-SkiaOrdering
  - CMap behaves correctly for single-byte fonts.

Also:
  - SkTestTypeface returns tounicode map for testing.
  - Unit test updated

All PDFs render the same

BUG=skia:5606
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2292303004

Committed: https://skia.googlesource.com/skia/+/3d01c62e19df9f369cdfaeff82ec8af2c0be75f1

[hal.canary, 2016-08-31 19:23:39]

Description was changed from

==========
SkPDF: Fix Type3 ToUnicode table.

This seems to fix text extraction on Adobe Reader

  - Registry/Ordering is now set to Skia/SkiaOrdering.
  - Type3 fonts now get a FontDescriptor (force symbolic font).
  - CMapName is now Skia-Identity-SkiaOrdering
  - CMap behaves correctly for single-byte fonts.

Also:
  - SkTestTypeface returns tounicode map for testing.
  - Unit test updated

All PDFs render the same

BUG=skia:5606
==========

to

==========
SkPDF: Fix Type3 ToUnicode table.

This seems to fix text extraction on Adobe Reader

  - Registry/Ordering is now set to Skia/SkiaOrdering.
  - Type3 fonts now get a FontDescriptor (force symbolic font).
  - CMapName is now Skia-Identity-SkiaOrdering
  - CMap behaves correctly for single-byte fonts.

Also:
  - SkTestTypeface returns tounicode map for testing.
  - Unit test updated

All PDFs render the same

BUG=skia:5606
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2292303004
==========

[hal.canary, 2016-08-31 19:24:53]

The CQ bit was checked by halcanary@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-08-31 19:25:03]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[hal.canary, 2016-08-31 19:25:04]

halcanary@google.com changed reviewers:
+ bungeman@google.com

[hal.canary, 2016-08-31 19:25:04]

ptal

[bungeman-skia, 2016-08-31 19:37:49]

lgtm

[bungeman-skia, 2016-08-31 19:38:18]

The CQ bit was unchecked by bungeman@google.com

[bungeman-skia, 2016-08-31 19:38:21]

The CQ bit was checked by bungeman@google.com

[commit-bot: I haz the power, 2016-08-31 19:38:31]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-31 19:52:38]

Description was changed from

==========
SkPDF: Fix Type3 ToUnicode table.

This seems to fix text extraction on Adobe Reader

  - Registry/Ordering is now set to Skia/SkiaOrdering.
  - Type3 fonts now get a FontDescriptor (force symbolic font).
  - CMapName is now Skia-Identity-SkiaOrdering
  - CMap behaves correctly for single-byte fonts.

Also:
  - SkTestTypeface returns tounicode map for testing.
  - Unit test updated

All PDFs render the same

BUG=skia:5606
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2292303004
==========

to

==========
SkPDF: Fix Type3 ToUnicode table.

This seems to fix text extraction on Adobe Reader

  - Registry/Ordering is now set to Skia/SkiaOrdering.
  - Type3 fonts now get a FontDescriptor (force symbolic font).
  - CMapName is now Skia-Identity-SkiaOrdering
  - CMap behaves correctly for single-byte fonts.

Also:
  - SkTestTypeface returns tounicode map for testing.
  - Unit test updated

All PDFs render the same

BUG=skia:5606
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2292303004

Committed:
https://skia.googlesource.com/skia/+/3d01c62e19df9f369cdfaeff82ec8af2c0be75f1
==========

[commit-bot: I haz the power, 2016-08-31 19:52:39]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/3d01c62e19df9f369cdfaeff82ec8af2c0be75f1

[Rik, 2016-09-01 17:41:21]

cabanier@adobe.com changed reviewers:
+ cabanier@adobe.com

[Rik, 2016-09-01 17:41:21]

https://codereview.chromium.org/2292303004/diff/1/src/pdf/SkPDFFont.cpp
File src/pdf/SkPDFFont.cpp (right):

https://codereview.chromium.org/2292303004/diff/1/src/pdf/SkPDFFont.cpp#newco...
src/pdf/SkPDFFont.cpp:429: sysInfo->insertString("Registry", "Skia");
This change isn't needed

https://codereview.chromium.org/2292303004/diff/1/src/pdf/SkPDFFont.cpp#newco...
src/pdf/SkPDFFont.cpp:431: sysInfo->insertString("Ordering", "SkiaOrdering");
Neither is this one

https://codereview.chromium.org/2292303004/diff/1/src/pdf/SkPDFMakeToUnicodeC...
File src/pdf/SkPDFMakeToUnicodeCmap.cpp (right):

https://codereview.chromium.org/2292303004/diff/1/src/pdf/SkPDFMakeToUnicodeC...
src/pdf/SkPDFMakeToUnicodeCmap.cpp:29: "/Ordering (SkiaOrdering)\n"
Please leave this as-is