fix clang compilation warning

fix clang compilation warning -Wsign-compare

../src/gpu/gl/../GrDrawState.h:88:75: warning: comparison of integers of different signs: 'unsigned long' and 'long' [-Wsign-compare]
                 sizeof(fSrcBlend) + sizeof(fDstBlend) + sizeof(GrMatrix) ==
                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~
../include/gpu/GrConfig.h:270:39: note: expanded from macro 'GrAssert'
                                      ^
../include/gpu/GrConfig.h:261:54: note: expanded from macro 'GR_DEBUGASSERT'
        #define GR_DEBUGASSERT(COND) GR_ALWAYSASSERT(COND)
                                                     ^
../include/gpu/GrConfig.h:249:19: note: expanded from macro 'GR_ALWAYSASSERT'
            if (!(COND)) {                                               \
                  ^~~~

    ../tests/UnicodeTest.cpp:72:36: warning: comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int
          [-Wsign-compare]
        REPORTER_ASSERT(reporter, len8 == count8);
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~
    ../tests/Test.h:116:15: note: expanded from macro 'REPORTER_ASSERT'
            if (!(cond)) {                                                  \
                  ^
    ../tests/UnicodeTest.cpp:73:36: warning: comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int
          [-Wsign-compare]
        REPORTER_ASSERT(reporter, len8 == count16);
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~
    ../tests/Test.h:116:15: note: expanded from macro 'REPORTER_ASSERT'
            if (!(cond)) {                                                  \
                  ^
    ../tests/UnicodeTest.cpp:74:36: warning: comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int
          [-Wsign-compare]
        REPORTER_ASSERT(reporter, len8 == count32);
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~
    ../tests/Test.h:116:15: note: expanded from macro 'REPORTER_ASSERT'
            if (!(cond)) {                                                  \
                  ^

BUG=
TEST=

[guanqun, 2012-03-22 04:32:03]

please review, thanks!

[reed1, 2012-03-22 12:39:00]

UnicodeTest changes are fine. Leaving the Gr changes to Brian

[bsalomon, 2012-03-22 13:03:37]

On 2012/03/22 12:39:00, reed1 wrote:
> UnicodeTest changes are fine. Leaving the Gr changes to Brian

LGTM

[guanqun.lu_gmail.com, 2012-03-22 14:13:48]

Please help land this CL.

BTW: there are lots of clang warning in src/animator/ and it complains
about subclass's add() function hides the overloaded functions.
1. are these intended interfaces for subclasses?
2. do we use animator code somewhere? That's why I'm not submitting
the fix here.

--guanqun

On Thu, Mar 22, 2012 at 9:03 PM,  <bsalomon@google.com> wrote:
> On 2012/03/22 12:39:00, reed1 wrote:
>>
>> UnicodeTest changes are fine. Leaving the Gr changes to Brian
>
>
> LGTM
>
> http://codereview.appspot.com/5833045/



-- 
Guanqun

[reed1, 2012-03-22 14:16:14]

animator is in flux right now -- best not to spend anytime tweaking it.

On Thu, Mar 22, 2012 at 10:13 AM, Lu Guanqun <guanqun.lu@gmail.com> wrote:

> Please help land this CL.
>
> BTW: there are lots of clang warning in src/animator/ and it complains
> about subclass's add() function hides the overloaded functions.
> 1. are these intended interfaces for subclasses?
> 2. do we use animator code somewhere? That's why I'm not submitting
> the fix here.
>
> --guanqun
>
> On Thu, Mar 22, 2012 at 9:03 PM,  <bsalomon@google.com> wrote:
> > On 2012/03/22 12:39:00, reed1 wrote:
> >>
> >> UnicodeTest changes are fine. Leaving the Gr changes to Brian
> >
> >
> > LGTM
> >
> > http://codereview.appspot.com/5833045/
>
>
>
> --
> Guanqun
>

[bsalomon, 2012-03-22 21:29:31]

On 2012/03/22 14:16:14, reed1 wrote:
> animator is in flux right now -- best not to spend anytime tweaking it.
> 
> On Thu, Mar 22, 2012 at 10:13 AM, Lu Guanqun <mailto:guanqun.lu@gmail.com>
wrote:
> 
> > Please help land this CL.
> >
> > BTW: there are lots of clang warning in src/animator/ and it complains
> > about subclass's add() function hides the overloaded functions.
> > 1. are these intended interfaces for subclasses?
> > 2. do we use animator code somewhere? That's why I'm not submitting
> > the fix here.
> >
> > --guanqun
> >
> > On Thu, Mar 22, 2012 at 9:03 PM,  <mailto:bsalomon@google.com> wrote:
> > > On 2012/03/22 12:39:00, reed1 wrote:
> > >>
> > >> UnicodeTest changes are fine. Leaving the Gr changes to Brian
> > >
> > >
> > > LGTM
> > >
> > > http://codereview.appspot.com/5833045/
> >
> >
> >
> > --
> > Guanqun
> >

Committed as r3472