roll skia to 2193

roll skia to 2193
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=98845

[reed1, 2011-08-30 17:13:13]

adding owner for webkit/glue/media

[reed1, 2011-08-30 17:14:43]

Removed check against device bounds, since by definition the global clip must be
contained within the bounds of the root device. This was also prompted by the
removal of SkDevice::getBounds() in the latest skia, and replaced with
getGlobalBounds() which returns the bounds including the origin offset in case
the device is part of a saveLayer construct.

[scherkus (not reviewing), 2011-08-30 17:21:38]

LGTM w/ nit

http://codereview.chromium.org/7792041/diff/3/webkit/glue/media/video_rendere...
File webkit/glue/media/video_renderer_impl.cc (right):

http://codereview.chromium.org/7792041/diff/3/webkit/glue/media/video_rendere...
webkit/glue/media/video_renderer_impl.cc:118: // Get the properties of the
SkDevice
I was going to nit about adding a period, but this comment and the one below are
utterly useless -- mind removing them while you're in here?

[epoger, 2011-08-30 17:34:45]

On 2011/08/30 17:21:38, scherkus wrote:
> LGTM w/ nit
> 
>
http://codereview.chromium.org/7792041/diff/3/webkit/glue/media/video_rendere...
> File webkit/glue/media/video_renderer_impl.cc (right):
> 
>
http://codereview.chromium.org/7792041/diff/3/webkit/glue/media/video_rendere...
> webkit/glue/media/video_renderer_impl.cc:118: // Get the properties of the
> SkDevice
> I was going to nit about adding a period, but this comment and the one below
are
> utterly useless -- mind removing them while you're in here?

LGTM pending trybot results

[reed1, 2011-08-30 17:42:16]

linux_layout failure unrelated to this CL

http://codereview.chromium.org/7792041/diff/3/webkit/glue/media/video_rendere...
File webkit/glue/media/video_renderer_impl.cc (right):

http://codereview.chromium.org/7792041/diff/3/webkit/glue/media/video_rendere...
webkit/glue/media/video_renderer_impl.cc:118: // Get the properties of the
SkDevice
On 2011/08/30 17:21:38, scherkus wrote:
> I was going to nit about adding a period, but this comment and the one below
are
> utterly useless -- mind removing them while you're in here?

Done.