SkPDF: PDF/A runtime switch

SkPDF: PDF/A runtime switch

TODO: remove gyp variable and modify API  in SkDocument.h

SkMD5 now moved into core as pdf depends on it now.

BUG=skia:3110
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1891873002

Committed: https://skia.googlesource.com/skia/+/570f18b43417d73c3fbd113cc0b4258e08b14c82

Committed: https://skia.googlesource.com/skia/+/488165e689baf0f215d5798c87d0031b58e4bc8d

[hal.canary, 2016-04-14 18:26:26]

Description was changed from

==========
SkPDF: PDF/A runtime switch

BUG=skia:3110
==========

to

==========
SkPDF: PDF/A runtime switch

BUG=skia:3110
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[hal.canary, 2016-04-14 18:27:09]

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

[commit-bot: I haz the power, 2016-04-14 18:27:19]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1891873002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1891873002/1

[commit-bot: I haz the power, 2016-04-14 18:34:25]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-14 18:34:26]

Dry run: Try jobs failed on following builders:
  Build-Ubuntu-GCC-Arm7-Debug-Android-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-Arm...)

[hal.canary, 2016-04-14 19:07:17]

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

[commit-bot: I haz the power, 2016-04-14 19:07:25]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1891873002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1891873002/20001

[commit-bot: I haz the power, 2016-04-14 19:08:32]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-14 19:08:33]

Dry run: Try jobs failed on following builders:
  Build-Mac-Clang-x86_64-Release-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Mac-Clang-x86_...)
  Build-Ubuntu-GCC-x86_64-Release-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-x86...)

[hal.canary, 2016-04-14 19:09:59]

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

[commit-bot: I haz the power, 2016-04-14 19:10:03]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1891873002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1891873002/40001

[hal.canary, 2016-04-14 19:10:05]

Patchset #2 (id:20001) has been deleted

[hal.canary, 2016-04-14 19:10:09]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2016-04-14 19:19:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-14 19:19:14]

Dry run: This issue passed the CQ dry run.

[hal.canary, 2016-04-14 20:53:48]

Description was changed from

==========
SkPDF: PDF/A runtime switch

BUG=skia:3110
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
SkPDF: PDF/A runtime switch

TODO: remove gyp variable and modify API  in SkDocument.h

BUG=skia:3110
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[hal.canary, 2016-04-14 20:53:56]

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

[hal.canary, 2016-04-14 20:54:00]

Patchset #1 (id:40001) has been deleted

[commit-bot: I haz the power, 2016-04-14 20:54:02]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1891873002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1891873002/60001

[hal.canary, 2016-04-14 21:02:17]

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

[hal.canary, 2016-04-14 21:02:25]

Patchset #1 (id:60001) has been deleted

[commit-bot: I haz the power, 2016-04-14 21:02:28]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1891873002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1891873002/100001

[hal.canary, 2016-04-14 21:02:38]

Patchset #1 (id:80001) has been deleted

[commit-bot: I haz the power, 2016-04-14 21:07:58]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-14 21:07:59]

Dry run: Try jobs failed on following builders:
  Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Debug-Trybot on client.skia (JOB_FAILED,
http://build.chromium.org/p/client.skia/builders/Test-Ubuntu-GCC-GCE-CPU-AVX2...)

[hal.canary, 2016-04-15 17:05:31]

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

[hal.canary, 2016-04-15 17:05:31]

PTAL
reed@ wants a runtime switch for PDF/A.  This makes sense since it will prevent
bit-rot.

A later CL will remove the compile-time master switch and replace it with a new
PDF SkDocument factory function.

[tomhudson, 2016-04-15 18:58:13]

The CQ bit was checked by tomhudson@google.com

[tomhudson, 2016-04-15 18:58:14]

lgtm

[commit-bot: I haz the power, 2016-04-15 18:58:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1891873002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1891873002/100001

[commit-bot: I haz the power, 2016-04-15 18:59:33]

Description was changed from

==========
SkPDF: PDF/A runtime switch

TODO: remove gyp variable and modify API  in SkDocument.h

BUG=skia:3110
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
SkPDF: PDF/A runtime switch

TODO: remove gyp variable and modify API  in SkDocument.h

BUG=skia:3110
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/570f18b43417d73c3fbd113cc0b4258e08b14c82
==========

[commit-bot: I haz the power, 2016-04-15 18:59:35]

Committed patchset #1 (id:100001) as
https://skia.googlesource.com/skia/+/570f18b43417d73c3fbd113cc0b4258e08b14c82

[hal.canary, 2016-04-15 19:30:02]

A revert of this CL (patchset #1 id:100001) has been created in
https://codereview.chromium.org/1892203002/ by halcanary@google.com.

The reason for reverting is: breaking something..

[hal.canary, 2016-04-18 14:25:29]

Description was changed from

==========
SkPDF: PDF/A runtime switch

TODO: remove gyp variable and modify API  in SkDocument.h

BUG=skia:3110
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/570f18b43417d73c3fbd113cc0b4258e08b14c82
==========

to

==========
SkPDF: PDF/A runtime switch

TODO: remove gyp variable and modify API  in SkDocument.h

BUG=skia:3110
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/570f18b43417d73c3fbd113cc0b4258e08b14c82
==========

[hal.canary, 2016-04-21 20:22:06]

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

[commit-bot: I haz the power, 2016-04-21 20:22:14]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1891873002/120001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1891873002/120001

[hal.canary, 2016-04-21 20:23:58]

The CQ bit was unchecked by halcanary@google.com

[hal.canary, 2016-04-21 20:26:05]

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

[hal.canary, 2016-04-21 20:26:05]

This may fix this.

[tomhudson, 2016-04-21 20:33:04]

On 2016/04/21 20:26:05, Hal Canary wrote:
> This may fix this.

Moved MD5 from src/utils to src/core?

[hal.canary, 2016-04-21 20:49:47]

On 2016/04/21 20:33:04, tomhudson wrote:
> On 2016/04/21 20:26:05, Hal Canary wrote:
> > This may fix this.
> 
> Moved MD5 from src/utils to src/core?

Yep.  I say PDF should only depend on core.

[hal.canary, 2016-04-22 12:56:02]

Description was changed from

==========
SkPDF: PDF/A runtime switch

TODO: remove gyp variable and modify API  in SkDocument.h

BUG=skia:3110
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/570f18b43417d73c3fbd113cc0b4258e08b14c82
==========

to

==========
SkPDF: PDF/A runtime switch

TODO: remove gyp variable and modify API  in SkDocument.h

SkMD5 now moved into core as pdf depends on it now.

BUG=skia:3110
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/570f18b43417d73c3fbd113cc0b4258e08b14c82
==========

[hal.canary, 2016-04-22 13:09:09]

with no objections, I'm going to land this.

[hal.canary, 2016-04-22 13:09:21]

The CQ bit was checked by halcanary@google.com

[hal.canary, 2016-04-22 13:09:22]

The patchset sent to the CQ was uploaded after l-g-t-m from tomhudson@google.com
Link to the patchset: https://codereview.chromium.org/1891873002/#ps120001
(title: "2016-04-21 (Thursday) 16:21:36 EDT")

[commit-bot: I haz the power, 2016-04-22 13:09:26]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1891873002/120001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1891873002/120001

[commit-bot: I haz the power, 2016-04-22 13:10:25]

Description was changed from

==========
SkPDF: PDF/A runtime switch

TODO: remove gyp variable and modify API  in SkDocument.h

SkMD5 now moved into core as pdf depends on it now.

BUG=skia:3110
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/570f18b43417d73c3fbd113cc0b4258e08b14c82
==========

to

==========
SkPDF: PDF/A runtime switch

TODO: remove gyp variable and modify API  in SkDocument.h

SkMD5 now moved into core as pdf depends on it now.

BUG=skia:3110
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/570f18b43417d73c3fbd113cc0b4258e08b14c82

Committed:
https://skia.googlesource.com/skia/+/488165e689baf0f215d5798c87d0031b58e4bc8d
==========

[commit-bot: I haz the power, 2016-04-22 13:10:26]

Committed patchset #2 (id:120001) as
https://skia.googlesource.com/skia/+/488165e689baf0f215d5798c87d0031b58e4bc8d