Bug 87569 – [BlackBerry] Add a default tap highlight

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit BlackBerry

[Arvid Nilsson]

We used to require the embedder to implement tap highlight drawing.
            Now, a default tap highlight, implemented using the recently added
            accelerated compositing overlay layer support, can be used instead.
    
            The tap highlight zooms in when it appears, to provide visible feedback
            even though your finger obscures the whole link, and fades out when
            hidden. The appearance of the default tap highlight is not final.
    
            The default tap highlight can be overridden using the new
            WebPage::setTapHighlight() method.
    
            Reviewed by Mike Lattanzio and Mike Fenton.

Also

commit b4d678e9f2234041e2ccecea60968eca5ca0cf50
Author: Arvid Nilsson <[e-mail]>
Date:   Tue May 8 17:47:14 2012 +0200

    2012-05-08  Arvid Nilsson  <[e-mail]>
    
            Fix simulator build after AC change
    
            Reviewed by Commit Bot.
    
            * Api/WebPage_p.h:

[Arvid Nilsson]

Created attachment 144231 [details]
Patch

[Mike Fenton]

Reviewed internally, no new issues.

[Rob Buis]

Comment on attachment 144231 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=144231&action=review

Looks good.

> Source/WebKit/blackberry/Api/WebTapHighlight.h:33
> +    virtual void draw(const Platform::IntRectRegion&, int red, int green, int blue, int alpha, bool hideAfterScroll) = 0;

Have you considered RGBA32?

> Source/WebKit/blackberry/WebKitSupport/DefaultTapHighlight.cpp:37
> +#include <SkCornerPathEffect.h>

Is this one needed?

[Arvid Nilsson]

(In reply to comment #3)
> (From update of attachment 144231 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=144231&action=review
> 
> Looks good.
> 
> > Source/WebKit/blackberry/Api/WebTapHighlight.h:33
> > +    virtual void draw(const Platform::IntRectRegion&, int red, int green, int blue, int alpha, bool hideAfterScroll) = 0;
> 
> Have you considered RGBA32?
> 

In this case, it's public api, and there's no public color class or typedef afaik. The reason four inta are used, and not oe int, or 4 chars, is that the method signature is modelled after the old WebPageClient::drawTapHighlight() method. Or was it showTapHighlight?

> > Source/WebKit/blackberry/WebKitSupport/DefaultTapHighlight.cpp:37
> > +#include <SkCornerPathEffect.h>
> 
> Is this one needed?

Nope, this one is popping up like a jack in the box, I'll have to upload a new patch without it.

[WebKit Review Bot]

Comment on attachment 144231 [details]
Patch

Clearing flags on attachment: 144231

Committed r118701: <http://trac.webkit.org/changeset/118701>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.