Dedup SkPaths by gen-ID when going to .skp

Dedup SkPaths by gen-ID when going to .skp

We're all set up to serialize and deserialze paths by an integer ID,
but we're just not deduping any paths (every path gets a new ID).

This turns on deduping.  This should mean the number of paths in a
deserialized SkPicture are the same as when it was recorded.

No diffs:
https://gold.skia.org/search2?issue=1409373011&unt=true&query=source_type%3Dgm&master=false

BUG=skia:4527
Committed: https://skia.googlesource.com/skia/+/c2e2977014c4581aed5da435fc2b7eba38e777b6

[mtklein, 2015-10-28 20:01:14]

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

[commit-bot: I haz the power, 2015-10-28 20:01:23]

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

[commit-bot: I haz the power, 2015-10-28 20:12:23]

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

[commit-bot: I haz the power, 2015-10-28 20:12:23]

Dry run: This issue passed the CQ dry run.

[mtklein, 2015-10-29 14:13:07]

Description was changed from

==========
Dedup SkPaths by gen-ID when going to .skp

We're all set up to serialize and deserialze paths by an integer ID,
but we're just not deduping any paths (every path gets a new ID).

This turns on deduping.  This should mean the number of paths in a
deserialized SkPicture are the same as when it was recorded.

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

to

==========
Dedup SkPaths by gen-ID when going to .skp

We're all set up to serialize and deserialze paths by an integer ID,
but we're just not deduping any paths (every path gets a new ID).

This turns on deduping.  This should mean the number of paths in a
deserialized SkPicture are the same as when it was recorded.

BUG=skia:4527
==========

[mtklein, 2015-10-29 14:13:24]

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

[commit-bot: I haz the power, 2015-10-29 14:13:36]

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

[commit-bot: I haz the power, 2015-10-29 14:15:17]

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

[commit-bot: I haz the power, 2015-10-29 14:15:18]

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

[mtklein, 2015-10-29 14:15:29]

https://codereview.chromium.org/1409373011/diff/20001/src/core/SkPictureData.cpp
File src/core/SkPictureData.cpp (right):

https://codereview.chromium.org/1409373011/diff/20001/src/core/SkPictureData....
src/core/SkPictureData.cpp:50: fPaths.reset(record.fPaths.count() + 1);  // 0 is
reserved.
Oops, note to self, remove this.  0's not reserved, I don't think.

[mtklein, 2015-10-29 14:48:49]

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

[commit-bot: I haz the power, 2015-10-29 14:48:53]

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

[commit-bot: I haz the power, 2015-10-29 14:59:22]

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

[commit-bot: I haz the power, 2015-10-29 14:59:22]

Dry run: This issue passed the CQ dry run.

[mtklein, 2015-10-29 21:06:26]

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

[commit-bot: I haz the power, 2015-10-29 21:06:35]

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

[commit-bot: I haz the power, 2015-10-29 21:17:06]

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

[commit-bot: I haz the power, 2015-10-29 21:17:06]

Dry run: This issue passed the CQ dry run.

[mtklein, 2015-10-29 21:26:51]

Description was changed from

==========
Dedup SkPaths by gen-ID when going to .skp

We're all set up to serialize and deserialze paths by an integer ID,
but we're just not deduping any paths (every path gets a new ID).

This turns on deduping.  This should mean the number of paths in a
deserialized SkPicture are the same as when it was recorded.

BUG=skia:4527
==========

to

==========
Dedup SkPaths by gen-ID when going to .skp

We're all set up to serialize and deserialze paths by an integer ID,
but we're just not deduping any paths (every path gets a new ID).

This turns on deduping.  This should mean the number of paths in a
deserialized SkPicture are the same as when it was recorded.

No diffs:
https://gold.skia.org/search2?issue=1409373011&unt=true&query=source_type%3Dg...

BUG=skia:4527
==========

[mtklein_C, 2015-10-29 21:27:51]

mtklein@chromium.org changed reviewers:
+ reed@google.com, robertphillips@google.com
- mtklein@google.com

[robertphillips, 2015-10-30 11:59:36]

lgtm

[mtklein, 2015-10-30 12:24:11]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2015-10-30 12:24:16]

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

[commit-bot: I haz the power, 2015-10-30 12:25:01]

Committed patchset #5 (id:80001) as
https://skia.googlesource.com/skia/+/c2e2977014c4581aed5da435fc2b7eba38e777b6