The great shader refactor: Clean up GLRenderer

The great shader refactor: Clean up GLRenderer

Get rid of the functions to get each individual program type, and
replace them with calls to GetProgram with the appropriate key.

Move uniform tracking to ProgramBinding, and get rid of all of the
various "uniform cache" structures that were created to handle getting
uniforms from all of the different shader types.

BUG=667966
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

[ccameron, 2017-01-12 01:10:07]

Description was changed from

==========
The great shader refactor: Clean up GLRenderer

Get rid of the functions to get each individual program type, and
replace them with calls to GetProgram with the appropriate key.

Move uniform tracking to ProgramBinding, and get rid of all of the
various "uniform cache" structures that were created to handle getting
uniforms from all of the different shader types.

BUG=667966
==========

to

==========
The great shader refactor: Clean up GLRenderer

Get rid of the functions to get each individual program type, and
replace them with calls to GetProgram with the appropriate key.

Move uniform tracking to ProgramBinding, and get rid of all of the
various "uniform cache" structures that were created to handle getting
uniforms from all of the different shader types.

BUG=667966
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[ccameron, 2017-01-12 01:10:39]

The CQ bit was checked by ccameron@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-12 01:11:53]

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

[ccameron, 2017-01-12 02:01:08]

The CQ bit was checked by ccameron@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-12 02:01:21]

Dry run: 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, 2017-01-12 02:39:06]

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

[commit-bot: I haz the power, 2017-01-12 02:39:07]

Dry run: This issue passed the CQ dry run.

[ccameron, 2017-01-12 09:27:14]

ptal -- more salami slices

[enne (OOO), 2017-01-12 18:30:36]

enne@chromium.org changed reviewers:
+ enne@chromium.org

[enne (OOO), 2017-01-12 18:30:36]

Ahhh this patch is so great.

lgtm

[ccameron, 2017-01-12 20:31:06]

The CQ bit was checked by ccameron@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-12 20:31:49]

Dry run: 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, 2017-01-12 21:54:37]

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

[commit-bot: I haz the power, 2017-01-12 21:54:38]

Dry run: This issue passed the CQ dry run.

[ccameron, 2017-01-12 22:20:22]

:D

[ccameron, 2017-01-12 22:20:27]

The CQ bit was checked by ccameron@chromium.org

[ccameron, 2017-01-12 22:20:28]

The patchset sent to the CQ was uploaded after l-g-t-m from enne@chromium.org
Link to the patchset: https://codereview.chromium.org/2622243003/#ps20001
(title: "Rebase")

[commit-bot: I haz the power, 2017-01-12 22:21:03]

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, 2017-01-12 22:31:23]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484259627553730,
"parent_rev": "34ff66f6a7ed64c19c9494b0327a7a4037b7b2ff", "commit_rev":
"eb97055cd797c4fce7e56afdb010ea8c8657f772"}

[commit-bot: I haz the power, 2017-01-12 22:38:51]

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

[commit-bot: I haz the power, 2017-01-12 22:38:52]

Prior attempt to commit was detected, but we were not able to check whether the
issue was successfully committed. Please check Git history manually and re-check
CQ or close this issue as needed.

[ccameron, 2017-01-13 07:36:42]

The CQ bit was checked by ccameron@chromium.org

[commit-bot: I haz the power, 2017-01-13 07:37:01]

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, 2017-01-13 07:43:00]

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

[commit-bot: I haz the power, 2017-01-13 07:43:01]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[ccameron, 2017-01-13 21:34:07]

On 2017/01/13 07:43:01, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

Oh I am so confused ... this looks like it landed, but there's no commit
message.

Whelp, it's in the tree, that's good enough for me.