Text context mess cleanup attempt + GMs

[bsalomon, 2012-07-10 21:30:09]

On 2012/07/10 21:21:45, TomH wrote:

LGTM. We'll prolly want to watch the bench graphs after this lands. You can
ignore my comment about the names if there's no simple explanation of what these
are testing.

[bsalomon, 2012-07-11 12:39:55]

I forgot to publish this comment before

http://codereview.appspot.com/6347043/diff/2001/gm/gradtext.cpp
File gm/gradtext.cpp (right):

http://codereview.appspot.com/6347043/diff/2001/gm/gradtext.cpp#newcode41
gm/gradtext.cpp:41: class CGM : public GM {
Maybe we should give these somewhat descriptive names?

[TomH, 2012-07-11 13:42:27]

On 2012/07/11 12:39:55, bsalomon wrote:
> I forgot to publish this comment before
> 
> http://codereview.appspot.com/6347043/diff/2001/gm/gradtext.cpp
> File gm/gradtext.cpp (right):
> 
> http://codereview.appspot.com/6347043/diff/2001/gm/gradtext.cpp#newcode41
> gm/gradtext.cpp:41: class CGM : public GM {
> Maybe we should give these somewhat descriptive names?

Absolutely.

[TomH, 2012-07-11 14:41:34]

Closed with r4531.