Fix includes in SkLayerDrawLooper.h. It was previously not possible to

Fix includes in SkLayerDrawLooper.h. It was previously not possible to
include this header unless SkPoint and SkPaint were previously defined.

BUG=

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

[jbroman, 2013-05-14 18:41:06]

I was trying to move a usage of SkLayerDrawLooper in Blink, and I had to include
SkPaint.h and SkPoint.h beforehand because they are used in struct definitions
in this file.

That shouldn't be necessary (if this header needs those struct definitions, it
should include them), hence this CL. Removed the unused forward declaration of
SkPoint.

Skia still builds and tests still pass.

Please commit for me if approved; I'm not a Skia committer.

[reed1, 2013-05-14 19:32:04]

lgtm

[commit-bot: I haz the power, 2013-05-14 19:43:40]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/jbroman@chromium.org/14607015/1

[commit-bot: I haz the power, 2013-05-14 20:11:25]

Change committed as 9130

[scroggo, 2013-05-14 21:06:05]

https://chromiumcodereview.appspot.com/14607015/diff/1/include/effects/SkLaye...
File include/effects/SkLayerDrawLooper.h (right):

https://chromiumcodereview.appspot.com/14607015/diff/1/include/effects/SkLaye...
include/effects/SkLayerDrawLooper.h:13: #include "SkPoint.h"
It appears to me that #include "SkPoint.h" can be removed entirely. I do not see
SkPoint referenced anywhere in this file or in the cpp file.

[jbroman, 2013-05-14 21:15:23]

On 2013/05/14 21:06:05, scroggo wrote:
>
https://chromiumcodereview.appspot.com/14607015/diff/1/include/effects/SkLaye...
> File include/effects/SkLayerDrawLooper.h (right):
> 
>
https://chromiumcodereview.appspot.com/14607015/diff/1/include/effects/SkLaye...
> include/effects/SkLayerDrawLooper.h:13: #include "SkPoint.h"
> It appears to me that #include "SkPoint.h" can be removed entirely. I do not
see
> SkPoint referenced anywhere in this file or in the cpp file.

SkVector (a typedef for SkPoint) is defined in SkPoint.h.

[scroggo, 2013-05-14 21:17:36]

On 2013/05/14 21:15:23, jbroman wrote:
> On 2013/05/14 21:06:05, scroggo wrote:
> >
>
https://chromiumcodereview.appspot.com/14607015/diff/1/include/effects/SkLaye...
> > File include/effects/SkLayerDrawLooper.h (right):
> > 
> >
>
https://chromiumcodereview.appspot.com/14607015/diff/1/include/effects/SkLaye...
> > include/effects/SkLayerDrawLooper.h:13: #include "SkPoint.h"
> > It appears to me that #include "SkPoint.h" can be removed entirely. I do not
> see
> > SkPoint referenced anywhere in this file or in the cpp file.
> 
> SkVector (a typedef for SkPoint) is defined in SkPoint.h.

Touche.