Addressed Windows compiler complaints

This CL will require rebaselining of the imagemagnifier GM due to 3-4 different pixels.

[robertphillips, 2012-08-15 14:24:53]

http://codereview.appspot.com/6462062/diff/1/gm/typeface.cpp
File gm/typeface.cpp (left):

http://codereview.appspot.com/6462062/diff/1/gm/typeface.cpp#oldcode37
gm/typeface.cpp:37: }
Allocated above @ line 28

[TomH, 2012-08-15 14:30:19]

LGTM. Sad about the constructor signature change, though.

http://codereview.appspot.com/6462062/diff/1/src/gpu/GrGpu.cpp
File src/gpu/GrGpu.cpp (right):

http://codereview.appspot.com/6462062/diff/1/src/gpu/GrGpu.cpp#newcode43
src/gpu/GrGpu.cpp:43: fClipMaskManager.setGpu(this);
Why was this generating warnings? Can't pass this before all its fields are
initialized?

[robertphillips, 2012-08-15 14:32:54]

http://codereview.appspot.com/6462062/diff/1/src/gpu/GrGpu.cpp
File src/gpu/GrGpu.cpp (right):

http://codereview.appspot.com/6462062/diff/1/src/gpu/GrGpu.cpp#newcode43
src/gpu/GrGpu.cpp:43: fClipMaskManager.setGpu(this);
Yep - our usage is safe (since we're just caching the pointer) but it makes the
compiler worry.

[robertphillips, 2012-08-15 14:37:02]

committed as r5108