Fix the bug that long varying varibales are mapped into different names in fragment/vertex shaders.

Fix the bug that long varying varibales are mapped into different names in fragment/vertex shaders.

ANGLEBUG=144
TEST=the same long varying variable name in fragment/vertex shaders map to the same shortened name if using the same translator.
Committed: http://code.google.com/p/angleproject/source/detail?r=660

[Zhenyao Mo, 2011-05-26 23:11:12]

Please review.

[kbr1, 2011-05-27 02:14:40]

LGTM; nice work. Perhaps Nicolas should take a look too.

http://codereview.appspot.com/4547063/diff/1/src/compiler/ShHandle.h
File src/compiler/ShHandle.h (right):

http://codereview.appspot.com/4547063/diff/1/src/compiler/ShHandle.h#newcode94
src/compiler/ShHandle.h:94: // Pair of long varying varibale name <originalName,
mappedName>.
varibale -> variable

[Zhenyao Mo, 2011-05-27 03:59:47]

Alok, Nicolas, any further review comments?

On Thu, May 26, 2011 at 7:14 PM,  <kbr@chromium.org> wrote:
> LGTM; nice work. Perhaps Nicolas should take a look too.
>
>
>
> http://codereview.appspot.com/4547063/diff/1/src/compiler/ShHandle.h
> File src/compiler/ShHandle.h (right):
>
> http://codereview.appspot.com/4547063/diff/1/src/compiler/ShHandle.h#newcode94
> src/compiler/ShHandle.h:94: // Pair of long varying varibale name
> <originalName, mappedName>.
> varibale -> variable
>
> http://codereview.appspot.com/4547063/
>

[nicolas, 2011-05-27 14:54:04]

Looks fine to me.

[Alok Priyadarshi, 2011-05-27 15:07:21]

lgtm

http://codereview.appspot.com/4547063/diff/1/src/compiler/MapLongVariableNames.h
File src/compiler/MapLongVariableNames.h (right):

http://codereview.appspot.com/4547063/diff/1/src/compiler/MapLongVariableName...
src/compiler/MapLongVariableNames.h:22: MapLongVariableNames(TMap<TString,
TString>& varyingLongNameMap);
const argument?