Fix touch handling in SampleApp

Unify handleTouch and handleClick in SampleApp.

Allow each sample to handle clicks in Android SampleApp.

Remove SampleWindow::handleTouch.

Allow an SkWindow to have multiple clicks.

Add a pointer ID to SkView::Click.

Committed: https://code.google.com/p/skia/source/detail?r=1673

[Scroggo, 2011-06-21 21:18:25]

New patch up for review.

[reed1, 2011-06-21 21:24:53]

LGTM with minor suggestions

http://codereview.appspot.com/4643052/diff/6002/src/views/SkWindow.cpp
File src/views/SkWindow.cpp (left):

http://codereview.appspot.com/4643052/diff/6002/src/views/SkWindow.cpp#oldcode59
src/views/SkWindow.cpp:59: 
or

fClicks.deleteAll();

http://codereview.appspot.com/4643052/diff/6002/src/views/SkWindow.cpp
File src/views/SkWindow.cpp (right):

http://codereview.appspot.com/4643052/diff/6002/src/views/SkWindow.cpp#newcod...
src/views/SkWindow.cpp:381: // First, attempt to find an existing click with
this owner.
or

int index = fClicks.find(owner);

[Scroggo, 2011-06-21 21:32:14]

New patch uploaded.

http://codereview.appspot.com/4643052/diff/6002/src/views/SkWindow.cpp
File src/views/SkWindow.cpp (left):

http://codereview.appspot.com/4643052/diff/6002/src/views/SkWindow.cpp#oldcode59
src/views/SkWindow.cpp:59: 
On 2011/06/21 21:24:53, reed1 wrote:
> or
> 
> fClicks.deleteAll();

Done.

http://codereview.appspot.com/4643052/diff/6002/src/views/SkWindow.cpp
File src/views/SkWindow.cpp (right):

http://codereview.appspot.com/4643052/diff/6002/src/views/SkWindow.cpp#newcod...
src/views/SkWindow.cpp:381: // First, attempt to find an existing click with
this owner.
On 2011/06/21 21:24:53, reed1 wrote:
> or
> 
> int index = fClicks.find(owner);
This will not work (I would need to pass the Click itself, rather than the owner
pointer).

[Scroggo, 2011-06-22 13:02:25]

http://codereview.appspot.com/4643052/diff/6002/src/views/SkWindow.cpp
File src/views/SkWindow.cpp (right):

http://codereview.appspot.com/4643052/diff/6002/src/views/SkWindow.cpp#newcod...
src/views/SkWindow.cpp:381: // First, attempt to find an existing click with
this owner.
On 2011/06/21 21:32:14, Scroggo wrote:
> On 2011/06/21 21:24:53, reed1 wrote:
> > or
> > 
> > int index = fClicks.find(owner);
> This will not work (I would need to pass the Click itself, rather than the
owner
> pointer).
The reason I do not pass the Click is because in the case where we remove a
Click from fClicks, that Click never gets removed from SkTouchGesture.  If I
added a callback on SkWindow for onClickRemoved (which, in SampleWindow, would
call a new method on SkTouchGesture to remove a touch), I could pass the Click. 
However, that seems like an API which exposes our implementation, and doesn't
necessarily make sense in general.

[reed1, 2011-06-22 13:10:19]

http://codereview.appspot.com/4643052/diff/6002/src/views/SkWindow.cpp
File src/views/SkWindow.cpp (right):

http://codereview.appspot.com/4643052/diff/6002/src/views/SkWindow.cpp#newcod...
src/views/SkWindow.cpp:381: // First, attempt to find an existing click with
this owner.
Doh! You're right.

On 2011/06/21 21:32:14, Scroggo wrote:
> On 2011/06/21 21:24:53, reed1 wrote:
> > or
> > 
> > int index = fClicks.find(owner);
> This will not work (I would need to pass the Click itself, rather than the
owner
> pointer).