[Webkit-unassigned] [Bug 79318] [BlackBerry] Anchor mailto: with target set won't launch Messaging application when clicked

[bugzilla-daemon at webkit.org]

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


Charles Wei <charles.wei at torchmobile.com.cn> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #128397|commit-queue?               |commit-queue+
               Flag|                            |




--- Comment #4 from Charles Wei <charles.wei at torchmobile.com.cn>  2012-02-23 05:51:52 PST ---
(From update of attachment 128397)
View in context: https://bugs.webkit.org/attachment.cgi?id=128397&action=review

>>> Source/WebKit/blackberry/WebCoreSupport/FrameLoaderClientBlackBerry.cpp:282
>>> +    BlackBerry::Platform::NetworkRequest platformRequest;
>> 
>> Can probably remove the BlackBerry prefix.
> 
> No, we can't remove the BlackBerry prefix. We are using BlackBerry:WebKit, but not BlackBerry. Maybe we should use BlackBerry and add the WebKit prefix back in the code. We should do that with another patch.

No, we can't remove the BlackBerry prefix. We are using BlackBerry:WebKit, but not BlackBerry. Maybe we should use BlackBerry and add the WebKit prefix back in the code. We should do that with another patch.

>>> Source/WebKit/blackberry/WebCoreSupport/FrameLoaderClientBlackBerry.cpp:287
>>> +            || action.type() == NavigationTypeFormResubmitted)
>> 
>> Can be just one line.
> 
> Yes,  will fix this and submit.

Yes,  will fix this and submit.

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