Fix Ganesh perspective projection bug

Fix Ganesh perspective projection bug

BUG=skia:3326
Committed: https://skia.googlesource.com/skia/+/ef4ba3da0b1fe375b9631bcd17c43c645f4aa5a6

[robertphillips, 2015-09-17 16:30:04]

robertphillips@google.com changed reviewers:
+ joshualitt@chromium.org, jvanverth@google.com

[jvanverth1, 2015-09-17 16:31:33]

lgtm

[joshualitt, 2015-09-17 16:40:51]

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

[joshualitt, 2015-09-17 16:40:51]

nit

https://codereview.chromium.org/1352883002/diff/20001/src/gpu/gl/builders/GrG...
File src/gpu/gl/builders/GrGLVertexShaderBuilder.cpp (right):

https://codereview.chromium.org/1352883002/diff/20001/src/gpu/gl/builders/GrG...
src/gpu/gl/builders/GrGLVertexShaderBuilder.cpp:61: "gl_Position =
vec4(_posTmp.x * %s.x + %s.y, _posTmp.y * %s.z + %s.w, 0, 1);}",
line wrap :-p

[robertphillips, 2015-09-17 16:45:41]

https://codereview.chromium.org/1352883002/diff/20001/src/gpu/gl/builders/GrG...
File src/gpu/gl/builders/GrGLVertexShaderBuilder.cpp (right):

https://codereview.chromium.org/1352883002/diff/20001/src/gpu/gl/builders/GrG...
src/gpu/gl/builders/GrGLVertexShaderBuilder.cpp:61: "gl_Position =
vec4(_posTmp.x * %s.x + %s.y, _posTmp.y * %s.z + %s.w, 0, 1);}",
On 2015/09/17 16:40:51, joshualitt wrote:
> line wrap :-p

Done.

[robertphillips, 2015-09-17 16:46:20]

The CQ bit was checked by robertphillips@google.com to run a CQ dry run

[commit-bot: I haz the power, 2015-09-17 16:46:31]

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

[joshualitt, 2015-09-17 16:57:29]

On 2015/09/17 16:46:31, commit-bot: I haz the power wrote:
> Dry run: CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/1352883002/40001
> View timeline at
>  https://chromium-cq-status.appspot.com/patch-timeline/1352883002/40001

lgtm

[robertphillips, 2015-09-17 17:34:42]

The CQ bit was unchecked by robertphillips@google.com

[robertphillips, 2015-09-17 17:34:43]

The CQ bit was checked by robertphillips@google.com

[robertphillips, 2015-09-17 17:34:43]

The patchset sent to the CQ was uploaded after l-g-t-m from jvanverth@google.com
Link to the patchset: https://codereview.chromium.org/1352883002/#ps40001
(title: "fixed overlength line")

[robertphillips, 2015-09-17 17:34:50]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2015-09-17 17:34:56]

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

[commit-bot: I haz the power, 2015-09-17 18:21:10]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/ef4ba3da0b1fe375b9631bcd17c43c645f4aa5a6