[Webkit-unassigned] [Bug 81197] [BlackBerry] Can not open certain links from bridge applications

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=81197





--- Comment #3 from Robin Cao <robin.cao at torchmobile.com.cn>  2012-03-16 00:11:02 PST ---
(In reply to comment #2)
> (From update of attachment 132009 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=132009&action=review
> 
> > Source/WebKit/blackberry/WebCoreSupport/FrameLoaderClientBlackBerry.cpp:1068
> > +    // how to handle patterns such as 'mailto:'.
> 
> Where are we giving that chance iafter this patch? I thought the disabled block did that.
> 

The following is the updated comments, hope this explains things better.

    // FIXME: Enable these commented out when WebPageClient::handleStringPattern is implemented
    // and exposed to client. Before that, don't return PolicyIgnore so we can continue to
    // create new window and get to dispatchDecidePolicyForNavigationAction() where the client
    // is given a chance to decide how to handle patterns such as 'mailto:'.

> > Source/WebKit/blackberry/WebCoreSupport/FrameLoaderClientBlackBerry.cpp:1075
> > +    // }
> 
> I think putting this in a #if 0 block would make this look less messy.
Will use "#if 0" in the updated patch.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.