gm: add a flag to force perspective usage in all gms

gm: add a flag to force perspective usage in all gms

BUG=

Committed: http://code.google.com/p/skia/source/detail?r=11487

[reed1, 2013-09-23 18:48:40]

https://codereview.chromium.org/23587029/diff/4001/gm/gm.h
File gm/gm.h (right):

https://codereview.chromium.org/23587029/diff/4001/gm/gm.h#newcode79
gm/gm.h:79: SkMatrix getInitialTransform() const {
I think we could generalize this guy a little for this CL:

1. I think you should concat w/ onGetInitialTransform(), instead of either/or
2. Perhaps gm itself has a setter (which gmmain.cpp can call) that accepts an
initial matrix.

#2 means you don't need to know about any flags here, just a matrix data blob in
the base-class of gm. THen the code would be something like

SkMatrix getInitialMatrix() const {
    SkMatrix matrix = fStarterMatrix;   // identity or set by the new
setStarterMatrix(...) on base class
    matrix.preConcat(this->onGetInitialTransform());
    return matrix;
}

[edisonn, 2013-09-23 19:38:03]

https://codereview.chromium.org/23587029/diff/4001/gm/gm.h
File gm/gm.h (right):

https://codereview.chromium.org/23587029/diff/4001/gm/gm.h#newcode79
gm/gm.h:79: SkMatrix getInitialTransform() const {
On 2013/09/23 18:48:40, reed1 wrote:
> I think we could generalize this guy a little for this CL:
> 
> 1. I think you should concat w/ onGetInitialTransform(), instead of either/or
> 2. Perhaps gm itself has a setter (which gmmain.cpp can call) that accepts an
> initial matrix.
> 
> #2 means you don't need to know about any flags here, just a matrix data blob
in
> the base-class of gm. THen the code would be something like
> 
> SkMatrix getInitialMatrix() const {
>     SkMatrix matrix = fStarterMatrix;   // identity or set by the new
> setStarterMatrix(...) on base class
>     matrix.preConcat(this->onGetInitialTransform());
>     return matrix;
> }

Done.

[reed1, 2013-09-23 20:34:48]

where is GM constructor? It will need to call fStarterMatrix.reset();

[edisonn, 2013-09-24 20:53:44]

On 2013/09/23 20:34:48, reed1 wrote:
> where is GM constructor? It will need to call fStarterMatrix.reset();

done

[edisonn, 2013-09-26 19:58:08]

On 2013/09/24 20:53:44, edisonn wrote:
> On 2013/09/23 20:34:48, reed1 wrote:
> > where is GM constructor? It will need to call fStarterMatrix.reset();
> 
> done

ping

[reed1, 2013-09-26 20:33:54]

lgtm

[commit-bot: I haz the power, 2013-09-26 20:35:02]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/edisonn@google.com/23587029/17001

[commit-bot: I haz the power, 2013-09-26 20:44:28]

Change committed as 11487