drawBitmap* cleanup

This CL cleans up the drawBitmap* call stack a bit in preparation for the "bleed" change; it:

folds SkCanvas::commonDrawBitmap into internalDrawbitmap (its only caller)

removes the 'srcRect' parameter from SkCanvas::internalDrawbitmap (since it was always NULL)

removes the 'srcRectOrNull' parameter from SkDevice::drawBitmap (and derived classes) since it was always NULL

makes a minor change to SkGpuDevice::internalDrawBitmap to only perform checks to turn off the texture domain if it was turned on in the first place

[edisonn, 2013-07-24 17:16:31]

LGTM the pdf stuff

[bsalomon, 2013-07-24 17:19:42]

On 2013/07/24 17:16:31, edisonn wrote:
> LGTM the pdf stuff

gpu lgtm

[reed1, 2013-07-24 18:32:47]

did you find all of the canvas subclasses? I guess the compiler would have told
you. I'm thinking of DumpCanvas, LuaCanvas, plus any in chrome? Modulo that,
lgtm

[robertphillips, 2013-07-24 18:48:01]

On 2013/07/24 18:32:47, reed1 wrote:
> did you find all of the canvas subclasses? I guess the compiler would have
told
> you. I'm thinking of DumpCanvas, LuaCanvas, plus any in chrome? Modulo that,
> lgtm

The only canvases/devices I know of that still need updating are in skia/ext and
I will update them with the roll.

[robertphillips, 2013-07-25 12:11:53]

committed as r10342