minor SkSL changes to avoid compiler errors in Chromium

minor SkSL changes to avoid compiler errors in Chromium

* fixed a couple of spots where using { } instead of an explicit constructor call resulted in errors
* Type::Field had a deleted copy constructor and therefore was not working inside std::vector; had to remove const from fields and change fType from a reference to a pointer

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2300023002

Committed: https://skia.googlesource.com/skia/+/0730be7c303ac415484b15ef44ff1dce077a93b8

[ethannicholas, 2016-09-01 14:18:08]

Description was changed from

==========
minor SkSL changes to avoid compiler errors in Chromium
==========

to

==========
minor SkSL changes to avoid compiler errors in Chromium
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2300023002
==========

[ethannicholas, 2016-09-01 14:24:57]

Description was changed from

==========
minor SkSL changes to avoid compiler errors in Chromium
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2300023002
==========

to

==========
minor SkSL changes to avoid compiler errors in Chromium

* fixed a couple of spots where using { } instead of an explicit constructor
call resulted in errors
* Type::Field had a deleted copy constructor and therefore was not working
inside std::vector; had to remove const from fields and change fType from a
reference to a pointer

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2300023002
==========

[ethannicholas, 2016-09-01 14:28:24]

ethannicholas@google.com changed reviewers:
+ benjaminwagner@google.com

[dogben, 2016-09-01 14:39:17]

lgtm

[ethannicholas, 2016-09-01 14:43:17]

The CQ bit was checked by ethannicholas@google.com

[commit-bot: I haz the power, 2016-09-01 14:43:30]

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-09-01 14:59:04]

Description was changed from

==========
minor SkSL changes to avoid compiler errors in Chromium

* fixed a couple of spots where using { } instead of an explicit constructor
call resulted in errors
* Type::Field had a deleted copy constructor and therefore was not working
inside std::vector; had to remove const from fields and change fType from a
reference to a pointer

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2300023002
==========

to

==========
minor SkSL changes to avoid compiler errors in Chromium

* fixed a couple of spots where using { } instead of an explicit constructor
call resulted in errors
* Type::Field had a deleted copy constructor and therefore was not working
inside std::vector; had to remove const from fields and change fType from a
reference to a pointer

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2300023002

Committed:
https://skia.googlesource.com/skia/+/0730be7c303ac415484b15ef44ff1dce077a93b8
==========

[commit-bot: I haz the power, 2016-09-01 14:59:05]

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