Record concat as Concat.

Record concat as Concat.

This seems to cause very slight diffs on GMs.
https://gold.skia.org/search2?issue=1462983002&unt=true&query=source_type%3Dgm&master=false

I'm not sure I understand why.
The diffs to at least pictureshader look like improvements.

BUG=skia:4584
Committed: https://skia.googlesource.com/skia/+/e9d2052e4931eeade49042a855b9c1f5ab7c84c5

[mtklein, 2015-11-19 18:41:08]

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

[commit-bot: I haz the power, 2015-11-19 18:41:14]

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

[commit-bot: I haz the power, 2015-11-19 18:45:35]

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

[commit-bot: I haz the power, 2015-11-19 18:45:35]

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-11-19 18:49:37]

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

[commit-bot: I haz the power, 2015-11-19 18:49:45]

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

[mtklein, 2015-11-19 18:53:21]

Description was changed from

==========
Record concat as Concat.

This seems to cause very slight diffs on GMs.
I'm not sure I understand why.

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

to

==========
Record concat as Concat.

This seems to cause very slight diffs on GMs.
https://gold.skia.org/search2?issue=1462983002&unt=true&query=source_type%3Dg...

I'm not sure I understand why.
The diffs to at least pictureshader look like improvements.

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

[mtklein_C, 2015-11-19 18:55:12]

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

[commit-bot: I haz the power, 2015-11-19 19:03:37]

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

[commit-bot: I haz the power, 2015-11-19 19:03:37]

Dry run: This issue passed the CQ dry run.

[reed1, 2015-11-19 20:06:21]

lgtm

do we think this has any impact on performance when using RTree?

[mtklein, 2015-11-19 20:07:06]

On 2015/11/19 at 20:06:21, reed wrote:
> lgtm
> 
> do we think this has any impact on performance when using RTree?

Strongly doubt it.

[mtklein, 2015-11-19 20:07:34]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2015-11-19 20:07:45]

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

[commit-bot: I haz the power, 2015-11-19 20:08:27]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/e9d2052e4931eeade49042a855b9c1f5ab7c84c5