Update filter tool to allow more flexible filtering

This CL refocuses the filter app on filter (rather than dumping out textures & paths). It leverages the debugger's DrawCommand and DebugCanvas classes to allow more flexible filters to be written. It is intended to allow filter prototyping prior to being folded into SkPictureRecord as a peep-hole.

The FontDescriptor was moved from ports to core to allow filter to be compiled without adding an additional dependency. Additionally, FontDescriptor's residence in ports was preventing the addition of a toString method to SkPaint.

Ultimately I hope to move this more towards having individual filter objects with check and apply entry points.

[robertphillips, 2013-01-30 17:07:20]

https://codereview.appspot.com/7227055/diff/1/debugger/SkDebugCanvas.cpp
File debugger/SkDebugCanvas.cpp (left):

https://codereview.appspot.com/7227055/diff/1/debugger/SkDebugCanvas.cpp#oldc...
debugger/SkDebugCanvas.cpp:12: #include "SkDevice.h"
Since this file is now compiled in tools I didn't want it relying on anything in
debugger\QT.

https://codereview.appspot.com/7227055/diff/1/gyp/tools.gyp
File gyp/tools.gyp (right):

https://codereview.appspot.com/7227055/diff/1/gyp/tools.gyp#newcode208
gyp/tools.gyp:208: '../debugger/SkDebugCanvas.cpp',
Ideally we wouldn't need the ObjectParser but for now we are stuck with it

[DerekS, 2013-01-31 14:56:01]

https://codereview.appspot.com/7227055/diff/2001/debugger/SkDrawCommand.cpp
File debugger/SkDrawCommand.cpp (right):

https://codereview.appspot.com/7227055/diff/2001/debugger/SkDrawCommand.cpp#n...
debugger/SkDrawCommand.cpp:234: this->fPaint = *paint;
why store both?

https://codereview.appspot.com/7227055/diff/2001/debugger/SkDrawCommand.h
File debugger/SkDrawCommand.h (right):

https://codereview.appspot.com/7227055/diff/2001/debugger/SkDrawCommand.h#new...
debugger/SkDrawCommand.h:168: SkPaint* paint() { return fPaintPtr; }
do you really need both?

https://codereview.appspot.com/7227055/diff/2001/src/ports/SkFontDescriptor.cpp
File src/ports/SkFontDescriptor.cpp (left):

https://codereview.appspot.com/7227055/diff/2001/src/ports/SkFontDescriptor.c...
src/ports/SkFontDescriptor.cpp:8: #include "SkFontDescriptor.h"
you deleted the file, but I don't see it added to src/core in this CL

https://codereview.appspot.com/7227055/diff/2001/tools/filtermain.cpp
File tools/filtermain.cpp (right):

https://codereview.appspot.com/7227055/diff/2001/tools/filtermain.cpp#newcode19
tools/filtermain.cpp:19: #include "..\debugger\SkDebugCanvas.h"
add debugger to the gyp's include dirs instead of doing this.

[robertphillips, 2013-01-31 15:22:47]

https://codereview.appspot.com/7227055/diff/2001/debugger/SkDrawCommand.cpp
File debugger/SkDrawCommand.cpp (right):

https://codereview.appspot.com/7227055/diff/2001/debugger/SkDrawCommand.cpp#n...
debugger/SkDrawCommand.cpp:234: this->fPaint = *paint;
drawBitmapRect takes a SkPaint*. I could've stored a boolean to know that the
stored paint was valid but figured this would save some logic in execute and
getPaint.

https://codereview.appspot.com/7227055/diff/2001/debugger/SkDrawCommand.h
File debugger/SkDrawCommand.h (right):

https://codereview.appspot.com/7227055/diff/2001/debugger/SkDrawCommand.h#new...
debugger/SkDrawCommand.h:168: SkPaint* paint() { return fPaintPtr; }
The non-const version is a bit of a new idea for the DrawCommands. I certainly
need the non-const for the paint alteration and only added the const version as
a nicety.

https://codereview.appspot.com/7227055/diff/2001/src/ports/SkFontDescriptor.cpp
File src/ports/SkFontDescriptor.cpp (left):

https://codereview.appspot.com/7227055/diff/2001/src/ports/SkFontDescriptor.c...
src/ports/SkFontDescriptor.cpp:8: #include "SkFontDescriptor.h"
Appspot is just being evil. I just moved the two files to core.

https://codereview.appspot.com/7227055/diff/2001/tools/filtermain.cpp
File tools/filtermain.cpp (right):

https://codereview.appspot.com/7227055/diff/2001/tools/filtermain.cpp#newcode19
tools/filtermain.cpp:19: #include "..\debugger\SkDebugCanvas.h"
On 2013/01/31 14:56:01, DerekS wrote:
> add debugger to the gyp's include dirs instead of doing this.

Done.

[DerekS, 2013-01-31 15:39:36]

lgtm

[robertphillips, 2013-01-31 15:57:02]

Added comment to better explain DrawBitmapRect's paint handling.

committed as r7492