There is an unused rowBytes parameter being passed along.

There is an unused rowBytes parameter being passed along.

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

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

[cblume, 2016-01-26 00:27:24]

Description was changed from

==========
There is an unused rowBytes parameter being passed along.

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

to

==========
There is an unused rowBytes parameter being passed along.

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

[cblume, 2016-01-26 02:24:04]

cblume@chromium.org changed reviewers:
+ bsalomon@chromium.org, reed@chromium.org

[cblume, 2016-01-26 02:27:37]

The CQ bit was checked by cblume@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-01-26 02:27:50]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1636873002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1636873002/1

[commit-bot: I haz the power, 2016-01-26 02:27:51]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from valid reviewer(s) till 2016-01-26 08:27 UTC

[commit-bot: I haz the power, 2016-01-26 02:38:43]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-01-26 02:38:43]

Dry run: Try jobs failed on following builders:
  Test-Ubuntu-GCC-ShuttleA-GPU-GTX660-x86_64-Release-Trybot on client.skia
(JOB_FAILED,
http://build.chromium.org/p/client.skia/builders/Test-Ubuntu-GCC-ShuttleA-GPU...)

[bsalomon, 2016-01-26 14:32:10]

bsalomon@google.com changed reviewers:
+ bsalomon@google.com

[bsalomon, 2016-01-26 14:32:11]

lgtm

[cblume, 2016-01-26 17:09:44]

The CQ bit was checked by cblume@chromium.org

[commit-bot: I haz the power, 2016-01-26 17:09:55]

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

[commit-bot: I haz the power, 2016-01-26 17:14:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-01-26 17:14:20]

Try jobs failed on following builders:
  Test-Ubuntu-GCC-ShuttleA-GPU-GTX660-x86_64-Release-Trybot on client.skia
(JOB_FAILED,
http://build.chromium.org/p/client.skia/builders/Test-Ubuntu-GCC-ShuttleA-GPU...)

[cblume, 2016-01-26 17:40:52]

On 2016/01/26 17:14:20, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   Test-Ubuntu-GCC-ShuttleA-GPU-GTX660-x86_64-Release-Trybot on client.skia
> (JOB_FAILED,
>
http://build.chromium.org/p/client.skia/builders/Test-Ubuntu-GCC-ShuttleA-GPU...)

I wonder what is going on here.
It builds and runs fine on my local Ubuntu GPU machine.
I'll investigate.

[cblume, 2016-02-08 23:05:21]

The CQ bit was checked by cblume@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-08 23:05:31]

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

[commit-bot: I haz the power, 2016-02-08 23:45:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-08 23:45:58]

Dry run: This issue passed the CQ dry run.

[cblume, 2016-02-10 18:57:50]

The CQ bit was checked by cblume@chromium.org

[cblume, 2016-02-10 18:57:50]

The patchset sent to the CQ was uploaded after l-g-t-m from bsalomon@google.com
Link to the patchset: https://codereview.chromium.org/1636873002/#ps20001
(title: "Fixing bug exposed by WritePixelsTest -- default parameters allowed me
to accidentally pass the fla…")

[commit-bot: I haz the power, 2016-02-10 18:57:59]

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

[cblume, 2016-02-10 19:08:49]

The CQ bit was unchecked by cblume@chromium.org

[cblume, 2016-02-10 20:42:15]

The CQ bit was checked by cblume@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-10 20:42:21]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1636873002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1636873002/40001

[commit-bot: I haz the power, 2016-02-10 20:45:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-10 20:45:20]

Dry run: Try jobs failed on following builders:
  Build-Win-MSVC-x86_64-Debug-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Win-MSVC-x86_6...)

[cblume, 2016-02-10 21:11:00]

The CQ bit was checked by cblume@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-10 21:11:11]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1636873002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1636873002/60001

[commit-bot: I haz the power, 2016-02-10 21:21:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-10 21:21:40]

Dry run: Try jobs failed on following builders:
  Test-Ubuntu-GCC-ShuttleA-GPU-GTX660-x86_64-Release-Trybot on client.skia
(JOB_FAILED,
http://build.chromium.org/p/client.skia/builders/Test-Ubuntu-GCC-ShuttleA-GPU...)

[cblume, 2016-02-16 02:09:41]

The CQ bit was checked by cblume@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-16 02:09:49]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1636873002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1636873002/80001

[cblume, 2016-02-16 02:22:48]

The CQ bit was checked by cblume@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-16 02:22:57]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1636873002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1636873002/100001

[commit-bot: I haz the power, 2016-02-16 02:44:04]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-16 02:44:05]

Dry run: This issue passed the CQ dry run.

[cblume, 2016-02-16 02:51:06]

@bsalomon Does this still LGTY?
I believe it is ready to land.

[bsalomon, 2016-02-16 14:26:45]

lgtm

[cblume, 2016-02-16 20:58:48]

The CQ bit was checked by cblume@chromium.org

[commit-bot: I haz the power, 2016-02-16 20:59:01]

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

[commit-bot: I haz the power, 2016-02-16 21:00:04]

Description was changed from

==========
There is an unused rowBytes parameter being passed along.

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

to

==========
There is an unused rowBytes parameter being passed along.

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

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

[commit-bot: I haz the power, 2016-02-16 21:00:04]

Committed patchset #6 (id:100001) as
https://skia.googlesource.com/skia/+/ed828001da75f4a3a5d31d9f1c1935e3595ce920