Clean up interfaces between Shader / ShaderTranslator / ANGLE side.

Clean up interfaces between Shader / ShaderTranslator / ANGLE side.

First, it is a good cleanup to avoid an extra copy of all the variable info.
Second, this cleanup will make switching to ANGLE's new APIs to query variable
info much easier.

BUG=401311
TEST=gpu_unittests, webgl_conformance_tests
R=bajones@chromium.org,kbr@chromium.org,piman@chromium.org

Committed: https://crrev.com/576a049e2372611da05e03b8ba61f4b60338cd44
Cr-Commit-Position: refs/heads/master@{#294718}

[Zhenyao Mo, 2014-09-12 22:04:51]

This refactoring made ShaderTranslator a functional class, and do not hold
temporary compilation result data - it is passed to Shader and gets deleted soon
anyway, so we save one unnecessary copy.

bajones: please review.
piman, kbr: FYI (feel free to review if you want to)

[bajones, 2014-09-12 23:42:56]

On 2014/09/12 at 22:04:51, zmo wrote:
> This refactoring made ShaderTranslator a functional class, and do not hold
temporary compilation result data - it is passed to Shader and gets deleted soon
anyway, so we save one unnecessary copy.
> 
> bajones: please review.
> piman, kbr: FYI (feel free to review if you want to)

Lot's of code, but a reasonably simple change at it's core. LGTM!

[Zhenyao Mo, 2014-09-12 23:47:34]

The CQ bit was checked by zmo@chromium.org

[commit-bot: I haz the power, 2014-09-13 00:01:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/566023002/40001

[commit-bot: I haz the power, 2014-09-13 01:16:05]

Committed patchset #3 (id:40001) as 82f76e744241a85b376c652426abc6584458849e

[commit-bot: I haz the power, 2014-09-13 01:19:12]

Patchset 3 (id:??) landed as
https://crrev.com/576a049e2372611da05e03b8ba61f4b60338cd44
Cr-Commit-Position: refs/heads/master@{#294718}