Fix up picture clip bounds in SkiaServe

Fix up picture clip bounds in SkiaServe

BUG=skia:5067
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1781653002

Committed: https://skia.googlesource.com/skia/+/e0449cf9f43e179a809eb1b87ca3bf77cf6f1222

[joshua.litt, 2016-03-09 15:32:44]

Description was changed from

==========
Fix up picture clip bounds in SkiaServe

BUG=skia:
==========

to

==========
Fix up picture clip bounds in SkiaServe

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[joshualitt, 2016-03-09 15:33:22]

Description was changed from

==========
Fix up picture clip bounds in SkiaServe

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Fix up picture clip bounds in SkiaServe

BUG=skia:5067
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[joshualitt, 2016-03-09 15:34:46]

joshualitt@google.com changed reviewers:
+ ethannicholas@google.com, jcgregorio@google.com, joshualitt@google.com

[joshualitt, 2016-03-09 15:34:47]

ptal

https://codereview.chromium.org/1781653002/diff/20001/tools/skiaserve/Request...
File tools/skiaserve/Request.cpp (right):

https://codereview.chromium.org/1781653002/diff/20001/tools/skiaserve/Request...
tools/skiaserve/Request.cpp:269: uint8_t* start = ((uint8_t*)
bitmap->getPixels()) + (y * bitmap->width() + x) * 4;
I changed this to bitmap->width(), is that correct?

[ethannicholas, 2016-03-09 16:32:37]

lgtm

https://codereview.chromium.org/1781653002/diff/20001/tools/skiaserve/Request...
File tools/skiaserve/Request.cpp (right):

https://codereview.chromium.org/1781653002/diff/20001/tools/skiaserve/Request...
tools/skiaserve/Request.cpp:269: uint8_t* start = ((uint8_t*)
bitmap->getPixels()) + (y * bitmap->width() + x) * 4;
On 2016/03/09 15:34:47, joshualitt wrote:
> I changed this to bitmap->width(), is that correct?

Yep.

[joshualitt, 2016-03-09 16:32:49]

The CQ bit was checked by joshualitt@google.com

[commit-bot: I haz the power, 2016-03-09 16:33:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1781653002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1781653002/20001

[commit-bot: I haz the power, 2016-03-09 18:07:04]

Description was changed from

==========
Fix up picture clip bounds in SkiaServe

BUG=skia:5067
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Fix up picture clip bounds in SkiaServe

BUG=skia:5067
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/e0449cf9f43e179a809eb1b87ca3bf77cf6f1222
==========

[commit-bot: I haz the power, 2016-03-09 18:07:05]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/e0449cf9f43e179a809eb1b87ca3bf77cf6f1222